diff mbox series

selftests/vDSO: fix ABI selftest on riscv

Message ID 20210204145042.7345-1-tklauser@distanz.ch
State Accepted
Commit f405ac83fa252dd0e346f2715b66e7d2adba9027
Headers show
Series selftests/vDSO: fix ABI selftest on riscv | expand

Commit Message

Tobias Klauser Feb. 4, 2021, 2:50 p.m. UTC
Only older versions of the RISC-V GCC toolchain define __riscv__. Check
for __riscv as well, which is used by newer GCC toolchains. Also set
VDSO_32BIT based on __riscv_xlen.

Before (on riscv64):

$ ./vdso_test_abi
[vDSO kselftest] VDSO_VERSION: LINUX_4
Could not find __vdso_gettimeofday
Could not find __vdso_clock_gettime
Could not find __vdso_clock_getres
clock_id: CLOCK_REALTIME [PASS]
Could not find __vdso_clock_gettime
Could not find __vdso_clock_getres
clock_id: CLOCK_BOOTTIME [PASS]
Could not find __vdso_clock_gettime
Could not find __vdso_clock_getres
clock_id: CLOCK_TAI [PASS]
Could not find __vdso_clock_gettime
Could not find __vdso_clock_getres
clock_id: CLOCK_REALTIME_COARSE [PASS]
Could not find __vdso_clock_gettime
Could not find __vdso_clock_getres
clock_id: CLOCK_MONOTONIC [PASS]
Could not find __vdso_clock_gettime
Could not find __vdso_clock_getres
clock_id: CLOCK_MONOTONIC_RAW [PASS]
Could not find __vdso_clock_gettime
Could not find __vdso_clock_getres
clock_id: CLOCK_MONOTONIC_COARSE [PASS]
Could not find __vdso_time

After (on riscv32):

$ ./vdso_test_abi
[vDSO kselftest] VDSO_VERSION: LINUX_4.15
The time is 1612449376.015086
The time is 1612449376.18340784
The resolution is 0 1
clock_id: CLOCK_REALTIME [PASS]
The time is 774.842586182
The resolution is 0 1
clock_id: CLOCK_BOOTTIME [PASS]
The time is 1612449376.22536565
The resolution is 0 1
clock_id: CLOCK_TAI [PASS]
The time is 1612449376.20885172
The resolution is 0 4000000
clock_id: CLOCK_REALTIME_COARSE [PASS]
The time is 774.845491269
The resolution is 0 1
clock_id: CLOCK_MONOTONIC [PASS]
The time is 774.849534200
The resolution is 0 1
clock_id: CLOCK_MONOTONIC_RAW [PASS]
The time is 774.842139684
The resolution is 0 4000000
clock_id: CLOCK_MONOTONIC_COARSE [PASS]
Could not find __vdso_time

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
---
 tools/testing/selftests/vDSO/vdso_config.h | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Tobias Klauser Feb. 5, 2021, 7:57 a.m. UTC | #1
On 2021-02-05 at 08:06:37 +0100, Palmer Dabbelt <palmer@dabbelt.com> wrote:
> On Thu, 04 Feb 2021 06:50:42 PST (-0800), tklauser@distanz.ch wrote:

[...]

> Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com>
> Acked-by: Palmer Dabbelt <palmerdabbelt@google.com>

Thank you!

> Not sure if you want this through the RISC-V tree, so I'm leaving it out for
> now and assuming it'll go through a kselftest tree.

Either way is fine for me.
Shuah Khan Feb. 8, 2021, 11:37 p.m. UTC | #2
On 2/5/21 12:57 AM, Tobias Klauser wrote:
> On 2021-02-05 at 08:06:37 +0100, Palmer Dabbelt <palmer@dabbelt.com> wrote:

>> On Thu, 04 Feb 2021 06:50:42 PST (-0800), tklauser@distanz.ch wrote:

> 

> [...]

> 

>> Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com>

>> Acked-by: Palmer Dabbelt <palmerdabbelt@google.com>

> 

> Thank you!

> 

>> Not sure if you want this through the RISC-V tree, so I'm leaving it out for

>> now and assuming it'll go through a kselftest tree.

> 

> Either way is fine for me.

> 


Thank you. Applied to linux-kselftest next for 5.12-rc1

thanks,
-- Shuah
Palmer Dabbelt Feb. 9, 2021, 1:55 a.m. UTC | #3
On Mon, 08 Feb 2021 15:37:24 PST (-0800), skhan@linuxfoundation.org wrote:
> On 2/5/21 12:57 AM, Tobias Klauser wrote:

>> On 2021-02-05 at 08:06:37 +0100, Palmer Dabbelt <palmer@dabbelt.com> wrote:

>>> On Thu, 04 Feb 2021 06:50:42 PST (-0800), tklauser@distanz.ch wrote:

>>

>> [...]

>>

>>> Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com>

>>> Acked-by: Palmer Dabbelt <palmerdabbelt@google.com>

>>

>> Thank you!

>>

>>> Not sure if you want this through the RISC-V tree, so I'm leaving it out for

>>> now and assuming it'll go through a kselftest tree.

>>

>> Either way is fine for me.

>>

>

> Thank you. Applied to linux-kselftest next for 5.12-rc1


Thanks!
Tobias Klauser Feb. 10, 2021, 8:06 a.m. UTC | #4
On 2021-02-09 at 00:37:24 +0100, Shuah Khan <skhan@linuxfoundation.org> wrote:
> On 2/5/21 12:57 AM, Tobias Klauser wrote:

> > On 2021-02-05 at 08:06:37 +0100, Palmer Dabbelt <palmer@dabbelt.com> wrote:

> > > On Thu, 04 Feb 2021 06:50:42 PST (-0800), tklauser@distanz.ch wrote:

> > 

> > [...]

> > 

> > > Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com>

> > > Acked-by: Palmer Dabbelt <palmerdabbelt@google.com>

> > 

> > Thank you!

> > 

> > > Not sure if you want this through the RISC-V tree, so I'm leaving it out for

> > > now and assuming it'll go through a kselftest tree.

> > 

> > Either way is fine for me.

> > 

> 

> Thank you. Applied to linux-kselftest next for 5.12-rc1


Thank you Shuah!
diff mbox series

Patch

diff --git a/tools/testing/selftests/vDSO/vdso_config.h b/tools/testing/selftests/vDSO/vdso_config.h
index 6a6fe8d4ff55..6188b16827d1 100644
--- a/tools/testing/selftests/vDSO/vdso_config.h
+++ b/tools/testing/selftests/vDSO/vdso_config.h
@@ -47,10 +47,12 @@ 
 #elif defined(__x86_64__)
 #define VDSO_VERSION		0
 #define VDSO_NAMES		1
-#elif defined(__riscv__)
+#elif defined(__riscv__) || defined(__riscv)
 #define VDSO_VERSION		5
 #define VDSO_NAMES		1
+#if __riscv_xlen == 32
 #define VDSO_32BIT		1
+#endif
 #else /* nds32 */
 #define VDSO_VERSION		4
 #define VDSO_NAMES		1