[LTP] [PATCH 1/1] libswap: Change TWARN message to TINFO

Andrea Cervesato andrea.cervesato@suse.com
Wed Apr 17 11:03:57 CEST 2024


Hi Petr,

On 4/17/24 11:02, Petr Vorel wrote:
> This is hit on systems with 64kb page size (e.g. on aarch64 with
> CONFIG_ARM64_64K_PAGES=y or on ppc64le with CONFIG_PAGE_SIZE_64KB=y).
> Using TINFO causes test not "failing" with non-zero exit code.
>
> Fixes: f987ffff5 ("libswap: add two methods to create swapfile")
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>   libs/libltpswap/libswap.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libs/libltpswap/libswap.c b/libs/libltpswap/libswap.c
> index 313a15f24..6d4184ef9 100644
> --- a/libs/libltpswap/libswap.c
> +++ b/libs/libltpswap/libswap.c
> @@ -160,7 +160,7 @@ int make_swapfile_(const char *file, const int lineno,
>   
>   	/* To guarantee at least one page can be swapped out */
>   	if (blk_size * blocks < pg_size) {
> -		tst_res(TWARN, "Swapfile size is less than the system page size. "
> +		tst_res(TINFO, "Swapfile size is less than the system page size. "
>   			"Using page size (%lu bytes) instead of block size (%lu bytes).",
>   			(unsigned long)pg_size, blk_size);
>   		blk_size = pg_size;

Are we sure about this? TWARN wasn't a bad choice, considering the 
meaning of the message.

Andrea



More information about the ltp mailing list