Am Do., 10. Juli 2025 um 22:21 Uhr schrieb Ville Syrjälä
:
>
> On Mon, Jul 07, 2025 at 03:41:20PM +, Naveen Kumar wrote:
> > Allow asynchronous page flips on planes that either lack a framebuffer or
> > retain the same framebuffer, as long as no plane properties are modified.
> >
> > This avoid
== Series Details ==
Series: drm/i915: replace DRM_DEBUG_SELFTEST with DRM_KUNIT_TEST (rev3)
URL : https://patchwork.freedesktop.org/series/151085/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16893 -> Patchwork_151085v3
S
Quoting Imre Deak (2025-07-18 14:17:09-03:00)
>On Fri, Jul 18, 2025 at 01:33:26PM -0300, Gustavo Sousa wrote:
>> Quoting Imre Deak (2025-07-18 12:54:11-03:00)
>> >On Thu, Jul 17, 2025 at 09:02:45AM -0300, Gustavo Sousa wrote:
>> >> Quoting Chaitanya Kumar Borah (2025-07-17 02:16:03-03:00)
>> >> >So
== Series Details ==
Series: drm/i915: move struct_mutex from drm_device to drm_i915_private (rev3)
URL : https://patchwork.freedesktop.org/series/151828/
State : failure
== Summary ==
Address 'luiz.me...@estudante.ufscar.br' is not on the allowlist, which
prevents CI from being triggered for
On Fri, Jul 18, 2025 at 01:34:13PM -0300, Luiz Otavio Mello wrote:
Oi Luiz! :)
First of all thank you so much for the patch. A few comments below:
> Includes the missing file drm_device.h, which was unintentionally
> omitted in v1.
This version comment should be below, not in the beginning of t
== Series Details ==
Series: drm/i915/dp: Implement POST_LT_ADJ_REQ (rev3)
URL : https://patchwork.freedesktop.org/series/145348/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16892 -> Patchwork_145348v3
Summary
---
== Series Details ==
Series: drm/i915/dsi: Don't set/read the DSI C clock divider on GLK
URL : https://patchwork.freedesktop.org/series/151803/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16892 -> Patchwork_151803v1
Summa
Includes the missing file drm_device.h, which was unintentionally
omitted in v1.
i915 is the only remaining user of struct_mutex lock.
Move it from drm_device to drm_i915_private so it is only used within
the i915 driver.
Also update intel_guc_log.c to use the new location of struct_mutex.
Sign
== Series Details ==
Series: drm/ci: add new jobs, uprev IGT and mesa
URL : https://patchwork.freedesktop.org/series/151801/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16891 -> Patchwork_151801v1
Summary
---
**SUC
== Series Details ==
Series: i915_gem_execbuffer.c minor improved
URL : https://patchwork.freedesktop.org/series/151799/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16891 -> Patchwork_151799v1
Summary
---
**SUCCESS
== Series Details ==
Series: iosys-map: Fix undefined behavior in iosys_map_clear()
URL : https://patchwork.freedesktop.org/series/151798/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16891 -> Patchwork_151798v1
Summary
--
On Fri, Jul 18, 2025 at 01:34:13PM -0300, Luiz Otavio Mello wrote:
> Includes the missing file drm_device.h, which was unintentionally
> omitted in v1.
>
> i915 is the only remaining user of struct_mutex lock.
>
> Move it from drm_device to drm_i915_private so it is only used within
> the i915 dr
== Series Details ==
Series: drm/i915: move struct_mutex from drm_device to drm_i915_private (rev2)
URL : https://patchwork.freedesktop.org/series/151828/
State : failure
== Summary ==
Address 'luiz.me...@estudante.ufscar.br' is not on the allowlist, which
prevents CI from being triggered for
On Fri, Jul 18, 2025 at 01:33:26PM -0300, Gustavo Sousa wrote:
> Quoting Imre Deak (2025-07-18 12:54:11-03:00)
> >On Thu, Jul 17, 2025 at 09:02:45AM -0300, Gustavo Sousa wrote:
> >> Quoting Chaitanya Kumar Borah (2025-07-17 02:16:03-03:00)
> >> >Some power wells are only relevant for certain displa
Includes the missing file drm_device.h, which was unintentionally
omitted in v1.
i915 is the only remaining user of struct_mutex lock.
Move it from drm_device to drm_i915_private so it is only used within
the i915 driver.
Also update intel_guc_log.c to use the new location of struct_mutex.
Sign
Quoting Imre Deak (2025-07-18 12:54:11-03:00)
>On Thu, Jul 17, 2025 at 09:02:45AM -0300, Gustavo Sousa wrote:
>> Quoting Chaitanya Kumar Borah (2025-07-17 02:16:03-03:00)
>> >Some power wells are only relevant for certain display pipes. Add a check
>> >to ensure we only allocate and initialize powe
On Fri, Jul 18, 2025 at 02:59:10PM +0100, Ruben Wauters wrote:
> On Tue, 2025-07-01 at 12:54 +0100, Ruben Wauters wrote:
> > DRM_DEBUG_SELFTEST was removed in commit fc8d29e298cf (drm: selftest:
> > convert drm_mm selftest to KUnit) and all functions under it were
> > converted to KUnit, under the
From: Ruben Wauters
DRM_DEBUG_SELFTEST was removed in commit fc8d29e298cf (drm: selftest:
convert drm_mm selftest to KUnit) and all functions under it were
converted to KUnit, under the DRM_KUNIT_TEST option
This conversion however did not occur in the Kconfig.debug file in the
i915 directory.
i915 is the only remaining user of struct_mutex lock.
Move it from drm_device to drm_i915_private so it is only used within
the i915 driver.
Signed-off-by: Luiz Otavio Mello
---
drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 4 ++--
drivers/gpu/drm/i915/i915_drv.h| 8
2 files
On Thu, Jul 17, 2025 at 09:02:45AM -0300, Gustavo Sousa wrote:
> Quoting Chaitanya Kumar Borah (2025-07-17 02:16:03-03:00)
> >Some power wells are only relevant for certain display pipes. Add a check
> >to ensure we only allocate and initialize power wells whose associated
> >pipes are available on
Hi
Am 18.07.25 um 16:47 schrieb Andi Shyti:
Hi Nitin,
On Fri, Jul 18, 2025 at 04:20:51PM +0530, Nitin Gote wrote:
The current iosys_map_clear() implementation reads the potentially
uninitialized 'is_iomem' boolean field to decide which union member
to clear. This causes undefined behavior when
Hi,
On Fri, Jul 18, 2025 at 01:21:53PM +0200, Chris Wilson wrote:
> Quoting Sebastian Brzezinka (2025-07-18 12:28:26)
> > Introduce a separate `ret` variable to hold the return value in
> > `eb_relocate_vma()`, instead of overloading `remain` for both loop
> > control and error reporting.
>
> No
Hi Sebastian,
On Fri, Jul 18, 2025 at 10:28:19AM +, Sebastian Brzezinka wrote:
> Simplifies the control flow by replacing an empty
> `if (likely(offset == 0))` block with a `continue` statement. This
> improves readability and avoids unnecessary nesting.
>
> Signed-off-by: Sebastian Brzezinka
Hi Nitin,
On Fri, Jul 18, 2025 at 04:20:51PM +0530, Nitin Gote wrote:
> The current iosys_map_clear() implementation reads the potentially
> uninitialized 'is_iomem' boolean field to decide which union member
> to clear. This causes undefined behavior when called on uninitialized
> structures, as
On Fri, Jul 18, 2025 at 01:16:58PM +0200, Chris Wilson wrote:
> Quoting Sebastian Brzezinka (2025-07-18 12:28:11)
> > The value 4095 is intended to represent PAGE_SIZE - 1, as stated
> > in the commit that introduced this change:
> > commit 5f22cc0b134a ("drm/i915: Fix mismatch between misplaced
>
On Tue, 2025-07-01 at 12:54 +0100, Ruben Wauters wrote:
> DRM_DEBUG_SELFTEST was removed in commit fc8d29e298cf (drm: selftest:
> convert drm_mm selftest to KUnit) and all functions under it were
> converted to KUnit, under the DRM_KUNIT_TEST option
>
> This conversion however did not occur in the
Quoting Hogander, Jouni (2025-07-18 10:08:52-03:00)
>On Fri, 2025-07-18 at 09:58 -0300, Gustavo Sousa wrote:
>> Quoting Hogander, Jouni (2025-07-18 07:46:31-03:00)
>> > On Fri, 2025-07-18 at 09:05 +0300, Hogander, Jouni wrote:
>> > > On Thu, 2025-07-17 at 10:31 -0300, Gustavo Sousa wrote:
>> > > >
On Fri, 2025-07-18 at 09:58 -0300, Gustavo Sousa wrote:
> Quoting Hogander, Jouni (2025-07-18 07:46:31-03:00)
> > On Fri, 2025-07-18 at 09:05 +0300, Hogander, Jouni wrote:
> > > On Thu, 2025-07-17 at 10:31 -0300, Gustavo Sousa wrote:
> > > > Quoting Jouni Högander (2025-07-17 03:32:58-03:00)
> > >
Quoting Hogander, Jouni (2025-07-18 07:46:31-03:00)
>On Fri, 2025-07-18 at 09:05 +0300, Hogander, Jouni wrote:
>> On Thu, 2025-07-17 at 10:31 -0300, Gustavo Sousa wrote:
>> > Quoting Jouni Högander (2025-07-17 03:32:58-03:00)
>> > > We are seeing "dmesg-warn/abort - *ERROR* PHY * failed after 3
>>
On Fri, Jul 18, 2025 at 04:23:57PM +0530, Vignesh Raman wrote:
> Uprev IGT to the latest version and update expectation files.
>
> Signed-off-by: Vignesh Raman
> ---
> drivers/gpu/drm/ci/gitlab-ci.yml | 2 +-
> .../gpu/drm/ci/xfails/amdgpu-stoney-fails.txt | 2 +
> .../drm/ci/xf
From: Ville Syrjälä
Implement the POST_LT_ADJ_REQ sequence, which should be used
to further fine tune the link if TPS4 is not supported.
The POST_LT_ADJ_REQ sequence will be performed after
the normal link training has succeeded.
Only the final hop between the last LTTPR and DPRX will
perform th
From: Ville Syrjälä
Disable TPS4 in favor of POST_LT_ADJ_REQ for testing purposes.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_dp_link_training.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c
b/drivers/
From: Ville Syrjälä
Add the bit definitions needed for POST_LT_ADJ sequence.
v2: DP_POST_LT_ADJ_REQ_IN_PROGRESS is bit 1 not 5 (Jani)
Tested-by: Imre Deak
Reviewed-by: Imre Deak
Signed-off-by: Ville Syrjälä
---
include/drm/display/drm_dp.h | 3 +++
1 file changed, 3 insertions(+)
diff --gi
From: Ville Syrjälä
Add small helpers (drm_dp_post_lt_adj_req_supported() and
drm_dp_post_lt_adj_req_in_progress()) to help with implementing
the POST_LT_ADJ_REQ sequence.
Tested-by: Imre Deak
Reviewed-by: Imre Deak
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/display/drm_dp_helper.c | 8
From: Ville Syrjälä
Implement the POST_LT_ADJ_REQ sequence, which is supposed
to be used to further tune the link vswing/pre-emphasis
when TPS4 is not supported.
Unfortunately I don't have any displays/dongles that support
this so I wasn't able to test anything. Imre has tested this
with a DP te
== Series Details ==
Series: drm/i915/display: Write PHY_CMN1_CONTROL only when using AUXLess ALPM
(rev2)
URL : https://patchwork.freedesktop.org/series/151740/
State : failure
== Summary ==
Error: patch
https://patchwork.freedesktop.org/api/1.0/series/151740/revisions/2/mbox/ not
applied
A
Den 2025-07-18 kl. 07:21, skrev Dibin Moolakadan Subrahmanian:
> On 17-07-2025 17:24, Maarten Lankhorst wrote:
>> Hey,
>>
>> When I looked at enabling -RT for Xe yesterday, I noticed this trace.
>>
>> https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-151626v2/shard-bmg-6/igt@kms_cursor_crc@cursor-s
From: Ville Syrjälä
GLK doesn't use the DSI C clock at all, no need to program
the divider for it. Bspec even says: "Do not program this field".
However looks like some firmware versions program this and
some do not. In order to avoid bogus fastset mismatches
we should also filter it out during
Quoting Sebastian Brzezinka (2025-07-18 12:28:26)
> Introduce a separate `ret` variable to hold the return value in
> `eb_relocate_vma()`, instead of overloading `remain` for both loop
> control and error reporting.
No need.
-Chris
On Thu, Jul 17, 2025 at 01:54:49PM +0200, Maarten Lankhorst wrote:
> Hey,
>
> When I looked at enabling -RT for Xe yesterday, I noticed this trace.
>
> https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-151626v2/shard-bmg-6/igt@kms_cursor_crc@cursor-susp...@pipe-d-hdmi-a-3.html#dmesg-warnings488
>
Quoting Sebastian Brzezinka (2025-07-18 12:28:11)
> The value 4095 is intended to represent PAGE_SIZE - 1, as stated
> in the commit that introduced this change:
> commit 5f22cc0b134a ("drm/i915: Fix mismatch between misplaced
> vma check and vma insert")
No, that is exactly
Add job that executes the IGT test suite for mt8192-asurada-spherion-r0.
Signed-off-by: Deborah Brouwer
Signed-off-by: Vignesh Raman
---
drivers/gpu/drm/ci/arm64.config | 4 +++
drivers/gpu/drm/ci/test.yml | 13 +--
.../drm/ci/xfails/mediatek-mt8192-fails.tx
Add job that executes the IGT test suite for rk3588-rock-5b.
Signed-off-by: Vignesh Raman
---
MAINTAINERS | 1 +
drivers/gpu/drm/ci/arm64.config | 5
drivers/gpu/drm/ci/build.sh | 1 +
drivers/gpu/drm/ci/gitlab-ci.yml
Uprev IGT to the latest version and update expectation files.
Signed-off-by: Vignesh Raman
---
drivers/gpu/drm/ci/gitlab-ci.yml | 2 +-
.../gpu/drm/ci/xfails/amdgpu-stoney-fails.txt | 2 +
.../drm/ci/xfails/amdgpu-stoney-flakes.txt| 7 ++
drivers/gpu/drm/ci/xfails/i915-aml
Uprev mesa to adapt to the latest changes in Mesa CI, including support
for firmware via LAVA overlays, removal of the python-artifacts job,
split container and build rules, use of the alpine container for LAVA
jobs, and various misc fixes.
Signed-off-by: Vignesh Raman
---
drivers/gpu/drm/ci/bui
asus-C436FA-Flip-hatch has fewer devices available in the LAVA lab and
drm-ci uses only 2 DUTs, causing tests to time out. Update drm-ci to
use puff instead of hatch so the tests can run on 5 DUTs.
Also increase parallel count for amly jobs to 3.
Signed-off-by: Vignesh Raman
---
drivers/gpu/drm
The sm8350-hdk jobs are short and each test takes around 2–3 minutes and
the full job completes in about 10 minutes. Running 4 parallel jobs uses
4 devices at once, which is not needed. Set parallel to 2 to reduce
device usage.
Signed-off-by: Vignesh Raman
---
drivers/gpu/drm/ci/test.yml | 2 +-
These devices are being moved to LAVA. The jobs will be enabled and the
job definitions will be updated once the move is complete.
Signed-off-by: Vignesh Raman
---
drivers/gpu/drm/ci/test.yml | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/ci/test.yml b/
This series introduces new jobs to drm-ci for testing the following
devices:
- rk3588-rock-5b
- mt8192-asurada-spherion-r0
Other updates include:
- Uprev IGT and updating test expectations accordingly.
- Adapting to recent changes in Mesa CI, such as:
- LAVA overlay-based firmware handling
-
On Fri, 2025-07-18 at 09:05 +0300, Hogander, Jouni wrote:
> On Thu, 2025-07-17 at 10:31 -0300, Gustavo Sousa wrote:
> > Quoting Jouni Högander (2025-07-17 03:32:58-03:00)
> > > We are seeing "dmesg-warn/abort - *ERROR* PHY * failed after 3
> > > retries"
> > > since we started configuring LFPS send
Introduce a separate `ret` variable to hold the return value in
`eb_relocate_vma()`, instead of overloading `remain` for both loop
control and error reporting.
Signed-off-by: Sebastian Brzezinka
---
v2 -> v3:
- Split commit to isolate individual changes
---
drivers/gpu/drm/i915/gem/i915_
Simplifies the control flow by replacing an empty
`if (likely(offset == 0))` block with a `continue` statement. This
improves readability and avoids unnecessary nesting.
Signed-off-by: Sebastian Brzezinka
---
v2 -> v3
- Split commit to isolate individual changes
---
.../gpu/drm/i915/gem/
The value 4095 is intended to represent PAGE_SIZE - 1, as stated
in the commit that introduced this change:
commit 5f22cc0b134a ("drm/i915: Fix mismatch between misplaced
vma check and vma insert")
Signed-off-by: Sebastian Brzezinka
---
v2 -> v3:
- Reword commit mess
According to the kernel coding style, if only one branch of a
conditional statement is a single statement, braces should
still be used in both branches.
Reviewed-by: Andi Shyti
Reviewed-by: Krzysztof Karas
Signed-off-by: Sebastian Brzezinka
---
v2 -> v3:
- Nothing has changed
---
drive
While debugging, I identified and addressed a few minor issues and
style inconsistencies that could be improved.
I ran some tests on these changes, and pushed them to trybot
first, no regressions were found.
Sebastian Brzezinka (4):
drm/i915: Add braces around the else block in clflush_write32(
The current iosys_map_clear() implementation reads the potentially
uninitialized 'is_iomem' boolean field to decide which union member
to clear. This causes undefined behavior when called on uninitialized
structures, as 'is_iomem' may contain garbage values like 0xFF.
UBSAN detects this as:
UB
Hi Krzysztof
On Thu Jul 17, 2025 at 2:38 PM UTC, Krzysztof Karas wrote:
> Hi Sebastian,
>
>> +++ b/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c
>> @@ -1529,6 +1529,7 @@ static int eb_relocate_vma(struct i915_execbuffer *eb,
>> struct eb_vma *ev)
>> struct drm_i915_gem_relocation_entry __use
Hi Sebastian,
On Thu, Jul 17, 2025 at 12:38:47PM +, Sebastian Brzezinka wrote:
> According to the kernel coding style, if only one branch of a
> conditional statement is a single statement, braces should
> still be used in both branches.
>
> Signed-off-by: Sebastian Brzezinka
Reviewed-by: A
58 matches
Mail list logo
