This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [RFC v6 03/23] time: Add a timeval with a long tv_sec and tv_usec
- From: Joseph Myers <joseph at codesourcery dot com>
- To: Alistair Francis <alistair dot francis at wdc dot com>
- Cc: <libc-alpha at sourceware dot org>, <arnd at arndb dot de>, <adhemerval dot zanella at linaro dot org>, <fweimer at redhat dot com>, <palmerdabbelt at google dot com>, <macro at wdc dot com>, <zongbox at gmail dot com>, <alistair23 at gmail dot com>
- Date: Sun, 12 Jan 2020 14:48:29 +0000
- Subject: Re: [RFC v6 03/23] time: Add a timeval with a long tv_sec and tv_usec
- Ironport-sdr: ySQMzkqhQXMs35+eFfhPGl5pEEQf3juVffgbRj4XZMEcFSwVMC9UZtdRJrG2qL/teGfAbO1HnR 7dA9bFw00OO7N0myqdlHhjPJr+4NRiT4aLShljEhfRFY+nYVVddskjLHk6oN0NeAf+tGM59WI3 oUIR79Nm4CkShgENtIKPwzNmtKuvyZgPEFOUjD0X095Kh5w4stiC7VTVWhpcf3dglY5YIyGrRu x3PnP6iyXvy6PsYtxL/z5ke08y3xv7jRdZ8YXaFIvRfu8XKar4544TMmYhzPUeHUsBYm13aNme 33Q=
- Ironport-sdr: R/DxE8+B0v81Rs3GoOP4VdFwL7QdC/pPPpZWpLRpysSHViHoJofpUWGboqGU42k4X9qasYtscC lt4RyUqh80TjB6SRbsSPqJKPo4+tKmQNTQMyylGAlqPv8HRs4/ljpxgI9vfSSKfvJ5Z40isPqe TF+iKV4VjJr/OtTjhq8L4pqkX5BagvYgw434Qn3H8ydC4H83FhBbJL74mmcyGp6sZS9O/rOI/G LoY150JgYzaH0TseUIrZrId29/P3qZgAMMkfmXR6o6AQ4IlIalkZ6DuC18Jm8HcxDuyRhR5DtQ Gbc=
- References: <cover.1578824547.git.alistair.francis@wdc.com> <85291ec316dcd3f3a3155488a8c290298650dc77.1578824547.git.alistair.francis@wdc.com>
On Sun, 12 Jan 2020, Alistair Francis wrote:
> On y2038 safe 32-bit systems the Linux kernel expects itimerval to
> use a 32-bit time_t, even though the other time_t's are 64-bit. To
> address this let's add a timeval_long struct to be used internally.
> ---
> include/time.h | 29 +++++++++++++++++++++++++++++
> time/bits/types/struct_timeval.h | 8 ++++++++
bits/types/ headers are *installed* headers that should only define the
type in question. If there is some good reason to need __timeval_long in
*installed* headers (i.e. a *public* interface using this type) it should
go in bits/types/struct___timeval_long.h, otherwise it should go in a
non-installed header.
--
Joseph S. Myers
joseph@codesourcery.com