diff mbox series

[v2,03/18] RISC-V: Add support for 32-bit vDSO calls

Message ID	02bea72edffeb116ab4d139d1e14b89d1f58c7ce.1591201405.git.alistair.francis@wdc.com
State	Committed
Headers	DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 8AD0B388B018 IronPort-SDR: rCAa3RVuQfOTIo7OGtE82EKupiwGElCDt+JagxrM87KSflBJM1dpWQYannzAoqyhU8gyPKceNZ OtidvGVpWmsWlshGnrMLDHubCRYFeC67gdFOx5ICN/I7Z+Z4o3ypRlro7KXDS3O4jeWrSSGMOf fRabKYkd/FzJqUPFYpsb1te3PHxJfn0eB16tJgmjoNopVTQ3mhuXHixD88VLYN+K+2TAJQvVRU C2Xa00UPOwr5PXohUVnDbuv9nwsxtYKkjKIE0CDT1rWDFZrgWgIRtN3JA7qbA3wDoFFHTL31og 8vg= IronPort-SDR: wVgUi++v3VmgmH/68GSGAduOkPqvrjMjijHVqWjWyybuh8UuwRx0j3sdQbtF0lhn8Tjpg9l8AM qJTnPLcrgg/4ssAHg3hJGTPOOOi+GMVJQ= IronPort-SDR: 3MRL/lfnnYXX92TZ61Oq+Eqru41FCD1Xujs2txckcgh7bk1/SzvbIUc3Intmj0TmD5Vddu+E5z ClnivtZzLrHQ== WDCIronportException: Internal To: libc-alpha@sourceware.org Subject: [PATCH v2 03/18] RISC-V: Add support for 32-bit vDSO calls Date: Wed, 3 Jun 2020 09:25:31 -0700 Message-Id: <02bea72edffeb116ab4d139d1e14b89d1f58c7ce.1591201405.git.alistair.francis@wdc.com> In-Reply-To: <cover.1591201405.git.alistair.francis@wdc.com> References: <cover.1591201405.git.alistair.francis@wdc.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list From: Alistair Francis via Libc-alpha <libc-alpha@sourceware.org> Reply-To: Alistair Francis <alistair.francis@wdc.com> Cc: alistair.francis@wdc.com Errors-To: libc-alpha-bounces@sourceware.org Sender: "Libc-alpha" <libc-alpha-bounces@sourceware.org>
Series	glibc port for 32-bit RISC-V (RV32) \| [v2,00/18] glibc port for 32-bit RISC-V (RV32) [v2,01/18] RISC-V: Use 64-bit time_t and off_t for RV32 and RV64 [v2,02/18] RISC-V: Define __NR_* as __NR_*_time64/64 for 32-bit [v2,03/18] RISC-V: Add support for 32-bit vDSO calls [v2,04/18] RISC-V: Support dynamic loader for the 32-bit [v2,05/18] RISC-V: Add path of library directories for the 32-bit [v2,06/18] RISC-V: Add arch-syscall.h for RV32 [v2,07/18] RISC-V: nptl: update default pthread-offsets.h [v2,08/18] riscv32: Add an architecture ipctypes.h [v2,09/18] RISC-V: The ABI implementation for 32-bit [v2,10/18] RISC-V: Hard float support for 32-bit [v2,11/18] RISC-V: Add ABI lists [v2,12/18] RISC-V: Add the RV32 libm-test-ulps [v2,13/18] RISC-V: Fix llrint and llround missing exceptions on RV32 [v2,14/18] RISC-V: Build Infastructure for 32-bit [v2,15/18] riscv32: Specify the arch_minimum_kernel as 5.4 [v2,16/18] RISC-V: Add rv32 path to RTLDLIST in ldd [v2,17/18] Documentation for the RISC-V 32-bit port [v2,18/18] Add RISC-V 32-bit target to build-many-glibcs.py

Commit Message

Alistair Francis June 3, 2020, 4:25 p.m. UTC

  ---
 sysdeps/unix/sysv/linux/riscv/sysdep.h | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

Comments

Maciej W. Rozycki July 8, 2020, 1:01 a.m. UTC | #1

On Wed, 3 Jun 2020, Alistair Francis via Libc-alpha wrote:

> diff --git a/sysdeps/unix/sysv/linux/riscv/sysdep.h b/sysdeps/unix/sysv/linux/riscv/sysdep.h
> index aa61e8b04d..9f6ed2001b 100644
> --- a/sysdeps/unix/sysv/linux/riscv/sysdep.h
> +++ b/sysdeps/unix/sysv/linux/riscv/sysdep.h
> @@ -182,14 +182,21 @@
>  
>  #ifndef __ASSEMBLER__
>  
> -# define VDSO_NAME  "LINUX_4.15"
> -# define VDSO_HASH  182943605
> +# if __riscv_xlen == 32

 Same note on __riscv_xlen vs __WORDSIZE as with 02/18.

 Also given it is a two-way conditional we can choose which leg is first 
and I'd make that the RV64 variant, as the first to be there and more 
developed, as evident here.

>  /* List of system calls which are supported as vsyscalls.  */
> -# define HAVE_CLOCK_GETRES64_VSYSCALL	"__vdso_clock_getres"
> -# define HAVE_CLOCK_GETTIME64_VSYSCALL	"__vdso_clock_gettime"
> -# define HAVE_GETTIMEOFDAY_VSYSCALL	"__vdso_gettimeofday"
> -# define HAVE_GETCPU_VSYSCALL		"__vdso_getcpu"
> +# else
> +#  define VDSO_NAME  "LINUX_4.15"
> +#  define VDSO_HASH  182943605
> +
> +/* List of system calls which are supported as vsyscalls.  */
> +#  define HAVE_CLOCK_GETRES64_VSYSCALL   "__vdso_clock_getres"
> +#  define HAVE_CLOCK_GETTIME64_VSYSCALL  "__vdso_clock_gettime"
> +#  define HAVE_GETTIMEOFDAY_VSYSCALL     "__vdso_gettimeofday"
> +#  define HAVE_GETCPU_VSYSCALL           "__vdso_getcpu"
> +# endif

 Also "__vdso_getcpu" does get exported unlike the time calls with RV32 
AFAICT:

# Symbols present in the vdso
vdso-syms  = rt_sigreturn
ifdef CONFIG_64BIT
vdso-syms += vgettimeofday
endif
vdso-syms += getcpu
vdso-syms += flush_icache

so it should be included (and placed outside the conditional, unlike with 
an earlier version of this change).  Then the change description (and 
preferably a comment as well) ought to explain why the remaining ones have 
been left out for RV32.

  Maciej

Alistair Francis July 8, 2020, 6:17 p.m. UTC | #2

On Tue, Jul 7, 2020 at 6:02 PM Maciej W. Rozycki via Libc-alpha
<libc-alpha@sourceware.org> wrote:
>
> On Wed, 3 Jun 2020, Alistair Francis via Libc-alpha wrote:
>
> > diff --git a/sysdeps/unix/sysv/linux/riscv/sysdep.h b/sysdeps/unix/sysv/linux/riscv/sysdep.h
> > index aa61e8b04d..9f6ed2001b 100644
> > --- a/sysdeps/unix/sysv/linux/riscv/sysdep.h
> > +++ b/sysdeps/unix/sysv/linux/riscv/sysdep.h
> > @@ -182,14 +182,21 @@
> >
> >  #ifndef __ASSEMBLER__
> >
> > -# define VDSO_NAME  "LINUX_4.15"
> > -# define VDSO_HASH  182943605
> > +# if __riscv_xlen == 32
>
>  Same note on __riscv_xlen vs __WORDSIZE as with 02/18.

Fixed

>
>  Also given it is a two-way conditional we can choose which leg is first
> and I'd make that the RV64 variant, as the first to be there and more
> developed, as evident here.

Fixed.

>
> >  /* List of system calls which are supported as vsyscalls.  */
> > -# define HAVE_CLOCK_GETRES64_VSYSCALL        "__vdso_clock_getres"
> > -# define HAVE_CLOCK_GETTIME64_VSYSCALL       "__vdso_clock_gettime"
> > -# define HAVE_GETTIMEOFDAY_VSYSCALL  "__vdso_gettimeofday"
> > -# define HAVE_GETCPU_VSYSCALL                "__vdso_getcpu"
> > +# else
> > +#  define VDSO_NAME  "LINUX_4.15"
> > +#  define VDSO_HASH  182943605
> > +
> > +/* List of system calls which are supported as vsyscalls.  */
> > +#  define HAVE_CLOCK_GETRES64_VSYSCALL   "__vdso_clock_getres"
> > +#  define HAVE_CLOCK_GETTIME64_VSYSCALL  "__vdso_clock_gettime"
> > +#  define HAVE_GETTIMEOFDAY_VSYSCALL     "__vdso_gettimeofday"
> > +#  define HAVE_GETCPU_VSYSCALL           "__vdso_getcpu"
> > +# endif
>
>  Also "__vdso_getcpu" does get exported unlike the time calls with RV32
> AFAICT:
>
> # Symbols present in the vdso
> vdso-syms  = rt_sigreturn
> ifdef CONFIG_64BIT
> vdso-syms += vgettimeofday
> endif
> vdso-syms += getcpu
> vdso-syms += flush_icache
>
> so it should be included (and placed outside the conditional, unlike with
> an earlier version of this change).  Then the change description (and
> preferably a comment as well) ought to explain why the remaining ones have
> been left out for RV32.

Ok, I have fixed this as well.

Alistair

>
>   Maciej

diff mbox series

Patch

diff --git a/sysdeps/unix/sysv/linux/riscv/sysdep.h b/sysdeps/unix/sysv/linux/riscv/sysdep.h
index aa61e8b04d..9f6ed2001b 100644
--- a/sysdeps/unix/sysv/linux/riscv/sysdep.h
+++ b/sysdeps/unix/sysv/linux/riscv/sysdep.h
@@ -182,14 +182,21 @@ 
 
 #ifndef __ASSEMBLER__
 
-# define VDSO_NAME  "LINUX_4.15"
-# define VDSO_HASH  182943605
+# if __riscv_xlen == 32
+#  define VDSO_NAME  "LINUX_5.4"
+#  define VDSO_HASH  61765876
 
 /* List of system calls which are supported as vsyscalls.  */
-# define HAVE_CLOCK_GETRES64_VSYSCALL	"__vdso_clock_getres"
-# define HAVE_CLOCK_GETTIME64_VSYSCALL	"__vdso_clock_gettime"
-# define HAVE_GETTIMEOFDAY_VSYSCALL	"__vdso_gettimeofday"
-# define HAVE_GETCPU_VSYSCALL		"__vdso_getcpu"
+# else
+#  define VDSO_NAME  "LINUX_4.15"
+#  define VDSO_HASH  182943605
+
+/* List of system calls which are supported as vsyscalls.  */
+#  define HAVE_CLOCK_GETRES64_VSYSCALL   "__vdso_clock_getres"
+#  define HAVE_CLOCK_GETTIME64_VSYSCALL  "__vdso_clock_gettime"
+#  define HAVE_GETTIMEOFDAY_VSYSCALL     "__vdso_gettimeofday"
+#  define HAVE_GETCPU_VSYSCALL           "__vdso_getcpu"
+# endif
 
 # define INTERNAL_SYSCALL(name, nr, args...) \
 	internal_syscall##nr (SYS_ify (name), args)