Skip to content

Commit

Permalink
drm/i915/gvt: move workload destroy out of execlist complete
Browse files Browse the repository at this point in the history
To let execlist.c only handle execlist handling and keep other
workload cleanup function in scheduler.c to align with other
workload specific handling there. This doesn't change current
code behavior.

Reviewed-by: Yan Zhao <yan.y.zhao@intel.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20200506094318.105604-1-zhenyuw@linux.intel.com
  • Loading branch information
Zhenyu Wang committed May 8, 2020
1 parent fb55c73 commit 40dcee1
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 0 additions & 2 deletions drivers/gpu/drm/i915/gvt/execlist.c
Original file line number Diff line number Diff line change
@@ -424,8 +424,6 @@ static int complete_execlist_workload(struct intel_vgpu_workload *workload)

ret = emulate_execlist_ctx_schedule_out(execlist, &workload->ctx_desc);
out:
intel_vgpu_unpin_mm(workload->shadow_mm);
intel_vgpu_destroy_workload(workload);
return ret;
}

3 changes: 3 additions & 0 deletions drivers/gpu/drm/i915/gvt/scheduler.c
Original file line number Diff line number Diff line change
@@ -1014,6 +1014,9 @@ static void complete_current_workload(struct intel_gvt *gvt, int ring_id)

workload->complete(workload);

intel_vgpu_unpin_mm(workload->shadow_mm);
intel_vgpu_destroy_workload(workload);

atomic_dec(&s->running_workload_num);
wake_up(&scheduler->workload_complete_wq);

0 comments on commit 40dcee1

Please sign in to comment.