| Message ID | 20220128130701.17511-1-vincenzo.frascino@arm.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v3] kselftest: Fix vdso_test_abi return status | expand |
On Fri, Jan 28, 2022 at 01:07:01PM +0000, Vincenzo Frascino wrote: > vdso_test_abi contains a batch of tests that verify the validity of the > vDSO ABI. > > When a vDSO symbol is not found the relevant test is skipped reporting > KSFT_SKIP. All the tests return values are then added in a single > variable which is checked to verify failures. This approach can have > side effects which result in reporting the wrong kselftest exit status. > > Fix vdso_test_abi verifying the return code of each test separately. > > Cc: Shuah Khan <shuah@kernel.org> > Cc: Andy Lutomirski <luto@kernel.org> > Cc: Thomas Gleixner <tglx@linutronix.de> > Reported-by: Cristian Marussi <cristian.marussi@arm.com> > Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> > --- Hi, LGTM. Tested-by: Cristian Marussi <cristian.marussi@arm.com> Thanks, Cristian > tools/testing/selftests/vDSO/vdso_test_abi.c | 135 +++++++++---------- > 1 file changed, 62 insertions(+), 73 deletions(-) > > diff --git a/tools/testing/selftests/vDSO/vdso_test_abi.c b/tools/testing/selftests/vDSO/vdso_test_abi.c > index 3d603f1394af..d670d5b2a626 100644 > --- a/tools/testing/selftests/vDSO/vdso_test_abi.c > +++ b/tools/testing/selftests/vDSO/vdso_test_abi.c > @@ -33,110 +33,114 @@ typedef long (*vdso_clock_gettime_t)(clockid_t clk_id, struct timespec *ts); > typedef long (*vdso_clock_getres_t)(clockid_t clk_id, struct timespec *ts); > typedef time_t (*vdso_time_t)(time_t *t); > > -static int vdso_test_gettimeofday(void) > +#define VDSO_TEST_PASS_MSG() "\n%s(): PASS\n", __func__ > +#define VDSO_TEST_FAIL_MSG(x) "\n%s(): %s FAIL\n", __func__, x > +#define VDSO_TEST_SKIP_MSG(x) "\n%s(): SKIP: Could not find %s\n", __func__, x > + > +static void vdso_test_gettimeofday(void) > { > /* Find gettimeofday. */ > vdso_gettimeofday_t vdso_gettimeofday = > (vdso_gettimeofday_t)vdso_sym(version, name[0]); > > if (!vdso_gettimeofday) { > - printf("Could not find %s\n", name[0]); > - return KSFT_SKIP; > + ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[0])); > + return; > } > > struct timeval tv; > long ret = vdso_gettimeofday(&tv, 0); > > if (ret == 0) { > - printf("The time is %lld.%06lld\n", > - (long long)tv.tv_sec, (long long)tv.tv_usec); > + ksft_print_msg("The time is %lld.%06lld\n", > + (long long)tv.tv_sec, (long long)tv.tv_usec); > + ksft_test_result_pass(VDSO_TEST_PASS_MSG()); > } else { > - printf("%s failed\n", name[0]); > - return KSFT_FAIL; > + ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[0])); > } > - > - return KSFT_PASS; > } > > -static int vdso_test_clock_gettime(clockid_t clk_id) > +static void vdso_test_clock_gettime(clockid_t clk_id) > { > /* Find clock_gettime. */ > vdso_clock_gettime_t vdso_clock_gettime = > (vdso_clock_gettime_t)vdso_sym(version, name[1]); > > if (!vdso_clock_gettime) { > - printf("Could not find %s\n", name[1]); > - return KSFT_SKIP; > + ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[1])); > + return; > } > > struct timespec ts; > long ret = vdso_clock_gettime(clk_id, &ts); > > if (ret == 0) { > - printf("The time is %lld.%06lld\n", > - (long long)ts.tv_sec, (long long)ts.tv_nsec); > + ksft_print_msg("The time is %lld.%06lld\n", > + (long long)ts.tv_sec, (long long)ts.tv_nsec); > + ksft_test_result_pass(VDSO_TEST_PASS_MSG());; > } else { > - printf("%s failed\n", name[1]); > - return KSFT_FAIL; > + ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[1])); > } > - > - return KSFT_PASS; > } > > -static int vdso_test_time(void) > +static void vdso_test_time(void) > { > /* Find time. */ > vdso_time_t vdso_time = > (vdso_time_t)vdso_sym(version, name[2]); > > if (!vdso_time) { > - printf("Could not find %s\n", name[2]); > - return KSFT_SKIP; > + ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[2])); > + return; > } > > long ret = vdso_time(NULL); > > if (ret > 0) { > - printf("The time in hours since January 1, 1970 is %lld\n", > + ksft_print_msg("The time in hours since January 1, 1970 is %lld\n", > (long long)(ret / 3600)); > + ksft_test_result_pass(VDSO_TEST_PASS_MSG()); > } else { > - printf("%s failed\n", name[2]); > - return KSFT_FAIL; > + ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[2])); > } > - > - return KSFT_PASS; > } > > -static int vdso_test_clock_getres(clockid_t clk_id) > +static void vdso_test_clock_getres(clockid_t clk_id) > { > + int clock_getres_fail = 0; > + > /* Find clock_getres. */ > vdso_clock_getres_t vdso_clock_getres = > (vdso_clock_getres_t)vdso_sym(version, name[3]); > > if (!vdso_clock_getres) { > - printf("Could not find %s\n", name[3]); > - return KSFT_SKIP; > + ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[3])); > + return; > } > > struct timespec ts, sys_ts; > long ret = vdso_clock_getres(clk_id, &ts); > > if (ret == 0) { > - printf("The resolution is %lld %lld\n", > - (long long)ts.tv_sec, (long long)ts.tv_nsec); > + ksft_print_msg("The vdso resolution is %lld %lld\n", > + (long long)ts.tv_sec, (long long)ts.tv_nsec); > } else { > - printf("%s failed\n", name[3]); > - return KSFT_FAIL; > + clock_getres_fail++; > } > > ret = syscall(SYS_clock_getres, clk_id, &sys_ts); > > - if ((sys_ts.tv_sec != ts.tv_sec) || (sys_ts.tv_nsec != ts.tv_nsec)) { > - printf("%s failed\n", name[3]); > - return KSFT_FAIL; > - } > + ksft_print_msg("The syscall resolution is %lld %lld\n", > + (long long)sys_ts.tv_sec, (long long)sys_ts.tv_nsec); > > - return KSFT_PASS; > + if ((sys_ts.tv_sec != ts.tv_sec) || (sys_ts.tv_nsec != ts.tv_nsec)) > + clock_getres_fail++; > + > + if (clock_getres_fail > 0) { > + ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[3])); > + } else { > + ksft_test_result_pass(VDSO_TEST_PASS_MSG()); > + } > } > > const char *vdso_clock_name[12] = { > @@ -158,36 +162,23 @@ const char *vdso_clock_name[12] = { > * This function calls vdso_test_clock_gettime and vdso_test_clock_getres > * with different values for clock_id. > */ > -static inline int vdso_test_clock(clockid_t clock_id) > +static inline void vdso_test_clock(clockid_t clock_id) > { > - int ret0, ret1; > - > - ret0 = vdso_test_clock_gettime(clock_id); > - /* A skipped test is considered passed */ > - if (ret0 == KSFT_SKIP) > - ret0 = KSFT_PASS; > - > - ret1 = vdso_test_clock_getres(clock_id); > - /* A skipped test is considered passed */ > - if (ret1 == KSFT_SKIP) > - ret1 = KSFT_PASS; > + ksft_print_msg("\nclock_id: %s\n", vdso_clock_name[clock_id]); > > - ret0 += ret1; > + vdso_test_clock_gettime(clock_id); > > - printf("clock_id: %s", vdso_clock_name[clock_id]); > - > - if (ret0 > 0) > - printf(" [FAIL]\n"); > - else > - printf(" [PASS]\n"); > - > - return ret0; > + vdso_test_clock_getres(clock_id); > } > > +#define VDSO_TEST_PLAN 16 > + > int main(int argc, char **argv) > { > unsigned long sysinfo_ehdr = getauxval(AT_SYSINFO_EHDR); > - int ret; > + > + ksft_print_header(); > + ksft_set_plan(VDSO_TEST_PLAN); > > if (!sysinfo_ehdr) { > printf("AT_SYSINFO_EHDR is not present!\n"); > @@ -201,44 +192,42 @@ int main(int argc, char **argv) > > vdso_init_from_sysinfo_ehdr(getauxval(AT_SYSINFO_EHDR)); > > - ret = vdso_test_gettimeofday(); > + vdso_test_gettimeofday(); > > #if _POSIX_TIMERS > 0 > > #ifdef CLOCK_REALTIME > - ret += vdso_test_clock(CLOCK_REALTIME); > + vdso_test_clock(CLOCK_REALTIME); > #endif > > #ifdef CLOCK_BOOTTIME > - ret += vdso_test_clock(CLOCK_BOOTTIME); > + vdso_test_clock(CLOCK_BOOTTIME); > #endif > > #ifdef CLOCK_TAI > - ret += vdso_test_clock(CLOCK_TAI); > + vdso_test_clock(CLOCK_TAI); > #endif > > #ifdef CLOCK_REALTIME_COARSE > - ret += vdso_test_clock(CLOCK_REALTIME_COARSE); > + vdso_test_clock(CLOCK_REALTIME_COARSE); > #endif > > #ifdef CLOCK_MONOTONIC > - ret += vdso_test_clock(CLOCK_MONOTONIC); > + vdso_test_clock(CLOCK_MONOTONIC); > #endif > > #ifdef CLOCK_MONOTONIC_RAW > - ret += vdso_test_clock(CLOCK_MONOTONIC_RAW); > + vdso_test_clock(CLOCK_MONOTONIC_RAW); > #endif > > #ifdef CLOCK_MONOTONIC_COARSE > - ret += vdso_test_clock(CLOCK_MONOTONIC_COARSE); > + vdso_test_clock(CLOCK_MONOTONIC_COARSE); > #endif > > #endif > > - ret += vdso_test_time(); > - > - if (ret > 0) > - return KSFT_FAIL; > + vdso_test_time(); > > - return KSFT_PASS; > + ksft_print_cnts(); > + return ksft_get_fail_cnt() == 0 ? KSFT_PASS : KSFT_FAIL; > } > -- > 2.34.1 >
On 1/28/22 7:44 AM, Cristian Marussi wrote: > On Fri, Jan 28, 2022 at 01:07:01PM +0000, Vincenzo Frascino wrote: >> vdso_test_abi contains a batch of tests that verify the validity of the >> vDSO ABI. >> >> When a vDSO symbol is not found the relevant test is skipped reporting >> KSFT_SKIP. All the tests return values are then added in a single >> variable which is checked to verify failures. This approach can have >> side effects which result in reporting the wrong kselftest exit status. >> >> Fix vdso_test_abi verifying the return code of each test separately. >> >> Cc: Shuah Khan <shuah@kernel.org> >> Cc: Andy Lutomirski <luto@kernel.org> >> Cc: Thomas Gleixner <tglx@linutronix.de> >> Reported-by: Cristian Marussi <cristian.marussi@arm.com> >> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> >> --- > > Hi, > > LGTM. > > Tested-by: Cristian Marussi <cristian.marussi@arm.com> > > Thanks, > Cristian Thank you both. Will queue this up for 5.17 rc3. thanks, -- Shuah
On 1/28/22 10:35 AM, Shuah Khan wrote: > On 1/28/22 7:44 AM, Cristian Marussi wrote: >> On Fri, Jan 28, 2022 at 01:07:01PM +0000, Vincenzo Frascino wrote: >>> vdso_test_abi contains a batch of tests that verify the validity of the >>> vDSO ABI. >>> >>> When a vDSO symbol is not found the relevant test is skipped reporting >>> KSFT_SKIP. All the tests return values are then added in a single >>> variable which is checked to verify failures. This approach can have >>> side effects which result in reporting the wrong kselftest exit status. >>> >>> Fix vdso_test_abi verifying the return code of each test separately. >>> >>> Cc: Shuah Khan <shuah@kernel.org> >>> Cc: Andy Lutomirski <luto@kernel.org> >>> Cc: Thomas Gleixner <tglx@linutronix.de> >>> Reported-by: Cristian Marussi <cristian.marussi@arm.com> >>> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> >>> --- >> >> Hi, >> >> LGTM. >> >> Tested-by: Cristian Marussi <cristian.marussi@arm.com> >> >> Thanks, >> Cristian > > Thank you both. Will queue this up for 5.17 rc3. > Vincenzo, A couple of checkpatch warns. Please fix them and send me v4. thanks, -- Shuah
Hi Shuah, On 1/28/22 5:41 PM, Shuah Khan wrote: > On 1/28/22 10:35 AM, Shuah Khan wrote: >> On 1/28/22 7:44 AM, Cristian Marussi wrote: >>> On Fri, Jan 28, 2022 at 01:07:01PM +0000, Vincenzo Frascino wrote: >>>> vdso_test_abi contains a batch of tests that verify the validity of the >>>> vDSO ABI. >>>> >>>> When a vDSO symbol is not found the relevant test is skipped reporting >>>> KSFT_SKIP. All the tests return values are then added in a single >>>> variable which is checked to verify failures. This approach can have >>>> side effects which result in reporting the wrong kselftest exit status. >>>> >>>> Fix vdso_test_abi verifying the return code of each test separately. >>>> >>>> Cc: Shuah Khan <shuah@kernel.org> >>>> Cc: Andy Lutomirski <luto@kernel.org> >>>> Cc: Thomas Gleixner <tglx@linutronix.de> >>>> Reported-by: Cristian Marussi <cristian.marussi@arm.com> >>>> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> >>>> --- >>> >>> Hi, >>> >>> LGTM. >>> >>> Tested-by: Cristian Marussi <cristian.marussi@arm.com> >>> >>> Thanks, >>> Cristian >> >> Thank you both. Will queue this up for 5.17 rc3. >> > > Vincenzo, > > A couple of checkpatch warns. Please fix them and send me v4. > It should be sorted now, I just posted v4. Please let me know if anything else is required from my side. Thanks! --->8--- $ scripts/checkpatch.pl v4-0001-kselftest-Fix-vdso_test_abi-return-status.patch total: 0 errors, 0 warnings, 252 lines checked > thanks, > -- Shuah
diff --git a/tools/testing/selftests/vDSO/vdso_test_abi.c b/tools/testing/selftests/vDSO/vdso_test_abi.c index 3d603f1394af..d670d5b2a626 100644 --- a/tools/testing/selftests/vDSO/vdso_test_abi.c +++ b/tools/testing/selftests/vDSO/vdso_test_abi.c @@ -33,110 +33,114 @@ typedef long (*vdso_clock_gettime_t)(clockid_t clk_id, struct timespec *ts); typedef long (*vdso_clock_getres_t)(clockid_t clk_id, struct timespec *ts); typedef time_t (*vdso_time_t)(time_t *t); -static int vdso_test_gettimeofday(void) +#define VDSO_TEST_PASS_MSG() "\n%s(): PASS\n", __func__ +#define VDSO_TEST_FAIL_MSG(x) "\n%s(): %s FAIL\n", __func__, x +#define VDSO_TEST_SKIP_MSG(x) "\n%s(): SKIP: Could not find %s\n", __func__, x + +static void vdso_test_gettimeofday(void) { /* Find gettimeofday. */ vdso_gettimeofday_t vdso_gettimeofday = (vdso_gettimeofday_t)vdso_sym(version, name[0]); if (!vdso_gettimeofday) { - printf("Could not find %s\n", name[0]); - return KSFT_SKIP; + ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[0])); + return; } struct timeval tv; long ret = vdso_gettimeofday(&tv, 0); if (ret == 0) { - printf("The time is %lld.%06lld\n", - (long long)tv.tv_sec, (long long)tv.tv_usec); + ksft_print_msg("The time is %lld.%06lld\n", + (long long)tv.tv_sec, (long long)tv.tv_usec); + ksft_test_result_pass(VDSO_TEST_PASS_MSG()); } else { - printf("%s failed\n", name[0]); - return KSFT_FAIL; + ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[0])); } - - return KSFT_PASS; } -static int vdso_test_clock_gettime(clockid_t clk_id) +static void vdso_test_clock_gettime(clockid_t clk_id) { /* Find clock_gettime. */ vdso_clock_gettime_t vdso_clock_gettime = (vdso_clock_gettime_t)vdso_sym(version, name[1]); if (!vdso_clock_gettime) { - printf("Could not find %s\n", name[1]); - return KSFT_SKIP; + ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[1])); + return; } struct timespec ts; long ret = vdso_clock_gettime(clk_id, &ts); if (ret == 0) { - printf("The time is %lld.%06lld\n", - (long long)ts.tv_sec, (long long)ts.tv_nsec); + ksft_print_msg("The time is %lld.%06lld\n", + (long long)ts.tv_sec, (long long)ts.tv_nsec); + ksft_test_result_pass(VDSO_TEST_PASS_MSG());; } else { - printf("%s failed\n", name[1]); - return KSFT_FAIL; + ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[1])); } - - return KSFT_PASS; } -static int vdso_test_time(void) +static void vdso_test_time(void) { /* Find time. */ vdso_time_t vdso_time = (vdso_time_t)vdso_sym(version, name[2]); if (!vdso_time) { - printf("Could not find %s\n", name[2]); - return KSFT_SKIP; + ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[2])); + return; } long ret = vdso_time(NULL); if (ret > 0) { - printf("The time in hours since January 1, 1970 is %lld\n", + ksft_print_msg("The time in hours since January 1, 1970 is %lld\n", (long long)(ret / 3600)); + ksft_test_result_pass(VDSO_TEST_PASS_MSG()); } else { - printf("%s failed\n", name[2]); - return KSFT_FAIL; + ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[2])); } - - return KSFT_PASS; } -static int vdso_test_clock_getres(clockid_t clk_id) +static void vdso_test_clock_getres(clockid_t clk_id) { + int clock_getres_fail = 0; + /* Find clock_getres. */ vdso_clock_getres_t vdso_clock_getres = (vdso_clock_getres_t)vdso_sym(version, name[3]); if (!vdso_clock_getres) { - printf("Could not find %s\n", name[3]); - return KSFT_SKIP; + ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[3])); + return; } struct timespec ts, sys_ts; long ret = vdso_clock_getres(clk_id, &ts); if (ret == 0) { - printf("The resolution is %lld %lld\n", - (long long)ts.tv_sec, (long long)ts.tv_nsec); + ksft_print_msg("The vdso resolution is %lld %lld\n", + (long long)ts.tv_sec, (long long)ts.tv_nsec); } else { - printf("%s failed\n", name[3]); - return KSFT_FAIL; + clock_getres_fail++; } ret = syscall(SYS_clock_getres, clk_id, &sys_ts); - if ((sys_ts.tv_sec != ts.tv_sec) || (sys_ts.tv_nsec != ts.tv_nsec)) { - printf("%s failed\n", name[3]); - return KSFT_FAIL; - } + ksft_print_msg("The syscall resolution is %lld %lld\n", + (long long)sys_ts.tv_sec, (long long)sys_ts.tv_nsec); - return KSFT_PASS; + if ((sys_ts.tv_sec != ts.tv_sec) || (sys_ts.tv_nsec != ts.tv_nsec)) + clock_getres_fail++; + + if (clock_getres_fail > 0) { + ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[3])); + } else { + ksft_test_result_pass(VDSO_TEST_PASS_MSG()); + } } const char *vdso_clock_name[12] = { @@ -158,36 +162,23 @@ const char *vdso_clock_name[12] = { * This function calls vdso_test_clock_gettime and vdso_test_clock_getres * with different values for clock_id. */ -static inline int vdso_test_clock(clockid_t clock_id) +static inline void vdso_test_clock(clockid_t clock_id) { - int ret0, ret1; - - ret0 = vdso_test_clock_gettime(clock_id); - /* A skipped test is considered passed */ - if (ret0 == KSFT_SKIP) - ret0 = KSFT_PASS; - - ret1 = vdso_test_clock_getres(clock_id); - /* A skipped test is considered passed */ - if (ret1 == KSFT_SKIP) - ret1 = KSFT_PASS; + ksft_print_msg("\nclock_id: %s\n", vdso_clock_name[clock_id]); - ret0 += ret1; + vdso_test_clock_gettime(clock_id); - printf("clock_id: %s", vdso_clock_name[clock_id]); - - if (ret0 > 0) - printf(" [FAIL]\n"); - else - printf(" [PASS]\n"); - - return ret0; + vdso_test_clock_getres(clock_id); } +#define VDSO_TEST_PLAN 16 + int main(int argc, char **argv) { unsigned long sysinfo_ehdr = getauxval(AT_SYSINFO_EHDR); - int ret; + + ksft_print_header(); + ksft_set_plan(VDSO_TEST_PLAN); if (!sysinfo_ehdr) { printf("AT_SYSINFO_EHDR is not present!\n"); @@ -201,44 +192,42 @@ int main(int argc, char **argv) vdso_init_from_sysinfo_ehdr(getauxval(AT_SYSINFO_EHDR)); - ret = vdso_test_gettimeofday(); + vdso_test_gettimeofday(); #if _POSIX_TIMERS > 0 #ifdef CLOCK_REALTIME - ret += vdso_test_clock(CLOCK_REALTIME); + vdso_test_clock(CLOCK_REALTIME); #endif #ifdef CLOCK_BOOTTIME - ret += vdso_test_clock(CLOCK_BOOTTIME); + vdso_test_clock(CLOCK_BOOTTIME); #endif #ifdef CLOCK_TAI - ret += vdso_test_clock(CLOCK_TAI); + vdso_test_clock(CLOCK_TAI); #endif #ifdef CLOCK_REALTIME_COARSE - ret += vdso_test_clock(CLOCK_REALTIME_COARSE); + vdso_test_clock(CLOCK_REALTIME_COARSE); #endif #ifdef CLOCK_MONOTONIC - ret += vdso_test_clock(CLOCK_MONOTONIC); + vdso_test_clock(CLOCK_MONOTONIC); #endif #ifdef CLOCK_MONOTONIC_RAW - ret += vdso_test_clock(CLOCK_MONOTONIC_RAW); + vdso_test_clock(CLOCK_MONOTONIC_RAW); #endif #ifdef CLOCK_MONOTONIC_COARSE - ret += vdso_test_clock(CLOCK_MONOTONIC_COARSE); + vdso_test_clock(CLOCK_MONOTONIC_COARSE); #endif #endif - ret += vdso_test_time(); - - if (ret > 0) - return KSFT_FAIL; + vdso_test_time(); - return KSFT_PASS; + ksft_print_cnts(); + return ksft_get_fail_cnt() == 0 ? KSFT_PASS : KSFT_FAIL; }
vdso_test_abi contains a batch of tests that verify the validity of the vDSO ABI. When a vDSO symbol is not found the relevant test is skipped reporting KSFT_SKIP. All the tests return values are then added in a single variable which is checked to verify failures. This approach can have side effects which result in reporting the wrong kselftest exit status. Fix vdso_test_abi verifying the return code of each test separately. Cc: Shuah Khan <shuah@kernel.org> Cc: Andy Lutomirski <luto@kernel.org> Cc: Thomas Gleixner <tglx@linutronix.de> Reported-by: Cristian Marussi <cristian.marussi@arm.com> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> --- tools/testing/selftests/vDSO/vdso_test_abi.c | 135 +++++++++---------- 1 file changed, 62 insertions(+), 73 deletions(-)