All control files created by cgroup subsystems are given a prefix corresponding to their subsystem name. But control files provided by cgroups itself have no prefix. Currently that set of files is just "tasks", "notify_on_release" and "release_agent", but that set is likely to expand in the future. To reduce the risk of clashes, it would make sense to prefix these files and any future ones with the "cgroup." prefix. The only reason that I can see *not* to do this would be for compatibility with 2.6.24. Do people think this is a strong enough reason to leave the existing names? If distros are planning to ship products based on 2.6.24, presumably they'd be adding their own patches anyway, so they could add a trivial prefix change patch too. (I realise this discussion would have been more useful *before* 2.6.24 shipped, but I didn't quite get round to it ...) A compromise might be to keep "tasks" unprefixed, and say that future names get the "cgroup." prefix; in this case I'd be inclined to add the prefix to notify_on_release and release_agent on the grounds that there's much less chance of breaking anyone with those files since (I suspect) they're much less used. Note that if you mount a cgroup filesystem with the "noprefix" option, which is what the cpuset filesystem wrapper does, no subsystems have prefixes, and in this case the "cgroup." prefix wouldn't be used either. So this doesn't affect any users that explicitly mount cpusets rather than cgroups. Thoughts? Paul --
| David Newall | Re: Slow DOWN, please!!! |
| Linus Torvalds | Re: O_DIRECT question |
| Ingo Molnar | Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23 |
| Greg KH | [GIT PATCH] driver core patches against 2.6.24 |
git: | |
| Junio C Hamano | Re: [RFC] Git User's Survey 2008 |
| Junichi Uekawa | Re: [ANNOUNCE] GIT 1.5.4 |
| Marcus Griep | [PATCH] git-svn: Make it scream by minimizing temp files |
| Bill Lear | Meaning of "fatal: protocol error: bad line length character"? |
| Richard Stallman | Real men don't attack straw men |
| GVG GVG | ssh_exchange_identification: Connection closed by remote host |
| farhan ahmed | Re: bash for root? (was: Re: libiconv problem ) |
| Tony Sarendal | bgpd causing black-holes with bgp-only setup |
| Krishna Kumar | [PATCH 9/10 REV5] [IPoIB] Implement batching |
| jamal | Re: [PATCH 2/3][NET_BATCH] net core use batching |
| Andi Kleen | [PATCH] Disable TSO for non standard qdiscs |
| James Chapman | Re: [PATCH][PPPOL2TP]: Fix SMP oops in pppol2tp driver |
