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: signature.asc
Description: PGP signature

Reply via email to