linux-arch.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
	Christoph Hellwig <hch@infradead.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Ley Foon Tan <lftan@altera.com>,
	Linux-Arch <linux-arch@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	LeyFoon Tan <lftan.linux@gmail.com>,
	Chung-Lin Tang <cltang@codesourcery.com>
Subject: Re: [PATCH 00/25] Change time_t and clock_t to 64 bit
Date: Thu, 15 May 2014 11:47:03 -0400	[thread overview]
Message-ID: <1400168823.7862.9.camel@dabdike> (raw)
In-Reply-To: <8468354.y2kUZdOz9e@wuerfel>

On Wed, 2014-05-14 at 13:00 +0200, Arnd Bergmann wrote:
> On Tuesday 13 May 2014 22:35:08 Geert Uytterhoeven wrote:
> > 
> > On Tue, May 13, 2014 at 9:32 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> > > I think we have three categories:
> > 
> > Thanks for the list!
> > 
> > > a) interfaces that uses relative time_t/timespec/timeval:
> > > b) interfaces that don't make sense for times in the past:
> > 
> > > c) interfaces that require absolute times:
> > >    - stat/lstat/fstatat/
> > >    - utime/utimes/futimesat
> > >
> > > These absolutely have to use something better than time_t
> > > both in user space and in the kernel so we can deal with
> > > old files. A lot of file systems need to be fixed as well so
> > > we can actually store the times, regardless of whether we
> > > are running a 32 or 64 bit kernel.
> > 
> > So these are the ones we have to worry about.
> > It looks like they all involve I/O? Apart from the case of using block data
> > from the buffer cache, the 64-bit operations should disappear in the
> > actual I/O noise, right?
> 
> Right. Also there have been proposals for a better 'stat' replacement
> for years, which would solve half of the interface problem for the
> file system interfaces.
> 
> However, we also need to find a solution for category b), I only put
> them into a different category above because we can treat them
> differently in the kernel. For instance, we could use ktime_t for
> the kernel code in category b) and a new struct timespec64 for
> the times in struct inode.
> On the user interface side, using timespec64 would be a reasonable
> choice for both categories, because we already have two implementations
> of all those syscalls in order to handle 32-on-64 compat tasks,
> and we could use the same set of syscall implementations for time64-on-32.

To step back a bit, is a 64 bit time_t actually a good solution on 32
bits?  Paying a 64 bit penalty on every time operation does seem a bit
overkill.  Most eventual uses are either monotonic counting or relative
addition/subtraction.  If we added an additional 32 bit quantity called
epoch, this would increase once every 68 years.  Within the kernel, we
could do intelligent interpolation about what the epoch is, so in 2039
if we get a low 32 bit time_t value, we assume epoch == 1, conversely if
we see a high one, we assume epoch == 0.  We could add epoch on the end
of the syscalls and detect if it's not present and fill in an
interpolated value.  Interfaces which truly want 64 bit time_t would get
it from epoch + 32 bit time_t.

James

  parent reply	other threads:[~2014-05-15 15:47 UTC|newest]

Thread overview: 132+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-13  8:57 [PATCH 00/25] Change time_t and clock_t to 64 bit Ley Foon Tan
2014-05-13  9:02 ` [PATCH 01/25] asm-generic: " Ley Foon Tan
2014-05-13  9:15   ` James Hogan
2014-05-13  9:30     ` Thomas Gleixner
2014-05-13  9:30       ` Thomas Gleixner
2014-05-13  9:32     ` Ley Foon Tan
2014-05-13  9:32       ` Ley Foon Tan
2014-05-13  9:50       ` James Hogan
2014-05-13  9:50         ` James Hogan
2014-05-13 10:17         ` Ley Foon Tan
2014-05-13 10:21           ` Arnd Bergmann
2014-05-13 10:21             ` Arnd Bergmann
2014-05-13 12:07             ` Thomas Gleixner
2014-05-13 13:11               ` Geert Uytterhoeven
2014-05-13 13:38                 ` Arnd Bergmann
2014-05-13 14:37                   ` Thomas Gleixner
2014-05-13 10:22           ` James Hogan
2014-05-13 10:22             ` James Hogan
2014-05-14  3:47             ` H. Peter Anvin
2014-05-14 10:44               ` Geert Uytterhoeven
2014-05-13  9:02 ` [PATCH 02/25] arc: Add 32 bit time_t and clock_t Ley Foon Tan
2014-05-13  9:02   ` Ley Foon Tan
2014-05-13  9:02 ` [PATCH 03/25] arm: " Ley Foon Tan
2014-05-13  9:02   ` Ley Foon Tan
2014-05-13  9:02 ` [PATCH 04/25] avr32: " Ley Foon Tan
2014-05-13  9:02   ` Ley Foon Tan
2014-05-13 20:19   ` Hans-Christian Egtvedt
2014-05-14  1:54     ` Ley Foon Tan
2014-05-14  7:44       ` Hans-Christian Egtvedt
2014-05-14 21:39         ` H. Peter Anvin
2014-05-13  9:02 ` [PATCH 05/25] blackfin: " Ley Foon Tan
2014-05-13  9:02   ` Ley Foon Tan
2014-05-13  9:03 ` [PATCH 06/25] c6x: " Ley Foon Tan
2014-05-13  9:03   ` Ley Foon Tan
2014-05-13  9:03 ` [PATCH 08/25] frv: " Ley Foon Tan
2014-05-13  9:03   ` Ley Foon Tan
2014-05-13  9:03 ` [PATCH 09/25] hexagon: " Ley Foon Tan
2014-05-13  9:03   ` Ley Foon Tan
2014-05-13  9:03 ` [PATCH 10/25] m32r: " Ley Foon Tan
2014-05-13  9:03   ` Ley Foon Tan
2014-05-13  9:03 ` [PATCH 11/25] m68k: " Ley Foon Tan
2014-05-13  9:03   ` Ley Foon Tan
2014-05-13  9:04 ` [PATCH 12/25] metag: " Ley Foon Tan
2014-05-13  9:04   ` Ley Foon Tan
2014-05-13  9:22   ` James Hogan
2014-05-13  9:22     ` James Hogan
2014-05-13  9:28     ` Ley Foon Tan
2014-05-13  9:28       ` Ley Foon Tan
2014-05-13  9:05 ` [PATCH 13/25] microblaze: " Ley Foon Tan
2014-05-13  9:05   ` Ley Foon Tan
2014-05-13  9:05 ` [PATCH 14/25] mips: " Ley Foon Tan
2014-05-13  9:05   ` Ley Foon Tan
2014-05-13  9:05 ` [PATCH 15/25] mn10300: " Ley Foon Tan
2014-05-13  9:05   ` Ley Foon Tan
2014-05-13  9:05 ` [PATCH 16/25] openrisc: " Ley Foon Tan
2014-05-13  9:05   ` Ley Foon Tan
2014-05-13  9:30   ` Jonas Bonn
2014-05-13  9:30     ` Jonas Bonn
2014-05-13  9:41     ` Ley Foon Tan
2014-05-13  9:41       ` Ley Foon Tan
2014-05-13  9:06 ` [PATCH 17/25] parisc: " Ley Foon Tan
2014-05-13  9:06   ` Ley Foon Tan
2014-05-13 20:17   ` Helge Deller
2014-05-14  1:57     ` Ley Foon Tan
2014-05-15  0:34     ` H. Peter Anvin
2014-05-15  0:34       ` H. Peter Anvin
2014-05-13  9:06 ` [PATCH 19/25] s390: " Ley Foon Tan
2014-05-13  9:06 ` [PATCH 20/25] score: " Ley Foon Tan
2014-05-13  9:06   ` Ley Foon Tan
2014-05-13  9:06 ` [PATCH 21/25] sh: " Ley Foon Tan
2014-05-13  9:06   ` Ley Foon Tan
2014-05-13  9:06 ` [PATCH 22/25] sparc: " Ley Foon Tan
2014-05-13  9:06   ` Ley Foon Tan
2014-05-13  9:06 ` [PATCH 23/25] unicore32: " Ley Foon Tan
2014-05-13  9:06   ` Ley Foon Tan
2014-05-13  9:06 ` [PATCH 25/25] xtense: " Ley Foon Tan
2014-05-13  9:06   ` Ley Foon Tan
2014-05-13  9:10 ` [PATCH 07/25] cris: " Ley Foon Tan
2014-05-13  9:10 ` [PATCH 00/25] Change time_t and clock_t to 64 bit Geert Uytterhoeven
2014-05-13  9:23   ` Ley Foon Tan
2014-05-13 15:24 ` Christoph Hellwig
2014-05-13 15:33   ` Thomas Gleixner
2014-05-13 17:46     ` Christoph Hellwig
2014-05-13 18:10       ` Arnd Bergmann
2014-05-13 18:24         ` Geert Uytterhoeven
2014-05-13 19:32           ` Arnd Bergmann
2014-05-13 20:35             ` Geert Uytterhoeven
2014-05-14 11:00               ` Arnd Bergmann
2014-05-14 12:21                 ` Thomas Gleixner
2014-05-14 14:01                   ` Arnd Bergmann
2014-05-14 14:01                     ` Arnd Bergmann
2014-05-14 14:46                     ` Thomas Gleixner
2014-05-14 15:48                       ` Arnd Bergmann
2014-05-15 15:47                 ` James Bottomley [this message]
2014-05-15 16:36                   ` Arnd Bergmann
2014-05-15 21:07                     ` One Thousand Gnomes
2014-05-16  7:48                       ` Christoph Hellwig
2014-05-16 23:06                         ` H. Peter Anvin
2014-05-16 23:06                           ` H. Peter Anvin
2014-05-14 21:33             ` John Stultz
2014-05-15  0:14               ` H. Peter Anvin
2014-05-15  3:32                 ` Nicolas Pitre
2014-05-15 11:08               ` Arnd Bergmann
2014-05-15 11:18                 ` Christoph Hellwig
2014-05-15 17:11                 ` Chung-Lin Tang
2014-05-15 17:12                 ` Chung-Lin Tang
2014-05-15 17:12                   ` Chung-Lin Tang
2014-05-15 18:01                   ` Joseph S. Myers
2014-05-15 19:18                     ` Arnd Bergmann
2014-05-15 20:10                       ` Joseph S. Myers
2014-05-15 20:38                         ` Arnd Bergmann
2014-05-15 20:38                           ` Arnd Bergmann
2014-05-15 21:05                           ` Joseph S. Myers
2014-05-19  8:08                             ` Arnd Bergmann
2014-05-19  8:08                               ` Arnd Bergmann
2014-05-19 14:46                               ` Joseph S. Myers
2014-05-19 17:22                                 ` Arnd Bergmann
2014-05-19 18:12                                   ` Joseph S. Myers
2014-05-19 18:20                                     ` Arnd Bergmann
2014-05-16 23:11                           ` H. Peter Anvin
2014-05-16 23:47                             ` Joseph S. Myers
2014-05-17  0:32                               ` H. Peter Anvin
2014-05-18 13:46                                 ` Joseph S. Myers
2014-05-18 16:10                                   ` H. Peter Anvin
2014-05-18 18:18                                     ` Joseph S. Myers
2014-05-14 10:13         ` Ley Foon Tan
2014-05-14 10:33           ` Arnd Bergmann
2014-05-14 21:04             ` H. Peter Anvin
2014-05-13 18:15       ` Geert Uytterhoeven
2014-05-13 19:14       ` Thomas Gleixner
2014-05-14  3:36 ` [PATCH 24/25] x86: Add 32 bit time_t and clock_t Ley Foon Tan
2014-05-14  3:36   ` Ley Foon Tan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1400168823.7862.9.camel@dabdike \
    --to=james.bottomley@hansenpartnership.com \
    --cc=arnd@arndb.de \
    --cc=cltang@codesourcery.com \
    --cc=geert@linux-m68k.org \
    --cc=hch@infradead.org \
    --cc=lftan.linux@gmail.com \
    --cc=lftan@altera.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).