[PATCH 04/16] drm/mgag200: Don't unreference when handle creation failed

2013-07-17 Thread Egbert Eich
Chris Wilson writes: > On Wed, Jul 17, 2013 at 03:07:17PM +0200, Egbert Eich wrote: > > drm_gem_handle_create() should not have referenced an object when it's > > creation has failed for some reason. > > > > Signed-off-by: Egbert Eich > > Nak. The unreference here is to free the locally

[PATCH 04/16] drm/mgag200: Don't unreference when handle creation failed

2013-07-17 Thread Egbert Eich
drm_gem_handle_create() should not have referenced an object when it's creation has failed for some reason. Signed-off-by: Egbert Eich --- drivers/gpu/drm/mgag200/mgag200_main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c

[PATCH 04/16] drm/mgag200: Don't unreference when handle creation failed

2013-07-17 Thread Chris Wilson
On Wed, Jul 17, 2013 at 03:07:17PM +0200, Egbert Eich wrote: > drm_gem_handle_create() should not have referenced an object when it's > creation has failed for some reason. > > Signed-off-by: Egbert Eich Nak. The unreference here is to free the locally created resource before returning the

[PATCH 04/16] drm/mgag200: Don't unreference when handle creation failed

2013-07-17 Thread Egbert Eich
drm_gem_handle_create() should not have referenced an object when it's creation has failed for some reason. Signed-off-by: Egbert Eich e...@suse.com --- drivers/gpu/drm/mgag200/mgag200_main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 04/16] drm/mgag200: Don't unreference when handle creation failed

2013-07-17 Thread Chris Wilson
On Wed, Jul 17, 2013 at 03:07:17PM +0200, Egbert Eich wrote: drm_gem_handle_create() should not have referenced an object when it's creation has failed for some reason. Signed-off-by: Egbert Eich e...@suse.com Nak. The unreference here is to free the locally created resource before returning

Re: [PATCH 04/16] drm/mgag200: Don't unreference when handle creation failed

2013-07-17 Thread Egbert Eich
Chris Wilson writes: On Wed, Jul 17, 2013 at 03:07:17PM +0200, Egbert Eich wrote: drm_gem_handle_create() should not have referenced an object when it's creation has failed for some reason. Signed-off-by: Egbert Eich e...@suse.com Nak. The unreference here is to free the