Messages in this thread | | | Date | Tue, 26 Feb 2008 17:05:14 +0800 | From | Li Zefan <> | Subject | Re: [PATCH] Memory Resource Controller Add Boot Option |
| |
Paul Menage wrote: > On Mon, Feb 25, 2008 at 7:01 PM, Li Zefan <lizf@cn.fujitsu.com> wrote: >> > >> > - foo doesn't show up in /proc/cgroups >> >> Or we can print out the disable flag, maybe this will be better? >> Because we can distinguish from disabled and not compiled in from >> >> /proc/cgroups. > > Certainly possible, if people felt it was useful. > >> > - foo isn't auto-mounted if you mount all cgroups in a single hierarchy >> > - foo isn't visible as an individually mountable subsystem >> >> You mentioned in a previous mail if we mount a disabled subsystem we >> will get an error. Here we just ignore the mount option. Which makes >> more sense ? >> > > No, we don't ignore the mount option - we give an error since it > doesn't refer to a valid subsystem. (And in the first case there is no > mount option). >
You are write, -ENOENT will be returned in this case. I made a mistake when reading the prototype patch, thanks for the clarification. :)
| |