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-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 it doesn't

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

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

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

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

2013-01-30 Thread aris
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 -- 1 file changed, 16

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

2013-01-30 Thread aris
This patch is in preparation for hierarchy support. This patch doesn't introduce any functional changes. Acked-by: Tejun Heo t...@kernel.org Cc: Tejun Heo t...@kernel.org Cc: Serge Hallyn serge.hal...@canonical.com Signed-off-by: Aristeu Rozanski a...@redhat.com --- security/device_cgroup.c |

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 t...@kernel.org Cc: Tejun Heo t...@kernel.org Cc: Serge Hallyn serge.hal...@canonical.com Signed-off-by: Aristeu

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); right here,

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 of

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 affect the next