Re: [Intel-gfx] [PATCH v6 0/4] Apply Wa_16018031267 / Wa_16018063123

[Andrzej Hajda <andrzej.hajda@intel.com>]

On 26.10.2023 20:36, Andrzej Hajda wrote:
> Hi all,
> 
> This the series from Jonathan:
> [PATCH v12 0/4] Apply Wa_16018031267 / Wa_16018063123
> taken over by me.
> 
> This patchset requires IGT changes fixing handling gtt sizes
> not being power of 2:
> https://patchwork.freedesktop.org/series/125640/
> 
> Changes in this version are described in the patches, in short:
> v2:
> - use real memory as WABB destination,
> - address CI compains - do not decrease vm.total,
> - minor reordering.
> v3:
> - fixed typos,
> - removed spare defs,
> - added tags
> v4:
> - removed NULL PTE patch,
> - separate selftest to separate patch,
> - use BB only on BCS0
> v5:
> - fixed reserved memory allocation
> v6:
> - decresase vm->total by reserved size
> 
> Regards
> Andrzej
> 
> Andrzej Hajda (1):
>    drm/i915: Reserve some kernel space per vm
> 
> Jonathan Cavitt (3):
>    drm/i915: Enable NULL PTE support for vm scratch
>    drm/i915: Add WABB blit for Wa_16018031267 / Wa_16018063123
>    drm/i915: Set copy engine arbitration for Wa_16018031267 /
>      Wa_16018063123
> 
> .../drm/i915/gem/selftests/i915_gem_context.c |   6 ++
>   drivers/gpu/drm/i915/gt/gen8_ppgtt.c          |  41 +++++++
>   drivers/gpu/drm/i915/gt/intel_engine_regs.h   |   6 ++
>   drivers/gpu/drm/i915/gt/intel_gt.h            |   4 +
>   drivers/gpu/drm/i915/gt/intel_gt_types.h      |   2 +
>   drivers/gpu/drm/i915/gt/intel_gtt.h           |   2 +
>   drivers/gpu/drm/i915/gt/intel_lrc.c           | 100 +++++++++++++++++-
>   drivers/gpu/drm/i915/gt/intel_workarounds.c   |   5 +
>   drivers/gpu/drm/i915/gt/selftest_lrc.c        |  65 ++++++++----
>   drivers/gpu/drm/i915/i915_drv.h               |   2 +
>   drivers/gpu/drm/i915/i915_pci.c               |   2 +
>   drivers/gpu/drm/i915/intel_device_info.h      |   1 +
>   12 files changed, 215 insertions(+), 21 deletions(-)
> 
> Test-with: 20231026-gtt_size_fix-v1-2-882d0e0022ec@intel.com
> ---
> - Link to v3: https://lore.kernel.org/r/20231023-wabb-v3-0-1a4fbc632440@intel.com
> - Link to v4: https://lore.kernel.org/r/20231023-wabb-v4-0-f75dec962b7d@intel.com
> - Link to v5: https://lore.kernel.org/r/20231025-wabb-v5-0-398e42309e9d@intel.com
> 
> ---
> Andrzej Hajda (3):
>        drm/i915: Reserve some kernel space per vm
>        drm/i915: Add WABB blit for Wa_16018031267 / Wa_16018063123
>        drm/i915/gt: add selftest to exercise WABB
> 
> Jonathan Cavitt (1):
>        drm/i915: Set copy engine arbitration for Wa_16018031267 / Wa_16018063123
> 
>   drivers/gpu/drm/i915/gt/gen8_ppgtt.c        |  43 ++++++++++++
>   drivers/gpu/drm/i915/gt/intel_engine_regs.h |   6 ++
>   drivers/gpu/drm/i915/gt/intel_gt.h          |   4 ++
>   drivers/gpu/drm/i915/gt/intel_gtt.h         |   4 ++
>   drivers/gpu/drm/i915/gt/intel_lrc.c         | 100 +++++++++++++++++++++++++++-
>   drivers/gpu/drm/i915/gt/intel_workarounds.c |   5 ++
>   drivers/gpu/drm/i915/gt/selftest_lrc.c      |  65 +++++++++++++-----
>   7 files changed, 206 insertions(+), 21 deletions(-)
> ---
> base-commit: 201c8a7bd1f3f415920a2df4b8a8817e973f42fe
> change-id: 20231020-wabb-bbe9324a69a8
> 
> Best regards,


Thanks all for reviews, pushed.

Regards
Andrzej