On 11/25/22 02:42, Sebastian Andrzej Siewior wrote:
The vsyscall for getcpu has been wired up on 32bit so it can be warned now if missing. Cc: Shuah Khan <shuah@xxxxxxxxxx> Cc: linux-kselftest@xxxxxxxxxxxxxxx Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> --- tools/testing/selftests/x86/test_vsyscall.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/tools/testing/selftests/x86/test_vsyscall.c b/tools/testing/selftests/x86/test_vsyscall.c index 5b45e6986aeab..47cab972807c4 100644 --- a/tools/testing/selftests/x86/test_vsyscall.c +++ b/tools/testing/selftests/x86/test_vsyscall.c @@ -92,11 +92,8 @@ static void init_vdso(void) printf("[WARN]\tfailed to find time in vDSO\n");vdso_getcpu = (getcpu_t)dlsym(vdso, "__vdso_getcpu");- if (!vdso_getcpu) { - /* getcpu() was never wired up in the 32-bit vDSO. */ - printf("[%s]\tfailed to find getcpu in vDSO\n", - sizeof(long) == 8 ? "WARN" : "NOTE"); - } + if (!vdso_getcpu) + printf("[WARN]\tfailed to find getcpu in vDSO\n"); }static int init_vsys(void)
Assuming this is going through x86 tree due to dependencies, here is my ack Acked-by: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> Let me know if you would like to me pick this up. thanks, -- Shuah
