Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-06 Thread Benjamin LaHaise
On Fri, Dec 06, 2013 at 10:34:13AM -0500, Dave Jones wrote: > On Fri, Dec 06, 2013 at 10:26:54AM -0500, Benjamin LaHaise wrote: > > On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: > > > Clean up the aio ring file in the fail path of aio_setup_ring > > > and ioctx_alloc. And maybe it

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-06 Thread Dave Jones
On Fri, Dec 06, 2013 at 10:26:54AM -0500, Benjamin LaHaise wrote: > On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: > > Clean up the aio ring file in the fail path of aio_setup_ring > > and ioctx_alloc. And maybe it can fix the GPF issue reported by > > Dave Jones: > >

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-06 Thread Benjamin LaHaise
On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: > Clean up the aio ring file in the fail path of aio_setup_ring > and ioctx_alloc. And maybe it can fix the GPF issue reported by > Dave Jones: > https://lkml.org/lkml/2013/11/25/898 Good catch. I've applied this to my aio-next tree. It

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-06 Thread Benjamin LaHaise
On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: Clean up the aio ring file in the fail path of aio_setup_ring and ioctx_alloc. And maybe it can fix the GPF issue reported by Dave Jones: https://lkml.org/lkml/2013/11/25/898 Good catch. I've applied this to my aio-next tree. It does

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-06 Thread Dave Jones
On Fri, Dec 06, 2013 at 10:26:54AM -0500, Benjamin LaHaise wrote: On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: Clean up the aio ring file in the fail path of aio_setup_ring and ioctx_alloc. And maybe it can fix the GPF issue reported by Dave Jones:

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-06 Thread Benjamin LaHaise
On Fri, Dec 06, 2013 at 10:34:13AM -0500, Dave Jones wrote: On Fri, Dec 06, 2013 at 10:26:54AM -0500, Benjamin LaHaise wrote: On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: Clean up the aio ring file in the fail path of aio_setup_ring and ioctx_alloc. And maybe it can fix

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-04 Thread Gu Zheng
On 12/05/2013 09:14 AM, Gu Zheng wrote: > Hi Dave, > > On 12/04/2013 10:54 PM, Dave Jones wrote: > >> On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: >> > Clean up the aio ring file in the fail path of aio_setup_ring >> > and ioctx_alloc. And maybe it can fix the GPF issue reported

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-04 Thread Gu Zheng
Hi Dave, On 12/04/2013 10:54 PM, Dave Jones wrote: > On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: > > Clean up the aio ring file in the fail path of aio_setup_ring > > and ioctx_alloc. And maybe it can fix the GPF issue reported by > > Dave Jones: > >

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-04 Thread Dave Jones
On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: > Clean up the aio ring file in the fail path of aio_setup_ring > and ioctx_alloc. And maybe it can fix the GPF issue reported by > Dave Jones: > https://lkml.org/lkml/2013/11/25/898 > > > Signed-off-by: Gu Zheng > --- >

[PATCH] aio: clean up aio ring in the fail path

2013-12-04 Thread Gu Zheng
Clean up the aio ring file in the fail path of aio_setup_ring and ioctx_alloc. And maybe it can fix the GPF issue reported by Dave Jones: https://lkml.org/lkml/2013/11/25/898 Signed-off-by: Gu Zheng --- fs/aio.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git

[PATCH] aio: clean up aio ring in the fail path

2013-12-04 Thread Gu Zheng
Clean up the aio ring file in the fail path of aio_setup_ring and ioctx_alloc. And maybe it can fix the GPF issue reported by Dave Jones: https://lkml.org/lkml/2013/11/25/898 Signed-off-by: Gu Zheng guz.f...@cn.fujitsu.com --- fs/aio.c |8 ++-- 1 files changed, 6 insertions(+), 2

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-04 Thread Dave Jones
On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: Clean up the aio ring file in the fail path of aio_setup_ring and ioctx_alloc. And maybe it can fix the GPF issue reported by Dave Jones: https://lkml.org/lkml/2013/11/25/898 Signed-off-by: Gu Zheng guz.f...@cn.fujitsu.com

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-04 Thread Gu Zheng
Hi Dave, On 12/04/2013 10:54 PM, Dave Jones wrote: On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: Clean up the aio ring file in the fail path of aio_setup_ring and ioctx_alloc. And maybe it can fix the GPF issue reported by Dave Jones: https://lkml.org/lkml/2013/11/25/898

Re: [PATCH] aio: clean up aio ring in the fail path

2013-12-04 Thread Gu Zheng
On 12/05/2013 09:14 AM, Gu Zheng wrote: Hi Dave, On 12/04/2013 10:54 PM, Dave Jones wrote: On Wed, Dec 04, 2013 at 06:19:06PM +0800, Gu Zheng wrote: Clean up the aio ring file in the fail path of aio_setup_ring and ioctx_alloc. And maybe it can fix the GPF issue reported by Dave