On Wed, Oct 13, 2021 at 2:32 AM Matthew Wilcox (Oracle)
<willy@xxxxxxxxxxxxx> wrote:
>
> Instead of having an ifdef to decide whether to print a |, use the
> 'append' functionality of the main loop to print it.
>
> Signed-off-by: Matthew Wilcox (Oracle) <willy@xxxxxxxxxxxxx>
Reviewed-by: Yafang Shao <laoar.shao@xxxxxxxxx>
> ---
> lib/test_printf.c | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/lib/test_printf.c b/lib/test_printf.c
> index f744b0498672..60cdf4ba991e 100644
> --- a/lib/test_printf.c
> +++ b/lib/test_printf.c
> @@ -616,12 +616,7 @@ page_flags_test(int section, int node, int zone, int last_cpupid,
> if (flags & PAGEFLAGS_MASK) {
> snprintf(cmp_buf + size, BUF_SIZE - size, "%s", name);
> size = strlen(cmp_buf);
> -#if SECTIONS_WIDTH || NODES_WIDTH || ZONES_WIDTH || \
> - LAST_CPUPID_WIDTH || KASAN_TAG_WIDTH
> - /* Other information also included in page flags */
> - snprintf(cmp_buf + size, BUF_SIZE - size, "|");
> - size = strlen(cmp_buf);
> -#endif
> + append = true;
> }
>
> for (i = 0; i < ARRAY_SIZE(pft); i++) {
> --
> 2.32.0
>
--
Thanks
Yafang
