Re: [Intel-gfx] [PATCH v2] drm/drm_vma_manager.c: Remove useless goto statement

2017-11-02 Thread Daniel Vetter
On Wed, Nov 01, 2017 at 02:47:34PM +, Chris Wilson wrote: > Quoting Liviu Dudau (2017-11-01 14:44:58) > > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > > interval-tree in drm_mm") removed a line in drm_vma_offset_add() function > > that > > makes checking the result of call

[Intel-gfx] [PATCH v2] drm/drm_vma_manager.c: Remove useless goto statement

2017-11-01 Thread Liviu Dudau
Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") removed a line in drm_vma_offset_add() function that makes checking the result of calling drm_mm_insert_node() and the goto call redundant. Rework the function (as suggested by Chris Wilson) to eliminate the n

Re: [Intel-gfx] [PATCH v2] drm/drm_vma_manager.c: Remove useless goto statement

2017-11-01 Thread Chris Wilson
Quoting Liviu Dudau (2017-11-01 14:44:58) > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > interval-tree in drm_mm") removed a line in drm_vma_offset_add() function that > makes checking the result of calling drm_mm_insert_node() and the goto > call redundant. Rework the function