[Intel-gfx] [PATCH 1/3] drm: Balance error path for GEM handle allocation

2016-01-04 Thread Ville Syrjälä
On Mon, Jan 04, 2016 at 05:24:11PM +0200, Ville Syrjälä wrote: > On Mon, Jan 04, 2016 at 10:10:59AM +, Chris Wilson wrote: > > The current error path for failure when establishing a handle for a GEM > > object is unbalance, e.g. we call object_close() without calling first > > object_open().

[PATCH 1/3] drm: Balance error path for GEM handle allocation

2016-01-04 Thread Ville Syrjälä
On Mon, Jan 04, 2016 at 10:18:50AM +, Chris Wilson wrote: > On Mon, Jan 04, 2016 at 10:10:59AM +, Chris Wilson wrote: > > The current error path for failure when establishing a handle for a GEM > > object is unbalance, e.g. we call object_close() without calling first > > object_open().

[PATCH 1/3] drm: Balance error path for GEM handle allocation

2016-01-04 Thread Ville Syrjälä
On Mon, Jan 04, 2016 at 10:10:59AM +, Chris Wilson wrote: > The current error path for failure when establishing a handle for a GEM > object is unbalance, e.g. we call object_close() without calling first > object_open(). Use the typical onion structure to only undo what has > been set up

[PATCH 1/3] drm: Balance error path for GEM handle allocation

2016-01-04 Thread Chris Wilson
On Mon, Jan 04, 2016 at 05:33:45PM +0200, Ville Syrjälä wrote: > On Mon, Jan 04, 2016 at 10:18:50AM +, Chris Wilson wrote: > > On Mon, Jan 04, 2016 at 10:10:59AM +, Chris Wilson wrote: > > > The current error path for failure when establishing a handle for a GEM > > > object is

[PATCH 1/3] drm: Balance error path for GEM handle allocation

2016-01-04 Thread Chris Wilson
On Mon, Jan 04, 2016 at 10:10:59AM +, Chris Wilson wrote: > The current error path for failure when establishing a handle for a GEM > object is unbalance, e.g. we call object_close() without calling first > object_open(). Use the typical onion structure to only undo what has > been set up

[PATCH 1/3] drm: Balance error path for GEM handle allocation

2016-01-04 Thread Chris Wilson
The current error path for failure when establishing a handle for a GEM object is unbalance, e.g. we call object_close() without calling first object_open(). Use the typical onion structure to only undo what has been set up prior to the error. Signed-off-by: Chris Wilson ---