From a27a4f4721837a5fb36ace833764b06a64c5af1c Mon Sep 17 00:00:00 2001 From: "Albert ARIBAUD (3ADEV)" Date: Wed, 24 Oct 2018 11:34:26 +0200 Subject: Y2038: provide size of default time_t for target architecture To determine whether the default time_t interfaces are 32-bit and so need conversions, or are 64-bit and so are compatible with the internal 64-bit type without conversions, a macro giving the size of the default time_t is also required. This macro is called __TIMESIZE. This macro can then be used instead of __WORDSIZE in msq-pad.h and shm-pad.h files, which in turn allows removing their x86 variants, and in sem-pad.h files but keeping the x86 variant. This patch was tested by running 'make check' on branch master then applying this patch and running 'make check' again, and checking that both 'make check' yield identical results. This was done on x86_64-linux-gnu and i686-linux-gnu. * bits/timesize.h: New file. * stdlib/Makefile (headers): Add bits/timesize.h. * sysdeps/unix/sysv/linux/bits/msq-pad.h (__MSQ_PAD_AFTER_TIME): Use __TIMESIZE instead of __WORDSIZE. * sysdeps/unix/sysv/linux/bits/sem-pad.h (__SEM_PAD_AFTER_TIME): Likewise. * sysdeps/unix/sysv/linux/bits/shm-pad.h (__SHM_PAD_AFTER_TIME): Likewise. * sysdeps/unix/sysv/linux/hppa/bits/msq-pad.h (__MSQ_PAD_BEFORE_TIME): Likewise. * sysdeps/unix/sysv/linux/hppa/bits/sem-pad.h (__SEM_PAD_BEFORE_TIME): Likewise. * sysdeps/unix/sysv/linux/hppa/bits/shm-pad.h (__SHM_PAD_BEFORE_TIME, __SHM_PAD_BETWEEN_TIME_AND_SEGSZ): Likewise. * sysdeps/unix/sysv/linux/mips/bits/msq-pad.h (__MSQ_PAD_AFTER_TIME, __MSQ_PAD_BEFORE_TIME): Likewise. * sysdeps/unix/sysv/linux/powerpc/bits/msq-pad.h (__MSQ_PAD_BEFORE_TIME): Likewise. * sysdeps/unix/sysv/linux/powerpc/bits/sem-pad.h (__SEM_PAD_BEFORE_TIME): Likewise. * sysdeps/unix/sysv/linux/powerpc/bits/shm-pad.h (__SHM_PAD_BEFORE_TIME, __SHM_PAD_BETWEEN_TIME_AND_SEGSZ): Likewise. * sysdeps/unix/sysv/linux/sparc/bits/msq-pad.h (__MSQ_PAD_BEFORE_TIME): Likewise. * sysdeps/unix/sysv/linux/sparc/bits/sem-pad.h (__SEM_PAD_BEFORE_TIME): Likewise. * sysdeps/unix/sysv/linux/sparc/bits/shm-pad.h (__SHM_PAD_BEFORE_TIME): Likewise. * sysdeps/unix/sysv/linux/x86/bits/msq-pad.h: Delete file. * sysdeps/unix/sysv/linux/x86/bits/shm-pad.h: Likewise. * sysdeps/unix/sysv/linux/x86/bits/timesize.h: New file. --- ChangeLog | 34 ++++++++++++++++++++++++++ bits/timesize.h | 22 +++++++++++++++++ stdlib/Makefile | 2 +- sysdeps/unix/sysv/linux/bits/msq-pad.h | 4 +-- sysdeps/unix/sysv/linux/bits/sem-pad.h | 4 +-- sysdeps/unix/sysv/linux/bits/shm-pad.h | 4 +-- sysdeps/unix/sysv/linux/hppa/bits/msq-pad.h | 4 +-- sysdeps/unix/sysv/linux/hppa/bits/sem-pad.h | 4 +-- sysdeps/unix/sysv/linux/hppa/bits/shm-pad.h | 6 ++--- sysdeps/unix/sysv/linux/mips/bits/msq-pad.h | 6 ++--- sysdeps/unix/sysv/linux/powerpc/bits/msq-pad.h | 4 +-- sysdeps/unix/sysv/linux/powerpc/bits/sem-pad.h | 4 +-- sysdeps/unix/sysv/linux/powerpc/bits/shm-pad.h | 6 ++--- sysdeps/unix/sysv/linux/sparc/bits/msq-pad.h | 4 +-- sysdeps/unix/sysv/linux/sparc/bits/sem-pad.h | 4 +-- sysdeps/unix/sysv/linux/sparc/bits/shm-pad.h | 4 +-- sysdeps/unix/sysv/linux/x86/bits/msq-pad.h | 28 --------------------- sysdeps/unix/sysv/linux/x86/bits/shm-pad.h | 30 ----------------------- sysdeps/unix/sysv/linux/x86/bits/timesize.h | 25 +++++++++++++++++++ 19 files changed, 111 insertions(+), 88 deletions(-) create mode 100644 bits/timesize.h delete mode 100644 sysdeps/unix/sysv/linux/x86/bits/msq-pad.h delete mode 100644 sysdeps/unix/sysv/linux/x86/bits/shm-pad.h create mode 100644 sysdeps/unix/sysv/linux/x86/bits/timesize.h diff --git a/ChangeLog b/ChangeLog index cda75db..9b44e8d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,37 @@ +2018-10-24 Albert ARIBAUD + + * bits/timesize.h: New file. + * stdlib/Makefile (headers): Add bits/timesize.h. + * sysdeps/unix/sysv/linux/bits/msq-pad.h + (__MSQ_PAD_AFTER_TIME): Use __TIMESIZE instead of __WORDSIZE. + * sysdeps/unix/sysv/linux/bits/sem-pad.h + (__SEM_PAD_AFTER_TIME): Likewise. + * sysdeps/unix/sysv/linux/bits/shm-pad.h + (__SHM_PAD_AFTER_TIME): Likewise. + * sysdeps/unix/sysv/linux/hppa/bits/msq-pad.h + (__MSQ_PAD_BEFORE_TIME): Likewise. + * sysdeps/unix/sysv/linux/hppa/bits/sem-pad.h + (__SEM_PAD_BEFORE_TIME): Likewise. + * sysdeps/unix/sysv/linux/hppa/bits/shm-pad.h + (__SHM_PAD_BEFORE_TIME, __SHM_PAD_BETWEEN_TIME_AND_SEGSZ): Likewise. + * sysdeps/unix/sysv/linux/mips/bits/msq-pad.h + (__MSQ_PAD_AFTER_TIME, __MSQ_PAD_BEFORE_TIME): Likewise. + * sysdeps/unix/sysv/linux/powerpc/bits/msq-pad.h + (__MSQ_PAD_BEFORE_TIME): Likewise. + * sysdeps/unix/sysv/linux/powerpc/bits/sem-pad.h + (__SEM_PAD_BEFORE_TIME): Likewise. + * sysdeps/unix/sysv/linux/powerpc/bits/shm-pad.h + (__SHM_PAD_BEFORE_TIME, __SHM_PAD_BETWEEN_TIME_AND_SEGSZ): Likewise. + * sysdeps/unix/sysv/linux/sparc/bits/msq-pad.h + (__MSQ_PAD_BEFORE_TIME): Likewise. + * sysdeps/unix/sysv/linux/sparc/bits/sem-pad.h + (__SEM_PAD_BEFORE_TIME): Likewise. + * sysdeps/unix/sysv/linux/sparc/bits/shm-pad.h + (__SHM_PAD_BEFORE_TIME): Likewise. + * sysdeps/unix/sysv/linux/x86/bits/msq-pad.h: Delete file. + * sysdeps/unix/sysv/linux/x86/bits/shm-pad.h: Likewise. + * sysdeps/unix/sysv/linux/x86/bits/timesize.h: New file. + 2018-10-24 H.J. Lu * benchtests/Makefile (CPPFLAGS-nonlib): Add -DUSE_RDTSCP if diff --git a/bits/timesize.h b/bits/timesize.h new file mode 100644 index 0000000..cc47ff1 --- /dev/null +++ b/bits/timesize.h @@ -0,0 +1,22 @@ +/* Bit size of the time_t type at glibc build time, general case. + Copyright (C) 2018 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include + +/* Size in bits of the 'time_t' type of the default ABI. */ +#define __TIMESIZE __WORDSIZE diff --git a/stdlib/Makefile b/stdlib/Makefile index 01194bb..1773a34 100644 --- a/stdlib/Makefile +++ b/stdlib/Makefile @@ -24,7 +24,7 @@ include ../Makeconfig headers := stdlib.h bits/stdlib.h bits/stdlib-ldbl.h bits/stdlib-float.h \ monetary.h bits/monetary-ldbl.h \ - inttypes.h stdint.h bits/wordsize.h \ + inttypes.h stdint.h bits/wordsize.h bits/timesize.h \ errno.h sys/errno.h bits/errno.h bits/types/error_t.h \ ucontext.h sys/ucontext.h bits/indirect-return.h \ alloca.h fmtmsg.h \ diff --git a/sysdeps/unix/sysv/linux/bits/msq-pad.h b/sysdeps/unix/sysv/linux/bits/msq-pad.h index 97dee0b..53f367d 100644 --- a/sysdeps/unix/sysv/linux/bits/msq-pad.h +++ b/sysdeps/unix/sysv/linux/bits/msq-pad.h @@ -20,12 +20,12 @@ # error "Never use directly; include instead." #endif -#include +#include /* On most architectures, padding goes after time fields for 32-bit systems and is omitted for 64-bit systems. Some architectures pad before time fields instead, or omit padding despite being 32-bit. */ -#define __MSQ_PAD_AFTER_TIME (__WORDSIZE == 32) +#define __MSQ_PAD_AFTER_TIME (__TIMESIZE == 32) #define __MSQ_PAD_BEFORE_TIME 0 diff --git a/sysdeps/unix/sysv/linux/bits/sem-pad.h b/sysdeps/unix/sysv/linux/bits/sem-pad.h index c57fea4..488b591 100644 --- a/sysdeps/unix/sysv/linux/bits/sem-pad.h +++ b/sysdeps/unix/sysv/linux/bits/sem-pad.h @@ -20,7 +20,7 @@ # error "Never use directly; include instead." #endif -#include +#include /* On most architectures, padding goes after time fields for 32-bit systems and is omitted for 64-bit systems. Some architectures pad @@ -29,5 +29,5 @@ used for struct semid64_ds in , as glibc does not do layout conversions for this structure. */ -#define __SEM_PAD_AFTER_TIME (__WORDSIZE == 32) +#define __SEM_PAD_AFTER_TIME (__TIMESIZE == 32) #define __SEM_PAD_BEFORE_TIME 0 diff --git a/sysdeps/unix/sysv/linux/bits/shm-pad.h b/sysdeps/unix/sysv/linux/bits/shm-pad.h index 9233c95..d3cc5eb 100644 --- a/sysdeps/unix/sysv/linux/bits/shm-pad.h +++ b/sysdeps/unix/sysv/linux/bits/shm-pad.h @@ -20,7 +20,7 @@ # error "Never use directly; include instead." #endif -#include +#include /* On most architectures, padding goes after time fields for 32-bit systems and is omitted for 64-bit systems. Some architectures pad @@ -31,7 +31,7 @@ layout used for struct shmid64_ds in , as glibc does not do layout conversions for this structure. */ -#define __SHM_PAD_AFTER_TIME (__WORDSIZE == 32) +#define __SHM_PAD_AFTER_TIME (__TIMESIZE == 32) #define __SHM_PAD_BEFORE_TIME 0 #define __SHM_SEGSZ_AFTER_TIME 0 #define __SHM_PAD_BETWEEN_TIME_AND_SEGSZ 0 diff --git a/sysdeps/unix/sysv/linux/hppa/bits/msq-pad.h b/sysdeps/unix/sysv/linux/hppa/bits/msq-pad.h index 10b26b7..05db3fd 100644 --- a/sysdeps/unix/sysv/linux/hppa/bits/msq-pad.h +++ b/sysdeps/unix/sysv/linux/hppa/bits/msq-pad.h @@ -20,7 +20,7 @@ # error "Never use directly; include instead." #endif -#include +#include #define __MSQ_PAD_AFTER_TIME 0 -#define __MSQ_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __MSQ_PAD_BEFORE_TIME (__TIMESIZE == 32) diff --git a/sysdeps/unix/sysv/linux/hppa/bits/sem-pad.h b/sysdeps/unix/sysv/linux/hppa/bits/sem-pad.h index a938c2c..f250f64 100644 --- a/sysdeps/unix/sysv/linux/hppa/bits/sem-pad.h +++ b/sysdeps/unix/sysv/linux/hppa/bits/sem-pad.h @@ -20,7 +20,7 @@ # error "Never use directly; include instead." #endif -#include +#include #define __SEM_PAD_AFTER_TIME 0 -#define __SEM_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __SEM_PAD_BEFORE_TIME (__TIMESIZE == 32) diff --git a/sysdeps/unix/sysv/linux/hppa/bits/shm-pad.h b/sysdeps/unix/sysv/linux/hppa/bits/shm-pad.h index 6a9c5dd..8764bde 100644 --- a/sysdeps/unix/sysv/linux/hppa/bits/shm-pad.h +++ b/sysdeps/unix/sysv/linux/hppa/bits/shm-pad.h @@ -20,9 +20,9 @@ # error "Never use directly; include instead." #endif -#include +#include #define __SHM_PAD_AFTER_TIME 0 -#define __SHM_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __SHM_PAD_BEFORE_TIME (__TIMESIZE == 32) #define __SHM_SEGSZ_AFTER_TIME 1 -#define __SHM_PAD_BETWEEN_TIME_AND_SEGSZ (__WORDSIZE == 32) +#define __SHM_PAD_BETWEEN_TIME_AND_SEGSZ (__TIMESIZE == 32) diff --git a/sysdeps/unix/sysv/linux/mips/bits/msq-pad.h b/sysdeps/unix/sysv/linux/mips/bits/msq-pad.h index 1ddfb0d..530babe 100644 --- a/sysdeps/unix/sysv/linux/mips/bits/msq-pad.h +++ b/sysdeps/unix/sysv/linux/mips/bits/msq-pad.h @@ -20,12 +20,12 @@ # error "Never use directly; include instead." #endif -#include +#include #ifdef __MIPSEL__ -# define __MSQ_PAD_AFTER_TIME (__WORDSIZE == 32) +# define __MSQ_PAD_AFTER_TIME (__TIMESIZE == 32) # define __MSQ_PAD_BEFORE_TIME 0 #else # define __MSQ_PAD_AFTER_TIME 0 -# define __MSQ_PAD_BEFORE_TIME (__WORDSIZE == 32) +# define __MSQ_PAD_BEFORE_TIME (__TIMESIZE == 32) #endif diff --git a/sysdeps/unix/sysv/linux/powerpc/bits/msq-pad.h b/sysdeps/unix/sysv/linux/powerpc/bits/msq-pad.h index 9680176..d19dfd5 100644 --- a/sysdeps/unix/sysv/linux/powerpc/bits/msq-pad.h +++ b/sysdeps/unix/sysv/linux/powerpc/bits/msq-pad.h @@ -20,7 +20,7 @@ # error "Never use directly; include instead." #endif -#include +#include #define __MSQ_PAD_AFTER_TIME 0 -#define __MSQ_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __MSQ_PAD_BEFORE_TIME (__TIMESIZE == 32) diff --git a/sysdeps/unix/sysv/linux/powerpc/bits/sem-pad.h b/sysdeps/unix/sysv/linux/powerpc/bits/sem-pad.h index d4c9a93..d7571cc 100644 --- a/sysdeps/unix/sysv/linux/powerpc/bits/sem-pad.h +++ b/sysdeps/unix/sysv/linux/powerpc/bits/sem-pad.h @@ -20,7 +20,7 @@ # error "Never use directly; include instead." #endif -#include +#include #define __SEM_PAD_AFTER_TIME 0 -#define __SEM_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __SEM_PAD_BEFORE_TIME (__TIMESIZE == 32) diff --git a/sysdeps/unix/sysv/linux/powerpc/bits/shm-pad.h b/sysdeps/unix/sysv/linux/powerpc/bits/shm-pad.h index 69a7e4a..c8c2d44 100644 --- a/sysdeps/unix/sysv/linux/powerpc/bits/shm-pad.h +++ b/sysdeps/unix/sysv/linux/powerpc/bits/shm-pad.h @@ -20,9 +20,9 @@ # error "Never use directly; include instead." #endif -#include +#include #define __SHM_PAD_AFTER_TIME 0 -#define __SHM_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __SHM_PAD_BEFORE_TIME (__TIMESIZE == 32) #define __SHM_SEGSZ_AFTER_TIME 1 -#define __SHM_PAD_BETWEEN_TIME_AND_SEGSZ (__WORDSIZE == 32) +#define __SHM_PAD_BETWEEN_TIME_AND_SEGSZ (__TIMESIZE == 32) diff --git a/sysdeps/unix/sysv/linux/sparc/bits/msq-pad.h b/sysdeps/unix/sysv/linux/sparc/bits/msq-pad.h index 3dfeb07..72324e8 100644 --- a/sysdeps/unix/sysv/linux/sparc/bits/msq-pad.h +++ b/sysdeps/unix/sysv/linux/sparc/bits/msq-pad.h @@ -20,7 +20,7 @@ # error "Never use directly; include instead." #endif -#include +#include #define __MSQ_PAD_AFTER_TIME 0 -#define __MSQ_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __MSQ_PAD_BEFORE_TIME (__TIMESIZE == 32) diff --git a/sysdeps/unix/sysv/linux/sparc/bits/sem-pad.h b/sysdeps/unix/sysv/linux/sparc/bits/sem-pad.h index b85f6b7..27266d4 100644 --- a/sysdeps/unix/sysv/linux/sparc/bits/sem-pad.h +++ b/sysdeps/unix/sysv/linux/sparc/bits/sem-pad.h @@ -20,7 +20,7 @@ # error "Never use directly; include instead." #endif -#include +#include #define __SEM_PAD_AFTER_TIME 0 -#define __SEM_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __SEM_PAD_BEFORE_TIME (__TIMESIZE == 32) diff --git a/sysdeps/unix/sysv/linux/sparc/bits/shm-pad.h b/sysdeps/unix/sysv/linux/sparc/bits/shm-pad.h index c4d0c2f..fa3cc18 100644 --- a/sysdeps/unix/sysv/linux/sparc/bits/shm-pad.h +++ b/sysdeps/unix/sysv/linux/sparc/bits/shm-pad.h @@ -20,9 +20,9 @@ # error "Never use directly; include instead." #endif -#include +#include #define __SHM_PAD_AFTER_TIME 0 -#define __SHM_PAD_BEFORE_TIME (__WORDSIZE == 32) +#define __SHM_PAD_BEFORE_TIME (__TIMESIZE == 32) #define __SHM_SEGSZ_AFTER_TIME 1 #define __SHM_PAD_BETWEEN_TIME_AND_SEGSZ 0 diff --git a/sysdeps/unix/sysv/linux/x86/bits/msq-pad.h b/sysdeps/unix/sysv/linux/x86/bits/msq-pad.h deleted file mode 100644 index 5d46956..0000000 --- a/sysdeps/unix/sysv/linux/x86/bits/msq-pad.h +++ /dev/null @@ -1,28 +0,0 @@ -/* Define where padding goes in struct msqid_ds. x86 version. - Copyright (C) 2018 Free Software Foundation, Inc. - This file is part of the GNU C Library. - - The GNU C Library is free software; you can redistribute it and/or - modify it under the terms of the GNU Lesser General Public - License as published by the Free Software Foundation; either - version 2.1 of the License, or (at your option) any later version. - - The GNU C Library is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - Lesser General Public License for more details. - - You should have received a copy of the GNU Lesser General Public - License along with the GNU C Library; if not, see - . */ - -#ifndef _SYS_MSG_H -# error "Never use directly; include instead." -#endif - -#ifdef __x86_64__ -# define __MSQ_PAD_AFTER_TIME 0 -#else -# define __MSQ_PAD_AFTER_TIME 1 -#endif -#define __MSQ_PAD_BEFORE_TIME 0 diff --git a/sysdeps/unix/sysv/linux/x86/bits/shm-pad.h b/sysdeps/unix/sysv/linux/x86/bits/shm-pad.h deleted file mode 100644 index 277a70f..0000000 --- a/sysdeps/unix/sysv/linux/x86/bits/shm-pad.h +++ /dev/null @@ -1,30 +0,0 @@ -/* Define where padding goes in struct shmid_ds. x86 version. - Copyright (C) 2018 Free Software Foundation, Inc. - This file is part of the GNU C Library. - - The GNU C Library is free software; you can redistribute it and/or - modify it under the terms of the GNU Lesser General Public - License as published by the Free Software Foundation; either - version 2.1 of the License, or (at your option) any later version. - - The GNU C Library is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - Lesser General Public License for more details. - - You should have received a copy of the GNU Lesser General Public - License along with the GNU C Library; if not, see - . */ - -#ifndef _SYS_SHM_H -# error "Never use directly; include instead." -#endif - -#ifdef __x86_64__ -# define __SHM_PAD_AFTER_TIME 0 -#else -# define __SHM_PAD_AFTER_TIME 1 -#endif -#define __SHM_PAD_BEFORE_TIME 0 -#define __SHM_SEGSZ_AFTER_TIME 0 -#define __SHM_PAD_BETWEEN_TIME_AND_SEGSZ 0 diff --git a/sysdeps/unix/sysv/linux/x86/bits/timesize.h b/sysdeps/unix/sysv/linux/x86/bits/timesize.h new file mode 100644 index 0000000..8b88ab8 --- /dev/null +++ b/sysdeps/unix/sysv/linux/x86/bits/timesize.h @@ -0,0 +1,25 @@ +/* Bit size of the time_t type at glibc build time, x86-64 and x32 case. + Copyright (C) 2018 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#if defined __x86_64__ && defined __ILP32__ +/* For x32, time is 64-bit even though word size is 32-bit. */ +# define __TIMESIZE 64 +#else +/* For others, time size is word size. */ +# define __TIMESIZE __WORDSIZE +#endif -- cgit v1.1