Re: [PATCH 3/3] agp/intel: Destroy the scatterlist on allocation failure

2010-08-01 Thread Eric Anholt
On Sat, 24 Jul 2010 18:29:37 +0100, Chris Wilson wrote: > A side-effect of being able to use custom page allocations with the > sg_table is that it cannot reap the partially constructed scatterlist if > fails to allocate a page. So we need to call sg_free_table() ourselves > if sg_alloc_table() f

[PATCH 3/3] agp/intel: Destroy the scatterlist on allocation failure

2010-08-01 Thread Eric Anholt
On Sat, 24 Jul 2010 18:29:37 +0100, Chris Wilson wrote: > A side-effect of being able to use custom page allocations with the > sg_table is that it cannot reap the partially constructed scatterlist if > fails to allocate a page. So we need to call sg_free_table() ourselves > if sg_alloc_table() f

[PATCH 3/3] agp/intel: Destroy the scatterlist on allocation failure

2010-07-27 Thread Zhenyu Wang
On 2010.07.24 18:29:37 +0100, Chris Wilson wrote: > A side-effect of being able to use custom page allocations with the > sg_table is that it cannot reap the partially constructed scatterlist if > fails to allocate a page. So we need to call sg_free_table() ourselves > if sg_alloc_table() fails. W

[PATCH 3/3] agp/intel: Destroy the scatterlist on allocation failure

2010-07-27 Thread Chris Wilson
On Tue, 27 Jul 2010 10:00:07 +0800, Zhenyu Wang wrote: > On 2010.07.24 18:29:37 +0100, Chris Wilson wrote: > > A side-effect of being able to use custom page allocations with the > > sg_table is that it cannot reap the partially constructed scatterlist if > > fails to allocate a page. So we need

Re: [PATCH 3/3] agp/intel: Destroy the scatterlist on allocation failure

2010-07-27 Thread Chris Wilson
On Tue, 27 Jul 2010 10:00:07 +0800, Zhenyu Wang wrote: > On 2010.07.24 18:29:37 +0100, Chris Wilson wrote: > > A side-effect of being able to use custom page allocations with the > > sg_table is that it cannot reap the partially constructed scatterlist if > > fails to allocate a page. So we need t

Re: [PATCH 3/3] agp/intel: Destroy the scatterlist on allocation failure

2010-07-26 Thread Zhenyu Wang
On 2010.07.24 18:29:37 +0100, Chris Wilson wrote: > A side-effect of being able to use custom page allocations with the > sg_table is that it cannot reap the partially constructed scatterlist if > fails to allocate a page. So we need to call sg_free_table() ourselves > if sg_alloc_table() fails. W

[PATCH 3/3] agp/intel: Destroy the scatterlist on allocation failure

2010-07-24 Thread Chris Wilson
A side-effect of being able to use custom page allocations with the sg_table is that it cannot reap the partially constructed scatterlist if fails to allocate a page. So we need to call sg_free_table() ourselves if sg_alloc_table() fails. Signed-off-by: Chris Wilson Cc Dave Airlie --- drivers/c

[PATCH 3/3] agp/intel: Destroy the scatterlist on allocation failure

2010-07-24 Thread Chris Wilson
A side-effect of being able to use custom page allocations with the sg_table is that it cannot reap the partially constructed scatterlist if fails to allocate a page. So we need to call sg_free_table() ourselves if sg_alloc_table() fails. Signed-off-by: Chris Wilson Cc Dave Airlie --- drivers/c