* [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build @ 2020-12-29 19:49 Adhemerval Zanella 2021-01-20 16:52 ` Adhemerval Zanella 2021-01-21 9:21 ` Andreas Schwab 0 siblings, 2 replies; 7+ messages in thread From: Adhemerval Zanella @ 2020-12-29 19:49 UTC (permalink / raw) To: libc-alpha, Andreas Schwab, Florian Weimer This reverts commit 20b39d59467b0c1d858e89ded8b0cebe55e22f60 for static library. This avoids the need to rebuild the world for the case where libstdc++ (and potentially other libraries) are linked to a old glibc. To avoid requering to provide xstat symbols for newer ABIs (such as riscv32) a new LIB_COMPAT macro is added. It is similar to SHLIB_COMPAT but also works for static case (thus evaluating similar to SHLIB_COMPAT for both shared and static case). Checked with a check-abi on all affected ABIs. I also check if the static library does contains the xstat symbols. --- include/shlib-compat.h | 23 ++++++++++++++--------- sysdeps/mach/hurd/fxstat.c | 5 ----- sysdeps/mach/hurd/fxstatat.c | 3 +-- sysdeps/mach/hurd/fxstatat64.c | 3 +-- sysdeps/mach/hurd/lxstat64.c | 3 +-- sysdeps/mach/hurd/xstat.c | 2 +- sysdeps/unix/sysv/linux/fxstat.c | 4 ++-- sysdeps/unix/sysv/linux/fxstat64.c | 4 ++-- sysdeps/unix/sysv/linux/fxstatat.c | 4 ++-- sysdeps/unix/sysv/linux/fxstatat64.c | 4 ++-- sysdeps/unix/sysv/linux/lxstat.c | 2 +- sysdeps/unix/sysv/linux/lxstat64.c | 4 ++-- sysdeps/unix/sysv/linux/xstat.c | 6 +++--- sysdeps/unix/sysv/linux/xstat64.c | 5 +++-- 14 files changed, 35 insertions(+), 37 deletions(-) diff --git a/include/shlib-compat.h b/include/shlib-compat.h index b58281857a..d6dae3bea9 100644 --- a/include/shlib-compat.h +++ b/include/shlib-compat.h @@ -21,8 +21,6 @@ # include <abi-versions.h> -#ifdef SHARED - /* The file abi-versions.h (generated by scripts/abi-versions.awk) defines symbols like `ABI_libm_GLIBC_2_0' for each version set in the source code for each library. For a version set that is subsumed by a later @@ -34,17 +32,24 @@ when the two version sets named are in fact two different ABIs we are supporting. If these do not differ, then there is no need to compile in extra code to support this version set where it has been superseded by a - newer version. The compatibility code should be conditionalized with - e.g. `#if SHLIB_COMPAT (libm, GLIBC_2_0, GLIBC_2_2)' for code introduced - in the GLIBC_2.0 version and obsoleted in the GLIBC_2.2 version. */ - -# define SHLIB_COMPAT(lib, introduced, obsoleted) \ - _SHLIB_COMPAT (lib, introduced, obsoleted) -# define _SHLIB_COMPAT(lib, introduced, obsoleted) \ + newer version. */ +#define LIB_COMPAT(lib, introduced, obsoleted) \ + _LIB_COMPAT (lib, introduced, obsoleted) +#define _LIB_COMPAT(lib, introduced, obsoleted) \ (IS_IN (lib) \ && (!(ABI_##lib##_##obsoleted - 0) \ || ((ABI_##lib##_##introduced - 0) < (ABI_##lib##_##obsoleted - 0)))) +#ifdef SHARED + +/* Similar to LIB_COMPAT, but evaluate to 0 for static build. The + compatibility code should be conditionalized with e.g. + `#if SHLIB_COMPAT (libm, GLIBC_2_0, GLIBC_2_2)' for code introduced + in the GLIBC_2.0 version and obsoleted in the GLIBC_2.2 version. */ + +# define SHLIB_COMPAT(lib, introduced, obsoleted) \ + _LIB_COMPAT (lib, introduced, obsoleted) + /* That header also defines symbols like `VERSION_libm_GLIBC_2_1' to the version set name to use for e.g. symbols first introduced into libm in the GLIBC_2.1 version. Definitions of symbols with explicit diff --git a/sysdeps/mach/hurd/fxstat.c b/sysdeps/mach/hurd/fxstat.c index 3201bb8cf1..e7e4b6e65e 100644 --- a/sysdeps/mach/hurd/fxstat.c +++ b/sysdeps/mach/hurd/fxstat.c @@ -19,9 +19,6 @@ #include <stddef.h> #include <sys/stat.h> #include <hurd.h> -#include <shlib-compat.h> - -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file descriptor FD in BUF. */ int @@ -33,5 +30,3 @@ __fxstat (int vers, int fd, struct stat *buf) return __fstat (fd, buf); } weak_alias (__fxstat, _fxstat) - -#endif diff --git a/sysdeps/mach/hurd/fxstatat.c b/sysdeps/mach/hurd/fxstatat.c index b71ab9fad5..b7f6ca742a 100644 --- a/sysdeps/mach/hurd/fxstatat.c +++ b/sysdeps/mach/hurd/fxstatat.c @@ -22,7 +22,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) int __fxstatat (int vers, int fd, const char *filename, struct stat *buf, int flag) @@ -32,5 +32,4 @@ __fxstatat (int vers, int fd, const char *filename, struct stat *buf, int flag) return __fstatat (fd, filename, buf, flag); } - #endif diff --git a/sysdeps/mach/hurd/fxstatat64.c b/sysdeps/mach/hurd/fxstatat64.c index f0c171f64c..8f401ddff2 100644 --- a/sysdeps/mach/hurd/fxstatat64.c +++ b/sysdeps/mach/hurd/fxstatat64.c @@ -23,7 +23,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) /* Get information about the file descriptor FD in BUF. */ int @@ -35,5 +35,4 @@ __fxstatat64 (int vers, int fd, const char *filename, struct stat64 *buf, return __fstatat64 (fd, filename, buf, flag); } - #endif diff --git a/sysdeps/mach/hurd/lxstat64.c b/sysdeps/mach/hurd/lxstat64.c index 7059bceb11..4975044ac3 100644 --- a/sysdeps/mach/hurd/lxstat64.c +++ b/sysdeps/mach/hurd/lxstat64.c @@ -22,7 +22,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) /* Get information about the file descriptor FD in BUF. */ int @@ -33,5 +33,4 @@ __lxstat64 (int vers, const char *file, struct stat64 *buf) return __lstat64 (file, buf); } - #endif diff --git a/sysdeps/mach/hurd/xstat.c b/sysdeps/mach/hurd/xstat.c index 8d56fe3c44..03b86ccb24 100644 --- a/sysdeps/mach/hurd/xstat.c +++ b/sysdeps/mach/hurd/xstat.c @@ -20,7 +20,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get file information about FILE in BUF. */ int diff --git a/sysdeps/unix/sysv/linux/fxstat.c b/sysdeps/unix/sysv/linux/fxstat.c index 528eedc272..0b89250b8b 100644 --- a/sysdeps/unix/sysv/linux/fxstat.c +++ b/sysdeps/unix/sysv/linux/fxstat.c @@ -26,7 +26,7 @@ # include <xstatover.h> # include <shlib-compat.h> -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file FD in BUF. */ int @@ -61,6 +61,6 @@ __fxstat (int vers, int fd, struct stat *buf) } } -# endif /* SHLIB_COMPAT */ +# endif /* LIB_COMPAT */ #endif /* XSTAT_IS_XSTAT64 */ diff --git a/sysdeps/unix/sysv/linux/fxstat64.c b/sysdeps/unix/sysv/linux/fxstat64.c index 9471d0be45..177233d759 100644 --- a/sysdeps/unix/sysv/linux/fxstat64.c +++ b/sysdeps/unix/sysv/linux/fxstat64.c @@ -26,7 +26,7 @@ #include <statx_cp.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file FD in BUF. */ @@ -78,4 +78,4 @@ strong_alias (___fxstat64, __fxstat64) strong_alias (___fxstat64, __fxstat) #endif -#endif /* SHLIB_COMPAT */ +#endif /* LIB_COMPAT */ diff --git a/sysdeps/unix/sysv/linux/fxstatat.c b/sysdeps/unix/sysv/linux/fxstatat.c index bd78971dea..78856742ef 100644 --- a/sysdeps/unix/sysv/linux/fxstatat.c +++ b/sysdeps/unix/sysv/linux/fxstatat.c @@ -26,7 +26,7 @@ # include <xstatover.h> # include <shlib-compat.h> -# if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) +# if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) /* Get information about the file FD in BUF. */ int @@ -50,6 +50,6 @@ __fxstatat (int vers, int fd, const char *file, struct stat *st, int flag) #endif } -# endif /* SHLIB_COMPAT */ +# endif /* LIB_COMPAT */ #endif /* XSTAT_IS_XSTAT64 */ diff --git a/sysdeps/unix/sysv/linux/fxstatat64.c b/sysdeps/unix/sysv/linux/fxstatat64.c index 091da83f1f..f23f877109 100644 --- a/sysdeps/unix/sysv/linux/fxstatat64.c +++ b/sysdeps/unix/sysv/linux/fxstatat64.c @@ -26,7 +26,7 @@ #include <statx_cp.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) /* Get information about the file FD in BUF. */ @@ -69,4 +69,4 @@ __fxstatat64 (int vers, int fd, const char *file, struct stat64 *st, int flag) strong_alias (__fxstatat64, __fxstatat) #endif -#endif /* SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) */ +#endif /* LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) */ diff --git a/sysdeps/unix/sysv/linux/lxstat.c b/sysdeps/unix/sysv/linux/lxstat.c index c7d4507d7c..a11f58b453 100644 --- a/sysdeps/unix/sysv/linux/lxstat.c +++ b/sysdeps/unix/sysv/linux/lxstat.c @@ -26,7 +26,7 @@ # include <xstatconv.h> # include <xstatover.h> -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file NAME in BUF. */ int diff --git a/sysdeps/unix/sysv/linux/lxstat64.c b/sysdeps/unix/sysv/linux/lxstat64.c index 74e11f3467..4d9d058d4b 100644 --- a/sysdeps/unix/sysv/linux/lxstat64.c +++ b/sysdeps/unix/sysv/linux/lxstat64.c @@ -26,7 +26,7 @@ #include <statx_cp.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) /* Get information about the file NAME in BUF. */ @@ -95,4 +95,4 @@ strong_alias (___lxstat64, __lxstat64); strong_alias (___lxstat64,__lxstat) #endif -#endif /* SHLIB_COMPAT */ +#endif /* LIB_COMPAT */ diff --git a/sysdeps/unix/sysv/linux/xstat.c b/sysdeps/unix/sysv/linux/xstat.c index 87556af616..ff95479612 100644 --- a/sysdeps/unix/sysv/linux/xstat.c +++ b/sysdeps/unix/sysv/linux/xstat.c @@ -20,13 +20,13 @@ #include <fcntl.h> #include <kernel_stat.h> #include <sysdep.h> -#include <shlib-compat.h> #if !XSTAT_IS_XSTAT64 # include <xstatconv.h> # include <xstatover.h> +# include <shlib-compat.h> -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file NAME in BUF. */ int @@ -61,6 +61,6 @@ __xstat (int vers, const char *name, struct stat *buf) } } -# endif /* SHLIB_COMPAT */ +# endif /* LIB_COMPAT */ #endif /* XSTAT_IS_XSTAT64 */ diff --git a/sysdeps/unix/sysv/linux/xstat64.c b/sysdeps/unix/sysv/linux/xstat64.c index ec912fd2b9..9b05d1528b 100644 --- a/sysdeps/unix/sysv/linux/xstat64.c +++ b/sysdeps/unix/sysv/linux/xstat64.c @@ -26,7 +26,7 @@ #include <statx_cp.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file NAME in BUF. */ @@ -92,4 +92,5 @@ compat_symbol (libc, __old__xstat64, __xstat64, GLIBC_2_1); strong_alias (___xstat64, __xstat64) #endif -#endif /* SHLIB_COMPAT */ + +#endif /* LIB_COMPAT */ -- 2.25.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build 2020-12-29 19:49 [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build Adhemerval Zanella @ 2021-01-20 16:52 ` Adhemerval Zanella 2021-01-20 18:03 ` Florian Weimer 2021-01-21 9:21 ` Andreas Schwab 1 sibling, 1 reply; 7+ messages in thread From: Adhemerval Zanella @ 2021-01-20 16:52 UTC (permalink / raw) To: libc-alpha, Andreas Schwab, Florian Weimer Florian, Do we want this for 2.33? If so it would be good if someone could take a look and check if I am missing something here. On 29/12/2020 16:49, Adhemerval Zanella wrote: > This reverts commit 20b39d59467b0c1d858e89ded8b0cebe55e22f60 for static > library. This avoids the need to rebuild the world for the case where > libstdc++ (and potentially other libraries) are linked to a old glibc. > > To avoid requering to provide xstat symbols for newer ABIs (such as > riscv32) a new LIB_COMPAT macro is added. It is similar to SHLIB_COMPAT > but also works for static case (thus evaluating similar to SHLIB_COMPAT > for both shared and static case). > > Checked with a check-abi on all affected ABIs. I also check if the > static library does contains the xstat symbols. > --- > include/shlib-compat.h | 23 ++++++++++++++--------- > sysdeps/mach/hurd/fxstat.c | 5 ----- > sysdeps/mach/hurd/fxstatat.c | 3 +-- > sysdeps/mach/hurd/fxstatat64.c | 3 +-- > sysdeps/mach/hurd/lxstat64.c | 3 +-- > sysdeps/mach/hurd/xstat.c | 2 +- > sysdeps/unix/sysv/linux/fxstat.c | 4 ++-- > sysdeps/unix/sysv/linux/fxstat64.c | 4 ++-- > sysdeps/unix/sysv/linux/fxstatat.c | 4 ++-- > sysdeps/unix/sysv/linux/fxstatat64.c | 4 ++-- > sysdeps/unix/sysv/linux/lxstat.c | 2 +- > sysdeps/unix/sysv/linux/lxstat64.c | 4 ++-- > sysdeps/unix/sysv/linux/xstat.c | 6 +++--- > sysdeps/unix/sysv/linux/xstat64.c | 5 +++-- > 14 files changed, 35 insertions(+), 37 deletions(-) > > diff --git a/include/shlib-compat.h b/include/shlib-compat.h > index b58281857a..d6dae3bea9 100644 > --- a/include/shlib-compat.h > +++ b/include/shlib-compat.h > @@ -21,8 +21,6 @@ > > # include <abi-versions.h> > > -#ifdef SHARED > - > /* The file abi-versions.h (generated by scripts/abi-versions.awk) defines > symbols like `ABI_libm_GLIBC_2_0' for each version set in the source > code for each library. For a version set that is subsumed by a later > @@ -34,17 +32,24 @@ > when the two version sets named are in fact two different ABIs we are > supporting. If these do not differ, then there is no need to compile in > extra code to support this version set where it has been superseded by a > - newer version. The compatibility code should be conditionalized with > - e.g. `#if SHLIB_COMPAT (libm, GLIBC_2_0, GLIBC_2_2)' for code introduced > - in the GLIBC_2.0 version and obsoleted in the GLIBC_2.2 version. */ > - > -# define SHLIB_COMPAT(lib, introduced, obsoleted) \ > - _SHLIB_COMPAT (lib, introduced, obsoleted) > -# define _SHLIB_COMPAT(lib, introduced, obsoleted) \ > + newer version. */ > +#define LIB_COMPAT(lib, introduced, obsoleted) \ > + _LIB_COMPAT (lib, introduced, obsoleted) > +#define _LIB_COMPAT(lib, introduced, obsoleted) \ > (IS_IN (lib) \ > && (!(ABI_##lib##_##obsoleted - 0) \ > || ((ABI_##lib##_##introduced - 0) < (ABI_##lib##_##obsoleted - 0)))) > > +#ifdef SHARED > + > +/* Similar to LIB_COMPAT, but evaluate to 0 for static build. The > + compatibility code should be conditionalized with e.g. > + `#if SHLIB_COMPAT (libm, GLIBC_2_0, GLIBC_2_2)' for code introduced > + in the GLIBC_2.0 version and obsoleted in the GLIBC_2.2 version. */ > + > +# define SHLIB_COMPAT(lib, introduced, obsoleted) \ > + _LIB_COMPAT (lib, introduced, obsoleted) > + > /* That header also defines symbols like `VERSION_libm_GLIBC_2_1' to > the version set name to use for e.g. symbols first introduced into > libm in the GLIBC_2.1 version. Definitions of symbols with explicit > diff --git a/sysdeps/mach/hurd/fxstat.c b/sysdeps/mach/hurd/fxstat.c > index 3201bb8cf1..e7e4b6e65e 100644 > --- a/sysdeps/mach/hurd/fxstat.c > +++ b/sysdeps/mach/hurd/fxstat.c > @@ -19,9 +19,6 @@ > #include <stddef.h> > #include <sys/stat.h> > #include <hurd.h> > -#include <shlib-compat.h> > - > -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > > /* Get information about the file descriptor FD in BUF. */ > int > @@ -33,5 +30,3 @@ __fxstat (int vers, int fd, struct stat *buf) > return __fstat (fd, buf); > } > weak_alias (__fxstat, _fxstat) > - > -#endif > diff --git a/sysdeps/mach/hurd/fxstatat.c b/sysdeps/mach/hurd/fxstatat.c > index b71ab9fad5..b7f6ca742a 100644 > --- a/sysdeps/mach/hurd/fxstatat.c > +++ b/sysdeps/mach/hurd/fxstatat.c > @@ -22,7 +22,7 @@ > #include <hurd.h> > #include <shlib-compat.h> > > -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) > +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) > > int > __fxstatat (int vers, int fd, const char *filename, struct stat *buf, int flag) > @@ -32,5 +32,4 @@ __fxstatat (int vers, int fd, const char *filename, struct stat *buf, int flag) > > return __fstatat (fd, filename, buf, flag); > } > - > #endif > diff --git a/sysdeps/mach/hurd/fxstatat64.c b/sysdeps/mach/hurd/fxstatat64.c > index f0c171f64c..8f401ddff2 100644 > --- a/sysdeps/mach/hurd/fxstatat64.c > +++ b/sysdeps/mach/hurd/fxstatat64.c > @@ -23,7 +23,7 @@ > #include <hurd.h> > #include <shlib-compat.h> > > -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) > +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) > > /* Get information about the file descriptor FD in BUF. */ > int > @@ -35,5 +35,4 @@ __fxstatat64 (int vers, int fd, const char *filename, struct stat64 *buf, > > return __fstatat64 (fd, filename, buf, flag); > } > - > #endif > diff --git a/sysdeps/mach/hurd/lxstat64.c b/sysdeps/mach/hurd/lxstat64.c > index 7059bceb11..4975044ac3 100644 > --- a/sysdeps/mach/hurd/lxstat64.c > +++ b/sysdeps/mach/hurd/lxstat64.c > @@ -22,7 +22,7 @@ > #include <hurd.h> > #include <shlib-compat.h> > > -#if SHLIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) > +#if LIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) > > /* Get information about the file descriptor FD in BUF. */ > int > @@ -33,5 +33,4 @@ __lxstat64 (int vers, const char *file, struct stat64 *buf) > > return __lstat64 (file, buf); > } > - > #endif > diff --git a/sysdeps/mach/hurd/xstat.c b/sysdeps/mach/hurd/xstat.c > index 8d56fe3c44..03b86ccb24 100644 > --- a/sysdeps/mach/hurd/xstat.c > +++ b/sysdeps/mach/hurd/xstat.c > @@ -20,7 +20,7 @@ > #include <hurd.h> > #include <shlib-compat.h> > > -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > > /* Get file information about FILE in BUF. */ > int > diff --git a/sysdeps/unix/sysv/linux/fxstat.c b/sysdeps/unix/sysv/linux/fxstat.c > index 528eedc272..0b89250b8b 100644 > --- a/sysdeps/unix/sysv/linux/fxstat.c > +++ b/sysdeps/unix/sysv/linux/fxstat.c > @@ -26,7 +26,7 @@ > # include <xstatover.h> > # include <shlib-compat.h> > > -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > > /* Get information about the file FD in BUF. */ > int > @@ -61,6 +61,6 @@ __fxstat (int vers, int fd, struct stat *buf) > } > } > > -# endif /* SHLIB_COMPAT */ > +# endif /* LIB_COMPAT */ > > #endif /* XSTAT_IS_XSTAT64 */ > diff --git a/sysdeps/unix/sysv/linux/fxstat64.c b/sysdeps/unix/sysv/linux/fxstat64.c > index 9471d0be45..177233d759 100644 > --- a/sysdeps/unix/sysv/linux/fxstat64.c > +++ b/sysdeps/unix/sysv/linux/fxstat64.c > @@ -26,7 +26,7 @@ > #include <statx_cp.h> > #include <shlib-compat.h> > > -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > > /* Get information about the file FD in BUF. */ > > @@ -78,4 +78,4 @@ strong_alias (___fxstat64, __fxstat64) > strong_alias (___fxstat64, __fxstat) > #endif > > -#endif /* SHLIB_COMPAT */ > +#endif /* LIB_COMPAT */ > diff --git a/sysdeps/unix/sysv/linux/fxstatat.c b/sysdeps/unix/sysv/linux/fxstatat.c > index bd78971dea..78856742ef 100644 > --- a/sysdeps/unix/sysv/linux/fxstatat.c > +++ b/sysdeps/unix/sysv/linux/fxstatat.c > @@ -26,7 +26,7 @@ > # include <xstatover.h> > # include <shlib-compat.h> > > -# if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) > +# if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) > > /* Get information about the file FD in BUF. */ > int > @@ -50,6 +50,6 @@ __fxstatat (int vers, int fd, const char *file, struct stat *st, int flag) > #endif > } > > -# endif /* SHLIB_COMPAT */ > +# endif /* LIB_COMPAT */ > > #endif /* XSTAT_IS_XSTAT64 */ > diff --git a/sysdeps/unix/sysv/linux/fxstatat64.c b/sysdeps/unix/sysv/linux/fxstatat64.c > index 091da83f1f..f23f877109 100644 > --- a/sysdeps/unix/sysv/linux/fxstatat64.c > +++ b/sysdeps/unix/sysv/linux/fxstatat64.c > @@ -26,7 +26,7 @@ > #include <statx_cp.h> > #include <shlib-compat.h> > > -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) > +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) > > /* Get information about the file FD in BUF. */ > > @@ -69,4 +69,4 @@ __fxstatat64 (int vers, int fd, const char *file, struct stat64 *st, int flag) > strong_alias (__fxstatat64, __fxstatat) > #endif > > -#endif /* SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) */ > +#endif /* LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) */ > diff --git a/sysdeps/unix/sysv/linux/lxstat.c b/sysdeps/unix/sysv/linux/lxstat.c > index c7d4507d7c..a11f58b453 100644 > --- a/sysdeps/unix/sysv/linux/lxstat.c > +++ b/sysdeps/unix/sysv/linux/lxstat.c > @@ -26,7 +26,7 @@ > # include <xstatconv.h> > # include <xstatover.h> > > -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > > /* Get information about the file NAME in BUF. */ > int > diff --git a/sysdeps/unix/sysv/linux/lxstat64.c b/sysdeps/unix/sysv/linux/lxstat64.c > index 74e11f3467..4d9d058d4b 100644 > --- a/sysdeps/unix/sysv/linux/lxstat64.c > +++ b/sysdeps/unix/sysv/linux/lxstat64.c > @@ -26,7 +26,7 @@ > #include <statx_cp.h> > #include <shlib-compat.h> > > -#if SHLIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) > +#if LIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) > > /* Get information about the file NAME in BUF. */ > > @@ -95,4 +95,4 @@ strong_alias (___lxstat64, __lxstat64); > strong_alias (___lxstat64,__lxstat) > #endif > > -#endif /* SHLIB_COMPAT */ > +#endif /* LIB_COMPAT */ > diff --git a/sysdeps/unix/sysv/linux/xstat.c b/sysdeps/unix/sysv/linux/xstat.c > index 87556af616..ff95479612 100644 > --- a/sysdeps/unix/sysv/linux/xstat.c > +++ b/sysdeps/unix/sysv/linux/xstat.c > @@ -20,13 +20,13 @@ > #include <fcntl.h> > #include <kernel_stat.h> > #include <sysdep.h> > -#include <shlib-compat.h> > > #if !XSTAT_IS_XSTAT64 > # include <xstatconv.h> > # include <xstatover.h> > +# include <shlib-compat.h> > > -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > > /* Get information about the file NAME in BUF. */ > int > @@ -61,6 +61,6 @@ __xstat (int vers, const char *name, struct stat *buf) > } > } > > -# endif /* SHLIB_COMPAT */ > +# endif /* LIB_COMPAT */ > > #endif /* XSTAT_IS_XSTAT64 */ > diff --git a/sysdeps/unix/sysv/linux/xstat64.c b/sysdeps/unix/sysv/linux/xstat64.c > index ec912fd2b9..9b05d1528b 100644 > --- a/sysdeps/unix/sysv/linux/xstat64.c > +++ b/sysdeps/unix/sysv/linux/xstat64.c > @@ -26,7 +26,7 @@ > #include <statx_cp.h> > #include <shlib-compat.h> > > -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > > /* Get information about the file NAME in BUF. */ > > @@ -92,4 +92,5 @@ compat_symbol (libc, __old__xstat64, __xstat64, GLIBC_2_1); > strong_alias (___xstat64, __xstat64) > #endif > > -#endif /* SHLIB_COMPAT */ > + > +#endif /* LIB_COMPAT */ > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build 2021-01-20 16:52 ` Adhemerval Zanella @ 2021-01-20 18:03 ` Florian Weimer 2021-01-20 18:28 ` Adhemerval Zanella 0 siblings, 1 reply; 7+ messages in thread From: Florian Weimer @ 2021-01-20 18:03 UTC (permalink / raw) To: Adhemerval Zanella via Libc-alpha; +Cc: Andreas Schwab, Adhemerval Zanella * Adhemerval Zanella via Libc-alpha: > Do we want this for 2.33? If so it would be good if someone could take > a look and check if I am missing something here. Sorry, I assumed that Andreas was going to review it. I think we need it for GCC compatibility, at least for a few releases, given that libstdc++.a references those static symbols. Thanks, Florian -- Red Hat GmbH, https://de.redhat.com/ , Registered seat: Grasbrunn, Commercial register: Amtsgericht Muenchen, HRB 153243, Managing Directors: Charles Cachera, Brian Klemm, Laurie Krebs, Michael O'Neill ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build 2021-01-20 18:03 ` Florian Weimer @ 2021-01-20 18:28 ` Adhemerval Zanella 0 siblings, 0 replies; 7+ messages in thread From: Adhemerval Zanella @ 2021-01-20 18:28 UTC (permalink / raw) To: Florian Weimer, Adhemerval Zanella via Libc-alpha; +Cc: Andreas Schwab On 20/01/2021 15:03, Florian Weimer wrote: > * Adhemerval Zanella via Libc-alpha: > >> Do we want this for 2.33? If so it would be good if someone could take >> a look and check if I am missing something here. > > Sorry, I assumed that Andreas was going to review it. > > I think we need it for GCC compatibility, at least for a few releases, > given that libstdc++.a references those static symbols. Right, so we will need someone to review it during this week, next week top. I will focus on reviewing the v5 static-pie ifunc. I would like to start the freeze next week to give us at least one week for testing. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build 2020-12-29 19:49 [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build Adhemerval Zanella 2021-01-20 16:52 ` Adhemerval Zanella @ 2021-01-21 9:21 ` Andreas Schwab 2021-01-21 16:41 ` Adhemerval Zanella 1 sibling, 1 reply; 7+ messages in thread From: Andreas Schwab @ 2021-01-21 9:21 UTC (permalink / raw) To: Adhemerval Zanella; +Cc: libc-alpha, Florian Weimer On Dez 29 2020, Adhemerval Zanella wrote: > diff --git a/sysdeps/mach/hurd/fxstat.c b/sysdeps/mach/hurd/fxstat.c > index 3201bb8cf1..e7e4b6e65e 100644 > --- a/sysdeps/mach/hurd/fxstat.c > +++ b/sysdeps/mach/hurd/fxstat.c > @@ -19,9 +19,6 @@ > #include <stddef.h> > #include <sys/stat.h> > #include <hurd.h> > -#include <shlib-compat.h> > - > -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) > > /* Get information about the file descriptor FD in BUF. */ > int > @@ -33,5 +30,3 @@ __fxstat (int vers, int fd, struct stat *buf) > return __fstat (fd, buf); > } > weak_alias (__fxstat, _fxstat) > - > -#endif Should this use LIB_COMPAT? Andreas. -- Andreas Schwab, schwab@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E780 A9DA AEC1 "And now for something completely different." ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build 2021-01-21 9:21 ` Andreas Schwab @ 2021-01-21 16:41 ` Adhemerval Zanella 2021-01-21 17:04 ` Andreas Schwab 0 siblings, 1 reply; 7+ messages in thread From: Adhemerval Zanella @ 2021-01-21 16:41 UTC (permalink / raw) To: Andreas Schwab; +Cc: libc-alpha, Florian Weimer On 21/01/2021 06:21, Andreas Schwab wrote: > On Dez 29 2020, Adhemerval Zanella wrote: > >> diff --git a/sysdeps/mach/hurd/fxstat.c b/sysdeps/mach/hurd/fxstat.c >> index 3201bb8cf1..e7e4b6e65e 100644 >> --- a/sysdeps/mach/hurd/fxstat.c >> +++ b/sysdeps/mach/hurd/fxstat.c >> @@ -19,9 +19,6 @@ >> #include <stddef.h> >> #include <sys/stat.h> >> #include <hurd.h> >> -#include <shlib-compat.h> >> - >> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) >> >> /* Get information about the file descriptor FD in BUF. */ >> int >> @@ -33,5 +30,3 @@ __fxstat (int vers, int fd, struct stat *buf) >> return __fstat (fd, buf); >> } >> weak_alias (__fxstat, _fxstat) >> - >> -#endif > > Should this use LIB_COMPAT? > > Andreas. > Indeed, the Hurd part was incomplete. Below the fixed patch: --- [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build This reverts commit 20b39d59467b0c1d858e89ded8b0cebe55e22f60 for static library. This avoids the need to rebuild the world for the case where libstdc++ (and potentially other libraries) are linked to a old glibc. To avoid requering to provide xstat symbols for newer ABIs (such as riscv32) a new LIB_COMPAT macro is added. It is similar to SHLIB_COMPAT but also works for static case (thus evaluating similar to SHLIB_COMPAT for both shared and static case). Checked with a check-abi on all affected ABIs. I also check if the static library does contains the xstat symbols. --- include/shlib-compat.h | 23 ++++++++++++++--------- sysdeps/mach/hurd/fxstat.c | 4 ++-- sysdeps/mach/hurd/fxstat64.c | 2 +- sysdeps/mach/hurd/fxstatat.c | 3 +-- sysdeps/mach/hurd/fxstatat64.c | 3 +-- sysdeps/mach/hurd/lxstat.c | 2 +- sysdeps/mach/hurd/lxstat64.c | 3 +-- sysdeps/mach/hurd/xstat.c | 2 +- sysdeps/mach/hurd/xstat64.c | 2 +- sysdeps/unix/sysv/linux/fxstat.c | 4 ++-- sysdeps/unix/sysv/linux/fxstat64.c | 4 ++-- sysdeps/unix/sysv/linux/fxstatat.c | 4 ++-- sysdeps/unix/sysv/linux/fxstatat64.c | 4 ++-- sysdeps/unix/sysv/linux/lxstat.c | 2 +- sysdeps/unix/sysv/linux/lxstat64.c | 4 ++-- sysdeps/unix/sysv/linux/xstat.c | 6 +++--- sysdeps/unix/sysv/linux/xstat64.c | 5 +++-- 17 files changed, 40 insertions(+), 37 deletions(-) diff --git a/include/shlib-compat.h b/include/shlib-compat.h index b58281857a..d6dae3bea9 100644 --- a/include/shlib-compat.h +++ b/include/shlib-compat.h @@ -21,8 +21,6 @@ # include <abi-versions.h> -#ifdef SHARED - /* The file abi-versions.h (generated by scripts/abi-versions.awk) defines symbols like `ABI_libm_GLIBC_2_0' for each version set in the source code for each library. For a version set that is subsumed by a later @@ -34,17 +32,24 @@ when the two version sets named are in fact two different ABIs we are supporting. If these do not differ, then there is no need to compile in extra code to support this version set where it has been superseded by a - newer version. The compatibility code should be conditionalized with - e.g. `#if SHLIB_COMPAT (libm, GLIBC_2_0, GLIBC_2_2)' for code introduced - in the GLIBC_2.0 version and obsoleted in the GLIBC_2.2 version. */ - -# define SHLIB_COMPAT(lib, introduced, obsoleted) \ - _SHLIB_COMPAT (lib, introduced, obsoleted) -# define _SHLIB_COMPAT(lib, introduced, obsoleted) \ + newer version. */ +#define LIB_COMPAT(lib, introduced, obsoleted) \ + _LIB_COMPAT (lib, introduced, obsoleted) +#define _LIB_COMPAT(lib, introduced, obsoleted) \ (IS_IN (lib) \ && (!(ABI_##lib##_##obsoleted - 0) \ || ((ABI_##lib##_##introduced - 0) < (ABI_##lib##_##obsoleted - 0)))) +#ifdef SHARED + +/* Similar to LIB_COMPAT, but evaluate to 0 for static build. The + compatibility code should be conditionalized with e.g. + `#if SHLIB_COMPAT (libm, GLIBC_2_0, GLIBC_2_2)' for code introduced + in the GLIBC_2.0 version and obsoleted in the GLIBC_2.2 version. */ + +# define SHLIB_COMPAT(lib, introduced, obsoleted) \ + _LIB_COMPAT (lib, introduced, obsoleted) + /* That header also defines symbols like `VERSION_libm_GLIBC_2_1' to the version set name to use for e.g. symbols first introduced into libm in the GLIBC_2.1 version. Definitions of symbols with explicit diff --git a/sysdeps/mach/hurd/fxstat.c b/sysdeps/mach/hurd/fxstat.c index 3201bb8cf1..d3bf41a0dd 100644 --- a/sysdeps/mach/hurd/fxstat.c +++ b/sysdeps/mach/hurd/fxstat.c @@ -21,7 +21,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file descriptor FD in BUF. */ int @@ -34,4 +34,4 @@ __fxstat (int vers, int fd, struct stat *buf) } weak_alias (__fxstat, _fxstat) -#endif +#endif /* LIB_COMPAT */ diff --git a/sysdeps/mach/hurd/fxstat64.c b/sysdeps/mach/hurd/fxstat64.c index 6b404b296c..9beb2950ce 100644 --- a/sysdeps/mach/hurd/fxstat64.c +++ b/sysdeps/mach/hurd/fxstat64.c @@ -21,7 +21,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) /* Get information about the file descriptor FD in BUF. */ int diff --git a/sysdeps/mach/hurd/fxstatat.c b/sysdeps/mach/hurd/fxstatat.c index b71ab9fad5..b7f6ca742a 100644 --- a/sysdeps/mach/hurd/fxstatat.c +++ b/sysdeps/mach/hurd/fxstatat.c @@ -22,7 +22,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) int __fxstatat (int vers, int fd, const char *filename, struct stat *buf, int flag) @@ -32,5 +32,4 @@ __fxstatat (int vers, int fd, const char *filename, struct stat *buf, int flag) return __fstatat (fd, filename, buf, flag); } - #endif diff --git a/sysdeps/mach/hurd/fxstatat64.c b/sysdeps/mach/hurd/fxstatat64.c index f0c171f64c..8f401ddff2 100644 --- a/sysdeps/mach/hurd/fxstatat64.c +++ b/sysdeps/mach/hurd/fxstatat64.c @@ -23,7 +23,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) /* Get information about the file descriptor FD in BUF. */ int @@ -35,5 +35,4 @@ __fxstatat64 (int vers, int fd, const char *filename, struct stat64 *buf, return __fstatat64 (fd, filename, buf, flag); } - #endif diff --git a/sysdeps/mach/hurd/lxstat.c b/sysdeps/mach/hurd/lxstat.c index ba40612f50..12e10f0106 100644 --- a/sysdeps/mach/hurd/lxstat.c +++ b/sysdeps/mach/hurd/lxstat.c @@ -21,7 +21,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) int __lxstat (int vers, const char *file, struct stat *buf) diff --git a/sysdeps/mach/hurd/lxstat64.c b/sysdeps/mach/hurd/lxstat64.c index 7059bceb11..4975044ac3 100644 --- a/sysdeps/mach/hurd/lxstat64.c +++ b/sysdeps/mach/hurd/lxstat64.c @@ -22,7 +22,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) /* Get information about the file descriptor FD in BUF. */ int @@ -33,5 +33,4 @@ __lxstat64 (int vers, const char *file, struct stat64 *buf) return __lstat64 (file, buf); } - #endif diff --git a/sysdeps/mach/hurd/xstat.c b/sysdeps/mach/hurd/xstat.c index 8d56fe3c44..03b86ccb24 100644 --- a/sysdeps/mach/hurd/xstat.c +++ b/sysdeps/mach/hurd/xstat.c @@ -20,7 +20,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get file information about FILE in BUF. */ int diff --git a/sysdeps/mach/hurd/xstat64.c b/sysdeps/mach/hurd/xstat64.c index 26bf5e2c79..e293c8adec 100644 --- a/sysdeps/mach/hurd/xstat64.c +++ b/sysdeps/mach/hurd/xstat64.c @@ -21,7 +21,7 @@ #include <hurd.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) /* Get information about the file descriptor FD in BUF. */ int diff --git a/sysdeps/unix/sysv/linux/fxstat.c b/sysdeps/unix/sysv/linux/fxstat.c index 528eedc272..0b89250b8b 100644 --- a/sysdeps/unix/sysv/linux/fxstat.c +++ b/sysdeps/unix/sysv/linux/fxstat.c @@ -26,7 +26,7 @@ # include <xstatover.h> # include <shlib-compat.h> -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file FD in BUF. */ int @@ -61,6 +61,6 @@ __fxstat (int vers, int fd, struct stat *buf) } } -# endif /* SHLIB_COMPAT */ +# endif /* LIB_COMPAT */ #endif /* XSTAT_IS_XSTAT64 */ diff --git a/sysdeps/unix/sysv/linux/fxstat64.c b/sysdeps/unix/sysv/linux/fxstat64.c index 9471d0be45..177233d759 100644 --- a/sysdeps/unix/sysv/linux/fxstat64.c +++ b/sysdeps/unix/sysv/linux/fxstat64.c @@ -26,7 +26,7 @@ #include <statx_cp.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file FD in BUF. */ @@ -78,4 +78,4 @@ strong_alias (___fxstat64, __fxstat64) strong_alias (___fxstat64, __fxstat) #endif -#endif /* SHLIB_COMPAT */ +#endif /* LIB_COMPAT */ diff --git a/sysdeps/unix/sysv/linux/fxstatat.c b/sysdeps/unix/sysv/linux/fxstatat.c index bd78971dea..78856742ef 100644 --- a/sysdeps/unix/sysv/linux/fxstatat.c +++ b/sysdeps/unix/sysv/linux/fxstatat.c @@ -26,7 +26,7 @@ # include <xstatover.h> # include <shlib-compat.h> -# if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) +# if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) /* Get information about the file FD in BUF. */ int @@ -50,6 +50,6 @@ __fxstatat (int vers, int fd, const char *file, struct stat *st, int flag) #endif } -# endif /* SHLIB_COMPAT */ +# endif /* LIB_COMPAT */ #endif /* XSTAT_IS_XSTAT64 */ diff --git a/sysdeps/unix/sysv/linux/fxstatat64.c b/sysdeps/unix/sysv/linux/fxstatat64.c index 091da83f1f..f23f877109 100644 --- a/sysdeps/unix/sysv/linux/fxstatat64.c +++ b/sysdeps/unix/sysv/linux/fxstatat64.c @@ -26,7 +26,7 @@ #include <statx_cp.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) /* Get information about the file FD in BUF. */ @@ -69,4 +69,4 @@ __fxstatat64 (int vers, int fd, const char *file, struct stat64 *st, int flag) strong_alias (__fxstatat64, __fxstatat) #endif -#endif /* SHLIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) */ +#endif /* LIB_COMPAT(libc, GLIBC_2_4, GLIBC_2_33) */ diff --git a/sysdeps/unix/sysv/linux/lxstat.c b/sysdeps/unix/sysv/linux/lxstat.c index c7d4507d7c..a11f58b453 100644 --- a/sysdeps/unix/sysv/linux/lxstat.c +++ b/sysdeps/unix/sysv/linux/lxstat.c @@ -26,7 +26,7 @@ # include <xstatconv.h> # include <xstatover.h> -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file NAME in BUF. */ int diff --git a/sysdeps/unix/sysv/linux/lxstat64.c b/sysdeps/unix/sysv/linux/lxstat64.c index 74e11f3467..4d9d058d4b 100644 --- a/sysdeps/unix/sysv/linux/lxstat64.c +++ b/sysdeps/unix/sysv/linux/lxstat64.c @@ -26,7 +26,7 @@ #include <statx_cp.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_1, GLIBC_2_33) /* Get information about the file NAME in BUF. */ @@ -95,4 +95,4 @@ strong_alias (___lxstat64, __lxstat64); strong_alias (___lxstat64,__lxstat) #endif -#endif /* SHLIB_COMPAT */ +#endif /* LIB_COMPAT */ diff --git a/sysdeps/unix/sysv/linux/xstat.c b/sysdeps/unix/sysv/linux/xstat.c index 87556af616..ff95479612 100644 --- a/sysdeps/unix/sysv/linux/xstat.c +++ b/sysdeps/unix/sysv/linux/xstat.c @@ -20,13 +20,13 @@ #include <fcntl.h> #include <kernel_stat.h> #include <sysdep.h> -#include <shlib-compat.h> #if !XSTAT_IS_XSTAT64 # include <xstatconv.h> # include <xstatover.h> +# include <shlib-compat.h> -# if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +# if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file NAME in BUF. */ int @@ -61,6 +61,6 @@ __xstat (int vers, const char *name, struct stat *buf) } } -# endif /* SHLIB_COMPAT */ +# endif /* LIB_COMPAT */ #endif /* XSTAT_IS_XSTAT64 */ diff --git a/sysdeps/unix/sysv/linux/xstat64.c b/sysdeps/unix/sysv/linux/xstat64.c index ec912fd2b9..9b05d1528b 100644 --- a/sysdeps/unix/sysv/linux/xstat64.c +++ b/sysdeps/unix/sysv/linux/xstat64.c @@ -26,7 +26,7 @@ #include <statx_cp.h> #include <shlib-compat.h> -#if SHLIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) +#if LIB_COMPAT(libc, GLIBC_2_0, GLIBC_2_33) /* Get information about the file NAME in BUF. */ @@ -92,4 +92,5 @@ compat_symbol (libc, __old__xstat64, __xstat64, GLIBC_2_1); strong_alias (___xstat64, __xstat64) #endif -#endif /* SHLIB_COMPAT */ + +#endif /* LIB_COMPAT */ -- 2.25.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build 2021-01-21 16:41 ` Adhemerval Zanella @ 2021-01-21 17:04 ` Andreas Schwab 0 siblings, 0 replies; 7+ messages in thread From: Andreas Schwab @ 2021-01-21 17:04 UTC (permalink / raw) To: Adhemerval Zanella; +Cc: libc-alpha, Florian Weimer On Jan 21 2021, Adhemerval Zanella wrote: > [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for > static build > > This reverts commit 20b39d59467b0c1d858e89ded8b0cebe55e22f60 for static > library. This avoids the need to rebuild the world for the case where > libstdc++ (and potentially other libraries) are linked to a old glibc. > > To avoid requering to provide xstat symbols for newer ABIs (such as > riscv32) a new LIB_COMPAT macro is added. It is similar to SHLIB_COMPAT > but also works for static case (thus evaluating similar to SHLIB_COMPAT > for both shared and static case). > > Checked with a check-abi on all affected ABIs. I also check if the > static library does contains the xstat symbols. Ok. Andreas. -- Andreas Schwab, schwab@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E780 A9DA AEC1 "And now for something completely different." ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-01-21 17:04 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-12-29 19:49 [PATCH] Revert "linux: Move {f}xstat{at} to compat symbols" for static build Adhemerval Zanella 2021-01-20 16:52 ` Adhemerval Zanella 2021-01-20 18:03 ` Florian Weimer 2021-01-20 18:28 ` Adhemerval Zanella 2021-01-21 9:21 ` Andreas Schwab 2021-01-21 16:41 ` Adhemerval Zanella 2021-01-21 17:04 ` Andreas Schwab
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 read-only IMAP folder(s) and NNTP newsgroup(s).