From: Yafang Shao <laoar.shao@gmail.com>
To: Michal Hocko <mhocko@suse.com>
Cc: akpm@linux-foundation.org, hannes@cmpxchg.org,
roman.gushchin@linux.dev, shakeelb@google.com,
muchun.song@linux.dev, linux-mm@kvack.org
Subject: Re: [RFC PATCH] mm: Add reclaim type to memory.reclaim
Date: Tue, 27 Feb 2024 13:48:31 +0800 [thread overview]
Message-ID: <CALOAHbD83eE4+RR5cHizmkcz5etNLx8Khc3QXcNFW8fj4uSWcQ@mail.gmail.com> (raw)
In-Reply-To: <Zdyai0Z-ftqNcyZ9@tiehlicka>
On Mon, Feb 26, 2024 at 10:05 PM Michal Hocko <mhocko@suse.com> wrote:
>
> On Sun 25-02-24 19:42:04, Yafang Shao wrote:
> > In our container environment, we've observed that certain containers may
> > accumulate more than 40GB of slabs, predominantly negative dentries. These
> > negative dentries remain unreclaimed unless there is memory pressure. Even
> > after the containers exit, these negative dentries persist.
>
> Have you considered using memory.high limit to trigger that memory
> reclaim? That is surely not focused on the neg. dentry cache but it
> should keep the overal memory consumption under control.
Upon reaching the high watermark, both pagecache and slabs are
reclaimed. However, it tends to prioritize reclaiming more pagecache
pages over slab pages, leading to persistent memory consumption by
slabs. Setting memory.high too low may negatively impact the working
set.
>
> > To manage disk
> > storage efficiently, we employ an agent that identifies container images
> > eligible for destruction once all instances of that image exit.
> >
> > However, during destruction, dealing with directories containing numerous
> > negative dentries can significantly impact performance.
>
> Performance of what. I have to say I am kind of lost here. We are
> talking about memory or a disk storage?
Removing an empty directory with numerous dentries can significantly
prolong the process of freeing associated dentries, leading to high
system CPU usage that adversely affects overall system performance.
>
> > To mitigate this
> > issue, we aim to proactively reclaim these dentries using a user agent.
> > Extending the memory.reclaim functionality to specifically target slabs
> > aligns with our requirements.
>
> Matthew has already pointed out that this has been proposed several
> times already and rejected.
With that being said, we haven't come up with any superior solutions
compared to the proposals mentioned.
> Dedicated slab shrinking interface is
> especially tricky because you would need a way to tell which shrinkers
> to invoke and that would be very kernel version specific.
The persistence of this issue over several years without any
discernible progress suggests that we might be heading in the wrong
direction. Perhaps we could consider providing a kernel interface to
users, allowing them to tailor the reclamation process based on their
workload requirements.
--
Regards
Yafang
next prev parent reply other threads:[~2024-02-27 5:49 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-25 11:42 Yafang Shao
2024-02-26 4:17 ` Matthew Wilcox
2024-02-26 12:34 ` Yafang Shao
2024-02-26 13:16 ` Matthew Wilcox
2024-02-26 13:56 ` Yafang Shao
2024-02-26 14:04 ` Michal Hocko
2024-02-27 5:48 ` Yafang Shao [this message]
2024-02-27 9:05 ` Michal Hocko
2024-02-27 9:39 ` Yafang Shao
2024-02-27 9:43 ` Yosry Ahmed
2024-02-27 11:37 ` Yafang Shao
2024-02-27 11:40 ` Yosry Ahmed
2024-02-27 11:58 ` Yafang Shao
2024-02-27 12:09 ` Yafang Shao
2024-02-27 12:12 ` Yafang Shao
2024-02-27 13:17 ` Michal Hocko
2024-02-27 14:05 ` Yafang Shao
2024-02-27 9:45 ` Michal Hocko
2024-02-27 10:06 ` Yafang Shao
2024-02-27 13:20 ` Michal Hocko
2024-02-27 14:03 ` Yafang Shao
2024-02-27 14:51 ` Yafang Shao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CALOAHbD83eE4+RR5cHizmkcz5etNLx8Khc3QXcNFW8fj4uSWcQ@mail.gmail.com \
--to=laoar.shao@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=hannes@cmpxchg.org \
--cc=linux-mm@kvack.org \
--cc=mhocko@suse.com \
--cc=muchun.song@linux.dev \
--cc=roman.gushchin@linux.dev \
--cc=shakeelb@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox