Re: [PATCH] drm/i915: Skip engine serialisation for no-op seqno reset

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes:

> If the engine's seqno is already at our target seqno (most likely it
> hasn't been used since the last reset), we can skip serialising the
> engine and leave it as is.
>
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/i915_request.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_request.c b/drivers/gpu/drm/i915/i915_request.c
> index 71107540581d..ca95ab2f4cfa 100644
> --- a/drivers/gpu/drm/i915/i915_request.c
> +++ b/drivers/gpu/drm/i915/i915_request.c
> @@ -136,6 +136,9 @@ static int reset_all_global_seqno(struct drm_i915_private *i915, u32 seqno)
>  			  intel_engine_get_seqno(engine),
>  			  seqno);
>  
> +		if (seqno == engine->timeline.seqno)
> +			continue;
> +

Patch looks valid. Only concern is possibility of hiding
the idle assertion deeper. But as that is for icl
and debugfs stomping, I am in favour.

Reviewed-by: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx>

>  		kthread_park(engine->breadcrumbs.signaler);
>  
>  		if (!i915_seqno_passed(seqno, engine->timeline.seqno)) {
> -- 
> 2.20.0.rc1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux