Commit ec049891 authored by Vincenzo Frascino's avatar Vincenzo Frascino Committed by Shuah Khan

kselftest: Fix vdso_test_abi return status

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: default avatarCristian Marussi <cristian.marussi@arm.com>
Signed-off-by: default avatarVincenzo Frascino <vincenzo.frascino@arm.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent dae1d8ac
...@@ -33,110 +33,114 @@ typedef long (*vdso_clock_gettime_t)(clockid_t clk_id, struct timespec *ts); ...@@ -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 long (*vdso_clock_getres_t)(clockid_t clk_id, struct timespec *ts);
typedef time_t (*vdso_time_t)(time_t *t); 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. */ /* Find gettimeofday. */
vdso_gettimeofday_t vdso_gettimeofday = vdso_gettimeofday_t vdso_gettimeofday =
(vdso_gettimeofday_t)vdso_sym(version, name[0]); (vdso_gettimeofday_t)vdso_sym(version, name[0]);
if (!vdso_gettimeofday) { if (!vdso_gettimeofday) {
printf("Could not find %s\n", name[0]); ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[0]));
return KSFT_SKIP; return;
} }
struct timeval tv; struct timeval tv;
long ret = vdso_gettimeofday(&tv, 0); long ret = vdso_gettimeofday(&tv, 0);
if (ret == 0) { if (ret == 0) {
printf("The time is %lld.%06lld\n", ksft_print_msg("The time is %lld.%06lld\n",
(long long)tv.tv_sec, (long long)tv.tv_usec); (long long)tv.tv_sec, (long long)tv.tv_usec);
ksft_test_result_pass(VDSO_TEST_PASS_MSG());
} else { } else {
printf("%s failed\n", name[0]); ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[0]));
return KSFT_FAIL;
} }
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. */ /* Find clock_gettime. */
vdso_clock_gettime_t vdso_clock_gettime = vdso_clock_gettime_t vdso_clock_gettime =
(vdso_clock_gettime_t)vdso_sym(version, name[1]); (vdso_clock_gettime_t)vdso_sym(version, name[1]);
if (!vdso_clock_gettime) { if (!vdso_clock_gettime) {
printf("Could not find %s\n", name[1]); ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[1]));
return KSFT_SKIP; return;
} }
struct timespec ts; struct timespec ts;
long ret = vdso_clock_gettime(clk_id, &ts); long ret = vdso_clock_gettime(clk_id, &ts);
if (ret == 0) { if (ret == 0) {
printf("The time is %lld.%06lld\n", ksft_print_msg("The time is %lld.%06lld\n",
(long long)ts.tv_sec, (long long)ts.tv_nsec); (long long)ts.tv_sec, (long long)ts.tv_nsec);
ksft_test_result_pass(VDSO_TEST_PASS_MSG());
} else { } else {
printf("%s failed\n", name[1]); ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[1]));
return KSFT_FAIL;
} }
return KSFT_PASS;
} }
static int vdso_test_time(void) static void vdso_test_time(void)
{ {
/* Find time. */ /* Find time. */
vdso_time_t vdso_time = vdso_time_t vdso_time =
(vdso_time_t)vdso_sym(version, name[2]); (vdso_time_t)vdso_sym(version, name[2]);
if (!vdso_time) { if (!vdso_time) {
printf("Could not find %s\n", name[2]); ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[2]));
return KSFT_SKIP; return;
} }
long ret = vdso_time(NULL); long ret = vdso_time(NULL);
if (ret > 0) { 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)); (long long)(ret / 3600));
ksft_test_result_pass(VDSO_TEST_PASS_MSG());
} else { } else {
printf("%s failed\n", name[2]); ksft_test_result_fail(VDSO_TEST_FAIL_MSG(name[2]));
return KSFT_FAIL;
} }
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. */ /* Find clock_getres. */
vdso_clock_getres_t vdso_clock_getres = vdso_clock_getres_t vdso_clock_getres =
(vdso_clock_getres_t)vdso_sym(version, name[3]); (vdso_clock_getres_t)vdso_sym(version, name[3]);
if (!vdso_clock_getres) { if (!vdso_clock_getres) {
printf("Could not find %s\n", name[3]); ksft_test_result_skip(VDSO_TEST_SKIP_MSG(name[3]));
return KSFT_SKIP; return;
} }
struct timespec ts, sys_ts; struct timespec ts, sys_ts;
long ret = vdso_clock_getres(clk_id, &ts); long ret = vdso_clock_getres(clk_id, &ts);
if (ret == 0) { if (ret == 0) {
printf("The resolution is %lld %lld\n", ksft_print_msg("The vdso resolution is %lld %lld\n",
(long long)ts.tv_sec, (long long)ts.tv_nsec); (long long)ts.tv_sec, (long long)ts.tv_nsec);
} else { } else {
printf("%s failed\n", name[3]); clock_getres_fail++;
return KSFT_FAIL;
} }
ret = syscall(SYS_clock_getres, clk_id, &sys_ts); ret = syscall(SYS_clock_getres, clk_id, &sys_ts);
if ((sys_ts.tv_sec != ts.tv_sec) || (sys_ts.tv_nsec != ts.tv_nsec)) { ksft_print_msg("The syscall resolution is %lld %lld\n",
printf("%s failed\n", name[3]); (long long)sys_ts.tv_sec, (long long)sys_ts.tv_nsec);
return KSFT_FAIL;
}
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] = { const char *vdso_clock_name[12] = {
...@@ -158,36 +162,23 @@ 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 * This function calls vdso_test_clock_gettime and vdso_test_clock_getres
* with different values for clock_id. * 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; ksft_print_msg("\nclock_id: %s\n", vdso_clock_name[clock_id]);
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;
ret0 += ret1; vdso_test_clock_gettime(clock_id);
printf("clock_id: %s", vdso_clock_name[clock_id]); vdso_test_clock_getres(clock_id);
if (ret0 > 0)
printf(" [FAIL]\n");
else
printf(" [PASS]\n");
return ret0;
} }
#define VDSO_TEST_PLAN 16
int main(int argc, char **argv) int main(int argc, char **argv)
{ {
unsigned long sysinfo_ehdr = getauxval(AT_SYSINFO_EHDR); unsigned long sysinfo_ehdr = getauxval(AT_SYSINFO_EHDR);
int ret;
ksft_print_header();
ksft_set_plan(VDSO_TEST_PLAN);
if (!sysinfo_ehdr) { if (!sysinfo_ehdr) {
printf("AT_SYSINFO_EHDR is not present!\n"); printf("AT_SYSINFO_EHDR is not present!\n");
...@@ -201,44 +192,42 @@ int main(int argc, char **argv) ...@@ -201,44 +192,42 @@ int main(int argc, char **argv)
vdso_init_from_sysinfo_ehdr(getauxval(AT_SYSINFO_EHDR)); vdso_init_from_sysinfo_ehdr(getauxval(AT_SYSINFO_EHDR));
ret = vdso_test_gettimeofday(); vdso_test_gettimeofday();
#if _POSIX_TIMERS > 0 #if _POSIX_TIMERS > 0
#ifdef CLOCK_REALTIME #ifdef CLOCK_REALTIME
ret += vdso_test_clock(CLOCK_REALTIME); vdso_test_clock(CLOCK_REALTIME);
#endif #endif
#ifdef CLOCK_BOOTTIME #ifdef CLOCK_BOOTTIME
ret += vdso_test_clock(CLOCK_BOOTTIME); vdso_test_clock(CLOCK_BOOTTIME);
#endif #endif
#ifdef CLOCK_TAI #ifdef CLOCK_TAI
ret += vdso_test_clock(CLOCK_TAI); vdso_test_clock(CLOCK_TAI);
#endif #endif
#ifdef CLOCK_REALTIME_COARSE #ifdef CLOCK_REALTIME_COARSE
ret += vdso_test_clock(CLOCK_REALTIME_COARSE); vdso_test_clock(CLOCK_REALTIME_COARSE);
#endif #endif
#ifdef CLOCK_MONOTONIC #ifdef CLOCK_MONOTONIC
ret += vdso_test_clock(CLOCK_MONOTONIC); vdso_test_clock(CLOCK_MONOTONIC);
#endif #endif
#ifdef CLOCK_MONOTONIC_RAW #ifdef CLOCK_MONOTONIC_RAW
ret += vdso_test_clock(CLOCK_MONOTONIC_RAW); vdso_test_clock(CLOCK_MONOTONIC_RAW);
#endif #endif
#ifdef CLOCK_MONOTONIC_COARSE #ifdef CLOCK_MONOTONIC_COARSE
ret += vdso_test_clock(CLOCK_MONOTONIC_COARSE); vdso_test_clock(CLOCK_MONOTONIC_COARSE);
#endif #endif
#endif #endif
ret += vdso_test_time(); vdso_test_time();
if (ret > 0)
return KSFT_FAIL;
return KSFT_PASS; ksft_print_cnts();
return ksft_get_fail_cnt() == 0 ? KSFT_PASS : KSFT_FAIL;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment