Re: [PATCH v2 19/53] selftests/mm: va_high_addr_switch: use kselftest framework

Mark Brown
2026-04-20T18:11:38Z

[-- Attachment #1: Type: text/plain, Size: 579 bytes --]

On Sat, Apr 18, 2026 at 01:55:04PM +0300, Mike Rapoport wrote:

> if (p == MAP_FAILED) {
> - printf("FAILED\n");
> - ret = KSFT_FAIL;
> + ksft_test_result_fail("%s: MAP_FAILED\n", t->msg);
> continue;
> }
>
> if (t->low_addr_required && p >= (void *)(switch_hint)) {
> - printf("FAILED\n");
> - ret = KSFT_FAIL;
> + ksft_test_result_fail("%s: %p not below switch hint\n", t->msg, p);

The test result reports should use a consistent name for each test and
log any diagnostic information about the source of failure in separate
prints.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]