* [Intel-gfx] [PATCH i-g-t] i915/perf_pmu: Update inter-engine semaphore detection
@ 2020-05-26 14:17 Chris Wilson
2020-05-26 15:58 ` Tvrtko Ursulin
0 siblings, 1 reply; 3+ messages in thread
From: Chris Wilson @ 2020-05-26 14:17 UTC (permalink / raw)
To: intel-gfx; +Cc: igt-dev, Chris Wilson
The kernel no longer uses semaphores between engines, unless it can do
so by preempting them with timeslices. Update the semaphore-busy to only
run when we expect semaphore usage, i.e. not on bdw/bsw.
Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/1939
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
tests/perf_pmu.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c
index e54a0d47e..e80f730cf 100644
--- a/tests/perf_pmu.c
+++ b/tests/perf_pmu.c
@@ -822,6 +822,7 @@ sema_busy(int gem_fd,
int fd;
igt_require(gem_scheduler_has_semaphores(gem_fd));
+ igt_require(gem_scheduler_has_preemption(gem_fd));
fd = open_group(gem_fd,
I915_PMU_ENGINE_SEMA(e->class, e->instance), -1);
--
2.27.0.rc0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Intel-gfx] [PATCH i-g-t] i915/perf_pmu: Update inter-engine semaphore detection
2020-05-26 14:17 [Intel-gfx] [PATCH i-g-t] i915/perf_pmu: Update inter-engine semaphore detection Chris Wilson
@ 2020-05-26 15:58 ` Tvrtko Ursulin
2020-05-26 16:05 ` Chris Wilson
0 siblings, 1 reply; 3+ messages in thread
From: Tvrtko Ursulin @ 2020-05-26 15:58 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev
On 26/05/2020 15:17, Chris Wilson wrote:
> The kernel no longer uses semaphores between engines, unless it can do
> so by preempting them with timeslices. Update the semaphore-busy to only
> run when we expect semaphore usage, i.e. not on bdw/bsw.
>
> Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/1939
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
> tests/perf_pmu.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c
> index e54a0d47e..e80f730cf 100644
> --- a/tests/perf_pmu.c
> +++ b/tests/perf_pmu.c
> @@ -822,6 +822,7 @@ sema_busy(int gem_fd,
> int fd;
>
> igt_require(gem_scheduler_has_semaphores(gem_fd));
> + igt_require(gem_scheduler_has_preemption(gem_fd));
>
> fd = open_group(gem_fd,
> I915_PMU_ENGINE_SEMA(e->class, e->instance), -1);
>
Test with perhaps too intimate knowledge of i915..
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Intel-gfx] [PATCH i-g-t] i915/perf_pmu: Update inter-engine semaphore detection
2020-05-26 15:58 ` Tvrtko Ursulin
@ 2020-05-26 16:05 ` Chris Wilson
0 siblings, 0 replies; 3+ messages in thread
From: Chris Wilson @ 2020-05-26 16:05 UTC (permalink / raw)
To: Tvrtko Ursulin, intel-gfx; +Cc: igt-dev
Quoting Tvrtko Ursulin (2020-05-26 16:58:05)
>
> On 26/05/2020 15:17, Chris Wilson wrote:
> > The kernel no longer uses semaphores between engines, unless it can do
> > so by preempting them with timeslices. Update the semaphore-busy to only
> > run when we expect semaphore usage, i.e. not on bdw/bsw.
> >
> > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/1939
> > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> > ---
> > tests/perf_pmu.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c
> > index e54a0d47e..e80f730cf 100644
> > --- a/tests/perf_pmu.c
> > +++ b/tests/perf_pmu.c
> > @@ -822,6 +822,7 @@ sema_busy(int gem_fd,
> > int fd;
> >
> > igt_require(gem_scheduler_has_semaphores(gem_fd));
> > + igt_require(gem_scheduler_has_preemption(gem_fd));
> >
> > fd = open_group(gem_fd,
> > I915_PMU_ENGINE_SEMA(e->class, e->instance), -1);
> >
>
> Test with perhaps too intimate knowledge of i915..
iirc, part of the intent of the test was to understand how the kernel's
choice of inter-engine sync would show up in the pmu. The challenge we
have is in declaring the expected outcomes, that requires intimate
knowledge of what the kernel might decide to do.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-05-26 16:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-26 14:17 [Intel-gfx] [PATCH i-g-t] i915/perf_pmu: Update inter-engine semaphore detection Chris Wilson
2020-05-26 15:58 ` Tvrtko Ursulin
2020-05-26 16:05 ` Chris Wilson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).