Re: [PATCH] drm/drm_vma_manager.c: Remove useless goto statement

2017-11-01 Thread Chris Wilson
Quoting Liviu Dudau (2017-11-01 14:20:53) > On Wed, Nov 01, 2017 at 02:11:44PM +, Chris Wilson wrote: > > Quoting Liviu Dudau (2017-11-01 14:04:24) > > > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > > > interval-tree in drm_mm") removed a line in drm_vma_offset_add() functi

Re: [PATCH] drm/drm_vma_manager.c: Remove useless goto statement

2017-11-01 Thread Liviu Dudau
On Wed, Nov 01, 2017 at 02:11:44PM +, Chris Wilson wrote: > Quoting Liviu Dudau (2017-11-01 14:04:24) > > 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 d

Re: [PATCH] drm/drm_vma_manager.c: Remove useless goto statement

2017-11-01 Thread Liviu Dudau
On Wed, Nov 01, 2017 at 02:11:44PM +, Chris Wilson wrote: > Quoting Liviu Dudau (2017-11-01 14:04:24) > > 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 d

Re: [PATCH] drm/drm_vma_manager.c: Remove useless goto statement

2017-11-01 Thread Chris Wilson
Quoting Liviu Dudau (2017-11-01 14:04:24) > 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. Remove them. > > F

[PATCH] 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. Remove them. Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to