[Intel-gfx] ✗ Fi.CI.BAT: failure for Some debugfs refactoring and improvements (rev3)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev3) URL : https://patchwork.freedesktop.org/series/114510/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12879 -> Patchwork_114510v3 Summary ---

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Some debugfs refactoring and improvements (rev3)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev3) URL : https://patchwork.freedesktop.org/series/114510/ State : warning == Summary == Error: git fetch origin failed

[Intel-gfx] ✗ Fi.CI.BUILD: warning for Some debugfs refactoring and improvements (rev3)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev3) URL : https://patchwork.freedesktop.org/series/114510/ State : warning == Summary == Error: git fetch origin failed

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Some debugfs refactoring and improvements (rev3)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev3) URL : https://patchwork.freedesktop.org/series/114510/ State : warning == Summary == Error: dim checkpatch failed 89ea655c4c11 drm/i915/gt: Create per-gt debugfs files 98f2b7d2ac07 drm/i915/debugfs: Enable upper laye

[Intel-gfx] ✗ Fi.CI.BAT: failure for Some debugfs refactoring and improvements (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev2) URL : https://patchwork.freedesktop.org/series/114510/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12879 -> Patchwork_114510v2 Summary ---

[Intel-gfx] ✗ Fi.CI.DOCS: warning for Some debugfs refactoring and improvements (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev2) URL : https://patchwork.freedesktop.org/series/114510/ State : warning == Summary == Error: git fetch origin failed

[Intel-gfx] ✗ Fi.CI.BUILD: warning for Some debugfs refactoring and improvements (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev2) URL : https://patchwork.freedesktop.org/series/114510/ State : warning == Summary == Error: git fetch origin failed

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Some debugfs refactoring and improvements (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev2) URL : https://patchwork.freedesktop.org/series/114510/ 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 Some debugfs refactoring and improvements (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Some debugfs refactoring and improvements (rev2) URL : https://patchwork.freedesktop.org/series/114510/ State : warning == Summary == Error: dim checkpatch failed 0a415051f3a4 drm/i915/gt: Create per-gt debugfs files 17ad07b6f3e0 drm/i915/debugfs: Enable upper laye

[Intel-gfx] [PATCH v3 2/2] drm/i915/debugfs: Enable upper layer interfaces to act on all gt's

2023-03-18 Thread Andi Shyti
The commit 82a149a62b6b5 ('drm/i915/gt: move remaining debugfs interfaces into gt') moved gt-related debugfs files in the gtX/ directories to operate on individual gt's. However, the original files were only functioning on the root GT (GT 0) and have been left in the same location to maintain comp

[Intel-gfx] [PATCH v3 1/2] drm/i915/gt: Create per-tile debugfs files

2023-03-18 Thread Andi Shyti
To support multi-GT configurations, we need to generate independent debug files for each GT. To achieve this create a separate directory for each GT under the debugfs directory. For instance, in a system with two tiles, the debugfs structure would look like this: /sys/kernel/debug/dri

[Intel-gfx] [PATCH v3 1/2] drm/i915/gt: Create per-gt debugfs files

2023-03-18 Thread Andi Shyti
To support multi-GT configurations, we need to generate independent debug files for each GT. To achieve this create a separate directory for each GT under the debugfs directory. For instance, in a system with two GTs, the debugfs structure would look like this: /sys/kernel/debug/dri

[Intel-gfx] [PATCH v3 0/2] Some debugfs refactoring and improvements

2023-03-18 Thread Andi Shyti
Hi, These two patches aim to enhance the multi-GT capabilities of the debugfs. The first patch reorganizes the file structure, while the second patch extends the functionality of the original files in the upper directories to operate on all tiles with a single write, providing an or'ed value amon

[Intel-gfx] ✓ Fi.CI.IGT: success for Revert "drm/i915/hwmon: Enable PL1 power limit" (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Revert "drm/i915/hwmon: Enable PL1 power limit" (rev2) URL : https://patchwork.freedesktop.org/series/113793/ State : success == Summary == CI Bug Log - changes from CI_DRM_12879_full -> Patchwork_113793v2_full

Re: [Intel-gfx] [PATCH v10 09/15] drm/syncobj: Add deadline support for syncobj waits

2023-03-18 Thread Rob Clark
On Fri, Mar 17, 2023 at 12:08 PM Faith Ekstrand wrote: > > > On Wed, Mar 8, 2023 at 9:54 AM Rob Clark wrote: >> >> From: Rob Clark >> >> Add a new flag to let userspace provide a deadline as a hint for syncobj >> and timeline waits. This gives a hint to the driver signaling the >> backing fence

[Intel-gfx] ✓ Fi.CI.BAT: success for Revert "drm/i915/hwmon: Enable PL1 power limit" (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Revert "drm/i915/hwmon: Enable PL1 power limit" (rev2) URL : https://patchwork.freedesktop.org/series/113793/ State : success == Summary == CI Bug Log - changes from CI_DRM_12879 -> Patchwork_113793v2 Summary --

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Revert "drm/i915/hwmon: Enable PL1 power limit" (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Revert "drm/i915/hwmon: Enable PL1 power limit" (rev2) URL : https://patchwork.freedesktop.org/series/113793/ State : warning == Summary == Error: git fetch origin failed

[Intel-gfx] ✗ Fi.CI.BUILD: warning for Revert "drm/i915/hwmon: Enable PL1 power limit" (rev2)

2023-03-18 Thread Patchwork
== Series Details == Series: Revert "drm/i915/hwmon: Enable PL1 power limit" (rev2) URL : https://patchwork.freedesktop.org/series/113793/ State : warning == Summary == Error: git fetch origin failed

Re: [Intel-gfx] Reverted patch reappears!

2023-03-18 Thread Dixit, Ashutosh
On Fri, 17 Mar 2023 20:28:58 -0700, Dixit, Ashutosh wrote: > > Jani/Rodrigo, > > Original Subject: Re: [Intel-gfx] [PATCH] Revert "drm/i915/hwmon: Enable PL1 > power limit" > > On Wed, 15 Feb 2023 09:19:07 -0800, Rodrigo Vivi wrote: > > > > On Wed, Feb 15, 2023 at 08:24:51AM -0800, Dixit, Ashutosh

[Intel-gfx] [PATCH] Revert "drm/i915/hwmon: Enable PL1 power limit"

2023-03-18 Thread Ashutosh Dixit
This reverts commit ee892ea83d99610fa33bea612de058e0955eec3a. 0349c41b0596 ("drm/i915/hwmon: Enable PL1 power limit") was reverted in 05d5562e401e ("Revert "drm/i915/hwmon: Enable PL1 power limit"") but has appeared again as ee892ea83d99 ("drm/i915/hwmon: Enable PL1 power limit"). Revert it again.

[Intel-gfx] ✗ Fi.CI.BUILD: failure for series starting with [xf86-video-intel,1/8] uxa: Switch to using crtc index instead of pipe

2023-03-18 Thread Patchwork
== Series Details == Series: series starting with [xf86-video-intel,1/8] uxa: Switch to using crtc index instead of pipe URL : https://patchwork.freedesktop.org/series/115348/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/115348/revisions/1/mbox/

[Intel-gfx] [PATCH xf86-video-intel 8/8] intel: Fix some theoretical buffer overflow

2023-03-18 Thread Ville Syrjala
From: Ville Syrjälä Looks to me like the theoretical max the sprintf()s need here is about 34+4+9+sizeof(de->d_name) bytes. Let's just make that 64+sizeof(de->d_name) for simplicity. This shuts up the compiler: ../src/intel_device.c: In function ‘__intel_open_device__pci’: ../src/intel_device.c:

[Intel-gfx] [PATCH xf86-video-intel 7/8] sna/gen3: Silence compiler warn

2023-03-18 Thread Ville Syrjala
From: Ville Syrjälä ../src/sna/kgem_debug_gen3.c:1289:50: warning: ‘%03d’ directive writing between 3 and 10 bytes into a region of size 8 [-Wformat-overflow=] 1289 | sprintf(instr_prefix, "PS%03d", instr); | ^~~~ ..

[Intel-gfx] [PATCH xf86-video-intel 6/8] sna/gen3: Fix 3DSTATE_PIXEL_SHADER_PROGRAM debugs

2023-03-18 Thread Ville Syrjala
From: Ville Syrjälä 3DSTATE_PIXEL_SHADER_PROGRAM instruction length is 9 bits, not 8 bits. Signed-off-by: Ville Syrjälä --- src/sna/kgem_debug_gen3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/sna/kgem_debug_gen3.c b/src/sna/kgem_debug_gen3.c index 1634225c4193..4a

[Intel-gfx] [PATCH xf86-video-intel 3/8] sna/video: Use crtc index instead of pipe

2023-03-18 Thread Ville Syrjala
From: Ville Syrjälä For consistency with most other code use the kms crtc index instead of the hardware pipe number where either will do. Signed-off-by: Ville Syrjälä --- src/sna/sna_video_sprite.c | 44 +++--- 1 file changed, 22 insertions(+), 22 deletions(-)

[Intel-gfx] [PATCH xf86-video-intel 5/8] sna/gen2: Silence compiler warn

2023-03-18 Thread Ville Syrjala
From: Ville Syrjälä ../src/sna/kgem_debug_gen2.c:625:5: warning: ‘static’ is not at beginning of declaration [-Wold-style-declaration] 625 | const static struct { | ^ Signed-off-by: Ville Syrjälä --- src/sna/kgem_debug_gen2.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[Intel-gfx] [PATCH xf86-video-intel 2/8] sna: Switch to using crtc index instead of pipe

2023-03-18 Thread Ville Syrjala
From: Ville Syrjälä Start using the kms crtc index rather than the pipe almost everywhere. The two numbers could in theory be different if the hardware has some pipes fused off. Though I think such non-contiguous fusing won't actually happen on the hardware generations the driver fully supports.

[Intel-gfx] [PATCH xf86-video-intel 4/8] sna: Switch debugs/errors to use crtc index rather than pipe

2023-03-18 Thread Ville Syrjala
From: Ville Syrjälä Let's the limit the use of hardware pipe numbers to absolutely the only place where it's needed (MI_SCANLINE_WAIT). Everywhere else just use the crtc index. Signed-off-by: Ville Syrjälä --- src/sna/sna_display.c | 220 +++-- src/sna/sna_

[Intel-gfx] [PATCH xf86-video-intel 1/8] uxa: Switch to using crtc index instead of pipe

2023-03-18 Thread Ville Syrjala
From: Ville Syrjälä Start using the kms crtc index rather than the pipe almost everywhere. The two numbers could in theory be different if the hardware has some pipes fused off. Though I think such non-contiguous fusing won't actually happen on the hardware generations the driver fully supports.

[Intel-gfx] ✗ Fi.CI.IGT: failure for Add crtc i915_pipe debugfs file

2023-03-18 Thread Patchwork
== Series Details == Series: Add crtc i915_pipe debugfs file URL : https://patchwork.freedesktop.org/series/115343/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12878_full -> Patchwork_115343v1_full Summary --- **FA