Re: [PATCH 09/11] cgroups: Add a task counter subsystem

From: Kirill A. Shutemov
Date: Fri Sep 30 2011 - 17:03:39 EST


On Fri, Sep 30, 2011 at 02:54:46PM +0200, Frederic Weisbecker wrote:
> On Sun, Sep 18, 2011 at 11:27:10PM +0300, Kirill A. Shutemov wrote:
> > On Tue, Sep 13, 2011 at 01:11:29AM +0200, Frederic Weisbecker wrote:
> > > +static struct cftype files[] = {
> > > + {
> > > + .name = "limit",
> > > + .read_u64 = task_counter_read_u64,
> > > + .write_u64 = task_counter_write_u64,
> > > + .private = RES_LIMIT,
> > > + },
> > > +
> > > + {
> > > + .name = "usage",
> > > + .read_u64 = task_counter_read_u64,
> >
> > It would be nice to implement .register_event/.unregister_event as well.
>
> It adds several complications. I prefer to wait for someone requesting
> that feature before doing it.

It may be a good replacement for release_agent. Or not?

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