Commit 17f298cf authored by Chris Wilson's avatar Chris Wilson

drm/i915: Move setting of request->batch into its single callsite

request->batch_obj is only set by execbuffer for the convenience of
debugging hangs. By moving that operation to the callsite, we can
simplify all other callers and future patches. We also move the
complications of reference handling of the request->batch_obj next to
where the active tracking is set up for the request.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1470832906-13972-2-git-send-email-chris@chris-wilson.co.uk
parent 737aac24
...@@ -1702,6 +1702,14 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data, ...@@ -1702,6 +1702,14 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data,
goto err_batch_unpin; goto err_batch_unpin;
} }
/* Whilst this request exists, batch_obj will be on the
* active_list, and so will hold the active reference. Only when this
* request is retired will the the batch_obj be moved onto the
* inactive_list and lose its active reference. Hence we do not need
* to explicitly hold another reference here.
*/
params->request->batch_obj = params->batch->obj;
ret = i915_gem_request_add_to_client(params->request, file); ret = i915_gem_request_add_to_client(params->request, file);
if (ret) if (ret)
goto err_request; goto err_request;
...@@ -1720,7 +1728,7 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data, ...@@ -1720,7 +1728,7 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data,
ret = execbuf_submit(params, args, &eb->vmas); ret = execbuf_submit(params, args, &eb->vmas);
err_request: err_request:
__i915_add_request(params->request, params->batch->obj, ret == 0); __i915_add_request(params->request, ret == 0);
err_batch_unpin: err_batch_unpin:
/* /*
......
...@@ -461,9 +461,7 @@ static void i915_gem_mark_busy(const struct intel_engine_cs *engine) ...@@ -461,9 +461,7 @@ static void i915_gem_mark_busy(const struct intel_engine_cs *engine)
* request is not being tracked for completion but the work itself is * request is not being tracked for completion but the work itself is
* going to happen on the hardware. This would be a Bad Thing(tm). * going to happen on the hardware. This would be a Bad Thing(tm).
*/ */
void __i915_add_request(struct drm_i915_gem_request *request, void __i915_add_request(struct drm_i915_gem_request *request, bool flush_caches)
struct drm_i915_gem_object *obj,
bool flush_caches)
{ {
struct intel_engine_cs *engine; struct intel_engine_cs *engine;
struct intel_ring *ring; struct intel_ring *ring;
...@@ -504,14 +502,6 @@ void __i915_add_request(struct drm_i915_gem_request *request, ...@@ -504,14 +502,6 @@ void __i915_add_request(struct drm_i915_gem_request *request,
request->head = request_start; request->head = request_start;
/* Whilst this request exists, batch_obj will be on the
* active_list, and so will hold the active reference. Only when this
* request is retired will the the batch_obj be moved onto the
* inactive_list and lose its active reference. Hence we do not need
* to explicitly hold another reference here.
*/
request->batch_obj = obj;
/* Seal the request and mark it as pending execution. Note that /* Seal the request and mark it as pending execution. Note that
* we may inspect this state, without holding any locks, during * we may inspect this state, without holding any locks, during
* hangcheck. Hence we apply the barrier to ensure that we do not * hangcheck. Hence we apply the barrier to ensure that we do not
......
...@@ -225,13 +225,11 @@ static inline void i915_gem_request_assign(struct drm_i915_gem_request **pdst, ...@@ -225,13 +225,11 @@ static inline void i915_gem_request_assign(struct drm_i915_gem_request **pdst,
*pdst = src; *pdst = src;
} }
void __i915_add_request(struct drm_i915_gem_request *req, void __i915_add_request(struct drm_i915_gem_request *req, bool flush_caches);
struct drm_i915_gem_object *batch_obj,
bool flush_caches);
#define i915_add_request(req) \ #define i915_add_request(req) \
__i915_add_request(req, NULL, true) __i915_add_request(req, true)
#define i915_add_request_no_flush(req) \ #define i915_add_request_no_flush(req) \
__i915_add_request(req, NULL, false) __i915_add_request(req, false)
struct intel_rps_client; struct intel_rps_client;
#define NO_WAITBOOST ERR_PTR(-1) #define NO_WAITBOOST ERR_PTR(-1)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment