FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu Kernel Team

 
 
LinkBack Thread Tools
 
Old 06-22-2012, 08:01 AM
Bryan Wu
 
Default cgroups configs review

Hi Serge and Stefan,

I've been reviewing cgroups configs for a while and help to sync ARM
cgroups configs with x86.
So right now in our Ubuntu kernel master-next branch, cgroups configs
should be all enabled for LXC usage:
--
ubuntu/quantal.git$ grep -r CGROUP debian.master/config/
debian.master/config/config.common.ubuntu:CONFIG_BLK_CGROUP=y
debian.master/config/config.common.ubuntu:CONFIG_CGROUPS=y
debian.master/config/config.common.ubuntu:CONFIG_CGROUP_CPUACCT=y

This is for debugging
debian.master/config/config.common.ubuntu:# CONFIG_CGROUP_DEBUG is not set

debian.master/config/config.common.ubuntu:CONFIG_CGROUP_DEVICE=y
debian.master/config/config.common.ubuntu:CONFIG_CGROUP_FREEZER=y
debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR=y

This one is experimental
debian.master/config/config.common.ubuntu:#
CONFIG_CGROUP_MEM_RES_CTLR_KMEM is not set

debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR_SW AP=y
debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR_SW AP_ENABLED=y
debian.master/config/config.common.ubuntu:CONFIG_CGROUP_PERF=y
debian.master/config/config.common.ubuntu:CONFIG_CGROUP_SCHED=y

This is for debugging
debian.master/config/config.common.ubuntu:# CONFIG_DEBUG_BLK_CGROUP is not set

These 2 are for networking cgroups, I'm not sure whether we need
built-in these 2 in kernel.
debian.master/config/config.common.ubuntu:CONFIG_NETPRIO_CGROUP=m
debian.master/config/config.common.ubuntu:CONFIG_NET_CLS_CGROUP=m
--

And for ti-omap4 branch we got the same CGROUPS configs as master now.

Thanks,
--
Bryan Wu <bryan.wu@canonical.com>
Kernel Developer * *+86.186-168-78255 Mobile
Canonical Ltd. * * *www.canonical.com
Ubuntu - Linux for human beings | www.ubuntu.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 06-22-2012, 12:53 PM
Serge Hallyn
 
Default cgroups configs review

Quoting Bryan Wu (bryan.wu@canonical.com):
> Hi Serge and Stefan,
>
> I've been reviewing cgroups configs for a while and help to sync ARM
> cgroups configs with x86.
> So right now in our Ubuntu kernel master-next branch, cgroups configs
> should be all enabled for LXC usage:
> --
> ubuntu/quantal.git$ grep -r CGROUP debian.master/config/
> debian.master/config/config.common.ubuntu:CONFIG_BLK_CGROUP=y
> debian.master/config/config.common.ubuntu:CONFIG_CGROUPS=y
> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_CPUACCT=y
>
> This is for debugging
> debian.master/config/config.common.ubuntu:# CONFIG_CGROUP_DEBUG is not set
>
> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_DEVICE=y
> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_FREEZER=y
> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR=y
>
> This one is experimental
> debian.master/config/config.common.ubuntu:#
> CONFIG_CGROUP_MEM_RES_CTLR_KMEM is not set

Thanks for reviewing these, Bryan.

I'm fine with CONFIG_CGROUP_MEM_RES_CTLR_KMEM not being set for
now. If/when at some point it becomes harder for containers to affect
the host in other ways, then DOS through kernel memory hog might be
worth worrying about, but not just yet.

> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR_SW AP=y
> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR_SW AP_ENABLED=y
> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_PERF=y
> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_SCHED=y
>
> This is for debugging
> debian.master/config/config.common.ubuntu:# CONFIG_DEBUG_BLK_CGROUP is not set
>
> These 2 are for networking cgroups, I'm not sure whether we need
> built-in these 2 in kernel.
> debian.master/config/config.common.ubuntu:CONFIG_NETPRIO_CGROUP=m
> debian.master/config/config.common.ubuntu:CONFIG_NET_CLS_CGROUP=m

What would be the advantage of having these built-in? AFAIK a module
is fine, but maybe I'm overlooking something.

> --
>
> And for ti-omap4 branch we got the same CGROUPS configs as master now.
>
> Thanks,
> --
> Bryan Wu <bryan.wu@canonical.com>
> Kernel Developer * *+86.186-168-78255 Mobile
> Canonical Ltd. * * *www.canonical.com
> Ubuntu - Linux for human beings | www.ubuntu.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 06-22-2012, 03:49 PM
Bryan Wu
 
Default cgroups configs review

On Fri, Jun 22, 2012 at 8:53 PM, Serge Hallyn
<serge.hallyn@canonical.com> wrote:
> Quoting Bryan Wu (bryan.wu@canonical.com):
>> Hi Serge and Stefan,
>>
>> I've been reviewing cgroups configs for a while and help to sync ARM
>> cgroups configs with x86.
>> So right now in our Ubuntu kernel master-next branch, cgroups configs
>> should be all enabled for LXC usage:
>> --
>> ubuntu/quantal.git$ grep -r CGROUP debian.master/config/
>> debian.master/config/config.common.ubuntu:CONFIG_BLK_CGROUP=y
>> debian.master/config/config.common.ubuntu:CONFIG_CGROUPS=y
>> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_CPUACCT=y
>>
>> This is for debugging
>> debian.master/config/config.common.ubuntu:# CONFIG_CGROUP_DEBUG is not set
>>
>> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_DEVICE=y
>> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_FREEZER=y
>> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR=y
>>
>> This one is experimental
>> debian.master/config/config.common.ubuntu:#
>> CONFIG_CGROUP_MEM_RES_CTLR_KMEM is not set
>
> Thanks for reviewing these, Bryan.
>
> I'm fine with CONFIG_CGROUP_MEM_RES_CTLR_KMEM not being set for
> now. *If/when at some point it becomes harder for containers to affect
> the host in other ways, then DOS through kernel memory hog might be
> worth worrying about, but not just yet.
>

OK, no problem, I will keep eyes on this.

>> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR_SW AP=y
>> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_MEM_RES_CTLR_SW AP_ENABLED=y
>> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_PERF=y
>> debian.master/config/config.common.ubuntu:CONFIG_CGROUP_SCHED=y
>>
>> This is for debugging
>> debian.master/config/config.common.ubuntu:# CONFIG_DEBUG_BLK_CGROUP is not set
>>
>> These 2 are for networking cgroups, I'm not sure whether we need
>> built-in these 2 in kernel.
>> debian.master/config/config.common.ubuntu:CONFIG_NETPRIO_CGROUP=m
>> debian.master/config/config.common.ubuntu:CONFIG_NET_CLS_CGROUP=m
>
> What would be the advantage of having these built-in? *AFAIK a module
> is fine, but maybe I'm overlooking something.
>

Looks like LXC doesn't need to built-in these 2 modules, so it's
better to keep them as modules.

Thanks,
-Bryan

>> --
>>
>> And for ti-omap4 branch we got the same CGROUPS configs as master now.
>>
>> Thanks,
>> --
>> Bryan Wu <bryan.wu@canonical.com>
>> Kernel Developer * *+86.186-168-78255 Mobile
>> Canonical Ltd. * * *www.canonical.com
>> Ubuntu - Linux for human beings | www.ubuntu.com



--
Bryan Wu <bryan.wu@canonical.com>
Kernel Developer * *+86.186-168-78255 Mobile
Canonical Ltd. * * *www.canonical.com
Ubuntu - Linux for human beings | www.ubuntu.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 

Thread Tools




All times are GMT. The time now is 06:33 PM.

VBulletin, Copyright ©2000 - 2013, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright ©2007 - 2008, www.linux-archive.org