| Message ID | 20240905134942.2463850-1-adhemerval.zanella@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | [v2] linux: Use GLRO(dl_vdso_gettimeofday) on gettimeofday | expand |
Ping. On 05/09/24 10:49, Adhemerval Zanella wrote: > The BZ#24967 fix (1bdda52fe92fd01b424c) missed the gettimeofday for > architectures that define USE_IFUNC_GETTIMEOFDAY. Although it is not > an issue, since there is no pointer mangling, there is also no need > to call dl_vdso_vsym since the vDSO setup was already done by the > loader. > > Checked on x86_64-linux-gnu and i686-linux-gnu. > -- > Change from v1: > * Fixed build on some architectures due a missing INIT_ARCH. > --- > sysdeps/unix/sysv/linux/gettimeofday.c | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/sysdeps/unix/sysv/linux/gettimeofday.c b/sysdeps/unix/sysv/linux/gettimeofday.c > index 7ab147c614..afee658e7b 100644 > --- a/sysdeps/unix/sysv/linux/gettimeofday.c > +++ b/sysdeps/unix/sysv/linux/gettimeofday.c > @@ -36,12 +36,11 @@ __gettimeofday_syscall (struct timeval *restrict tv, void *restrict tz) > return INLINE_SYSCALL_CALL (gettimeofday, tv, tz); > } > > -# undef INIT_ARCH > -# define INIT_ARCH() \ > - void *vdso_gettimeofday = dl_vdso_vsym (HAVE_GETTIMEOFDAY_VSYSCALL) > +# define INIT_ARCH() > libc_ifunc (__gettimeofday, > - vdso_gettimeofday ? VDSO_IFUNC_RET (vdso_gettimeofday) > - : (void *) __gettimeofday_syscall) > + GLRO(dl_vdso_gettimeofday) != NULL > + ? VDSO_IFUNC_RET (GLRO(dl_vdso_gettimeofday)) > + : (void *) __gettimeofday_syscall) > > # else > int
On Sep 05 2024, Adhemerval Zanella wrote: > The BZ#24967 fix (1bdda52fe92fd01b424c) missed the gettimeofday for > architectures that define USE_IFUNC_GETTIMEOFDAY. Although it is not > an issue, since there is no pointer mangling, there is also no need > to call dl_vdso_vsym since the vDSO setup was already done by the > loader. Ok.
diff --git a/sysdeps/unix/sysv/linux/gettimeofday.c b/sysdeps/unix/sysv/linux/gettimeofday.c index 7ab147c614..afee658e7b 100644 --- a/sysdeps/unix/sysv/linux/gettimeofday.c +++ b/sysdeps/unix/sysv/linux/gettimeofday.c @@ -36,12 +36,11 @@ __gettimeofday_syscall (struct timeval *restrict tv, void *restrict tz) return INLINE_SYSCALL_CALL (gettimeofday, tv, tz); } -# undef INIT_ARCH -# define INIT_ARCH() \ - void *vdso_gettimeofday = dl_vdso_vsym (HAVE_GETTIMEOFDAY_VSYSCALL) +# define INIT_ARCH() libc_ifunc (__gettimeofday, - vdso_gettimeofday ? VDSO_IFUNC_RET (vdso_gettimeofday) - : (void *) __gettimeofday_syscall) + GLRO(dl_vdso_gettimeofday) != NULL + ? VDSO_IFUNC_RET (GLRO(dl_vdso_gettimeofday)) + : (void *) __gettimeofday_syscall) # else int