Re: [PATCH v2 3/6] mm/page_owner: hoist CONFIG_MEMCG to function level for print_page_owner_memcg()

[Vlastimil Babka (SUSE)]

On 6/26/26 04:45, Ye Liu wrote:
> The print_page_owner_memcg() function has CONFIG_MEMCG guarding its
> entire body via #ifdef inside the function, which leaves a no-op
> { return ret; } when the config is disabled.  Hoist the #ifdef to the

IMHO it's not a big deal for an internal function with the ifdef block being
so straightforward. But whatever.

> top level so the real implementation and the empty stub are two clearly
> separated definitions.
> 
> No functional change.
> 
> Signed-off-by: Ye Liu <ye.liu@xxxxxxxxx>

Reviewed-by: Vlastimil Babka (SUSE) <vbabka@xxxxxxxxxx>