On Mon, Oct 19, 2026 at 02:23:58PM +0300, Andy Shevchenko wrote:
> On Mon, Oct 19, 2026 at 09:59:01AM +0800, Qiliang Yuan wrote:

...

> > -   if (IS_ENABLED(CONFIG_KMSAN))
> > +   if (IS_ENABLED(CONFIG_KMSAN) || IS_ENABLED(CONFIG_KASAN_GENERIC) ||
> > +       IS_ENABLED(CONFIG_KASAN_SW_TAGS))
> >             max = 0;
> 
> Logically better to have them grouped by SANitizer:
> 
>       if (IS_ENABLED(CONFIG_KASAN_GENERIC) || 
> IS_ENABLED(CONFIG_KASAN_SW_TAGS) ||
>           IS_ENABLED(CONFIG_KMSAN))

Or other way around (I think it will be shorter)

        if (IS_ENABLED(CONFIG_KMSAN) ||
            IS_ENABLED(CONFIG_KASAN_GENERIC) || 
IS_ENABLED(CONFIG_KASAN_SW_TAGS))

-- 
With Best Regards,
Andy Shevchenko



Reply via email to