Re: [PATCH v2 2/3] mm: huge_memory: refactor anon_enabled_store() with change_anon_orders()

[Breno Leitao <leitao@debian.org>]

On Fri, Mar 06, 2026 at 02:07:50PM +0800, Baolin Wang wrote:
> > +	mode = sysfs_match_string(enabled_mode_strings, buf);
> > +	if (mode < 0)
> > +		return -EINVAL;
> > -	if (ret > 0) {
> > -		int err;
> > +	if (change_anon_orders(order, mode)) {
> > +		int err = start_stop_khugepaged();
>
> Thanks for the cleanup, and the code looks better.
>
> > -		err = start_stop_khugepaged();
> >   		if (err)
> > -			ret = err;
> > +			return err;
> > +	} else {
> > +		/*
> > +		 * Recalculate watermarks even when the mode didn't
> > +		 * change, as the previous code always called
> > +		 * start_stop_khugepaged() which does this internally.
> > +		 */
> > +		set_recommended_min_free_kbytes();
>
> However, this won't fix your issue. You will still get lots of warning
> messages even if no hugepage options are changed.

Correct — this was part of the earlier discussion, where the suggestion
was to retain the set_recommended_min_free_kbytes() call even when it
is a no-op.

As for the warnings, I see a few possible approaches:

 * Remove them entirely — are they providing any real value?
 * Apply rate limiting to reduce the noise
 * Something else?

Thanks for bringing this up,
--breno