[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAHVum0e+68df7b_e0RoCGgA1h_SLdTt9t2MZ0VgA9afPW9iEvg@mail.gmail.com>
Date: Fri, 10 Sep 2021 10:31:02 -0700
From: Vipin Sharma <vipinsh@...gle.com>
To: Tejun Heo <tj@...nel.org>
Cc: Michal Koutný <mkoutny@...e.com>,
"brookxu.cn" <brookxu.cn@...il.com>, lizefan.x@...edance.com,
hannes@...xchg.org, linux-kernel@...r.kernel.org,
cgroups@...r.kernel.org
Subject: Re: [RFC PATCH 3/3] misc_cgroup: remove error log to avoid log flood
On Fri, Sep 10, 2021 at 10:19 AM Tejun Heo <tj@...nel.org> wrote:
>
> Hello,
>
> On Fri, Sep 10, 2021 at 10:16:46AM -0700, Vipin Sharma wrote:
> > For now, I think we can just have one file, events.local
> > (non-hierarchical) which has %s.max type entries. This will tell us
> > which cgroup is under pressure and I believe this is helpful.
>
> How about just sticking with hierarchical events? Do you see a lot of
> downsides with that compared to .local?
>
Local would have made it easier to identify constrained cgroup and the
lesser number of file notify events will be fired.
I don't have a strong aversion to hierarchical events only as well if
that is more suited for the cgroup v2 style.
Powered by blists - more mailing lists