[Intel-gfx] [PATCH] drm/i915/psr: Add continuous full frame bit together with single

2022-11-28 Thread Jouni Högander
Currently we are observing occasionally display flickering or complete freeze. This is narrowed down to be caused by single full frame update (SFF). SFF bit after it's written gets cleared by HW in subsequent vblank i.e. when the update is sent to the panel. SFF bit is required to be written toget

[Intel-gfx] [PATCH v8 13/22] drm/i915/vm_bind: Update i915_vma_verify_bind_complete()

2022-11-28 Thread Niranjana Vishwanathapura
Ensure i915_vma_verify_bind_complete() handles case where bind is not initiated. Also make it non static, add documentation and move it out of CONFIG_DRM_I915_DEBUG_GEM. v2: Fix fence leak Reviewed-by: Matthew Auld Signed-off-by: Niranjana Vishwanathapura Signed-off-by: Andi Shyti --- drivers

[Intel-gfx] [PATCH v8 16/22] drm/i915/vm_bind: userptr dma-resv changes

2022-11-28 Thread Niranjana Vishwanathapura
For persistent (vm_bind) vmas of userptr BOs, handle the user page pinning by using the i915_gem_object_userptr_submit_init() /done() functions v2: Do not double add vma to vm->userptr_invalidated_list v3: Initialize vma->userptr_invalidated_link Reviewed-by: Matthew Auld Signed-off-by: Niranjan

[Intel-gfx] [PATCH v8 17/22] drm/i915/vm_bind: Limit vm_bind mode to non-recoverable contexts

2022-11-28 Thread Niranjana Vishwanathapura
Only support vm_bind mode with non-recoverable contexts. With new vm_bind mode with eb3 submission path, we need not support older recoverable contexts. Reviewed-by: Matthew Auld Signed-off-by: Niranjana Vishwanathapura --- drivers/gpu/drm/i915/gem/i915_gem_context.c | 6 ++ 1 file changed,

[Intel-gfx] [PATCH v8 07/22] drm/i915/vm_bind: Add support to handle object evictions

2022-11-28 Thread Niranjana Vishwanathapura
Support eviction by maintaining a list of evicted persistent vmas for rebinding during next submission. Ensure the list do not include persistent vmas that are being purged. v2: Remove unused I915_VMA_PURGED definition. v3: Properly handle __i915_vma_unbind_async() case. Reviewed-by: Matthew Auld

[Intel-gfx] [PATCH v8 10/22] drm/i915/vm_bind: Abstract out common execbuf functions

2022-11-28 Thread Niranjana Vishwanathapura
The new execbuf3 ioctl path and the legacy execbuf ioctl paths have many common functionalities. Abstract out the common execbuf functionalities into a separate file where possible, thus allowing code sharing. v2: Use drm_dbg instead of DRM_DEBUG Reviewed-by: Andi Shyti Reviewed-by: Matthew Auld

[Intel-gfx] [PATCH v8 21/22] drm/i915/vm_bind: Properly build persistent map sg table

2022-11-28 Thread Niranjana Vishwanathapura
Properly build the sg table for persistent mapping which can be partial map of the underlying object. Ensure the sg pages are properly set for page backed regions. The dump capture support requires this for page backed regions. Signed-off-by: Niranjana Vishwanathapura --- drivers/gpu/drm/i915/i9

[Intel-gfx] [PATCH v8 08/22] drm/i915/vm_bind: Support persistent vma activeness tracking

2022-11-28 Thread Niranjana Vishwanathapura
Do not use i915_vma activeness tracking for persistent vmas. As persistent vmas are part of working set for each execbuf submission on that address space (VM), a persistent vma is active if the VM active. As vm->root_obj->base.resv will be updated for each submission on that VM, it correctly repre

[Intel-gfx] [PATCH v8 06/22] drm/i915/vm_bind: Support for VM private BOs

2022-11-28 Thread Niranjana Vishwanathapura
Each VM creates a root_obj and shares it with all of its private objects to use it as dma_resv object. This has a performance advantage as it requires a single dma_resv object update for all private BOs vs list of dma_resv objects update for shared BOs, in the execbuf path. VM private BOs can be o

[Intel-gfx] [PATCH v8 02/22] drm/i915/vm_bind: Add __i915_sw_fence_await_reservation()

2022-11-28 Thread Niranjana Vishwanathapura
Add function __i915_sw_fence_await_reservation() for asynchronous wait on a dma-resv object with specified dma_resv_usage. This is required for async vma unbind with vm_bind. Reviewed-by: Matthew Auld Signed-off-by: Niranjana Vishwanathapura --- drivers/gpu/drm/i915/i915_sw_fence.c | 28 +++

[Intel-gfx] [PATCH v8 04/22] drm/i915/vm_bind: Add support to create persistent vma

2022-11-28 Thread Niranjana Vishwanathapura
Add i915_vma_instance_persistent() to create persistent vmas. Persistent vmas will use i915_gtt_view to support partial binding. vma_lookup is tied to segment of the object instead of section of VA space. Hence, it do not support aliasing. ie., multiple mappings (at different VA) point to the same

[Intel-gfx] [PATCH v8 05/22] drm/i915/vm_bind: Implement bind and unbind of object

2022-11-28 Thread Niranjana Vishwanathapura
Add uapi and implement support for bind and unbind of an object at the specified GPU virtual addresses. The vm_bind mode is not supported in legacy execbuf2 ioctl. It will be supported only in the newer execbuf3 ioctl. v2: On older platforms ctx->vm is not set, check for it. In vm_bind call,

[Intel-gfx] [PATCH v8 20/22] drm/i915/vm_bind: Async vm_unbind support

2022-11-28 Thread Niranjana Vishwanathapura
Asynchronously unbind the vma upon vm_unbind call. Fall back to synchronous unbind if backend doesn't support async unbind or if async unbind fails. No need for vm_unbind out fence support as i915 will internally handle all sequencing and user need not try to sequence any operation with the unbind

[Intel-gfx] [PATCH v8 19/22] drm/i915/vm_bind: Render VM_BIND documentation

2022-11-28 Thread Niranjana Vishwanathapura
Update i915 documentation to include VM_BIND changes and render all VM_BIND related documentation. Reviewed-by: Matthew Auld Signed-off-by: Niranjana Vishwanathapura --- Documentation/gpu/i915.rst | 78 -- 1 file changed, 59 insertions(+), 19 deletions(-) di

[Intel-gfx] [PATCH v8 11/22] drm/i915/vm_bind: Use common execbuf functions in execbuf path

2022-11-28 Thread Niranjana Vishwanathapura
Update the execbuf path to use common execbuf functions to reduce code duplication with the newer execbuf3 path. Reviewed-by: Matthew Auld Signed-off-by: Niranjana Vishwanathapura --- .../gpu/drm/i915/gem/i915_gem_execbuffer.c| 513 ++ 1 file changed, 39 insertions(+), 474 d

[Intel-gfx] [PATCH v8 22/22] drm/i915/vm_bind: Support capture of persistent mappings

2022-11-28 Thread Niranjana Vishwanathapura
Support dump capture of persistent mappings upon user request. Signed-off-by: Brian Welty Signed-off-by: Niranjana Vishwanathapura --- .../drm/i915/gem/i915_gem_vm_bind_object.c| 11 +++ drivers/gpu/drm/i915/gt/intel_gtt.c | 3 +++ drivers/gpu/drm/i915/gt/intel_gtt.h

[Intel-gfx] [PATCH v8 12/22] drm/i915/vm_bind: Implement I915_GEM_EXECBUFFER3 ioctl

2022-11-28 Thread Niranjana Vishwanathapura
Implement new execbuf3 ioctl (I915_GEM_EXECBUFFER3) which only works in vm_bind mode. The vm_bind mode only works with this new execbuf3 ioctl. The new execbuf3 ioctl will not have any list of objects to validate bind as all required objects binding would have been requested by the userspace befor

[Intel-gfx] [PATCH v8 15/22] drm/i915/vm_bind: Handle persistent vmas in execbuf3

2022-11-28 Thread Niranjana Vishwanathapura
Handle persistent (VM_BIND) mappings during the request submission in the execbuf3 path. v2: Ensure requests wait for bindings to complete. v3: Remove short term pinning with PIN_VALIDATE flag. Individualize fences before adding to dma_resv obj. v4: Fix bind completion check, use PIN_NOEVICT,

[Intel-gfx] [PATCH v8 09/22] drm/i915/vm_bind: Add out fence support

2022-11-28 Thread Niranjana Vishwanathapura
Add support for handling out fence for vm_bind call. v2: Reset vma->vm_bind_fence.syncobj to NULL at the end of vm_bind call. v3: Remove vm_unbind out fence uapi which is not supported yet. v4: Return error if I915_TIMELINE_FENCE_WAIT fence flag is set. Wait for bind to complete iff I915_T

[Intel-gfx] [PATCH v8 03/22] drm/i915/vm_bind: Expose i915_gem_object_max_page_size()

2022-11-28 Thread Niranjana Vishwanathapura
Expose i915_gem_object_max_page_size() function non-static which will be used by the vm_bind feature. Reviewed-by: Matthew Auld Signed-off-by: Niranjana Vishwanathapura Signed-off-by: Andi Shyti --- drivers/gpu/drm/i915/gem/i915_gem_create.c | 18 +- drivers/gpu/drm/i915/gem/i9

[Intel-gfx] [PATCH v8 18/22] drm/i915/vm_bind: Add uapi for user to enable vm_bind_mode

2022-11-28 Thread Niranjana Vishwanathapura
Add getparam support for VM_BIND capability version. Add VM creation time flag to enable vm_bind_mode for the VM. v2: update kernel-doc v3: create vm->root_obj only upon I915_VM_CREATE_FLAGS_USE_VM_BIND v4: replace vm->vm_bind_mode check with i915_gem_vm_is_vm_bind_mode() Reviewed-by: Matthew Aul

[Intel-gfx] [PATCH v8 01/22] drm/i915/vm_bind: Expose vm lookup function

2022-11-28 Thread Niranjana Vishwanathapura
Make i915_gem_vm_lookup() function non-static as it will be used by the vm_bind feature. Reviewed-by: Matthew Auld Signed-off-by: Niranjana Vishwanathapura Signed-off-by: Andi Shyti --- drivers/gpu/drm/i915/gem/i915_gem_context.c | 11 ++- drivers/gpu/drm/i915/gem/i915_gem_context.h |

[Intel-gfx] [PATCH v8 14/22] drm/i915/vm_bind: Expose i915_request_await_bind()

2022-11-28 Thread Niranjana Vishwanathapura
Rename __i915_request_await_bind() as i915_request_await_bind() and make it non-static as it will be used in execbuf3 ioctl path. v2: add documentation Reviewed-by: Matthew Auld Reviewed-by: Andi Shyti Signed-off-by: Niranjana Vishwanathapura --- drivers/gpu/drm/i915/i915_vma.c | 8 +---

[Intel-gfx] [PATCH v8 00/22] drm/i915/vm_bind: Add VM_BIND functionality

2022-11-28 Thread Niranjana Vishwanathapura
DRM_I915_GEM_VM_BIND/UNBIND ioctls allows UMD to bind/unbind GEM buffer objects (BOs) or sections of a BOs at specified GPU virtual addresses on a specified address space (VM). Multiple mappings can map to the same physical pages of an object (aliasing). These mappings (also referred to as persiste

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/perf: Do not parse context image for HSW (rev4)

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/perf: Do not parse context image for HSW (rev4) URL : https://patchwork.freedesktop.org/series/111231/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12440_full -> Patchwork_111231v4_full

Re: [Intel-gfx] [PATCH] drm/i915/mtl: Add support for 32 bit OAG formats in MTL

2022-11-28 Thread Lucas De Marchi
On Mon, Nov 28, 2022 at 05:21:46PM -0800, Umesh Nerlige Ramappa wrote: As part of OA support for MTL, - Enable 32 bit OAG formats for MTL. - 0x200c is repurposed on MTL. Use a separate mux table to verify oa configs passed by user. - Similar to ACM, OA/CS timestamp is mismatched on MTL. Add MTL

Re: [Intel-gfx] [PATCH 2/3] drm/i915/mtl: Define new PTE encode for MTL

2022-11-28 Thread Lucas De Marchi
On Tue, Nov 29, 2022 at 09:58:03AM +0530, Iddamsetty, Aravind wrote: On 29-11-2022 01:57, Lucas De Marchi wrote: On Mon, Nov 28, 2022 at 03:43:51PM +0530, Aravind Iddamsetty wrote: Add a separate PTE encode function for MTL. The number of PAT registers have increased to 16 on MTL. All 16 PAT

Re: [Intel-gfx] [PATCH v4] drm/i915/mtl: Media GT and Render GT share common GGTT

2022-11-28 Thread Lucas De Marchi
On Tue, Nov 29, 2022 at 11:33:15AM +0530, Iddamsetty, Aravind wrote: On 29-11-2022 11:24, Lucas De Marchi wrote: On Wed, Nov 23, 2022 at 09:47:03AM +0530, Iddamsetty, Aravind wrote: On 23-11-2022 05:29, Matt Roper wrote: On Tue, Nov 22, 2022 at 12:31:26PM +0530, Aravind Iddamsetty wrote:

[Intel-gfx] ✗ Fi.CI.IGT: failure for i915: dedicated MCR locking and hardware semaphore (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: i915: dedicated MCR locking and hardware semaphore (rev2) URL : https://patchwork.freedesktop.org/series/111220/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12440_full -> Patchwork_111220v2_full ===

Re: [Intel-gfx] [PATCH v4] drm/i915/mtl: Media GT and Render GT share common GGTT

2022-11-28 Thread Iddamsetty, Aravind
On 29-11-2022 11:24, Lucas De Marchi wrote: > On Wed, Nov 23, 2022 at 09:47:03AM +0530, Iddamsetty, Aravind wrote: >> >> >> On 23-11-2022 05:29, Matt Roper wrote: >>> On Tue, Nov 22, 2022 at 12:31:26PM +0530, Aravind Iddamsetty wrote: On XE_LPM+ platforms the media engines are carved out in

Re: [Intel-gfx] [PATCH v4] drm/i915/mtl: Media GT and Render GT share common GGTT

2022-11-28 Thread Lucas De Marchi
On Wed, Nov 23, 2022 at 09:47:03AM +0530, Iddamsetty, Aravind wrote: On 23-11-2022 05:29, Matt Roper wrote: On Tue, Nov 22, 2022 at 12:31:26PM +0530, Aravind Iddamsetty wrote: On XE_LPM+ platforms the media engines are carved out into a separate GT but have a common GGTMMADR address range whi

Re: [Intel-gfx] [PATCH 3/3] drm/i915/mtl/UAPI: Disable SET_CACHING IOCTL for MTL+

2022-11-28 Thread Iddamsetty, Aravind
On 29-11-2022 01:49, Lucas De Marchi wrote: > On Mon, Nov 28, 2022 at 03:43:52PM +0530, Aravind Iddamsetty wrote: >> From: Pallavi Mishra >> >> Caching mode for an object shall be selected via upcoming VM_BIND >> interface. > > last I've heard there was no plan to support this through VM_BIND.

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/huc: always init the delayed load fence (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/huc: always init the delayed load fence (rev2) URL : https://patchwork.freedesktop.org/series/111288/ State : success == Summary == CI Bug Log - changes from CI_DRM_12440_full -> Patchwork_111288v2_full

Re: [Intel-gfx] [PATCH 2/3] drm/i915/mtl: Define new PTE encode for MTL

2022-11-28 Thread Iddamsetty, Aravind
On 29-11-2022 01:57, Lucas De Marchi wrote: > On Mon, Nov 28, 2022 at 03:43:51PM +0530, Aravind Iddamsetty wrote: >> Add a separate PTE encode function for MTL. The number of PAT registers >> have increased to 16 on MTL. All 16 PAT registers are available for >> PPGTT mapped pages, but only the

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/mtl: Add support for 32 bit OAG formats in MTL

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/mtl: Add support for 32 bit OAG formats in MTL URL : https://patchwork.freedesktop.org/series/111411/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12440 -> Patchwork_111411v1 Summary -

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/mtl: Add support for 32 bit OAG formats in MTL

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/mtl: Add support for 32 bit OAG formats in MTL URL : https://patchwork.freedesktop.org/series/111411/ 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.BAT: success for drm/i915/pxp: Add missing cleanup steps for PXP global-teardown

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/pxp: Add missing cleanup steps for PXP global-teardown URL : https://patchwork.freedesktop.org/series/111409/ State : success == Summary == CI Bug Log - changes from CI_DRM_12440 -> Patchwork_111409v1 S

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/pxp: Add missing cleanup steps for PXP global-teardown

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/pxp: Add missing cleanup steps for PXP global-teardown URL : https://patchwork.freedesktop.org/series/111409/ 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.BAT: success for drm/i915/perf: Do not parse context image for HSW (rev4)

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/perf: Do not parse context image for HSW (rev4) URL : https://patchwork.freedesktop.org/series/111231/ State : success == Summary == CI Bug Log - changes from CI_DRM_12440 -> Patchwork_111231v4 Summary

[Intel-gfx] [PATCH] drm/i915/mtl: Add support for 32 bit OAG formats in MTL

2022-11-28 Thread Umesh Nerlige Ramappa
As part of OA support for MTL, - Enable 32 bit OAG formats for MTL. - 0x200c is repurposed on MTL. Use a separate mux table to verify oa configs passed by user. - Similar to ACM, OA/CS timestamp is mismatched on MTL. Add MTL to the WA. - On MTL, gt->scratch was using stolen lmem. An MI_SRM to

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/pxp: Prepare intel_pxp entry points for MTL

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/pxp: Prepare intel_pxp entry points for MTL URL : https://patchwork.freedesktop.org/series/111408/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12440 -> Patchwork_111408v1 Summary

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/pxp: Prepare intel_pxp entry points for MTL

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/pxp: Prepare intel_pxp entry points for MTL URL : https://patchwork.freedesktop.org/series/111408/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

Re: [Intel-gfx] [PATCH v5 0/1] drm/i915/pxp: Prepare intel_pxp entry points for MTL

2022-11-28 Thread Teres Alexis, Alan Previn
(++tagging Rodrigo) This series is a replacement for https://patchwork.freedesktop.org/series/109429/. Patchwork bestowed a new URL as the series is significantly different now with the new design approach direction from Rodrigo. ...alan On Mon, 2022-11-28 at 16:31 -0800, Alan Previn wrote: >

[Intel-gfx] [PATCH v2 0/3] drm/i915/pxp: Add missing cleanup steps for PXP global-teardown

2022-11-28 Thread Alan Previn
A customer issue was recently discovered and in the process a gap in i915's PXP interaction with HW+FW architecure was also realized. This series adds those missing pieces. The patches explain the details. Changes from prior revs: v1: - Dont need to teardown non arbitration sessions (Juston).

[Intel-gfx] [PATCH v2 3/3] drm/i915/pxp: Pxp hw init should be in resume_complete

2022-11-28 Thread Alan Previn
During suspend flow, i915 currently achors' on the pm_suspend_prepare callback as the location where we quiesce the entire GPU and perform all necessary cleanup in order to go into suspend. PXP is also called during this time to perform the arbitration session teardown (with the assurance no additi

[Intel-gfx] [PATCH v2 1/3] drm/i915/pxp: Invalidate all PXP fw sessions during teardown

2022-11-28 Thread Alan Previn
A gap was recently discovered where if an application did not invalidate all of the stream keys (intentionally or not), and the driver did a full PXP global teardown on the GT subsystem, we find that future session creation would fail on the security firmware's side of the equation. i915 is the ent

[Intel-gfx] [PATCH v2 2/3] drm/i915/pxp: Trigger the global teardown for before suspending

2022-11-28 Thread Alan Previn
A driver bug was recently discovered where the security firmware was receiving internal HW signals indicating that session key expirations had occurred. Architecturally, the firmware was expecting a response from the GuC to acknowledge the event with the firmware side. However the OS was in a suspe

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: fix exiting context timeout calculation

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915: fix exiting context timeout calculation URL : https://patchwork.freedesktop.org/series/111402/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12439_full -> Patchwork_111402v1_full Summa

[Intel-gfx] [PATCH v5 0/1] drm/i915/pxp: Prepare intel_pxp entry points for MTL

2022-11-28 Thread Alan Previn
MTL has two tiles that is represented by the intel_gt structure in the i915 code. The PXP feature has a control-structure that currently hangs off the intel_gt structure. In MTL, the standalone media tile (i.e. not the root tile) contains the VDBOX and KCR engine which are among several assets that

[Intel-gfx] [PATCH v5 1/1] drm/i915/pxp: Promote pxp subsystem to top-level of i915

2022-11-28 Thread Alan Previn
Starting with MTL, there will be two GT-tiles, a render and media tile. PXP as a service for supporting workloads with protected contexts and protected buffers can be subscribed by process workloads on any tile. However, depending on the platform, only one of the tiles is used for control events pe

Re: [Intel-gfx] [PATCH 01/20] drm/i915/mtl: Initial DDI port setup

2022-11-28 Thread Sripada, Radhakrishna
> -Original Message- > From: Kahola, Mika > Sent: Friday, October 14, 2022 5:47 AM > To: intel-gfx@lists.freedesktop.org > Cc: Kahola, Mika ; Taylor, Clinton A > ; Sripada, Radhakrishna > > Subject: [PATCH 01/20] drm/i915/mtl: Initial DDI port setup > > From: Clint Taylor > > Initia

[Intel-gfx] ✓ Fi.CI.BAT: success for i915: dedicated MCR locking and hardware semaphore (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: i915: dedicated MCR locking and hardware semaphore (rev2) URL : https://patchwork.freedesktop.org/series/111220/ State : success == Summary == CI Bug Log - changes from CI_DRM_12440 -> Patchwork_111220v2 Summary

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for i915: dedicated MCR locking and hardware semaphore (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: i915: dedicated MCR locking and hardware semaphore (rev2) URL : https://patchwork.freedesktop.org/series/111220/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

Re: [Intel-gfx] [PATCH 2/3] drm/i915/mtl: Define new PTE encode for MTL

2022-11-28 Thread Iddamsetty, Aravind
On 29-11-2022 01:22, Yang, Fei wrote: >> From: Iddamsetty, Aravind >> Sent: Monday, November 28, 2022 2:14 AM >> To: intel-gfx@lists.freedesktop.org >> Cc: De Marchi, Lucas ; Roper, Matthew D >> ; Yang, Fei >> Subject: [PATCH 2/3] drm/i915/mtl: Define new PTE encode for MTL >> >> Add a separa

[Intel-gfx] [PATCH v2 1/5] drm/i915/gt: Correct kerneldoc for intel_gt_mcr_wait_for_reg()

2022-11-28 Thread Matt Roper
The kerneldoc function name was not updated when this function was converted to a non-fw form. Fixes: 192bb40f030a ("drm/i915/gt: Manage uncore->lock while waiting on MCR register") Reported-by: kernel test robot Signed-off-by: Matt Roper --- drivers/gpu/drm/i915/gt/intel_gt_mcr.c | 2 +- 1 fi

[Intel-gfx] [PATCH v2 5/5] drm/i915/mtl: Hold forcewake and MCR lock over PPAT setup

2022-11-28 Thread Matt Roper
PPAT setup involves a series of multicast writes. This can be optimized slightly be acquiring forcewake and the steering lock just once for the entire sequence. Suggested-by: Balasubramani Vivekanandan Signed-off-by: Matt Roper --- drivers/gpu/drm/i915/gt/intel_gtt.c | 27 +++--

[Intel-gfx] [PATCH v2 4/5] drm/i915/mtl: Add hardware-level lock for steering

2022-11-28 Thread Matt Roper
Starting with MTL, the driver needs to not only protect the steering control register from simultaneous software accesses, but also protect against races with hardware/firmware agents. The hardware provides a dedicated locking mechanism to support this via the MTL_STEER_SEMAPHORE register. Readin

[Intel-gfx] [PATCH v2 2/5] drm/i915/gt: Pass gt rather than uncore to lowest-level reads/writes

2022-11-28 Thread Matt Roper
Passing the GT rather than uncore to the lowest level MCR read and write functions will make it easier to introduce dedicated MCR locking in a following patch. Signed-off-by: Matt Roper --- drivers/gpu/drm/i915/gt/intel_gt_mcr.c | 18 ++ 1 file changed, 10 insertions(+), 8 deleti

[Intel-gfx] [PATCH v2 3/5] drm/i915/gt: Add dedicated MCR lock

2022-11-28 Thread Matt Roper
We've been overloading uncore->lock to protect access to the MCR steering register. That's not really what uncore->lock is intended for, and it would be better if we didn't need to hold such a high-traffic spinlock for the whole sequence of (apply steering, access MCR register, restore steering).

[Intel-gfx] [PATCH v2 0/5] i915: dedicated MCR locking and hardware semaphore

2022-11-28 Thread Matt Roper
We've been overloading uncore->lock to protect access to the MCR steering register. That's not really what uncore->lock is intended for, and it would be better if we didn't need to hold such a high-traffic spinlock for the whole sequence of (apply steering, access MCR register, restore steering).

[Intel-gfx] ✓ Fi.CI.IGT: success for Align DDI_BUF_CTL Active timeouts with Bspec updates (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: Align DDI_BUF_CTL Active timeouts with Bspec updates (rev2) URL : https://patchwork.freedesktop.org/series/111373/ State : success == Summary == CI Bug Log - changes from CI_DRM_12439_full -> Patchwork_111373v2_full =

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/huc: always init the delayed load fence (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/huc: always init the delayed load fence (rev2) URL : https://patchwork.freedesktop.org/series/111288/ State : success == Summary == CI Bug Log - changes from CI_DRM_12440 -> Patchwork_111288v2 Summary -

Re: [Intel-gfx] [PATCH] drm/i915/gt: remove some limited use register access wrappers

2022-11-28 Thread Andrzej Hajda
On 23.11.2022 17:49, Jani Nikula wrote: Remove rmw_set(), rmw_clear(), clear_register(), rmw_set_fw(), and rmw_clear_fw(). They're just one too many levels of abstraction for register access, for very specific purposes. clear_register() seems like a micro-optimization bypassing the write when th

Re: [Intel-gfx] [PATCH 2/3] drm/i915/mtl: Define new PTE encode for MTL

2022-11-28 Thread Lucas De Marchi
On Mon, Nov 28, 2022 at 03:43:51PM +0530, Aravind Iddamsetty wrote: Add a separate PTE encode function for MTL. The number of PAT registers have increased to 16 on MTL. All 16 PAT registers are available for PPGTT mapped pages, but only the lower 4 are available for GGTT mapped pages. this woul

Re: [Intel-gfx] [PATCH 3/3] drm/i915/mtl/UAPI: Disable SET_CACHING IOCTL for MTL+

2022-11-28 Thread Lucas De Marchi
On Mon, Nov 28, 2022 at 03:43:52PM +0530, Aravind Iddamsetty wrote: From: Pallavi Mishra Caching mode for an object shall be selected via upcoming VM_BIND interface. last I've heard there was no plan to support this through VM_BIND. Did anything change? Otherwise this needs a better explanat

Re: [Intel-gfx] [PATCH 2/3] drm/i915/mtl: Define new PTE encode for MTL

2022-11-28 Thread Yang, Fei
> From: Iddamsetty, Aravind > Sent: Monday, November 28, 2022 2:14 AM > To: intel-gfx@lists.freedesktop.org > Cc: De Marchi, Lucas ; Roper, Matthew D > ; Yang, Fei > Subject: [PATCH 2/3] drm/i915/mtl: Define new PTE encode for MTL > > Add a separate PTE encode function for MTL. The number of PAT

Re: [Intel-gfx] [RFC 11/13] cgroup/drm: Introduce weight based drm cgroup control

2022-11-28 Thread Tejun Heo
Hello, On Thu, Nov 24, 2022 at 02:32:25PM +, Tvrtko Ursulin wrote: > > Soft limits is a bit of misnomer and can be confused with best-effort limits > > such as memory.high. Prolly best to not use the term. > > Are you suggesting "best effort limits" or "best effort "? It > would sounds good t

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/dsi: fix VBT send packet port selection for dual link DSI (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/dsi: fix VBT send packet port selection for dual link DSI (rev2) URL : https://patchwork.freedesktop.org/series/111366/ State : success == Summary == CI Bug Log - changes from CI_DRM_12438_full -> Patchwork_111366v2_full ==

Re: [Intel-gfx] [PATCH] drm/i915/huc: always init the delayed load fence

2022-11-28 Thread John Harrison
On 11/23/2022 15:54, Daniele Ceraolo Spurio wrote: The fence is only tracking if the HuC load is in progress or not and doesn't distinguish between already loaded, not supported or disabled, so we can always initialize it to completed, no matter the actual support. We already do that for most pla

Re: [Intel-gfx] [PATCH] drm/i915/gt: remove some limited use register access wrappers

2022-11-28 Thread Matt Roper
On Wed, Nov 23, 2022 at 06:49:16PM +0200, Jani Nikula wrote: > Remove rmw_set(), rmw_clear(), clear_register(), rmw_set_fw(), and > rmw_clear_fw(). They're just one too many levels of abstraction for > register access, for very specific purposes. > > clear_register() seems like a micro-optimizatio

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: fix exiting context timeout calculation

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915: fix exiting context timeout calculation URL : https://patchwork.freedesktop.org/series/111402/ State : success == Summary == CI Bug Log - changes from CI_DRM_12439 -> Patchwork_111402v1 Summary ---

Re: [Intel-gfx] signal: break out of wait loops on kthread_stop()

2022-11-28 Thread Jason A. Donenfeld
Hi Eric, On Mon, Nov 28, 2022 at 7:22 PM Eric W. Biederman wrote: > > Tvrtko Ursulin writes: > > > On 19/10/2022 21:19, Jason A. Donenfeld wrote: > >> On Wed, Oct 19, 2022 at 09:09:28PM +0100, Tvrtko Ursulin wrote: > >>> Hm why is kthread_stop() after kthread_run() abuse? I don't see it in > >>>

Re: [Intel-gfx] [PATCH v4 1/2] drm/i915/dg2: Introduce Wa_18018764978

2022-11-28 Thread Matt Roper
On Wed, Nov 23, 2022 at 04:45:25PM -0300, Gustavo Sousa wrote: > On Wed, Nov 23, 2022 at 10:36:47AM -0800, Matt Atwood wrote: > > Wa_18018764978 applies to specific steppings of DG2 (G10 C0+, > > G11 and G12 A0+). Clean up style in function at the same time. > > > > Bspec: 66622 > > > > Signed-of

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/bios: fix a memory leak in generate_lfp_data_ptrs

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/bios: fix a memory leak in generate_lfp_data_ptrs URL : https://patchwork.freedesktop.org/series/111392/ State : success == Summary == CI Bug Log - changes from CI_DRM_12438_full -> Patchwork_111392v1_full ==

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: fix exiting context timeout calculation

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915: fix exiting context timeout calculation URL : https://patchwork.freedesktop.org/series/111402/ State : warning == Summary == Error: dim checkpatch failed a049dd461775 drm/i915: fix exiting context timeout calculation -:10: WARNING:BAD_FIXES_TAG: Please us

[Intel-gfx] ✓ Fi.CI.BAT: success for Align DDI_BUF_CTL Active timeouts with Bspec updates (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: Align DDI_BUF_CTL Active timeouts with Bspec updates (rev2) URL : https://patchwork.freedesktop.org/series/111373/ State : success == Summary == CI Bug Log - changes from CI_DRM_12439 -> Patchwork_111373v2 Summa

Re: [Intel-gfx] [PATCH] drm/i915/psr: Use continuous full frame update instead of single

2022-11-28 Thread Hogander, Jouni
I found issue with this patch: There is a time window between psr_invalidate/flush and next vblank where atomic commit with selective update will cause full frame update being lost due to overwriting the configuration. I will send a new version. On Fri, 2022-11-25 at 15:43 +0200, Jouni Högander

[Intel-gfx] ✓ Fi.CI.IGT: success for Add DSC fractional bpp support

2022-11-28 Thread Patchwork
== Series Details == Series: Add DSC fractional bpp support URL : https://patchwork.freedesktop.org/series/111391/ State : success == Summary == CI Bug Log - changes from CI_DRM_12438_full -> Patchwork_111391v1_full Summary --- **SUC

[Intel-gfx] [PATCH] drm/i915: fix exiting context timeout calculation

2022-11-28 Thread Andrzej Hajda
In case context is exiting preempt_timeout_ms is used for timeout, but since introduction of DRM_I915_PREEMPT_TIMEOUT_COMPUTE it increases to 7.5 seconds. Heartbeat occurs earlier but it is still 2.5s. Fixes: d7a8680ec9fb21 ("drm/i915: Improve long running compute w/a for GuC submission") Closes:

Re: [Intel-gfx] [PATCH] drm/i915/huc: fix leak of debug object in huc load fence on driver unload

2022-11-28 Thread Ceraolo Spurio, Daniele
On 11/28/2022 5:08 AM, Ville Syrjälä wrote: On Mon, Nov 28, 2022 at 01:10:58AM -0800, Ceraolo Spurio, Daniele wrote: On 11/25/2022 5:54 AM, Ville Syrjälä wrote: On Thu, Nov 10, 2022 at 04:56:51PM -0800, Daniele Ceraolo Spurio wrote: The fence is always initialized in huc_init_early, but th

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/3] drm/i915/mtl: Define MOCS and PAT tables for MTL

2022-11-28 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915/mtl: Define MOCS and PAT tables for MTL URL : https://patchwork.freedesktop.org/series/111390/ State : success == Summary == CI Bug Log - changes from CI_DRM_12438_full -> Patchwork_111390v1_full

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dsi: fix VBT send packet port selection for dual link DSI (rev2)

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/dsi: fix VBT send packet port selection for dual link DSI (rev2) URL : https://patchwork.freedesktop.org/series/111366/ State : success == Summary == CI Bug Log - changes from CI_DRM_12438 -> Patchwork_111366v2

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/bios: fix a memory leak in generate_lfp_data_ptrs

2022-11-28 Thread Patchwork
== Series Details == Series: drm/i915/bios: fix a memory leak in generate_lfp_data_ptrs URL : https://patchwork.freedesktop.org/series/111392/ State : success == Summary == CI Bug Log - changes from CI_DRM_12438 -> Patchwork_111392v1 Summar

Re: [Intel-gfx] [PATCH] drm/i915/huc: fix leak of debug object in huc load fence on driver unload

2022-11-28 Thread Ville Syrjälä
On Mon, Nov 28, 2022 at 01:10:58AM -0800, Ceraolo Spurio, Daniele wrote: > > > On 11/25/2022 5:54 AM, Ville Syrjälä wrote: > > On Thu, Nov 10, 2022 at 04:56:51PM -0800, Daniele Ceraolo Spurio wrote: > >> The fence is always initialized in huc_init_early, but the cleanup in > >> huc_fini is only b

[Intel-gfx] ✓ Fi.CI.BAT: success for Add DSC fractional bpp support

2022-11-28 Thread Patchwork
== Series Details == Series: Add DSC fractional bpp support URL : https://patchwork.freedesktop.org/series/111391/ State : success == Summary == CI Bug Log - changes from CI_DRM_12438 -> Patchwork_111391v1 Summary --- **SUCCESS**

[Intel-gfx] ✗ Fi.CI.DOCS: warning for Add DSC fractional bpp support

2022-11-28 Thread Patchwork
== Series Details == Series: Add DSC fractional bpp support URL : https://patchwork.freedesktop.org/series/111391/ State : warning == Summary == Error: make htmldocs had i915 warnings ./drivers/gpu/drm/i915/gt/intel_gt_mcr.c:739: warning: expecting prototype for intel_gt_mcr_wait_for_reg_fw()

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Add DSC fractional bpp support

2022-11-28 Thread Patchwork
== Series Details == Series: Add DSC fractional bpp support URL : https://patchwork.freedesktop.org/series/111391/ State : warning == Summary == Error: dim checkpatch failed d0e4e48ec58b drm/i915/dp: Check if force dsc bpc <= max requested bpc 22a420e3ce87 drm/display/dp: Add helper function t

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/3] drm/i915/mtl: Define MOCS and PAT tables for MTL

2022-11-28 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915/mtl: Define MOCS and PAT tables for MTL URL : https://patchwork.freedesktop.org/series/111390/ State : success == Summary == CI Bug Log - changes from CI_DRM_12438 -> Patchwork_111390v1 ==

Re: [Intel-gfx] [PATCH 2/2] drm/i915/ddi: Add missing wait-for-active for HDMI aligning with bspec updates

2022-11-28 Thread Imre Deak
On Sun, Nov 27, 2022 at 10:52:32AM +0530, Ankit Nautiyal wrote: > After enabling DDI_BUF_CTL, wait for DDI_BUF_CTL to be active. > Bspec:4232,53339,49191,54145 > > Signed-off-by: Ankit Nautiyal Looks ok to me. On the series: Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/display/intel_d

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/3] drm/i915/mtl: Define MOCS and PAT tables for MTL

2022-11-28 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915/mtl: Define MOCS and PAT tables for MTL URL : https://patchwork.freedesktop.org/series/111390/ State : warning == Summary == Error: dim checkpatch failed 3a4cce54c504 drm/i915/mtl: Define MOCS and PAT tables for MTL 42fb2a23d315

Re: [Intel-gfx] [PATCH v3 3/3] drm/i915/selftests: Add hwmon support in libpower for dgfx

2022-11-28 Thread Tauro, Riana
On 11/22/2022 8:10 AM, Dixit, Ashutosh wrote: On Sun, 20 Nov 2022 23:29:46 -0800, Riana Tauro wrote: diff --git a/drivers/gpu/drm/i915/gt/selftest_rc6.c b/drivers/gpu/drm/i915/gt/selftest_rc6.c index 15b84c428f66..845058ed83ed 100644 --- a/drivers/gpu/drm/i915/gt/selftest_rc6.c +++ b/driver

[Intel-gfx] [PATCH] drm/i915/dsi: fix VBT send packet port selection for dual link DSI

2022-11-28 Thread Mikko Kovanen
intel_dsi->ports contains bitmask of enabled ports and correspondingly logic for selecting port for VBT packet sending must use port specific bitmask when deciding appropriate port. Fixes: 08c59dde71b7 ("drm/i915/dsi: fix VBT send packet port selection for ICL+") Cc: sta...@vger.kernel.org Signed

[Intel-gfx] [PATCH] drm/i915/bios: fix a memory leak in generate_lfp_data_ptrs

2022-11-28 Thread Xia Fukun
When (size != 0 || ptrs->lvds_ entries != 3), the program tries to free() the ptrs. However, the ptrs is not created by calling kzmalloc(), but is obtained by pointer offset operation. This may lead to memory leaks or undefined behavior. Fix this by replacing the arguments of kfree() with ptrs_blo

[Intel-gfx] [PATCH 11/11] drm/i915/dsc: Allow DSC only with fractional bpp when forced from debugfs

2022-11-28 Thread Ankit Nautiyal
From: Swati Sharma If force_dsc_fractional_bpp_en is set through debugfs allow DSC iff compressed bpp is fractional. Continue if we computed compressed bpp is computed as integer. Signed-off-by: Swati Sharma --- drivers/gpu/drm/i915/display/intel_dp.c | 6 ++ 1 file changed, 6 insertions(+

[Intel-gfx] [PATCH 08/11] drm/i915/dsc/mtl: Add support for fractional bpp

2022-11-28 Thread Ankit Nautiyal
From: Vandita Kulkarni Consider the fractional bpp while reading the qp values. Signed-off-by: Vandita Kulkarni Signed-off-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_qp_tables.c | 3 --- drivers/gpu/drm/i915/display/intel_vdsc.c | 12 +--- 2 files changed, 9 inser

[Intel-gfx] [PATCH 09/11] drm/i915/dp: Iterate over output bpp with fractional step size

2022-11-28 Thread Ankit Nautiyal
This patch adds support to iterate over compressed output bpp as per the fractional step, supported by DP sink. Signed-off-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp.c | 45 +++-- 1 file changed, 27 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/dr

[Intel-gfx] [PATCH 10/11] drm/i915/dsc: Add debugfs entry to validate DSC fractional bpp

2022-11-28 Thread Ankit Nautiyal
From: Swati Sharma DSC_Sink_BPP_Precision entry is added to i915_dsc_fec_support_show to depict sink's precision. Also, new debugfs entry is created to enforce fractional bpp. If Force_DSC_Fractional_BPP_en is set then while iterating over output bpp with fractional step size we will continue if

[Intel-gfx] [PATCH 07/11] drm/i915/audio : Consider fractional vdsc bpp while computing tu_data

2022-11-28 Thread Ankit Nautiyal
MTL+ supports fractional compressed bits_per_pixel, with precision of 1/16. This compressed bpp is stored in U6.4 format. Accommodate the precision during calculation of transfer unit data for hblank_early calculation. Signed-off-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_audio.c

[Intel-gfx] [PATCH 06/11] drm/i915/display: Consider fractional vdsc bpp while computing m_n values

2022-11-28 Thread Ankit Nautiyal
MTL+ supports fractional compressed bits_per_pixel, with precision of 1/16. This compressed bpp is stored in U6.4 format. Accommodate this precision while computing m_n values. Signed-off-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_display.c | 6 +- drivers/gpu/drm/i915/display

[Intel-gfx] [PATCH 04/11] drm/i915/dp: Get optimal link config to have best compressed bpp

2022-11-28 Thread Ankit Nautiyal
Currently, we take the max lane, rate and pipe bpp, to get the maximum compressed bpp possible. We then set the output bpp to this value. This patch provides support to have max bpp, min rate and min lanes, that can support the min compressed bpp. Signed-off-by: Ankit Nautiyal --- drivers/gpu/dr

  1   2   >