mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Tejun Heo <tj@kernel.org>
To: Vivek Goyal <vgoyal@redhat.com>
Cc: Li Zefan <lizefan@huawei.com>,
	containers@lists.linux-foundation.org, cgroups@vger.kernel.org,
	bsingharora@gmail.com, Kay Sievers <kay.sievers@vrfy.org>,
	lpoetter@redhat.com, linux-kernel@vger.kernel.org,
	dhaval.giani@gmail.com, workman-devel@redhat.com
Subject: Re: [Workman-devel] cgroup: status-quo and userland efforts
Date: Mon, 8 Apr 2013 12:20:24 -0700	[thread overview]
Message-ID: <20130408192024.GL3021@htj.dyndns.org> (raw)
In-Reply-To: <20130408191105.GG28292@redhat.com>

Hey,

On Mon, Apr 08, 2013 at 03:11:05PM -0400, Vivek Goyal wrote:
> >  What if the program crashes?
> 
> I am not sure about this. I guess when applications comes back after crash,
> it can go through all the children cgroups and reclaim empty cgroups.

Fragile, right?  What are you arguing here?

> >  Wouldn't it make more sense to just have
> > a central arbitrator that everyone talks to?
> 
> May be. Just that in the past folks have not liked the idea of talking
> to central authority to figure out resource group of an object they are
> managing.

What we've been doing seems tragically broken to me, so I'm not sure
"people didn't use to do it that way" is a good point.

> >  What's the benefit of
> > distributing the responsiblities here?  It's not like we can put them
> > in different security domains.
> 
> To me it makes sense in a way, as these resources associated with the
> service is just one another property and there does not seem to be
> anything special about this property that it should be managed using
> a single centralized authority.
> 
> For example, one might want to say that maximum IO bandwidth for 
> virtual machine virt1 on disk sda should be 10MB/s. Now libvirt
> should be able to save it in virtual machine specific configuration
> easily and whenever virtual machine is started, create a children
> cgroup, set the limits as specified.

Yes, sure, libvirt can *request* whatever it seems appropriate to the
central authority, which will decide whether it'll be able to honor
the request and grant it if possible and allowed by policies in
effect.

> That would make sense. systemd had this conflict with cgconfig
> too. Problem is that systemd starts first and sets up everything. Now
> if there is a service which sets up cgroups, after systemd startup,
> it is already late.

Come on, that's not a difficult or fundamental problem.  Whatever the
central authority may be, systemd can use it to setup the initial
hierarchy or set up bare-bone hierarchy in compatible manner.  This
isn't that different from udev.

Thanks.

-- 
tejun

  reply	other threads:[~2013-04-08 19:20 UTC|newest]

Thread overview: 88+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-06  1:21 Tejun Heo
2013-04-08 13:46 ` Glauber Costa
2013-04-08 18:00   ` [Workman-devel] " Vivek Goyal
2013-04-08 18:26   ` Tejun Heo
2013-04-08 23:32   ` Lennart Poettering
2013-04-09  7:37     ` Glauber Costa
2013-04-09 19:11     ` Tejun Heo
2013-04-08 17:59 ` [Workman-devel] " Vivek Goyal
2013-04-08 18:16   ` Tejun Heo
2013-04-08 18:49     ` Tejun Heo
2013-04-08 19:11     ` Vivek Goyal
2013-04-08 19:20       ` Tejun Heo [this message]
2013-04-08 19:46         ` Vivek Goyal
2013-04-08 20:02           ` Tejun Heo
2013-04-09  9:50 ` Daniel P. Berrange
2013-04-09 19:38   ` Tejun Heo
2013-04-09 19:46     ` Tejun Heo
2013-04-09 21:04       ` Serge Hallyn
2013-04-09 21:11         ` Tejun Heo
2013-04-16 11:17 ` Li Zefan
2013-04-16 17:10   ` Tejun Heo
2013-04-17  1:29     ` Li Zefan
2013-04-22 21:26 ` Tim Hockin
2013-04-22 21:41   ` Tejun Heo
2013-04-22 22:33     ` Tim Hockin
2013-06-22 23:13       ` Tim Hockin
2013-06-25  0:01         ` Tejun Heo
2013-06-25  4:07           ` Tim Hockin
2013-06-26 21:20             ` Tejun Heo
2013-06-27  0:06               ` Tim Hockin
2013-06-26 23:14                 ` David Lang
2013-06-27  1:04                 ` Tejun Heo
2013-06-27  3:42                   ` Tim Hockin
2013-06-27 17:38                     ` Tejun Heo
2013-06-27 20:46                       ` Tim Hockin
2013-06-27 21:04                         ` Tejun Heo
2013-06-28 18:44                           ` Tim Hockin
2013-06-29 16:40                             ` Tejun Heo
2015-03-03 21:53                               ` Luke Leighton
2015-03-03 21:38                       ` Luke Leighton
2015-03-03 21:17                   ` Luke Leighton
2015-03-04  5:08                     ` David Lang
2015-03-04 11:27                       ` Luke Kenneth Casson Leighton
2015-03-04 20:08                         ` David Lang
2013-06-27  5:45               ` Mike Galbraith
2013-06-27 13:22                 ` Serge Hallyn
2013-06-27 15:29                   ` Tim Hockin
2013-06-27 16:18                     ` Serge Hallyn
2015-03-03 22:00                       ` Luke Leighton
2013-06-27 17:48                   ` Tejun Heo
2013-06-27 18:14                     ` Serge Hallyn
2013-06-27 18:45                       ` Tejun Heo
2013-06-27 18:51                         ` Serge Hallyn
2013-06-27 18:52                           ` Tejun Heo
2013-06-27 20:52                       ` Tim Hockin
2015-03-03 22:08                     ` Luke Leighton
2013-06-28  9:09                   ` [Workman-devel] " Daniel P. Berrange
2013-06-28 15:53                     ` Serge Hallyn
2013-06-28 18:58                       ` Tim Hockin
2015-03-03 22:20                       ` Luke Leighton
2013-06-27 18:01                 ` Tejun Heo
2013-06-28  3:46                   ` Mike Galbraith
2013-06-28  4:09                     ` Tejun Heo
2013-06-28  4:49                       ` Mike Galbraith
2013-06-28  5:01                         ` Tejun Heo
2013-06-28  6:00                           ` Mike Galbraith
2013-06-28 15:05                           ` Michal Hocko
2013-06-28 18:01                             ` [Workman-devel] " Vivek Goyal
2013-06-28 19:59                               ` Daniel P. Berrange
2013-06-28 22:40                                 ` Serge Hallyn
2013-06-28 22:43                                   ` Tejun Heo
2013-06-30 18:38                               ` Michal Hocko
2013-07-15 18:49                                 ` Vivek Goyal
2013-07-23 14:48                                   ` Michal Hocko
2013-06-28 18:30                             ` Tejun Heo
2013-06-28 18:53                             ` Tim Hockin
2013-06-29  1:48                               ` Lennart Poettering
2013-06-29  3:05                                 ` Tim Hockin
2013-06-30 19:39                                   ` Lennart Poettering
2013-07-01  6:06                                     ` Tim Hockin
2013-07-02 23:57                                     ` Thomas Gleixner
2013-07-03  0:44                                       ` Kay Sievers
2013-07-03  7:37                                         ` Borislav Petkov
2013-07-03  9:30                                         ` Thomas Gleixner
2013-07-09 23:12                                         ` Jiri Kosina
2013-07-03 17:11                                       ` James Bottomley
2013-06-28 19:18                   ` Andy Lutomirski
2013-06-28 19:36                     ` Serge Hallyn

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130408192024.GL3021@htj.dyndns.org \
    --to=tj@kernel.org \
    --cc=bsingharora@gmail.com \
    --cc=cgroups@vger.kernel.org \
    --cc=containers@lists.linux-foundation.org \
    --cc=dhaval.giani@gmail.com \
    --cc=kay.sievers@vrfy.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lizefan@huawei.com \
    --cc=lpoetter@redhat.com \
    --cc=vgoyal@redhat.com \
    --cc=workman-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®