All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Wilson <chris@chris-wilson.co.uk>
To: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>,
	intel-gfx@lists.freedesktop.org
Subject: Re: Direct execlists submission
Date: Mon, 14 May 2018 15:54:34 +0100	[thread overview]
Message-ID: <152630967443.18532.14081887920198521982@mail.alporthouse.com> (raw)
In-Reply-To: <152629354524.18532.16724655744185362384@mail.alporthouse.com>

Quoting Chris Wilson (2018-05-14 11:25:45)
> Quoting Tvrtko Ursulin (2018-05-14 11:11:54)
> > 
> > On 14/05/2018 10:37, Chris Wilson wrote:
> > > Continuing the discussion with the latest refactorings, however I ran
> > > some tests to measure the impact on system (!i915) latency,
> > > using igt/benchmarks/gem_syslatency -t 120
> > > 
> > > drm-tip:
> > >       latency mean=1.211us max=10us (no load)
> > >       latency mean=2.611us max=83us (i915)
> > > 
> > >          latency mean=1.720us max=833us (no load, bg writeout)
> > >          latency mean=3.294us max=607us (i915, bg writeout)
> > > 
> > > this series:
> > >          latency mean=1.280us max=15us (no load)
> > >          latency mean=9.688us max=1271us (i915)
> > > 
> > >          latency mean=1.712us max=1026us (no load, bg writeout)
> > >          latency mean=14.347us max=489850us (i915, bg writeout)
> > > 
> > > That certainly takes the shine off directly using the tasklet for
> > > submission from the irq handler. Being selfish, I still think we can't
> > > allow the GPU to stall waiting for ksoftirqd, but at the same time we
> > > need to solve the latency issues introduced elsewhere.

> > But now with this data it looks like a quite significant regression even 
> > if it fixes the rthog test case. So I don't know where this leaves us. :I

The ginormous writeout latency is a bit inconsistent. What remains
consistent is that with direct submission, latency with i915 loaded
remains around 10us versus 1us unloaded. The difference for direct
submission is that the latency and i915 throughput remains the same even
with bg writeout (with ksoftirqd there is a very noticeable fall off).

I've still no clue as what causes the latency to spike so badly with bg
write out.

As for the 10us latency, I think that is just i915 hogging the cpu to
handle the flow onto GPU, so the "fairness" issue that ksoftirqd seeks
to prevent (rather than a side-effect of irqoff, as the irqoff periods
do not change that much due to execlists_dequeue being the brunt of the
work).

I no longer think the sky is falling, or that these bad results are
anything more than something we can and should improve. I think the
protection from ksoftirqd is definitely more important for us.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2018-05-14 14:54 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-14  9:37 Direct execlists submission Chris Wilson
2018-05-14  9:37 ` [PATCH 01/10] drm/i915: Mark up nested spinlocks Chris Wilson
2018-05-14 10:06   ` Tvrtko Ursulin
2018-05-14  9:37 ` [PATCH 02/10] drm/i915: Remove tasklet flush before disable Chris Wilson
2018-05-14  9:37 ` [PATCH 03/10] drm/i915: Wrap tasklet_struct for abuse Chris Wilson
2018-05-14  9:37 ` [PATCH 04/10] drm/i915: Only sync tasklets once for recursive reset preparation Chris Wilson
2018-05-14  9:37 ` [PATCH 05/10] drm/i915/execlists: Direct submit onto idle engines Chris Wilson
2018-05-14  9:37 ` [PATCH 06/10] drm/i915/execlists: Direct submission from irq handler Chris Wilson
2018-05-14  9:37 ` [PATCH 07/10] drm/i915: Rearrange gen8_cs_irq_handler Chris Wilson
2018-05-14  9:37 ` [PATCH 08/10] drm/i915: Remove USES_GUC_SUBMISSION() pointer chasing from gen8_cs_irq_handler Chris Wilson
2018-05-14 10:27   ` Tvrtko Ursulin
2018-05-14 11:45     ` Chris Wilson
2018-05-14  9:37 ` [PATCH 09/10] drm/i915: Speed up idle detection by kicking the tasklets Chris Wilson
2018-05-14  9:37 ` [PATCH 10/10] drm/i915: Detect if we missed kicking the execlists tasklet Chris Wilson
2018-05-14  9:42 ` ✗ Fi.CI.CHECKPATCH: warning for series starting with [01/10] drm/i915: Mark up nested spinlocks Patchwork
2018-05-14  9:45 ` ✗ Fi.CI.SPARSE: " Patchwork
2018-05-14  9:59 ` ✓ Fi.CI.BAT: success " Patchwork
2018-05-14 10:11 ` Direct execlists submission Tvrtko Ursulin
2018-05-14 10:25   ` Chris Wilson
2018-05-14 14:54     ` Chris Wilson [this message]
2018-05-14 12:45 ` ✓ Fi.CI.IGT: success for series starting with [01/10] drm/i915: Mark up nested spinlocks Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=152630967443.18532.14081887920198521982@mail.alporthouse.com \
    --to=chris@chris-wilson.co.uk \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=tvrtko.ursulin@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.