On Wed, Nov 16, 2016 at 09:32:56PM +0800, Min He wrote:
> For a singl_port_submission context, it can only be submitted to port 0,
> and there shouldn't be any other context in port 1 at the same time. This
> is required by GVT-g context to have an opportunity to save/restore some
> non-hw context render registers.
> This patch is to implement the correct logic in execlists_dequeue.
>
> V2: optimized code by following Chris's advice, and added more comments to
> explain the patch.
>
> Signed-off-by: Min He <min.he@xxxxxxxxx>
> Signed-off-by: Zhenyu Wang <zhenyuw@xxxxxxxxxxxxxxx>
If you wrote it, and sent it to the list, at what point did Zhenyu Wang
handle it?
That would be more valuable to us if that was either a reviewed-by or a
tested-by.
> drivers/gpu/drm/i915/intel_lrc.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
> index f50feaa..4f692a8 100644
> --- a/drivers/gpu/drm/i915/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/intel_lrc.c
> @@ -499,7 +499,8 @@ static void execlists_dequeue(struct intel_engine_cs *engine)
> * context (even though a different request) to
> * the second port.
> */
> - if (ctx_single_port_submission(cursor->ctx))
> + if (ctx_single_port_submission(cursor->ctx)
> + || ctx_single_port_submission(last->ctx))
if (ctx_single_port_submission(last->ctx) ||
ctx_single_port_submission(cursor->ctx))
CodingStyle: || goes at the end of the line, new line aligned to
bracket.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
