Re: [Y2038] [PATCH 11/23] y2038: rusage: use __kernel_old_timeval

2019-11-14 Thread Christian Brauner
[+Cc Florian, libc-alpha] On Thu, Nov 14, 2019 at 11:18:15AM +0100, Arnd Bergmann wrote: > On Thu, Nov 14, 2019 at 1:38 AM Christian Brauner > wrote: > > On Wed, Nov 13, 2019 at 11:02:12AM +0100, Arnd Bergmann wrote: > > > On Tue, Nov 12, 2019 at 10:09 PM Cyrill Gorcunov > > > wrote: > > > > >

Re: [Y2038] [PATCH 11/23] y2038: rusage: use __kernel_old_timeval

2019-11-14 Thread Arnd Bergmann
On Thu, Nov 14, 2019 at 1:38 AM Christian Brauner wrote: > On Wed, Nov 13, 2019 at 11:02:12AM +0100, Arnd Bergmann wrote: > > On Tue, Nov 12, 2019 at 10:09 PM Cyrill Gorcunov wrote: > > > > > > On Fri, Nov 08, 2019 at 10:12:10PM +0100, Arnd Bergmann wrote: > > > > > > --- > > > > Question:

Re: [Y2038] [PATCH 11/23] y2038: rusage: use __kernel_old_timeval

2019-11-13 Thread Cyrill Gorcunov
On Wed, Nov 13, 2019 at 11:02:12AM +0100, Arnd Bergmann wrote: ... > > There are clearly too many time types at the moment, but I'm in the > process of throwing out the ones we no longer need now. Cool! > I do have a number patches implementing other variants for the syscall, > and I suppose

Re: [Y2038] [PATCH 11/23] y2038: rusage: use __kernel_old_timeval

2019-11-13 Thread Arnd Bergmann
On Tue, Nov 12, 2019 at 10:09 PM Cyrill Gorcunov wrote: > > On Fri, Nov 08, 2019 at 10:12:10PM +0100, Arnd Bergmann wrote: > > --- > > Question: should we also rename 'struct rusage' into 'struct > > __kernel_rusage' > > here, to make them completely unambiguous? > > The patch looks ok to me. I

Re: [Y2038] [PATCH 11/23] y2038: rusage: use __kernel_old_timeval

2019-11-12 Thread Cyrill Gorcunov
On Fri, Nov 08, 2019 at 10:12:10PM +0100, Arnd Bergmann wrote: > There are two 'struct timeval' fields in 'struct rusage'. > > Unfortunately the definition of timeval is now ambiguous when used in > user space with a libc that has a 64-bit time_t, and this also changes > the 'rusage' definition

[Y2038] [PATCH 11/23] y2038: rusage: use __kernel_old_timeval

2019-11-08 Thread Arnd Bergmann
There are two 'struct timeval' fields in 'struct rusage'. Unfortunately the definition of timeval is now ambiguous when used in user space with a libc that has a 64-bit time_t, and this also changes the 'rusage' definition in user space in a way that is incompatible with the system call