Message-ID: <04d801ca08e3$9f9bd9b0$808410ac@zhaoleiwin>
From: "Zhaolei" <zhaolei@cn.fujitsu.com>
To: "Andi Kleen" <andi@firstfloor.org>
Cc: "Ingo Molnar" <mingo@elte.hu>, "Thomas Gleixner" <tglx@linutronix.de>,
       "Andrew Morton" <akpm@linux-foundation.org>,
       "OGAWA Hirofumi" <hirofumi@mail.parknet.co.jp>,
       <linux-kernel@vger.kernel.org>
References: <4A5C3BC0.6020701@cn.fujitsu.com> <87skguuqh3.fsf@basil.nowhere.org>
Subject: Re: [PATCH 1/2] Add function to convert between calendar time and broken-down time for universal use
Date: Mon, 20 Jul 2009 10:41:45 +0800
MIME-Version: 1.0
Content-Type: text/plain;
	charset="ISO-8859-1"
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 873
Lines: 26

* From: "Andi Kleen" <andi@firstfloor.org>
>> +
>> +static void __offtime(__kernel_time_t totalsecs, int offset,
>> +         unsigned int *year, unsigned int *mon, unsigned int *mday,
>> +         unsigned int *hour, unsigned int *min, unsigned int *sec,
>> +         unsigned int *wday, unsigned int *yday)
>> +{
>> +    long days, rem, y;
>> +    const unsigned short *ip;
>> +
>> +    days = totalsecs / SECS_PER_DAY;
> 
> Does this handle leap seconds? Doesn't seem to.

Hello, Andi

Thank for your attention.

It seems that we don't need to deal with leap seconds in conversion because
unix time is not counting leap seconds..

There is some detail in http://en.wikipedia.org/wiki/Unix_time

Thanks
Zhaolei
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?