Re: [Intel-gfx] [PATCH 1/5] drm/i915/guc: Don't capture Gen8 regs on Xe devices

2023-04-28 Thread John Harrison
On 4/26/2023 14:14, Teres Alexis, Alan Previn wrote: On Wed, 2023-04-26 at 10:23 -0700, Harrison, John C wrote: On 4/25/2023 10:55, Teres Alexis, Alan Previn wrote: On Thu, 2023-04-06 at 15:26 -0700, Harrison, John C wrote: From: John Harrison A pair of pre-Xe registers were being included i

Re: [Intel-gfx] [PATCH 1/5] drm/i915/guc: Don't capture Gen8 regs on Xe devices

2023-04-26 Thread Teres Alexis, Alan Previn
On Wed, 2023-04-26 at 10:23 -0700, Harrison, John C wrote: > On 4/25/2023 10:55, Teres Alexis, Alan Previn wrote: > > On Thu, 2023-04-06 at 15:26 -0700, Harrison, John C wrote: > > > From: John Harrison > > > > > > A pair of pre-Xe registers were being included in the Xe capture list. > > > GuC w

Re: [Intel-gfx] [PATCH 1/5] drm/i915/guc: Don't capture Gen8 regs on Xe devices

2023-04-26 Thread John Harrison
On 4/25/2023 10:55, Teres Alexis, Alan Previn wrote: On Thu, 2023-04-06 at 15:26 -0700, Harrison, John C wrote: From: John Harrison A pair of pre-Xe registers were being included in the Xe capture list. GuC was rejecting those as being invalid and logging errors about them. So, stop doing it.

Re: [Intel-gfx] [PATCH 1/5] drm/i915/guc: Don't capture Gen8 regs on Xe devices

2023-04-26 Thread John Harrison
On 4/25/2023 10:55, Teres Alexis, Alan Previn wrote: On Thu, 2023-04-06 at 15:26 -0700, Harrison, John C wrote: From: John Harrison A pair of pre-Xe registers were being included in the Xe capture list. GuC was rejecting those as being invalid and logging errors about them. So, stop doing it.

Re: [Intel-gfx] [PATCH 1/5] drm/i915/guc: Don't capture Gen8 regs on Xe devices

2023-04-25 Thread Teres Alexis, Alan Previn
On Thu, 2023-04-06 at 15:26 -0700, Harrison, John C wrote: > From: John Harrison > > A pair of pre-Xe registers were being included in the Xe capture list. > GuC was rejecting those as being invalid and logging errors about > them. So, stop doing it. > alan:snip > #define COMMON_GEN9BASE_GLOBAL

[Intel-gfx] [PATCH 1/5] drm/i915/guc: Don't capture Gen8 regs on Xe devices

2023-04-06 Thread John . C . Harrison
From: John Harrison A pair of pre-Xe registers were being included in the Xe capture list. GuC was rejecting those as being invalid and logging errors about them. So, stop doing it. Signed-off-by: John Harrison Fixes: dce2bd542337 ("drm/i915/guc: Add Gen9 registers for GuC error state capture.