Re: [PATCH v4 8/9] devcg: refactor dev_exception_clean()

2013-01-31 Thread Aristeu Rozanski
On Wed, Jan 30, 2013 at 12:50:38PM -0800, Tejun Heo wrote: > On Wed, Jan 30, 2013 at 12:49 PM, Aristeu Rozanski > that's not > intentional. thanks for catching this > > > > Tejun, you want me to resubmit the whole series or just the next patch > > (where I was supposed to move that chunk)? > > If

Re: [PATCH v4 8/9] devcg: refactor dev_exception_clean()

2013-01-30 Thread Li Zefan
On 2013/1/31 4:50, Tejun Heo wrote: > Hello, > > On Wed, Jan 30, 2013 at 12:49 PM, Aristeu Rozanski > that's not > intentional. thanks for catching this >> >> Tejun, you want me to resubmit the whole series or just the next patch >> (where I was supposed to move that chunk)? > > If it doesn't aff

Re: [PATCH v4 8/9] devcg: refactor dev_exception_clean()

2013-01-30 Thread Tejun Heo
Hello, On Wed, Jan 30, 2013 at 12:49 PM, Aristeu Rozanski > that's not intentional. thanks for catching this > > Tejun, you want me to resubmit the whole series or just the next patch > (where I was supposed to move that chunk)? If it doesn't affect the next patch, just posting an updated version

Re: [PATCH v4 8/9] devcg: refactor dev_exception_clean()

2013-01-30 Thread Aristeu Rozanski
On Wed, Jan 30, 2013 at 08:47:30PM +, Serge E. Hallyn wrote: > Quoting a...@redhat.com (a...@redhat.com): > > - > > - if (rc) > > - return rc; > > Was this intentional? > > I see that you next add > > rc = propagate_behavior(devcgroup); > >

Re: [PATCH v4 8/9] devcg: refactor dev_exception_clean()

2013-01-30 Thread Serge E. Hallyn
Quoting a...@redhat.com (a...@redhat.com): > This patch is in preparation for hierarchy support. > > This patch doesn't introduce any functional changes. > > Acked-by: Tejun Heo > Cc: Tejun Heo > Cc: Serge Hallyn > Signed-off-by: Aristeu Rozanski > > --- > security/device_cgroup.c | 34 ++