David Hildenbrand <david@xxxxxxxxxx> writes: >> +/* Verify that KSM can be enabled / queried with prctl. */ >> +static void test_prctl(void) >> +{ >> + int ret; >> + >> + ksft_print_msg("[RUN] %s\n", __func__); >> + >> + ret = prctl(PR_SET_MEMORY_MERGE, 1, 0, 0, 0); >> + if (ret < 0 && errno == EINVAL) { >> + ksft_test_result_skip("PR_SET_MEMORY_MERGE not supported\n"); >> + return; >> + } else if (ret) { >> + ksft_test_result_fail("PR_SET_MEMORY_MERGE=1 failed\n"); >> + } > > Just realized we're missing a "return;" in case of the failure here. > I'll fix it in the next version.