Skip to content

Commit

Permalink
selftests: arm64: Fix and enable test for setting current VL in vec-s…
Browse files Browse the repository at this point in the history
…yscfg

We had some test code for verifying that we can write the current VL via
the prctl() interface but the condition for the test was inverted which
wasn't noticed as it was never actually hooked up to the array of tests
we execute. Fix this.

Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20210929151925.9601-4-broonie@kernel.org
Signed-off-by: Will Deacon <will@kernel.org>
  • Loading branch information
Mark Brown authored and Will Deacon committed Sep 29, 2021
1 parent 4caf339 commit e423911
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions tools/testing/selftests/arm64/fp/vec-syscfg.c
Original file line number Diff line number Diff line change
Expand Up @@ -329,12 +329,9 @@ static void prctl_set_same(struct vec_data *data)
return;
}

if (cur_vl != data->rdvl())
ksft_test_result_pass("%s current VL is %d\n",
data->name, ret);
else
ksft_test_result_fail("%s prctl() VL %d but RDVL is %d\n",
data->name, ret, data->rdvl());
ksft_test_result(cur_vl == data->rdvl(),
"%s set VL %d and have VL %d\n",
data->name, cur_vl, data->rdvl());
}

/* Can we set a new VL for this process? */
Expand Down Expand Up @@ -555,6 +552,7 @@ static const test_type tests[] = {
proc_write_max,

prctl_get,
prctl_set_same,
prctl_set,
prctl_set_no_child,
prctl_set_for_child,
Expand Down

0 comments on commit e423911

Please sign in to comment.