linux-kernel - Re: + cgroups-forbid-noprefix-if-mounting-more-than-just-cpuset-subsystem. patch added to -mm tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 3 Jun 2009 09:02:43 +0900
From:	KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	Dhaval Giani <dhaval@...ux.vnet.ibm.com>, menage@...gle.com,
	lizf@...fujitsu.com, balbir@...ux.vnet.ibm.com,
	linux-kernel@...r.kernel.org, bharata@...ux.vnet.ibm.com,
	libcg-devel@...ts.sourceforge.net
Subject: Re: +
 cgroups-forbid-noprefix-if-mounting-more-than-just-cpuset-subsystem. patch
 added to -mm tree

On Tue, 2 Jun 2009 13:35:34 -0700
Andrew Morton <akpm@...ux-foundation.org> wrote:

> On Tue, 2 Jun 2009 23:10:42 +0530
> Dhaval Giani <dhaval@...ux.vnet.ibm.com> wrote:
> 
> > On Tue, Jun 02, 2009 at 09:08:04AM -0700, Paul Menage wrote:
> > > On Tue, Jun 2, 2009 at 2:09 AM, Dhaval Giani <dhaval@...ux.vnet.ibm.com> wrote:
> > > >
> > > > I am not sure if this is a good idea. For libcgroup, we would then be
> > > > adding a special case for just cpuset. I would rather that we allow it
> > > > either for all the subsystems or none of them.
> > > >
> > > 
> > > libcgroup shouldn't be using the noprefix option. Its only intentded
> > > use is to allow the legacy "cpuset" filesystem type to be mounted and
> > > to see the same fileset as it had before the cgroups transition.
> > > 
> > 
> > It does not. But if some user is using that option, we need to be in a
> > position to handle it.
> > 
> > I am quite happy not supporting the noprefix option in the library if it
> > is fine.
> > 
> 
> fyi, the above discussion transitions akpm into the "confused" state. 
> I'll keep the patch on hold until akpm transitions back out of that
> state.
> 
Traffic control...

[What "noprefix" is]
 - When "noprefix" is used, the name of file under cgroup is..
  [Without noprefix]  ... (dir)/subsysname.filename
  [With noprefix]     ... (dir)/filename

Then, cpuset's files will be
  [Without noprefix]  ... (dir)/cpuset.xxx
  [With noprefix]     ... (dir)/xxx

This is for _backward compatibility_.

[Problem]
 cpustat subsys has "stat" file.
 memory  subsys has "stat" file.

 So, these cannot be mounted at the same mount point with "noprefix".

Considering arbitrary subsys can be mounted at the same point, allowing "noprefix"
other than cpuset just makes user-land complex, and "noprefix" itself is
troublesome, it breaks naming rule for cgroup files.

Then, I vote for that this patch should go.

Thanks,
-Kame


 













> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/