[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