Re: [PATCH v4 3/7] cgroup: protect cgroup->nr_(dying_)descendants by css_set_lock

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Tejun Heo <tj@xxxxxxxxxx>
Subject: Re: [PATCH v4 3/7] cgroup: protect cgroup->nr_(dying_)descendants by css_set_lock
From: Roman Gushchin <guro@xxxxxx>
Date: Mon, 3 Dec 2018 17:55:32 +0000
Accept-language: en-US
Cc: Roman Gushchin <guroan@xxxxxxxxx>, Oleg Nesterov <oleg@xxxxxxxxxx>, "Dan Carpenter" <dan.carpenter@xxxxxxxxxx>, Mike Rapoport <rppt@xxxxxxxxxxxxxxxxxx>, "cgroups@xxxxxxxxxxxxxxx" <cgroups@xxxxxxxxxxxxxxx>, "linux-doc@xxxxxxxxxxxxxxx" <linux-doc@xxxxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, Kernel Team <Kernel-team@xxxxxx>
In-reply-to: <20181203161706.GL2509588@devbig004.ftw2.facebook.com>
References: <20181130234745.6756-1-guro@fb.com> <20181130234745.6756-4-guro@fb.com> <20181203161706.GL2509588@devbig004.ftw2.facebook.com>

On Mon, Dec 03, 2018 at 08:17:06AM -0800, Tejun Heo wrote:
> On Fri, Nov 30, 2018 at 03:47:41PM -0800, Roman Gushchin wrote:
> > +	 * nr_descendants and nr_dying_descendants are protected
> > +	 * by cgroup_mutex and css_set_lock.
> 
> Can you be a bit more specific - hold both for writes, either for
> reads.

Sure. Thanks!

References:
- [PATCH v4 0/7] freezer for cgroup v2
  - From: Roman Gushchin
- [PATCH v4 3/7] cgroup: protect cgroup->nr_(dying_)descendants by css_set_lock
  - From: Roman Gushchin
- Re: [PATCH v4 3/7] cgroup: protect cgroup->nr_(dying_)descendants by css_set_lock
  - From: Tejun Heo

Prev by Date: Re: [PATCH v3] code-of-conduct: Remove explicit list of discrimination factors
Next by Date: Re: [PATCH v5 1/9] clkdev: add managed clkdev lookup registration
Previous by thread: Re: [PATCH v4 3/7] cgroup: protect cgroup->nr_(dying_)descendants by css_set_lock
Next by thread: [PATCH v4 2/7] cgroup: implement __cgroup_task_count() helper
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux FS] [Yosemite Forum] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Video 4 Linux] [Device Mapper] [Linux Resources]