Re: [Intel-gfx] [RFC 10/10] drm/i915/vm_bind: Fix vm->vm_bind_mutex and vm->mutex nesting

On Tue, Jul 05, 2022 at 10:40:56AM +0200, Thomas Hellström wrote: On Fri, 2022-07-01 at 15:50 -0700, Niranjana Vishwanathapura wrote: VM_BIND functionality maintain that vm->vm_bind_mutex will never be taken while holding vm->mutex. However, while closing 'vm', vma is destroyed while holding

Re: [Intel-gfx] [RFC 02/10] drm/i915/vm_bind: Bind and unbind mappings

On Thu, Jul 07, 2022 at 03:41:26AM +0200, Andi Shyti wrote: Hi, [...] > @@ -28,7 +28,6 @@ static u32 object_max_page_size(struct > intel_memory_region **placements, > max_page_size = max_t(u32, max_page_size, mr- > >min_page_size); > } >   > -   

Re: [Intel-gfx] [RFC 02/10] drm/i915/vm_bind: Bind and unbind mappings

On Wed, Jul 06, 2022 at 06:21:03PM +0200, Thomas Hellström wrote: On Fri, 2022-07-01 at 15:50 -0700, Niranjana Vishwanathapura wrote: Bind and unbind the mappings upon VM_BIND and VM_UNBIND calls. Signed-off-by: Niranjana Vishwanathapura Signed-off-by: Prathap Kumar Valsan ---  

[Intel-gfx] [PATCH] drm/i915/hdmi: Prune modes that require HDMI2.1 FRL

HDMI2.1 requires some higher resolution video modes to be enumerated only if HDMI2.1 Fixed Rate Link (FRL) is supported. Current platforms do not support FRL transmission so prune modes that require HDMI2.1 FRL. Signed-off-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_hdmi.c | 21

Re: [Intel-gfx] [RFC 01/10] drm/i915/vm_bind: Introduce VM_BIND ioctl

On Thu, Jul 07, 2022 at 03:18:15AM +0200, Andi Shyti wrote: Hi, [...] > +/* > + * VM_BIND feature version supported. > + * > + * The following versions of VM_BIND have been defined: > + * > + * 0: No VM_BIND support. > + * > + * 1: In VM_UNBIND calls, the UMD must specify the exact mappings >

Re: [Intel-gfx] [RFC 01/10] drm/i915/vm_bind: Introduce VM_BIND ioctl

On Tue, Jul 05, 2022 at 02:59:24AM -0700, Hellstrom, Thomas wrote: Hi, On Fri, 2022-07-01 at 15:50 -0700, Niranjana Vishwanathapura wrote: Add VM_BIND and VM_UNBIND ioctls to bind/unbind a section of an object at the specified GPU virtual addresses. Add I915_PARAM_VM_BIND_VERSION to indicate

[Intel-gfx] ✗ Fi.CI.BUILD: failure for i915: Introduce Meteorlake (rev2)

== Series Details == Series: i915: Introduce Meteorlake (rev2) URL : https://patchwork.freedesktop.org/series/106000/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/106000/revisions/2/mbox/ not applied Applying: drm/i915/mtl: Add MeteorLake

Re: [Intel-gfx] [PATCH 2/2] drm/i915/mtl: Add MeteorLake PCI IDs

> -Original Message- > From: Intel-gfx On Behalf Of > Radhakrishna Sripada > Sent: Thursday, July 7, 2022 6:01 AM > To: intel-gfx@lists.freedesktop.org > Cc: Nikula, Jani ; De Marchi, Lucas > ; Vivi, Rodrigo > Subject: [Intel-gfx] [PATCH 2/2] drm/i915/mtl: Add MeteorLake PCI IDs > >

Re: [Intel-gfx] [PATCH 1/2] drm/i915/mtl: Add MeteorLake platform info

> -Original Message- > From: Intel-gfx On Behalf Of > Radhakrishna Sripada > Sent: Thursday, July 7, 2022 6:01 AM > To: intel-gfx@lists.freedesktop.org > Cc: Nikula, Jani ; De Marchi, Lucas > ; Vivi, Rodrigo > Subject: [Intel-gfx] [PATCH 1/2] drm/i915/mtl: Add MeteorLake platform info

[Intel-gfx] [PULL] drm-intel-next

Hi Dave and Daniel, Here goes drm-intel-next-2022-07-06 that shall be our last one towards 5.20, containing only a few things: - Suspend fixes for Display (Jose) - Properly block D3Cold for now (Anshuman) - Eliminate PIPECONF RMWs from .color_commit()(Ville) - Display info clean-up (Ville) - Fix

Re: [Intel-gfx] [RFC 02/10] drm/i915/vm_bind: Bind and unbind mappings

Hi, [...] > > @@ -28,7 +28,6 @@ static u32 object_max_page_size(struct > > intel_memory_region **placements, > > max_page_size = max_t(u32, max_page_size, mr- > > >min_page_size); > > } > >   > > -   GEM_BUG_ON(!max_page_size); > > return max_page_size; > >  }

Re: [Intel-gfx] [RFC 01/10] drm/i915/vm_bind: Introduce VM_BIND ioctl

Hi, [...] > > +/* > > + * VM_BIND feature version supported. > > + * > > + * The following versions of VM_BIND have been defined: > > + * > > + * 0: No VM_BIND support. > > + * > > + * 1: In VM_UNBIND calls, the UMD must specify the exact mappings > > created > > + *    previously with VM_BIND,

[Intel-gfx] ✗ Fi.CI.BUILD: failure for i915: Introduce Meteorlake

== Series Details == Series: i915: Introduce Meteorlake URL : https://patchwork.freedesktop.org/series/106000/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/106000/revisions/1/mbox/ not applied Applying: drm/i915/mtl: Add MeteorLake platform

[Intel-gfx] [PATCH 2/2] drm/i915/mtl: Add MeteorLake PCI IDs

Add Meteorlake PCI IDs. Split into M, and P subplatforms. Bspec: 55420 Signed-off-by: Radhakrishna Sripada Signed-off-by: Matt Roper --- drivers/gpu/drm/i915/i915_drv.h | 4 drivers/gpu/drm/i915/i915_pci.c | 1 + drivers/gpu/drm/i915/intel_device_info.c | 14

[Intel-gfx] [PATCH 0/2] i915: Introduce Meteorlake

Meteorlake is a new client platform following RPL S. Meteorlake introduces version 14 for Display, version 13 Media and version 12.70 for Graphics. This is the basic platform definition series to introduce Meteorlake. Radhakrishna Sripada (2): drm/i915/mtl: Add MeteorLake platform info

[Intel-gfx] [PATCH 1/2] drm/i915/mtl: Add MeteorLake platform info

MTL has Xe_LPD+ display IP (version = 14), MTL graphics IP (version = 12.70), and Xe_LPM+ media IP (version = 13). Bspec: 55413 Bspec: 55416 Bspec: 55417 Bspec: 55418 Bspec: 55726 Bspec: 45544 Bspec: 65380 Cc: Matt Roper Signed-off-by: Radhakrishna Sripada --- drivers/gpu/drm/i915/i915_drv.h

[Intel-gfx] ✗ Fi.CI.BAT: failure for i915/perf: Disable OA sseu config param for non-gen11 platforms

== Series Details == Series: i915/perf: Disable OA sseu config param for non-gen11 platforms URL : https://patchwork.freedesktop.org/series/105998/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11854 -> Patchwork_105998v1

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for i915/perf: Disable OA sseu config param for non-gen11 platforms

== Series Details == Series: i915/perf: Disable OA sseu config param for non-gen11 platforms URL : https://patchwork.freedesktop.org/series/105998/ State : warning == Summary == Error: dim checkpatch failed ba9ea66b787d i915/perf: Disable OA sseu config param for non-gen11 platforms -:30:

Re: [Intel-gfx] [PATCH] drm/i915/gem: Really move i915_gem_context.link under ref protection

Hi, [...] > @@ -1265,10 +1264,15 @@ static void i915_gem_context_release_work(struct > work_struct *work) > struct i915_gem_context *ctx = container_of(work, typeof(*ctx), > release_work); > struct i915_address_space *vm; > +

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/selftests: Grab the runtime pm in shrink_thp

== Series Details == Series: drm/i915/selftests: Grab the runtime pm in shrink_thp URL : https://patchwork.freedesktop.org/series/105978/ State : success == Summary == CI Bug Log - changes from CI_DRM_11852_full -> Patchwork_105978v1_full

[Intel-gfx] [PATCH] i915/perf: Disable OA sseu config param for non-gen11 platforms

The global sseu config is applicable only to gen11 platforms where concurrent media, render and OA use cases may cause some subslices to be turned off and hence lose NOA configuration. Return ENODEV for non-gen11 platforms. Signed-off-by: Umesh Nerlige Ramappa ---

Re: [Intel-gfx] [PATCH 00/15] HuC loading for DG2

On 7/6/2022 12:29 PM, Ceraolo Spurio, Daniele wrote: On 7/6/2022 10:26 AM, Ye, Tony wrote: On 7/5/2022 4:30 PM, Ceraolo Spurio, Daniele wrote: On 6/15/2022 7:28 PM, Zhang, Carl wrote: On From: Ye, Tony Sent: Thursday, June 16, 2022 12:15 AM On 6/15/2022 3:13 AM, Tvrtko Ursulin

Re: [Intel-gfx] [PATCH 00/15] HuC loading for DG2

On 7/6/2022 10:26 AM, Ye, Tony wrote: On 7/5/2022 4:30 PM, Ceraolo Spurio, Daniele wrote: On 6/15/2022 7:28 PM, Zhang, Carl wrote: On From: Ye, Tony Sent: Thursday, June 16, 2022 12:15 AM On 6/15/2022 3:13 AM, Tvrtko Ursulin wrote: On 15/06/2022 00:15, Ye, Tony wrote: On 6/14/2022

[Intel-gfx] ✗ Fi.CI.IGT: failure for GSC support for XeHP SDV and DG2 platforms (rev5)

== Series Details == Series: GSC support for XeHP SDV and DG2 platforms (rev5) URL : https://patchwork.freedesktop.org/series/102339/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11850_full -> Patchwork_102339v5_full

Re: [Intel-gfx] [PATCH 00/15] HuC loading for DG2

On 7/5/2022 4:30 PM, Ceraolo Spurio, Daniele wrote: On 6/15/2022 7:28 PM, Zhang, Carl wrote: On From: Ye, Tony Sent: Thursday, June 16, 2022 12:15 AM On 6/15/2022 3:13 AM, Tvrtko Ursulin wrote: On 15/06/2022 00:15, Ye, Tony wrote: On 6/14/2022 8:30 AM, Ceraolo Spurio, Daniele wrote:

Re: [Intel-gfx] [PATCH v2 2/7] drm/i915/gem: Typecheck page lookups

On Wed, 6 Jul 2022 19:33:22 +0300 Gwan-gyeong Mun wrote: > On 7/5/22 5:35 PM, Mauro Carvalho Chehab wrote: > > On Tue, 5 Jul 2022 15:24:50 +0300 > > Gwan-gyeong Mun wrote: > > > >> From: Chris Wilson > >> > >> We need to check that we avoid integer overflows when looking up a page, > >>

Re: [Intel-gfx] [PATCH v2 1/7] drm: Move and add a few utility macros into drm util header

On Wed, 6 Jul 2022 18:04:20 +0300 Gwan-gyeong Mun wrote: > On 7/5/22 5:23 PM, Mauro Carvalho Chehab wrote: > > On Tue, 5 Jul 2022 15:24:49 +0300 > > Gwan-gyeong Mun wrote: > > > >> It moves overflows_type utility macro into drm util header from i915_utils > >> header. The overflows_type can

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Grab the runtime pm in shrink_thp

== Series Details == Series: drm/i915/selftests: Grab the runtime pm in shrink_thp URL : https://patchwork.freedesktop.org/series/105978/ State : success == Summary == CI Bug Log - changes from CI_DRM_11852 -> Patchwork_105978v1 Summary

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/display: Add debug print for scaler filter

== Series Details == Series: drm/i915/display: Add debug print for scaler filter URL : https://patchwork.freedesktop.org/series/105954/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11850_full -> Patchwork_105954v1_full

Re: [Intel-gfx] [PATCH v2 2/7] drm/i915/gem: Typecheck page lookups

On 7/5/22 5:35 PM, Mauro Carvalho Chehab wrote: On Tue, 5 Jul 2022 15:24:50 +0300 Gwan-gyeong Mun wrote: From: Chris Wilson We need to check that we avoid integer overflows when looking up a page, and so fix all the instances where we have mistakenly used a plain integer instead of a

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gem: Really move i915_gem_context.link under ref protection

== Series Details == Series: drm/i915/gem: Really move i915_gem_context.link under ref protection URL : https://patchwork.freedesktop.org/series/105975/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11852 -> Patchwork_105975v1

Re: [Intel-gfx] [RFC 10/10] drm/i915/vm_bind: Fix vm->vm_bind_mutex and vm->mutex nesting

On 2022-07-05 at 10:40:56 +0200, Thomas Hellström wrote: > On Fri, 2022-07-01 at 15:50 -0700, Niranjana Vishwanathapura wrote: > > VM_BIND functionality maintain that vm->vm_bind_mutex will never be > > taken > > while holding vm->mutex. > > However, while closing 'vm', vma is destroyed while

Re: [Intel-gfx] [RFC 02/10] drm/i915/vm_bind: Bind and unbind mappings

On Fri, 2022-07-01 at 15:50 -0700, Niranjana Vishwanathapura wrote: > Bind and unbind the mappings upon VM_BIND and VM_UNBIND calls. > > Signed-off-by: Niranjana Vishwanathapura > > Signed-off-by: Prathap Kumar Valsan > --- >  drivers/gpu/drm/i915/Makefile |   1 + >  

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gem: Really move i915_gem_context.link under ref protection

== Series Details == Series: drm/i915/gem: Really move i915_gem_context.link under ref protection URL : https://patchwork.freedesktop.org/series/105975/ State : warning == Summary == Error: dim checkpatch failed 475c5a4e6fa4 drm/i915/gem: Really move i915_gem_context.link under ref

Re: [Intel-gfx] [PATCH] drm/i915/selftests: Grab the runtime pm in shrink_thp

Hi, On Wed, Jul 06, 2022 at 04:47:38PM +0100, Matthew Auld wrote: > From: Chris Wilson > > Since we are not holding a wakeref, shrinking a bound object is not > guaranteed. > > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/6370 > Signed-off-by: Chris Wilson > Signed-off-by:

Re: [Intel-gfx] [PATCH] drm/i915/selftests: Grab the runtime pm in shrink_thp

On Wed, 6 Jul 2022 at 16:48, Matthew Auld wrote: > > From: Chris Wilson > > Since we are not holding a wakeref, shrinking a bound object is not > guaranteed. > > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/6370 > Signed-off-by: Chris Wilson Reviewed-by: Matthew Auld

[Intel-gfx] [PATCH] drm/i915/selftests: Grab the runtime pm in shrink_thp

From: Chris Wilson Since we are not holding a wakeref, shrinking a bound object is not guaranteed. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/6370 Signed-off-by: Chris Wilson Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gem/selftests/huge_pages.c | 7 ++- 1 file

[Intel-gfx] [PATCH] drm/i915/gem: Really move i915_gem_context.link under ref protection

From: Chris Wilson i915_perf assumes that it can use the i915_gem_context reference to protect its i915->gem.contexts.list iteration. However, this requires that we do not remove the context from the list until after we drop the final reference and release the struct. If, as currently, we remove

Re: [Intel-gfx] [PATCH v2 1/7] drm: Move and add a few utility macros into drm util header

On 7/5/22 5:23 PM, Mauro Carvalho Chehab wrote: On Tue, 5 Jul 2022 15:24:49 +0300 Gwan-gyeong Mun wrote: It moves overflows_type utility macro into drm util header from i915_utils header. The overflows_type can be used to catch the truncation between data types. And it adds

[Intel-gfx] ✓ Fi.CI.BAT: success for GSC support for XeHP SDV and DG2 platforms (rev5)

== Series Details == Series: GSC support for XeHP SDV and DG2 platforms (rev5) URL : https://patchwork.freedesktop.org/series/102339/ State : success == Summary == CI Bug Log - changes from CI_DRM_11850 -> Patchwork_102339v5 Summary

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for GSC support for XeHP SDV and DG2 platforms (rev5)

== Series Details == Series: GSC support for XeHP SDV and DG2 platforms (rev5) URL : https://patchwork.freedesktop.org/series/102339/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for GSC support for XeHP SDV and DG2 platforms (rev5)

== Series Details == Series: GSC support for XeHP SDV and DG2 platforms (rev5) URL : https://patchwork.freedesktop.org/series/102339/ State : warning == Summary == Error: dim checkpatch failed 063c66ed39c4 drm/i915/gsc: skip irq initialization if using polling e199641a3bfd drm/i915/gsc: add

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: Add debug print for scaler filter

== Series Details == Series: drm/i915/display: Add debug print for scaler filter URL : https://patchwork.freedesktop.org/series/105954/ State : success == Summary == CI Bug Log - changes from CI_DRM_11850 -> Patchwork_105954v1 Summary

[Intel-gfx] [PATCH v5 14/14] HAX: drm/i915: force INTEL_MEI_GSC on for CI

From: Daniele Ceraolo Spurio After the new config option is merged we'll enable it by default in the CI config, but for now just force it on via the i915 Kconfig so we can get pre-merge CI results for it. Signed-off-by: Daniele Ceraolo Spurio Signed-off-by: Alexander Usyskin ---

[Intel-gfx] [PATCH v5 13/14] drm/i915/gsc: allocate extended operational memory in LMEM

From: Tomas Winkler GSC requires more operational memory than available on chip. Reserve 4M of LMEM for GSC operation. The memory is provided to the GSC as struct resource to the auxiliary data of the child device. Cc: Alan Previn Signed-off-by: Tomas Winkler Signed-off-by: Daniele Ceraolo

[Intel-gfx] [PATCH v5 12/14] mei: debugfs: add pxp mode to devstate in debugfs

From: Tomas Winkler Add pxp mode devstate to debugfs to monitor pxp state machine progress. This is useful to debug issues in scenarios in which the pxp state needs to be re-initialized, like during power transitions such as suspend/resume. With this debugfs the state could be monitored to

[Intel-gfx] [PATCH v5 11/14] mei: gsc: add transition to PXP mode in resume flow

From: Vitaly Lubart Added transition to PXP mode in resume flow. CC: Daniele Ceraolo Spurio Signed-off-by: Vitaly Lubart Signed-off-by: Tomas Winkler Signed-off-by: Alexander Usyskin --- drivers/misc/mei/gsc-me.c | 11 +++ 1 file changed, 11 insertions(+) diff --git

[Intel-gfx] [PATCH v5 10/14] mei: gsc: setup gsc extended operational memory

From: Tomas Winkler 1. Retrieve extended operational memory physical pointers from the auxiliary device info. 2. Setup memory registers. 3. Notify firmware that the memory is ready by sending the memory ready command. 4. Disable PXP device if GSC is not in PXP mode. CC: Daniele Ceraolo

[Intel-gfx] [PATCH v5 09/14] mei: mkhi: add memory ready command

From: Tomas Winkler Add GSC memory ready command. The command indicates to the firmware that extend operation memory was setup and the firmware may enter PXP mode. CC: Daniele Ceraolo Spurio Signed-off-by: Tomas Winkler Signed-off-by: Alexander Usyskin --- drivers/misc/mei/mkhi.h | 14

[Intel-gfx] [PATCH v5 08/14] mei: bus: export common mkhi definitions into a separate header

From: Vitaly Lubart Exported common mkhi definitions from bus-fixup.c into a separate header file mkhi.h for other driver usage. Signed-off-by: Vitaly Lubart Signed-off-by: Tomas Winkler Signed-off-by: Alexander Usyskin --- drivers/misc/mei/bus-fixup.c | 32 ++---

[Intel-gfx] [PATCH v5 07/14] mei: extend timeouts on slow devices.

Parametrize operational timeouts in order to support slow firmware on some graphic devices. Signed-off-by: Alexander Usyskin Signed-off-by: Tomas Winkler --- drivers/misc/mei/bus-fixup.c | 3 +-- drivers/misc/mei/client.c| 14 +++--- drivers/misc/mei/gsc-me.c| 2 +-

[Intel-gfx] [PATCH v5 06/14] mei: gsc: wait for reset thread on stop

Wait for reset work to complete before initiating stop reset flow sequence. Signed-off-by: Alexander Usyskin Signed-off-by: Tomas Winkler --- drivers/misc/mei/init.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/misc/mei/init.c b/drivers/misc/mei/init.c index

[Intel-gfx] [PATCH v5 05/14] mei: gsc: use polling instead of interrupts

From: Tomas Winkler A work-around for a HW issue in XEHPSDV that manifests itself when SW reads a gsc register when gsc is sending an interrupt. The work-around is to disable interrupts and to use polling instead. Cc: James Ausmus Signed-off-by: Vitaly Lubart Signed-off-by: Tomas Winkler

[Intel-gfx] [PATCH v5 04/14] drm/i915/gsc: add GSC XeHP SDV platform definition

Define GSC on XeHP SDV (Intel(R) dGPU without display) XeHP SDV uses the same hardware settings as DG1, but uses polling instead of interrupts and runs the firmware in slow pace due to hardware limitations. Signed-off-by: Vitaly Lubart Signed-off-by: Tomas Winkler Signed-off-by: Alexander

[Intel-gfx] [PATCH v5 03/14] drm/i915/gsc: add slow_fw flag to the gsc device definition

Add slow_fw flag to the gsc device definition and pass it to mei auxiliary device. Signed-off-by: Alexander Usyskin Reviewed-by: Daniele Ceraolo Spurio --- drivers/gpu/drm/i915/gt/intel_gsc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/i915/gt/intel_gsc.c

[Intel-gfx] [PATCH v5 02/14] drm/i915/gsc: add slow_fw flag to the mei auxiliary device

Add slow_fw flag to the mei auxiliary device info to inform the mei driver about slow underlying firmware. Such firmware will require to use larger operation timeouts. Signed-off-by: Alexander Usyskin Signed-off-by: Tomas Winkler Reviewed-by: Daniele Ceraolo Spurio --- include/linux/mei_aux.h

[Intel-gfx] [PATCH v5 01/14] drm/i915/gsc: skip irq initialization if using polling

From: Vitaly Lubart Some platforms require the host to poll on the GSC registers instead of relaying on the interrupts. For those platforms, irq initialization should be skipped Signed-off-by: Vitaly Lubart Signed-off-by: Tomas Winkler Signed-off-by: Alexander Usyskin Reviewed-by: Daniele

[Intel-gfx] [PATCH v5 00/14] GSC support for XeHP SDV and DG2 platforms

Add GSC support for XeHP SDV and DG2 platforms. The series includes changes for the mei driver: - add ability to use polling instead of interrupts - add ability to use extended timeouts - setup extended operational memory for GSC The series includes changes for the i915 driver: - allocate

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/display: stop HPD workers before display driver unregister (rev2)

On 01.07.2022 10:51, Patchwork wrote: *Patch Details* *Series:* drm/i915/display: stop HPD workers before display driver unregister (rev2) *URL:* https://patchwork.freedesktop.org/series/105557/ *State:*failure *Details:*

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/display: Add debug print for scaler filter

== Series Details == Series: drm/i915/display: Add debug print for scaler filter URL : https://patchwork.freedesktop.org/series/105954/ State : warning == Summary == Error: dim checkpatch failed 3531bcb837be drm/i915/display: Add debug print for scaler filter -:25: WARNING:LONG_LINE: line

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915: Add lmem_bar_size modparam

On 7/5/2022 1:54 PM, priyanka.dandam...@intel.com wrote: From: Priyanka Dandamudi For testing purposes, support forcing the lmem_bar_size through a new modparam. In CI we only have a limited number of configurations for DG2, but we still need to be reasonably sure we get a usable device

Re: [Intel-gfx] [PATCH v3 1/2] drm/i915/gt: Serialize GRDOM access between multiple engine resets

Hi Mauro and Chris, On Mon, Jul 04, 2022 at 09:09:28AM +0100, Mauro Carvalho Chehab wrote: > From: Chris Wilson > > Don't allow two engines to be reset in parallel, as they would both > try to select a reset bit (and send requests to common registers) > and wait on that register, at the same

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/gt: Serialize TLB invalidates with GT resets

Hi Mauro and Chris, On Mon, Jul 04, 2022 at 09:09:29AM +0100, Mauro Carvalho Chehab wrote: > From: Chris Wilson > > Avoid trying to invalidate the TLB in the middle of performing an > engine reset, as this may result in the reset timing out. Currently, > the TLB invalidate is only serialised by

[Intel-gfx] [PATCH] drm/i915/display: Add debug print for scaler filter

Add debug print statement to print scaler filter property value. Since property can be set as either default or integer scaler; its good if we can get debug print for the same in dmesg log. Cc: Juha-Pekka Heikkila Signed-off-by: Swati Sharma ---

Re: [Intel-gfx] [PATCH i-g-t v2] tests/i915_query: tweak the unallocated tracking test

Reviewed-by: Nirmoy Das On 7/6/2022 11:38 AM, Matthew Auld wrote: Make sure we always print the seed. Also make sure we use the correct helpers for the igt_assert when comparing values of type uint64. v2:(Nirmoy) - Fix broken conversion with s/lt/lte/ in a couple places. Signed-off-by:

[Intel-gfx] [PATCH i-g-t v2] tests/i915_query: tweak the unallocated tracking test

Make sure we always print the seed. Also make sure we use the correct helpers for the igt_assert when comparing values of type uint64. v2:(Nirmoy) - Fix broken conversion with s/lt/lte/ in a couple places. Signed-off-by: Matthew Auld Cc: Nirmoy Das --- tests/i915/i915_query.c | 36

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_query: tweak the unallocated tracking test

On 7/5/2022 12:15 PM, Matthew Auld wrote: Make sure we always print the seed. Also make sure we use the correct helpers for the igt_assert when comparing values of type uint64. Signed-off-by: Matthew Auld Cc: Nirmoy Das --- tests/i915/i915_query.c | 36 +++-

Re: [Intel-gfx] [RFT][PATCH v2 9/9] vfio: Replace phys_pfn with pages for vfio_pin_pages()

Looks good: Reviewed-by: Christoph Hellwig

Re: [Intel-gfx] [RFT][PATCH v2 7/9] vfio: Rename user_iova of vfio_dma_rw()

Looks good: Reviewed-by: Christoph Hellwig

Re: [Intel-gfx] [PATCH i-g-t] tests/gem_exec_capture: adjust the pi subtest

Reviewed-by: Nirmoy Das On 7/5/2022 12:49 PM, Matthew Auld wrote: This was broken by the kernel change: 71b1669ea9bd ("drm/i915/uapi: tweak error capture on recoverable contexts"). Update the test to use non-recoverable context if needed. Closes:

Re: [Intel-gfx] [RFT][PATCH v2 4/9] vfio: Pass in starting IOVA to vfio_pin/unpin_pages API

> - vfio_unpin_pages(>matrix_mdev->vdev, >saved_pfn, 1); > + vfio_unpin_pages(>matrix_mdev->vdev, q->saved_pfn << > PAGE_SHIFT, 1); Overly long line here. Otherwise looks good: Reviewed-by: Christoph Hellwig

Re: [Intel-gfx] [RFT][PATCH v2 1/9] vfio: Make vfio_unpin_pages() return void

> +void vfio_unpin_pages(struct vfio_device *device, unsigned long *user_pfn, > + int npage) > { > struct vfio_container *container; > struct vfio_iommu_driver *driver; > - int ret; > > - if (!user_pfn || !npage || !vfio_assert_device_open(device)) > -