Re: cls_cgroup: Store classid in struct sock

Neil Horman
2010-05-24T11:10:06Z

On Mon, May 24, 2010 at 12:14:29AM -0700, David Miller wrote:
> From: Herbert Xu <herbert@gondor.apana.org.au>
> Date: Mon, 24 May 2010 17:07:43 +1000
>
> > On Sun, May 23, 2010 at 11:55:57PM -0700, David Miller wrote:
> >>
> >> Probably you only tested the build with cgroups enabled?
> >
> > Indeed, that does seem to be the problem here. It turns out
> > that their struct is only declared when CONFIG_CGROUPS is defined,
> > how annoying.
> >
> > Oh well I guess I'll follow their example :)
> >
> > cls_cgroup: Store classid in struct sock
>
> Thanks for fixing this up, applied.
>

Excuse all my noise from before, Herberts patch works fine. Apparently the
problem was mine. QEMU creates several threads during startup, but adding the
processid to a cgroup doesn't cause its child threads to get added
automatically. Doing this by hand causes this to work.

Thanks
Neil