[PATCH v2 09/25] y2038: Add __USE_TIME_BITS64 support for struct timeval

Carlos O'Donell carlos@redhat.com
Fri Jun 4 19:31:14 GMT 2021


On 5/18/21 4:55 PM, Adhemerval Zanella wrote:
> The __USE_TIME_BITS64 is not defined internally yet.

LGTM.

No regressions on x86_64, i686, ppc64le, aarch64, s390x.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Tested-by: Carlos O'Donell <carlos@redhat.com>


> ---
>  time/bits/types/struct_timeval.h | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/time/bits/types/struct_timeval.h b/time/bits/types/struct_timeval.h
> index 70394ce886..3466137c35 100644
> --- a/time/bits/types/struct_timeval.h
> +++ b/time/bits/types/struct_timeval.h
> @@ -7,7 +7,12 @@
>     microsecond but also has a range of years.  */
>  struct timeval
>  {
> +#ifdef __USE_TIME_BITS64
> +  __time64_t tv_sec;		/* Seconds.  */
> +  __suseconds64_t tv_usec;	/* Microseconds.  */

OK.

> +#else
>    __time_t tv_sec;		/* Seconds.  */
>    __suseconds_t tv_usec;	/* Microseconds.  */
> +#endif
>  };
>  #endif
> 


-- 
Cheers,
Carlos.



More information about the Libc-alpha mailing list