Re: [RFC][PATCH] ppc misusing NTP's time_offset value

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I think you forgot the patch.

- kumar

On Jun 30, 2005, at 5:15 AM, Mikael Pettersson wrote:

On Wed, 29 Jun 2005 15:05:51 -0700, john stultz wrote:

    As part of my timeofday rework, I've been looking at the NTP

code and I

noticed that the PPC architecture is apparently misusing the NTP's
time_offset (it is a terrible name!) value as some form of timezone
offset. This could cause problems when time_offset changed by the NTP
code.

    This patch changes the PPC code so it uses a more clear local

variable:

timezone_offset.

Could a PPC maintainer verify this is correct?

Let me know if you have any comments or feedback.


arch/ppc/kernel/time.c used to have a 'static long time_offset;'
variable. Ulthough unrelated, this declaration clashed with the
one for kernel/time.c, causing compile-time errors with gcc4.
I submitted a fix for this in February, which renamed ppc's local
variable, and it was ACKed by Tom Rini and queued for 2.6.12.

However, the patch that actually went into 2.6.12 was different:
it just removed ppc's local variable, making arch/ppc/kernel/time.c
now share kernel/time.c's variable. At the time I assumed someone
had proved that the two modules _should_ share state, so I didn't
make a fuss about it.

Your patch brings the semantics back to what it was prior to 2.6.12.

/Mikael
-
To unsubscribe from this list: send the line "unsubscribe linux- kernel"
in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux