Re: [Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-31 Thread Umesh Nerlige Ramappa
On Fri, Jan 31, 2020 at 01:00:54PM +0200, Lionel Landwerlin wrote: On 31/01/2020 01:54, Umesh Nerlige Ramappa wrote: On Mon, Jan 27, 2020 at 11:16:32AM +0200, Lionel Landwerlin wrote: [snip] --- a/drivers/gpu/drm/i915/i915_perf.c +++ b/drivers/gpu/drm/i915/i915_perf.c @@ -1323,7 +1323,12 @@

Re: [Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-31 Thread Lionel Landwerlin
On 31/01/2020 01:54, Umesh Nerlige Ramappa wrote: On Mon, Jan 27, 2020 at 11:16:32AM +0200, Lionel Landwerlin wrote: [snip] --- a/drivers/gpu/drm/i915/i915_perf.c +++ b/drivers/gpu/drm/i915/i915_perf.c @@ -1323,7 +1323,12 @@ static int oa_get_render_ctx_id(struct i915_perf_stream *stream)   

Re: [Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-30 Thread Umesh Nerlige Ramappa
On Mon, Jan 27, 2020 at 11:16:32AM +0200, Lionel Landwerlin wrote: [snip] --- a/drivers/gpu/drm/i915/i915_perf.c +++ b/drivers/gpu/drm/i915/i915_perf.c @@ -1323,7 +1323,12 @@ static int oa_get_render_ctx_id(struct i915_perf_stream *stream) case 12: {

Re: [Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-27 Thread Lionel Landwerlin
On 27/01/2020 07:30, Umesh Nerlige Ramappa wrote: On Sat, Jan 25, 2020 at 03:37:38AM +0200, Lionel Landwerlin wrote: On 24/01/2020 03:37, Umesh Nerlige Ramappa wrote: Engine context pinned in perf OA was set to same context id as the idle context. Set the context id to an unused value. Clear

Re: [Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-26 Thread Umesh Nerlige Ramappa
On Sat, Jan 25, 2020 at 03:37:38AM +0200, Lionel Landwerlin wrote: On 24/01/2020 03:37, Umesh Nerlige Ramappa wrote: Engine context pinned in perf OA was set to same context id as the idle context. Set the context id to an unused value. Clear the sw context id field in lrc descriptor before

Re: [Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-25 Thread Chris Wilson
Quoting Lionel Landwerlin (2020-01-25 15:02:21) > On 25/01/2020 03:37, Lionel Landwerlin wrote: > > On 24/01/2020 03:37, Umesh Nerlige Ramappa wrote: > >> Engine context pinned in perf OA was set to same context id as > >> the idle context. Set the context id to an unused value. > >> > >> Clear

Re: [Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-25 Thread Lionel Landwerlin
On 25/01/2020 03:37, Lionel Landwerlin wrote: On 24/01/2020 03:37, Umesh Nerlige Ramappa wrote: Engine context pinned in perf OA was set to same context id as the idle context. Set the context id to an unused value. Clear the sw context id field in lrc descriptor before ORing with ce->tag

Re: [Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-24 Thread Lionel Landwerlin
On 24/01/2020 03:37, Umesh Nerlige Ramappa wrote: Engine context pinned in perf OA was set to same context id as the idle context. Set the context id to an unused value. Clear the sw context id field in lrc descriptor before ORing with ce->tag (Chris) Fixes:

[Intel-gfx] [PATCH] drm/i915/perf: Fix OA context id overlap with idle context id

2020-01-23 Thread Umesh Nerlige Ramappa
Engine context pinned in perf OA was set to same context id as the idle context. Set the context id to an unused value. Clear the sw context id field in lrc descriptor before ORing with ce->tag (Chris) Fixes: https://gitlab.freedesktop.org/drm/intel/issues/756 Signed-off-by: Umesh Nerlige