== Series Details ==
Series: drm/xe/display: check for error on drmm_mutex_init
URL : https://patchwork.freedesktop.org/series/131301/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_1 -> Patchwork_131301v1
Summary
The failures are not related to the changes made.
Regards,
Radhakrishna Sripada
From: Patchwork
Sent: Monday, March 18, 2024 5:15 PM
To: Sripada, Radhakrishna
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.BAT: failure for drm/i915/xelpg: Add Wa_14020495402
Patch Details
Series:
Check return value for drmm_mutex_init as it can fail and return on
failure.
Signed-off-by: Arun R Murthy
---
drivers/gpu/drm/xe/display/xe_display.c | 24 ++--
1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/xe/display/xe_display.c
https://patchwork.freedesktop.org/patch/582411/?series=130780=2 provides
the scenario and the details of DMESG in the commit message on rev#1. I'll
assume we'll go with rev#2.
Thanks,
Gareth
-Original Message-
From: Jani Nikula
Sent: Monday, March 18, 2024 9:45 PM
To: Dolan Liu ;
== Series Details ==
Series: drm/i915/xelpg: Add Wa_14020495402
URL : https://patchwork.freedesktop.org/series/131291/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14443 -> Patchwork_131291v1
Summary
---
== Series Details ==
Series: drm: Add plane SIZE_HINTS property (rev6)
URL : https://patchwork.freedesktop.org/series/113758/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14443 -> Patchwork_113758v6
Summary
---
== Series Details ==
Series: drm: Add plane SIZE_HINTS property (rev6)
URL : https://patchwork.freedesktop.org/series/113758/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/i915/mtl: Add Wa_14020495402
URL : https://patchwork.freedesktop.org/series/131284/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14443 -> Patchwork_131284v1
Summary
---
**SUCCESS**
== Series Details ==
Series: drm/i915/gt: Reset queue_priority_hint on parking
URL : https://patchwork.freedesktop.org/series/131268/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14443 -> Patchwork_131268v1
Summary
== Series Details ==
Series: drm/i915/gt: Reset queue_priority_hint on parking
URL : https://patchwork.freedesktop.org/series/131268/
State : warning
== Summary ==
Error: dim checkpatch failed
e1d8aba7c160 drm/i915/gt: Reset queue_priority_hint on parking
-:19: WARNING:COMMIT_LOG_LONG_LINE:
On Mon, Mar 18, 2024 at 02:00:25PM -0700, Radhakrishna Sripada wrote:
> Disable clockgating for TDL SVHS fub.
>
> v2: Implement in general render/compute wa's(MattR)
>
> Bspec: 46045
> Cc: Matt Roper
> Signed-off-by: Radhakrishna Sripada
Reviewed-by: Matt Roper
> ---
>
== Series Details ==
Series: drm/i915/display: DMC wakelock implementation (rev2)
URL : https://patchwork.freedesktop.org/series/131124/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14443 -> Patchwork_131124v2
Summary
== Series Details ==
Series: drm/i915/display: DMC wakelock implementation (rev2)
URL : https://patchwork.freedesktop.org/series/131124/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/i915/display: DMC wakelock implementation (rev2)
URL : https://patchwork.freedesktop.org/series/131124/
State : warning
== Summary ==
Error: dim checkpatch failed
eaf8c75d8c52 drm/i915/display: add support for DMC wakelocks
Traceback (most recent call last):
== Series Details ==
Series: drm/xe/display: fix potential overflow when multiplying 2 u32 (rev2)
URL : https://patchwork.freedesktop.org/series/131014/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14443 -> Patchwork_131014v2
== Series Details ==
Series: Enable Adaptive Sync SDP Support for DP (rev19)
URL : https://patchwork.freedesktop.org/series/126829/
State : failure
== Summary ==
Error: patch
https://patchwork.freedesktop.org/api/1.0/series/126829/revisions/19/mbox/ not
applied
Applying: drm/dp: Add support
Disable clockgating for TDL SVHS fub.
v2: Implement in general render/compute wa's(MattR)
Bspec: 46045
Cc: Matt Roper
Signed-off-by: Radhakrishna Sripada
---
drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 +
drivers/gpu/drm/i915/gt/intel_workarounds.c | 6 +-
2 files changed, 6
== Series Details ==
Series: drm/xe/display: fix potential overflow when multiplying 2 u32 (rev2)
URL : https://patchwork.freedesktop.org/series/131014/
State : warning
== Summary ==
Error: dim checkpatch failed
8f904ca289af drm/xe/display: fix potential overflow when multiplying 2 u32
-:20:
== Series Details ==
Series: Bigjoiner refactoring (rev12)
URL : https://patchwork.freedesktop.org/series/128311/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14443 -> Patchwork_128311v12
Summary
---
**FAILURE**
From: Ville Syrjälä
Advertize more suitable cursor sizes via the new SIZE_HINTS
plane property.
We can't really enumerate all supported cursor sizes on
the platforms where the cursor height can vary freely, so
for simplicity we'll just expose all square+POT sizes between
each platform's min and
From: Ville Syrjälä
Add a new immutable plane property by which a plane can advertise
a handful of recommended plane sizes. This would be mostly exposed
by cursor planes as a slightly more capable replacement for
the DRM_CAP_CURSOR_WIDTH/HEIGHT caps, which can only declare
a one size fits all
From: Ville Syrjälä
Final final version I hope. Mainly for CI to test against the
new IGTs.
Real userspace implementation:
https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/3165
IGT:
== Series Details ==
Series: Bigjoiner refactoring (rev12)
URL : https://patchwork.freedesktop.org/series/128311/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
+./arch/x86/include/asm/bitops.h:116:1:
== Series Details ==
Series: Bigjoiner refactoring (rev12)
URL : https://patchwork.freedesktop.org/series/128311/
State : warning
== Summary ==
Error: dim checkpatch failed
f7402570def3 drm/i915: Add a small helper to compute the set of pipes for crtc
7a0f16bdaae4 drm/i915: Extract
On Mon, Mar 18, 2024 at 02:58:47PM +0100, Janusz Krzysztofik wrote:
> From: Chris Wilson
>
> Originally, with strict in order execution, we could complete execution
> only when the queue was empty. Preempt-to-busy allows replacement of an
> active request that may complete before the preemption
On Mon, Mar 18, 2024 at 12:35:58PM -0700, Radhakrishna Sripada wrote:
> Disable clockgating for TDL SVHS fub.
>
> Bspec: 46045
> Cc: Matt Roper
> Signed-off-by: Radhakrishna Sripada
> ---
> drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 +
> drivers/gpu/drm/i915/gt/intel_workarounds.c | 3 +++
Disable clockgating for TDL SVHS fub.
Bspec: 46045
Cc: Matt Roper
Signed-off-by: Radhakrishna Sripada
---
drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 +
drivers/gpu/drm/i915/gt/intel_workarounds.c | 3 +++
2 files changed, 4 insertions(+)
diff --git
On Sat, Mar 16, 2024 at 12:05:59PM +, Chris Bainbridge wrote:
> Fix a regression when using nouveau and unplugging a StarTech MSTDP122DP
> DisplayPort 1.2 MST hub (the same regression does not appear when using
> a Cable Matters DisplayPort 1.4 MST hub). Trace:
Thanks for the report and fix,
On Mon, 11 Mar 2024, Mitul Golani wrote:
> An Adaptive-Sync-capable DP protocol converter indicates its
> support by setting the related bit in the DPCD register. This
> is valid for DP and edp as well.
>
> Computes AS SDP values based on the display configuration,
> ensuring proper handling of
From: Chris Wilson
Originally, with strict in order execution, we could complete execution
only when the queue was empty. Preempt-to-busy allows replacement of an
active request that may complete before the preemption is processed by
HW. If that happens, the request is retired from the queue,
On Mon, 18 Mar 2024, Dolan Liu wrote:
> this case have been there so many years, it's time to fix it if
> possible. And user-space software may improved by themselves in we
> didn't realize place.
>
> even if not, for the proof user-space setting 0, it's better to change to
>
>
> if
We only need DMC wakelocks when we allow DC5 and DC6 states. Add the
calls to enable and disable DMC wakelock accordingly.
Signed-off-by: Luca Coelho
---
drivers/gpu/drm/i915/display/intel_display_power_well.c | 7 +++
drivers/gpu/drm/i915/display/intel_dmc.c| 4
2
This feature should be disabled by default until properly tested and
mature. Add a module parameter to enable the feature for testing,
while keeping it disabled by default for now.
Signed-off-by: Luca Coelho
---
drivers/gpu/drm/i915/display/intel_display_params.c | 5 +
Only allow running DMC wakelock code if the display version is 20 or
greater.
Signed-off-by: Luca Coelho
---
drivers/gpu/drm/i915/display/intel_dmc_wl.c | 12
1 file changed, 12 insertions(+)
diff --git a/drivers/gpu/drm/i915/display/intel_dmc_wl.c
In order to reduce the DC5->DC2 restore time, wakelocks have been
introduced in DMC so the driver can tell it when registers and other
memory areas are going to be accessed and keep their respective blocks
awake.
Implement this in the driver by adding the concept of DMC wakelocks.
When the driver
Hi,
This is the fifth version of my series, the third as a proper
patchset.
These are the changes:
In v3:
* Fixed some checkpatch issues.
In v2:
* Enable/disable the wakelocks on DC5-6 entry and exit instead of on
DMC load and unload;
* Added bspec link to the commit message;
* A
On Thu, 2024-03-14 at 14:48 +0200, Joonas Lahtinen wrote:
> Quoting Rodrigo Vivi (2024-03-08 16:58:04)
> > On Fri, Mar 08, 2024 at 04:46:43PM +0200, Joonas Lahtinen wrote:
> > > Add standalone includes for BUG_ON and BUILD_BUG_ON to avoid
> > > build failure
> > > after linux-next include
this case have been there so many years, it's time to fix it if
possible. And user-space software may improved by themselves in we
didn't realize place.
even if not, for the proof user-space setting 0, it's better to change to
if (level < min || level == 0 )
level =max;
Compute vrr_vsync_start/end, which sets the position
for hardware to send the Vsync at a fixed position
relative to the end of the Vblank.
--v2:
- Updated VSYNC_START/END macros to VRR_VSYNC_START/END. (Ankit)
- Updated bit fields of VRR_VSYNC_START/END. (Ankit)
--v3:
- Add
Multiplying XE_PAGE_SIZE with another u32 and the product stored in
u64 can potentially lead to overflow, use mul_u32_u32 instead.
Signed-off-by: Arun R Murthy
---
drivers/gpu/drm/xe/display/xe_fb_pin.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
On Fri, Mar 15, 2024 at 01:56:19PM +0200, Ville Syrjälä wrote:
> On Fri, Mar 15, 2024 at 12:33:52PM +0100, Sebastian Wick wrote:
> > On Fri, Mar 15, 2024 at 10:35:30AM +0530, Suraj Kandpal wrote:
>
> > > @@ -318,11 +346,24 @@ intel_dp_aux_hdr_setup_backlight(struct
> > > intel_connector
Users who store a pointer to struct igt_ktap_results, obtained from
igt_ktap_alloc(), in a central location and then call igt_ktap_free() when
no longer needed, now have to reset that pointer to NULL to avoid double
free on final cleanup. For their convenience, teach igt_ktap_free() to
accept
A new helper has been introduced recently, used for fetching KTAP results
of a single test case. Since that helper is called for that purpose
only after the test module is loaded with all other test cases filtered
out, its actual implementation is as simple as collecting all results from
a single
Up to now we were loading a KUnit test module in test execution mode only
once per subtest, in background, and then, in parallel with execution of
test cases while the module was loading, we were looking through dmesg for
KTAP results from each expected test case. As a consequence, our IGT
If a test provides a subtest name that doesn't match any test suites
provided by the requested KUnit test module then no KTAP report appears in
dmesg, not even an empty one as one may expect. As a consequence, we now
loop endlessly around reading potential lines of the missing report from
To give more freedom to future enhancements of KUnit library (legacy path
excluded) in using IGT fails and skips, maintain a pointer to struct
igt_ktap_results, allocated by several functions, in a single central
location, and free it from a closing igt_fixture section before return.
Up to now we were loading a KUnit test module in test execution mode only
once per subtest, in background, and then, in parallel with execution of
test cases while the module was loading, we were looking through dmesg for
KTAP results from each expected test case. As a consequence, our IGT
On Wed, 13 Mar 2024, Jani Nikula wrote:
> On Wed, 13 Mar 2024, Jonathon Hall wrote:
>> Since commit 0c65dc062611 ("drm/i915/jsl: s/JSL/JASPERLAKE for
>> platform/subplatform defines"), boot freezes on a Jasper Lake tablet
>> (Librem 11), usually with graphical corruption on the eDP display,
>>
Handle only bigjoiner masters in skl_commit_modeset_enables/disables,
slave crtcs should be handled by master hooks. Same for encoders.
That way we can also remove a bunch of checks like
intel_crtc_is_bigjoiner_slave.
v2: - Moved skl_pfit_enable, intel_dsc_enable, intel_crtc_vblank_on to
49 matches
Mail list logo
