== Series Details ==
Series: Fix i915 pmu on bind/unbind (rev2)
URL : https://patchwork.freedesktop.org/series/136362/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15522 -> Patchwork_136362v2
Summary
---
**SUCCESS**
== Series Details ==
Series: Fix i915 pmu on bind/unbind (rev2)
URL : https://patchwork.freedesktop.org/series/136362/
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: Fix i915 pmu on bind/unbind (rev2)
URL : https://patchwork.freedesktop.org/series/136362/
State : warning
== Summary ==
Error: dim checkpatch failed
73e46040952e squashed series - 20241008183501.1354695-1-lucas.demar...@intel.com
-:7: WARNING:COMMIT_LOG_LONG_LINE:
== Series Details ==
Series: drm/i915: Fix irq related documentation
URL : https://patchwork.freedesktop.org/series/139915/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15520 -> Patchwork_139915v1
Summary
---
**SUCC
Setting event_init to NULL is mostly done to detect when the driver is
partially working: i915 probed, but pmu is not registered. However,
checking for event_init is odd as it was supposed to always be set and
kernel/events/ would just crash if it found it set to NULL.
Since there's already a "clo
Since i915 calls perf_pmu_register/perf_pmu_unregister, let's call the
variable "registered" so we can flip the logic and rely on it being
false by default. Looking at other drivers, it's also more common.
Examples: arch/x86/events/intel/uncore.c and
drivers/powercap/intel_rapl_common.c.
Signed-of
When unregistering the PMU, disable the active events. This
allows userspace to see the change and possibly react on it, like
reopening the fd.
With perf-stat, "" starts to be printed:
2.007597571500,909,500 ns i915__03_00.0/rc6-residency/
2.508569383
There's no need to free the resources during unbind. Since perf events
may still access them due to open events, it's safer to free them when
dropping the last i915 reference. It will also allow to ask perf to
release its own resources when dropping the last reference in a follow
up change.
Signed
Patches 2 and 3 of
https://lore.kernel.org/all/20241008183501.1354695-1-lucas.demar...@intel.com/
with the unused module var removed (pending for v2 after review).
Signed-off-by: Lucas De Marchi
---
include/linux/perf_event.h | 12
kernel/events/core.c | 37 ++
This is already done inside perf_pmu_unregister() - no need to do it
before.
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/i915_pmu.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_pmu.c b/drivers/gpu/drm/i915/i915_pmu.c
index e28c29
Both the documentation and most of other users call the return of
cpuhp_setup_state_multi() as "state". Follow that.
Signed-off-by: Lucas De Marchi
---
drivers/gpu/drm/i915/i915_pmu.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_pmu
When an i915 PMU counter is enabled and the driver is then unbound, the
PMU will be unregistered via perf_pmu_unregister(), however the event
will still be alive. i915 currently tries to deal with this situation
by:
a) Marking the pmu as "closed" and shortcut the calls from perf
b)
Add i915 changes on top of the fixes proposed at
https://lore.kernel.org/all/20241008183501.1354695-1-lucas.demar...@intel.com/
First patch is the patches needed from that series, squashed to avoid
reviewing them here and multiple versions.
Second and third patches fix the issues with i915 unbind
== Series Details ==
Series: drm/i915/guc: Enable PXP GuC autoteardown flow (rev4)
URL : https://patchwork.freedesktop.org/series/138337/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15519 -> Patchwork_138337v4
Summary
---
On Thu, Oct 10, 2024 at 03:43:04PM -0700, Matt Atwood wrote:
> From: Matt Roper
>
> Xe3's power well handling is similar to previous platforms, but there
> are a few changes that need to be handled to ensure optimal power
> management:
> - PGB now only depends on PG1, not PG2
> - Transcoder B i
On Thu, Oct 10, 2024 at 03:43:08PM -0700, Matt Atwood wrote:
> From: Suraj Kandpal
>
> From DISPLAY_VER() >= 30 C20 PHY consolidated programming table of
> DP and eDP been merged and now use the same rates and values. eDP
> over TypeC has also been introduced.
> Moreover it allows more granular a
Also update the function names in the documentation.
Reported-by: Stephen Rothwell
Closes:
https://lore.kernel.org/intel-gfx/20241001134331.7b4d4...@canb.auug.org.au/
Fixes: 3de5774cb8c0 ("drm/i915/irq: Rename suspend/resume functions")
Cc: Jonathan Cavitt
Cc: Andi Shyti
Signed-off-by: Rodrigo
== Series Details ==
Series: drm/i915/guc: Enable PXP GuC autoteardown flow (rev4)
URL : https://patchwork.freedesktop.org/series/138337/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
> -Original Message-
> From: Nikula, Jani
> Sent: Friday, October 11, 2024 1:28 AM
> To: Lin, Shuicheng ; intel...@lists.freedesktop.org;
> intel-gfx@lists.freedesktop.org
> Cc: Lin, Shuicheng
> Subject: Re: [PATCH] drm/i915/dp: Fix memory leak in parse_lfp_panel_dtd()
>
> On Thu, 10 Oct
On Thu, Oct 10, 2024 at 03:43:06PM -0700, Matt Atwood wrote:
> From: Radhakrishna Sripada
>
> Xe3_LPD has new max cdclk of 691200 which requires reusing the lnl table
> and modify/add higher frequencies. Updating the max cdclk supported by
> the platform and voltage_level determination is also up
== Series Details ==
Series: drm/i915: Async flip + compression, and some plane cleanups (rev3)
URL : https://patchwork.freedesktop.org/series/139807/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15518 -> Patchwork_139807v3
== Series Details ==
Series: Revert "perf/core: Only copy-to-user after completely unlocking all
locks, v3."
URL : https://patchwork.freedesktop.org/series/139903/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15518 -> Patchwork_139903v1
==
== Series Details ==
Series: drm/i915: Async flip + compression, and some plane cleanups (rev3)
URL : https://patchwork.freedesktop.org/series/139807/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
+./arch/
== Series Details ==
Series: drm/i915: Async flip + compression, and some plane cleanups (rev3)
URL : https://patchwork.freedesktop.org/series/139807/
State : warning
== Summary ==
Error: dim checkpatch failed
2f18535a21c4 drm/i915: Allow async flips with render compression on TGL+
9c7ea262ad1
This reverts commit 80e846398c695dc3d0d2eac6da91772ff2aade65.
Let's see what CI says about reverting this 4 ~ 5 years old commit. We
shouldn't be in the business of changing perf/core in our "for CI" drm
branch.
Signed-off-by: Lucas De Marchi
---
kernel/events/core.c | 49 ++
On Thu, 10 Oct 2024, Matt Atwood wrote:
> From: Matt Roper
>
> There are some minor changes to pmdemand handling on Xe3:
> - Active scalers are no longer tracked. We can simply skip the readout
>and programming of this field.
> - Active dbuf slices are no longer tracked. We should skip th
On Fri, Oct 11, 2024 at 03:17:29PM +0300, Jani Nikula wrote:
> This reverts commit 8d16a118950c ("ICL HACK: usb/icl: Work around ACPI
> boottime crash").
>
> There shouldn't be any ICL RVP's in CI anymore.
>
> Cc: Imre Deak
> Signed-off-by: Jani Nikula
Acked-by: Imre Deak
> ---
> drivers/us
On Fri, Oct 11, 2024 at 02:26:22PM +0300, Imre Deak wrote:
> On Thu, Oct 10, 2024 at 11:47:49AM +, Patchwork wrote:
> > == Series Details ==
> >
> > Series: drm/xe: Fix HPD interrupt enabling during runtime resume
> > URL : https://patchwork.freedesktop.org/series/139812/
> > State : failure
== Series Details ==
Series: Revert "ICL HACK: usb/icl: Work around ACPI boottime crash"
URL : https://patchwork.freedesktop.org/series/139886/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15517 -> Patchwork_139886v1
Summa
Quoting Matt Atwood (2024-10-10 19:43:05-03:00)
>From: Matt Roper
>
>There are some minor changes to pmdemand handling on Xe3:
> - Active scalers are no longer tracked. We can simply skip the readout
> and programming of this field.
> - Active dbuf slices are no longer tracked. We should skip
On 11-10-2024 11:52, Anirban, Sk wrote:
On 10-10-2024 23:45, Pottumuttu, Sai Teja wrote:
On 10-10-2024 23:04, Sk Anirban wrote:
Introduce RC6 & RC0 frequency logging mechanism to ensure accurate
energy readings aimed at addressing GPU energy leaks and power
measurement failures.
This enhan
This reverts commit 8d16a118950c ("ICL HACK: usb/icl: Work around ACPI
boottime crash").
There shouldn't be any ICL RVP's in CI anymore.
Cc: Imre Deak
Signed-off-by: Jani Nikula
---
drivers/usb/core/usb-acpi.c | 25 -
1 file changed, 25 deletions(-)
diff --git a/driver
On Fri, Oct 11, 2024 at 03:44:54AM +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/xe: Fix HPD interrupt enabling during runtime resume
> URL : https://patchwork.freedesktop.org/series/139813/
> State : failure
Thanks for the review, pushed patches 1 and 2 to drm-intel-next.
The
== Series Details ==
Series: Implement Wa_14022698537
URL : https://patchwork.freedesktop.org/series/139883/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_15514 -> Patchwork_139883v1
Summary
---
**FAILURE**
Seriou
== Series Details ==
Series: Implement Wa_14022698537
URL : https://patchwork.freedesktop.org/series/139883/
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: Implement Wa_14022698537
URL : https://patchwork.freedesktop.org/series/139883/
State : warning
== Summary ==
Error: dim checkpatch failed
132c044fd5e1 drm/i915/pciids: Refactor DG2 PCI IDs into workaround ranges
-:18: ERROR:COMPLEX_MACRO: Macros with complex value
Hi Dave & Sima -
Here's the first i915 feature pull towards v6.13.
drm-intel-next-2024-10-11:
drm/i915 features for v6.13:
Features and functionality:
- Enable BMG and LNL+ ultra joiner support to join 2+2 pipes (Ankit, Stan)
- Enable 10bpc+CCS scanout for ICL+ and fp16+CCS scanout for TGL+ (V
On Fri, 11 Oct 2024, Raag Jadav wrote:
> This series implements Wa_14022698537 along with its prerequisites.
> Detailed description in commit message.
The same workaround is also needed for the xe driver, right?
Please include that in the same series, so that we can unify the
solution in a way t
On Fri, 11 Oct 2024, Raag Jadav wrote:
> Having similar naming convention in intel-family.h and intel_device_info.h
> results in redefinition of a few platforms. Define CPU IDs in its own file
> to avoid this.
>
> Signed-off-by: Raag Jadav
> ---
> drivers/gpu/drm/i915/Makefile | 1
On Fri, 11 Oct 2024, Raag Jadav wrote:
> Introduce DG2_WA subplatform for the devices that will be used in a
> workaround and span across multiple DG2 subplatforms.
Ditto, give the subplatform a name other than "WA". Look up the specs,
what's in common?
> Signed-off-by: Raag Jadav
> ---
> driv
On Fri, 11 Oct 2024, Raag Jadav wrote:
> Refactor DG2 PCI IDs into device ranges that will be used in a workaround.
Give the PCI ID ranges a name other than "WA". What are they?
BR,
Jani.
>
> Signed-off-by: Raag Jadav
> ---
> include/drm/intel/i915_pciids.h | 34 +++---
== Series Details ==
Series: series starting with [v1,1/2] drm/dp: fix warning for target_rr_divide
missing comment
URL : https://patchwork.freedesktop.org/series/139881/
State : failure
== Summary ==
Error: patch
https://patchwork.freedesktop.org/api/1.0/series/139881/revisions/1/mbox/ not
G8 power state entry is disabled due to a limitation on DG2, so we
enable it from driver with Wa_14022698537. Fow now we enable it for
all DG2 devices with the exception of a few, for which, we enable
only when paired with whitelisted CPU models.
v2: Fix Wa_ID and include it in subject (Badal)
Having similar naming convention in intel-family.h and intel_device_info.h
results in redefinition of a few platforms. Define CPU IDs in its own file
to avoid this.
Signed-off-by: Raag Jadav
---
drivers/gpu/drm/i915/Makefile | 1 +
drivers/gpu/drm/i915/gt/intel_wa_cpu.c | 34
Refactor DG2 PCI IDs into device ranges that will be used in a workaround.
Signed-off-by: Raag Jadav
---
include/drm/intel/i915_pciids.h | 34 +++--
1 file changed, 24 insertions(+), 10 deletions(-)
diff --git a/include/drm/intel/i915_pciids.h b/include/drm/intel/i91
Introduce DG2_WA subplatform for the devices that will be used in a
workaround and span across multiple DG2 subplatforms.
Signed-off-by: Raag Jadav
---
drivers/gpu/drm/i915/i915_drv.h | 2 ++
drivers/gpu/drm/i915/intel_device_info.c | 34 +---
drivers/gpu/drm/i915/i
This series implements Wa_14022698537 along with its prerequisites.
Detailed description in commit message.
v2: Introduce DG2_WA subplatform for workaround (Jani)
Fix Wa_ID and include it in subject (Badal)
Rephrase commit message (Jani)
Move CPU whitelist to intel_wa_cpu.c
v1: https:/
== Series Details ==
Series: Some correction in the DP Link Training sequence (rev4)
URL : https://patchwork.freedesktop.org/series/139027/
State : warning
== Summary ==
Error: dim checkpatch failed
153513093c6d drm/i915/dp: use fsleep instead of usleep_range for LT
-:12: WARNING:BAD_SIGN_OFF:
== Series Details ==
Series: Some correction in the DP Link Training sequence (rev4)
URL : https://patchwork.freedesktop.org/series/139027/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15514 -> Patchwork_139027v4
Summary
-
On Thu, 2024-10-10 at 15:06 +0300, Ville Syrjälä wrote:
> On Thu, Oct 10, 2024 at 08:33:11AM +0300, Jouni Högander wrote:
> > Currently vrr code is overwriting possibly set PSR PR Frame Change
> > Enable
> > bit in TRANS_PUSH register. Avoid this by using rmw instead of
> > write.
>
> RMWs are not
== Series Details ==
Series: Add xe3lpd edp enabling (rev2)
URL : https://patchwork.freedesktop.org/series/139731/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15514 -> Patchwork_139731v2
Summary
---
**SUCCESS**
== Series Details ==
Series: Add xe3lpd edp enabling (rev2)
URL : https://patchwork.freedesktop.org/series/139731/
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: Add xe3lpd edp enabling (rev2)
URL : https://patchwork.freedesktop.org/series/139731/
State : warning
== Summary ==
Error: dim checkpatch failed
21482e49e44a drm/i915/xe3lpd: reuse xe2lpd definition
-:67: ERROR:COMPLEX_MACRO: Macros with complex values should be en
On Thu, 2024-10-03 at 17:44 +0200, Maarten Lankhorst wrote:
> This is already handled below by fixup_initial_plane_config.
>
> Signed-off-by: Maarten Lankhorst
> Fixes: a8153627520a ("drm/i915: Try to relocate the BIOS fb to the start of
> ggtt")
> Cc: Ville Syrjälä
> ---
> drivers/gpu/drm/xe/
== Series Details ==
Series: drm/i915/selftests: Implement frequency logging for energy reading
validation (rev4)
URL : https://patchwork.freedesktop.org/series/139235/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15514 -> Patchwork_139235v4
=
== Series Details ==
Series: drm/i915/selftests: Implement frequency logging for energy reading
validation (rev4)
URL : https://patchwork.freedesktop.org/series/139235/
State : warning
== Summary ==
Error: dim checkpatch failed
ff83961d3b48 drm/i915/selftests: Implement frequency logging for
== Series Details ==
Series: drm/i915: Async flip + compression, and some plane cleanups (rev2)
URL : https://patchwork.freedesktop.org/series/139807/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_15514 -> Patchwork_139807v2
== Series Details ==
Series: drm/i915: Async flip + compression, and some plane cleanups (rev2)
URL : https://patchwork.freedesktop.org/series/139807/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
+./arch/
== Series Details ==
Series: drm/i915: Async flip + compression, and some plane cleanups (rev2)
URL : https://patchwork.freedesktop.org/series/139807/
State : warning
== Summary ==
Error: dim checkpatch failed
ada3e3a900a5 drm/i915: Allow async flips with render compression on TGL+
2373e6a9ee6
On Thu, Oct 10, 2024 at 08:02:10AM -0500, Lucas De Marchi wrote:
> On Tue, Oct 08, 2024 at 06:02:43PM +0300, Raag Jadav wrote:
> > On Thu, Oct 03, 2024 at 03:23:22PM +0300, Raag Jadav wrote:
> > > On Tue, Oct 01, 2024 at 02:20:29PM +0200, Michal Wajdeczko wrote:
> > > > On 30.09.2024 09:38, Raag Ja
== Series Details ==
Series: drm/i915/dp: Fix memory leak in parse_lfp_panel_dtd()
URL : https://patchwork.freedesktop.org/series/139856/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15514 -> Patchwork_139856v1
Summary
---
On Thu, 10 Oct 2024, Shuicheng Lin wrote:
> The function parse_lfp_panel_dtd() is called when the driver
> attempts to initialize the eDP connector, and it allocates memory,
> which is recorded in panel->vbt.lfp_vbt_mode. However, since no
> eDP panel is connected, the driver fails at intel_edp_in
include/drm/display/drm_dp_helper.h:127: warning:
Function parameter or struct member 'target_rr_divider'
not described in 'drm_dp_as_sdp'
Signed-off-by: Sunil Khatri
Cc: Mitul Golani
Cc: Arun R Murthy
Cc: Suraj Kandpal
---
include/drm/display/drm_dp_helper.h | 1 +
1 file changed, 1 insertio
Fixes:
include/uapi/drm/drm_mode.h:869:
warning: Function parameter or struct member
'width' not described in 'drm_plane_size_hint'
include/uapi/drm/drm_mode.h:869:
warning: Function parameter or struct member
'height' not described in 'drm_plane_size_hint'
Signed-off-by: Sunil Khatri
Cc: Ville
On Thu, 10 Oct 2024, Matt Atwood wrote:
> From: Suraj Kandpal
>
> Read PICA register to see if edp over type C is possible and then
> add the appropriate tables for it.
>
> Bspec: 68846
> Signed-off-by: Suraj Kandpal
> Signed-off-by: Matt Atwood
This patch was already rejected. Please don't se
Hi Janusz,
On 2024-10-10 at 15:49:06 +0200, Janusz Krzysztofik wrote:
> A few tests depend on for_each_mmap_offset_type() helper used for
> selection of device supported mmap_offset types in loops either around
> dynamic sub-subtests or inside subtest bodies. That helper in turn
> calls gem_has_mm
On Thu, 10 Oct 2024, Matt Atwood wrote:
> From: Suraj Kandpal
>
> hblank restriction now includes all of xe3.
>
> v2: add additional definition instead of function, commit message typo
> fix and update.
>
> Signed-off-by: Suraj Kandpal
> Signed-off-by: Matt Atwood
> ---
> drivers/gpu/drm/i915/
== Series Details ==
Series: drm/i915: move segment iterator to match current offset (rev2)
URL : https://patchwork.freedesktop.org/series/139687/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15514 -> Patchwork_139687v2
Su
== Series Details ==
Series: drm/i915/ddi: use string choice helpers
URL : https://patchwork.freedesktop.org/series/139846/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_15514 -> Patchwork_139846v1
Summary
---
**FAIL
These regressions have nothing to do with my changes. My changes
affected only Xe driver.
/Juha-Pekka
On 9.10.2024 10.11, Patchwork wrote:
*Patch Details*
*Series:* drm/i915/display: Don't allow tile4 framebuffer to do hflip on
display20 or greater
*URL:* https://patchwork.freedesktop.org/ser
== Series Details ==
Series: drm/i915/ddi: use string choice helpers
URL : https://patchwork.freedesktop.org/series/139846/
State : warning
== Summary ==
Error: dim checkpatch failed
dd9ecbf84c0a drm/i915/ddi: use string choice helpers
-:9: WARNING:BAD_REPORTED_BY_LINK: Reported-by: should be
On 11-10-2024 03:00, Taylor, Clinton A wrote:
On Wed, 2024-10-09 at 22:32 +0530, Pottumuttu, Sai Teja wrote:
On 05-10-2024 02:38, Clint Taylor wrote:
Some devices NAK DPCD writes to the SOURCE OUI (0x300) DPCD registers.
Reduce the log level priority to prevent dmesg noise for these devices.
72 matches
Mail list logo
