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@kernel.org Cc: linux-kselftest@vger.kernel.org Signed-off-by: Sebastian Andrzej Siewior bigeasy@linutronix.de

---

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@linuxfoundation.org

Let me know if you would like to me pick this up.

thanks, -- Shuah