Hi,
https://patchwork.freedesktop.org/series/130966/ - Re-reported.
Thanks,
Tejasree
-Original Message-
From: Janusz Krzysztofik
Sent: Tuesday, March 12, 2024 8:45 PM
To: LGCI Bug Filing
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: ✗ Fi.CI.IGT: failure for drm/i915/hwmon: Fix lock
Hi,
https://patchwork.freedesktop.org/series/130643/ - Re-reported.
Thanks,
Tejasree
From: Chauhan, Shekhar
Sent: Wednesday, March 13, 2024 9:25 AM
To: LGCI Bug Filing
Cc: intel-gfx@lists.freedesktop.org; Roper, Matthew D
Subject: RE: ✗ Fi.CI.BAT: failure for drm/i915/dp: Increase idle patte
On Tue, 12 Mar 2024, "Sripada, Radhakrishna"
wrote:
> LGTM,
> Reviewed-by: Radhakrishna Sripada
Thanks for the review, pushed to drm-intel-next.
BR,
Jani.
--
Jani Nikula, Intel
On Tue, Mar 12, 2024 at 09:36:22PM -0700, Manasi Navare wrote:
> Thanks Stan for the cleanup around post disable non MST case, one comment
> below
>
> On Fri, Mar 8, 2024 at 5:11 AM Stanislav Lisovskiy
> wrote:
> >
> > Extract the "not-MST" stuff from intel_ddi_post_disable() so that
> > the who
From: Arnd Bergmann
The newly added function is not available without CONFIG_ACPI, causing
a build failure:
drivers/gpu/drm/i915/display/intel_bios.c:3424:24: error: implicit declaration
of function 'intel_opregion_vbt_present'; did you mean
'intel_opregion_asle_present'? [-Werror=implicit-fun
On Tue, 12 Mar 2024 16:26:00 +0200
Pekka Paalanen wrote:
> On Tue, 12 Mar 2024 08:30:34 +
> "Garg, Nemesa" wrote:
>
> > This KMS property is not implementing any formula
>
> Sure it is. Maybe Intel just does not want to tell what the algorithm
> is, or maybe it's even patented.
>
> > a
On Wed, 13 Mar 2024, Arnd Bergmann wrote:
> From: Arnd Bergmann
>
> The newly added function is not available without CONFIG_ACPI, causing
> a build failure:
>
> drivers/gpu/drm/i915/display/intel_bios.c:3424:24: error: implicit
> declaration of function 'intel_opregion_vbt_present'; did you mea
There are few things we need to do for bigjoiner, in order
to improve code maintenance and also make testing for Bigjoiner
easier.
Those series contain addition of bigjoiner force debugfs option,
in order to be able to force bigjoiner even if there is no display
support, also we refactor pipe vs tr
And we have at least one trivial place in
intel_ddi_update_active_dpll() where we can use it
immediately, so let's do that.
v2: - Fixed conflicts, part of patch didn't apply, because of master_crtc
rename(Stan)
Signed-off-by: Ville Syrjälä
Signed-off-by: Stanislav Lisovskiy
---
drivers/g
Reorganize the crtc disable path to only deal with the
master pipes/transcoders in intel_old_crtc_state_disables()
and offload the handling of joined pipes to hsw_crtc_disable().
This makes the whole thing much more sensible since we can
actually control the order in which we do the per-pipe vs.
pe
Extract the "not-MST" stuff from intel_ddi_post_disable() so that
the whole thing isn't so cluttered.
The bigjoiner slave handling was outside of the !MST check,
but it really should have been inside it as its the counterpart
to the master handling inside the check. So we pull that
in as well. The
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
intel_e
From: Vidya Srinivas
We need bigjoiner support with MST functionality
for MST monitor resolutions > 5K to work.
Adding support for the same.
v2: Addressed review comments from Jani.
Revert rejection of MST bigjoiner modes and add
functionality
v3: Fixed pipe_mismatch WARN for mst_master_transco
Unify the master vs. slave handling in
intel_ddi_post_disable_hdmi_or_sst() by looping over all the
pipes in one go.
This also lets us move the intel_crtc_vblank_off() calls to
happen in a consistent place vs. the transcoder disable.
Previously we did the master vs. slaves on different sides
of th
On Tue, Mar 12, 2024 at 09:48:33PM -0700, Manasi Navare wrote:
> Now when we enable bigjoiner for MST, in MST case
> intel_ddi_post_disable_hdmi_or_sst() function wont get called,
> Do we need similar changes for MST case to loop over the joined pipes
> in MST bigjoiner case?
>
> Manasi
Hi Manasi
> -Original Message-
> From: Hogander, Jouni
> Sent: Wednesday, March 6, 2024 4:15 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Ville Syrjälä ; Manna, Animesh
> ; Murthy, Arun R ;
> Hogander, Jouni
> Subject: [PATCH v3 1/6] drm/display: Add missing aux less alpm wake related
> bits
>
> -Original Message-
> From: Hogander, Jouni
> Sent: Wednesday, March 6, 2024 4:15 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Ville Syrjälä ; Manna, Animesh
> ; Murthy, Arun R ;
> Hogander, Jouni
> Subject: [PATCH v3 3/6] drm/i915/psr: Calculate aux less wake time
>
> Calculate aux
> -Original Message-
> From: Hogander, Jouni
> Sent: Wednesday, March 6, 2024 4:15 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Ville Syrjälä ; Manna, Animesh
> ; Murthy, Arun R ;
> Hogander, Jouni
> Subject: [PATCH v3 5/6] drm/i915/psr: Enable ALPM for eDP Panel replay
>
> Enable AL
> -Original Message-
> From: Hogander, Jouni
> Sent: Wednesday, March 6, 2024 4:15 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: Ville Syrjälä ; Manna, Animesh
> ; Murthy, Arun R ;
> Hogander, Jouni
> Subject: [PATCH v3 6/6] drm/i915/psr: Do not write ALPM configuration for
> PSR1 or D
On Fri, 08 Mar 2024, Thomas Zimmermann wrote:
> Acked-by: Thomas Zimmermann
>
> for the series.
Thanks, pushed the lot to drm-misc-next-fixes.
> Do you plan to make the return type an unsigned int eventually?
Not really. The hooks could still return -ENOMEM or something, with no
compiler warni
On Wed, 2024-03-13 at 11:14 +, Manna, Animesh wrote:
>
>
> > -Original Message-
> > From: Hogander, Jouni
> > Sent: Wednesday, March 6, 2024 4:15 PM
> > To: intel-gfx@lists.freedesktop.org
> > Cc: Ville Syrjälä ; Manna, Animesh
> > ; Murthy, Arun R
> > ;
> > Hogander, Jouni
> > Subj
On Wed, 2024-03-13 at 11:14 +, Manna, Animesh wrote:
>
>
> > -Original Message-
> > From: Hogander, Jouni
> > Sent: Wednesday, March 6, 2024 4:15 PM
> > To: intel-gfx@lists.freedesktop.org
> > Cc: Ville Syrjälä ; Manna, Animesh
> > ; Murthy, Arun R
> > ;
> > Hogander, Jouni
> > Subj
On Wed, 2024-03-13 at 11:15 +, Manna, Animesh wrote:
>
>
> > -Original Message-
> > From: Hogander, Jouni
> > Sent: Wednesday, March 6, 2024 4:15 PM
> > To: intel-gfx@lists.freedesktop.org
> > Cc: Ville Syrjälä ; Manna, Animesh
> > ; Murthy, Arun R
> > ;
> > Hogander, Jouni
> > Subj
Quoting Patchwork (2024-03-12 22:30:46-03:00)
>== Series Details ==
>
>Series: Enable LNL display (rev2)
>URL : https://patchwork.freedesktop.org/series/130689/
>State : failure
>
>== Summary ==
>
>CI Bug Log - changes from CI_DRM_14421_full -> Patchwork_130689v2_full
>===
On Wed, 2024-03-13 at 11:18 +, Manna, Animesh wrote:
>
>
> > -Original Message-
> > From: Hogander, Jouni
> > Sent: Wednesday, March 6, 2024 4:15 PM
> > To: intel-gfx@lists.freedesktop.org
> > Cc: Ville Syrjälä ; Manna, Animesh
> > ; Murthy, Arun R
> > ;
> > Hogander, Jouni
> > Subj
Hi!
On Mon, 2024-03-11 at 14:07 +0100, Thomas Hellström wrote:
> On Fri, 2024-03-08 at 13:13 +0530, Somalapuram, Amaranath wrote:
> > Patches are tested on AMD platform.
> > Repeated stress test on Unigine Heaven, memory full (VRAM + GTT +
> > system
> > SWAP), then free.
> > No errors/warning in
On Tue, 12 Mar 2024 13:36:31 -0300, Gustavo Sousa wrote:
> This series aims at providing the remaining patches for enabling display
> on Lunar Lake, which used Xe2LPD display IP.
>
> The first set of patches contains fixes and extra stuff required for
> supporting CDCLK on Xe2LPD:
>
> drm/i
On Tue, Mar 12, 2024 at 09:49:05AM -0700, Matt Roper wrote:
On Tue, Mar 12, 2024 at 01:36:33PM -0300, Gustavo Sousa wrote:
Currently, only Xe2LPD uses CDCLK PLL as the source of MDCLK and
previous display IPs use the CD2XCLK. There will be changes in code
paths common to those platforms that wil
This patch set is improving IO and fast wake lines calculation in PSR
code:
Use actual fast wake sync pulse count in calculation Implement getter
for IO buffer wake times and use that. Better presentation on how
these are calculated. Use calculation for display version < 12 as
well.
Also number
ALPM AUX-Wake fast wake sync pulse count is needed by PSR to calculate IO
wake and fast wake lines. Convert intel_dp_aux_fw_sync_len as non-static
to make it available for PSR code.
v2: use int instead of u8
Signed-off-by: Jouni Högander
Reviewed-by: Ville Syrjälä
---
drivers/gpu/drm/i915/disp
Bspec mentions 50 us for IO wake time and 32 us for fast wake time. 32 us
is most probably wrong as it doesn't meet the specification as fast wake
time is calculated in Bspec like this:
10..16 us (precharge) + 8 us (preamble) + 4 us (phy_wake) + 20 us
(tfw_exit_latency)
Instead of using these con
Current fast and IO wake lines calculation is assuming fast wake sync
length is 18 pulses. Let's improve this by checking actual length.
Add getter for IO buffer wake time and return 10 us there which was assumed
with static 42 us IO wake time. Upcoming patches will extent this for
different displ
Increasing number of fast wake sync pulses seem to fix problems with
certain PSR panels. This should be ok for other panels as well as the eDP
specification allows 10...16 precharge pulses and we are still within that
range.
v2: add comment explaining pulse count is increased
Closes: https://gitl
On Wed, Mar 13, 2024 at 03:32:21PM +0200, Jouni Högander wrote:
> Increasing number of fast wake sync pulses seem to fix problems with
> certain PSR panels. This should be ok for other panels as well as the eDP
> specification allows 10...16 precharge pulses and we are still within that
> range.
>
Hi Nirmoy,
On Tue, Mar 12, 2024 at 12:18:15PM +0100, Nirmoy Das wrote:
> Caching mode is HW dependent so pick a correct one using
> intel_gt_coherent_map_type().
>
> Cc: Andi Shyti
> Cc: Janusz Krzysztofik
> Cc: Jonathan Cavitt
> Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/10249
Hi Janusz,
On Mon, Mar 11, 2024 at 09:34:58PM +0100, Janusz Krzysztofik wrote:
> In i915 hwmon sysfs getter path we now take a hwmon_lock, then acquire an
> rpm wakeref. That results in lock inversion:
>
> <4> [197.079335] ==
> <4> [197.085473]
For Earlier platforms, the Pipe source size is 12-bits so
max pipe source width and height is 4096. For newer platforms it is
13-bits so theoretically max width/height is 8192. For few of the
earlier platforms the scaler did not use all bits of the PIPESRC,
so max scaler source size was used to mak
== Series Details ==
Series: drm/i915: add intel_opregion_vbt_present() stub function
URL : https://patchwork.freedesktop.org/series/131060/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
On Wed, Mar 13, 2024 at 07:27:36PM +0300, Nikita Kiryushin wrote:
>
> Remove IS_KABYLAKE and IS_SKYLAKE in special handlers for
> skylake and kabylake: the checks are done at hook initialization and are
> always true in corresponding handlers.
>
> Signed-off-by: Nikita Kiryushin
Reviewed-by: Ma
== Series Details ==
Series: drm/i915: add intel_opregion_vbt_present() stub function
URL : https://patchwork.freedesktop.org/series/131060/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14426 -> Patchwork_131060v1
Summary
== Series Details ==
Series: Bigjoiner refactoring (rev10)
URL : https://patchwork.freedesktop.org/series/128311/
State : warning
== Summary ==
Error: dim checkpatch failed
96075b92e19f drm/i915: Add a small helper to compute the set of pipes for crtc
8307ecb8447f drm/i915: Extract intel_ddi_p
== Series Details ==
Series: Bigjoiner refactoring (rev10)
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: warni
== Series Details ==
Series: Bigjoiner refactoring (rev10)
URL : https://patchwork.freedesktop.org/series/128311/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14426 -> Patchwork_128311v10
Summary
---
**FAILURE**
tree/branch:
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
branch HEAD: dad309222e4c3fc7f88b20ce725ce1e0eea07cc7 Add linux-next specific
files for 20240313
Error/Warning reports:
https://lore.kernel.org/oe-kbuild-all/202403131859.szdcjzfy-...@intel.com
Error
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,
> but sometimes just a black screen. This commit w
== Series Details ==
Series: drm/i915: Rename ICL_PORT_TX_DW6 bits (rev2)
URL : https://patchwork.freedesktop.org/series/130899/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14427 -> Patchwork_130899v2
Summary
---
*
Commit 9bb66c179f50 ("drm/i915: Reserve some kernel space per
vm") has reserved an object for kernel space usage.
Userspace, though, needs to know the full address range.
Fixes: 9bb66c179f50 ("drm/i915: Reserve some kernel space per vm")
Signed-off-by: Andi Shyti
Cc: Andrzej Hajda
Cc: Chris Wil
== Series Details ==
Series: drm/i915: Use drm_printer more (rev8)
URL : https://patchwork.freedesktop.org/series/129956/
State : warning
== Summary ==
Error: dim checkpatch failed
bfbc14be30be drm/i915: Indicate which pipe failed the fastset check overall
b4f30f44c52b drm/i915: Include CRTC i
== Series Details ==
Series: drm/i915: Use drm_printer more (rev8)
URL : https://patchwork.freedesktop.org/series/129956/
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: Use drm_printer more (rev8)
URL : https://patchwork.freedesktop.org/series/129956/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14427 -> Patchwork_129956v8
Summary
---
**SUCCES
Hi,
this series does basically two things:
1. Disables automatic load balancing as adviced by the hardware
workaround.
2. Assigns all the CCS slices to one single user engine. The user
will then be able to query only one CCS engine
>From v5 I have created a new file, gt/intel_gt_ccs_mode.
The hardware should not dynamically balance the load between CCS
engines. Wa_14019159160 recommends disabling it across all
platforms.
Fixes: d2eae8e98d59 ("drm/i915/dg2: Drop force_probe requirement")
Signed-off-by: Andi Shyti
Cc: Chris Wilson
Cc: Joonas Lahtinen
Cc: Matt Roper
Cc: # v6.2+
R
We want a fixed load CCS balancing consisting in all slices
sharing one single user engine. For this reason do not create the
intel_engine_cs structure with its dedicated command streamer for
CCS slices beyond the first.
Fixes: d2eae8e98d59 ("drm/i915/dg2: Drop force_probe requirement")
Signed-off
Enable only one CCS engine by default with all the compute sices
allocated to it.
While generating the list of UABI engines to be exposed to the
user, exclude any additional CCS engines beyond the first
instance.
This change can be tested with igt i915_query.
Fixes: d2eae8e98d59 ("drm/i915/dg2:
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:
divide error: [#1] PREEMPT SMP PTI
CPU: 7 PID: 2962 Comm: Xorg Not tainted 6.8.0-rc3+ #744
Hardw
== Series Details ==
Series: drm/i915: Use container_of_const() (rev2)
URL : https://patchwork.freedesktop.org/series/130868/
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: Use container_of_const() (rev2)
URL : https://patchwork.freedesktop.org/series/130868/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14428 -> Patchwork_130868v2
Summary
---
**SU
== Series Details ==
Series: IO and fast wake lines calculation and increase fw sync length (rev6)
URL : https://patchwork.freedesktop.org/series/130173/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
+./ar
== Series Details ==
Series: IO and fast wake lines calculation and increase fw sync length (rev6)
URL : https://patchwork.freedesktop.org/series/130173/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14428 -> Patchwork_130173v6
=
== Series Details ==
Series: drm/i915/scaler: Update Pipe src size check in skl_update_scaler
URL : https://patchwork.freedesktop.org/series/131078/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14428 -> Patchwork_131078v1
== Series Details ==
Series: drm/i915/gt: Report full vm address range
URL : https://patchwork.freedesktop.org/series/131095/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14428 -> Patchwork_131095v1
Summary
---
**FA
== Series Details ==
Series: Disable automatic load CCS load balancing (rev9)
URL : https://patchwork.freedesktop.org/series/129951/
State : warning
== Summary ==
Error: dim checkpatch failed
06497016416d drm/i915/gt: Disable HW load balancing for CCS
c3197ab37c84 drm/i915/gt: Do not generate
== Series Details ==
Series: Disable automatic load CCS load balancing (rev9)
URL : https://patchwork.freedesktop.org/series/129951/
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/bi
== Series Details ==
Series: Disable automatic load CCS load balancing (rev9)
URL : https://patchwork.freedesktop.org/series/129951/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14428 -> Patchwork_129951v9
Summary
---
== Series Details ==
Series: Fix divide-by-zero regression on DP MST unplug with nouveau (rev2)
URL : https://patchwork.freedesktop.org/series/131002/
State : warning
== Summary ==
Error: dim checkpatch failed
64a2dd5de4fd Fix divide-by-zero regression on DP MST unplug with nouveau
-:14: WARNI
== Series Details ==
Series: Fix divide-by-zero regression on DP MST unplug with nouveau (rev2)
URL : https://patchwork.freedesktop.org/series/131002/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14428 -> Patchwork_131002v2
== Series Details ==
Series: drm/i915/display: Fixed a screen flickering when turning on display
from off (rev3)
URL : https://patchwork.freedesktop.org/series/130780/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14428 -> Patchwork_130780v3
==
Commit 9bb66c179f50 ("drm/i915: Reserve some kernel space per
vm") has reserved an object for kernel space usage.
Userspace, though, needs to know the full address range.
Fixes: 9bb66c179f50 ("drm/i915: Reserve some kernel space per vm")
Signed-off-by: Andi Shyti
Cc: Andrzej Hajda
Cc: Chris Wil
Some of the backported intel_uncore_read*() functions used the wrong
types. Change the function declarations accordingly.
Reviewed-by: Gustavo Sousa
Signed-off-by: Luca Coelho
---
drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
69 matches
Mail list logo
