Chris Wilson <[email protected]> writes: > [ text/plain ] > Move the single line to the callsite as the name is now misleading, and > the purpose is solely to add the request to the execution queue. > > Signed-off-by: Chris Wilson <[email protected]>
Reviewed-by: Mika Kuoppala <[email protected]> > --- > drivers/gpu/drm/i915/i915_gem_execbuffer.c | 9 +-------- > 1 file changed, 1 insertion(+), 8 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c > b/drivers/gpu/drm/i915/i915_gem_execbuffer.c > index 1b19a36adedc..40937a09855d 100644 > --- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c > +++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c > @@ -1177,13 +1177,6 @@ i915_gem_execbuffer_move_to_active(struct list_head > *vmas, > } > } > > -static void > -i915_gem_execbuffer_retire_commands(struct i915_execbuffer_params *params) > -{ > - /* Add a breadcrumb for the completion of the batch buffer */ > - __i915_add_request(params->request, params->batch_obj, true); > -} > - > static int > i915_reset_gen7_sol_offsets(struct drm_i915_gem_request *req) > { > @@ -1677,7 +1670,7 @@ i915_gem_do_execbuffer(struct drm_device *dev, void > *data, > > ret = execbuf_submit(params, args, &eb->vmas); > err_request: > - i915_gem_execbuffer_retire_commands(params); > + __i915_add_request(params->request, params->batch_obj, ret == 0); > > err_batch_unpin: > /* > -- > 2.8.1 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
