Re: [PATCH 1/9] memcg: make the v1 soft limit knob inert
Shakeel Butt <[email protected]>
| Newsgroups | org.kernel.vger.cgroups,org.kernel.vger.linux-kernel,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
On Thu, Aug 13, 2026 at 10:22:57AM +0200, Michal Hocko wrote: > On Tue 11-08-26 13:31:55, Shakeel Butt wrote: > > The v1 soft limit has been deprecated since v6.12 and nobody has > > reported depending on it. Start the removal by decoupling the interface > > from the implementation: keep memory.soft_limit_in_bytes, but ignore > > writes to it and always report the maximum value on read similar to > > what memory.kmem.limit_in_bytes already does. > > > > Writes are still parsed, so malformed input keeps returning -EINVAL. > > The knob now also behaves the same everywhere: it used to return > > -EOPNOTSUPP on PREEMPT_RT, where soft limit reclaim has always been > > disabled. > > Is there any specific reason to not return EOPNOTSUPP for everybody now? I followed the memory.kmem.limit_in_bytes example. If you have strong opinion, I am ok with EOPNOTSUPP as well. > > > This also fixes the syzbot report linked below. Soft limit reclaim is > > the only caller that runs shrink_lruvec() from kswapd against a > > specific memcg, so it is the only way to reach lru_gen_shrink_lruvec() > > and in turn set_mm_walk(), which warns when called from kswapd. > > > > Reported-by: [email protected] > > Closes: https://lore.kernel.org/all/[email protected]/ > > Signed-off-by: Shakeel Butt <[email protected]> > > Anyway > Acked-by: Michal Hocko <[email protected]> Thanks for reviewing the series.