Re: [patch linux-util-ng 2.13.0.1] misc rtcwake cleanups

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

 



On Wed, Dec 05, 2007 at 11:22:25AM -0800, David Brownell wrote:
> 
> And for the record ... see the appended, which I needed to get this
> to work on an Ubuntu Feisty system (dual booting XP, so that the RTC
> must match wall-clock time).  That was the reason I ended up looking
> at this code again ... :)

 I have also system with "LOCAL" in /etc/adjtime.

> I'm not currently submitting this, since I've not had time to test
> this the ARM systems with the suspect uClibc version.  I'd like to
> understand a bit better how this bug arrived in the first place.

 If I well understand the alarm (RTC_[WK]ALM_SET) requires time in the
 same mode like /dev/rtc.

> -	tm = gmtime(wakeup);
> +	tm = localtime(wakeup);

 I think this change is correct. You needn't gmtime() in setup_alarm(),
 because:

    - for "UTC" systems you have TZ=UTC (defined in get_basetimes())
      so localtime() always returns UTC,
    - for "LOCAL" systems you need the wakeup time in real timezone so
      localtime() is also correct.

 Right?

    Karel

-- 
 Karel Zak  <kzak@xxxxxxxxxx>
-
To unsubscribe from this list: send the line "unsubscribe util-linux-ng" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux