Re: [PATCH 7/7] drm/i915/psr: Disable early transport by default

2024-01-09 Thread Hogander, Jouni
On Tue, 2024-01-09 at 12:57 +, Kahola, Mika wrote: > > -Original Message- > > From: Intel-gfx On Behalf > > Of Jouni Högander > > Sent: Monday, December 18, 2023 7:50 PM > > To: intel-gfx@lists.freedesktop.org > > Subject: [PATCH 7/7] drm/i915/psr: Disable early transport by > >

✓ Fi.CI.BAT: success for drm/i915/gt: Use rc6.supported flag from intel_gt for rc6_enable sysfs (rev2)

2024-01-09 Thread Patchwork
== Series Details == Series: drm/i915/gt: Use rc6.supported flag from intel_gt for rc6_enable sysfs (rev2) URL : https://patchwork.freedesktop.org/series/128343/ State : success == Summary == CI Bug Log - changes from CI_DRM_14103 -> Patchwork_128343v2

[PATCH] drm/i915/gt: Use rc6.supported flag from intel_gt for rc6_enable sysfs

2024-01-09 Thread Juan Escamilla
Currently if rc6 is supported, it gets enabled and the sysfs files for rc6_enable_show and rc6_enable_dev_show uses masks to check information from drm_i915_private. However rc6_support functions take more variables and conditions into consideration and thus these masks are not enough for most of

Re: ✗ Fi.CI.IGT: failure for Resolve suspend-resume racing with GuC destroy-context-worker (rev13)

2024-01-09 Thread Rodrigo Vivi
On Thu, Jan 04, 2024 at 05:39:16PM +, Teres Alexis, Alan Previn wrote: > On Thu, 2024-01-04 at 10:57 +, Patchwork wrote: > > Patch Details > > Series: Resolve suspend-resume racing with GuC destroy-context-worker > > (rev13) > > URL:https://patchwork.freedesktop.org/series/121916/ > >

Re: [PATCH 2/7] drm/uAPI: Add "active color format" drm property as feedback for userspace

2024-01-09 Thread Daniel Stone
Hi, On Tue, 9 Jan 2024 at 18:12, Andri Yngvason wrote: > + * active color format: > + * This read-only property tells userspace the color format actually used > + * by the hardware display engine "on the cable" on a connector. The > chosen > + * value depends on hardware

Re: [PATCH] drm/xe/display: Disable aux ccs framebuffers

2024-01-09 Thread Souza, Jose
On Mon, 2024-01-08 at 17:18 -0500, Rodrigo Vivi wrote: > On Tue, Jan 02, 2024 at 09:44:48PM +0200, Jani Nikula wrote: > > On Tue, 02 Jan 2024, Juha-Pekka Heikkila > > wrote: > > > Aux ccs framebuffers don't work on Xe driver hence disable them > > > from plane capabilities until they are fixed.

Re: [PATCH] drm/xe/display: Disable aux ccs framebuffers

2024-01-09 Thread Souza, Jose
On Tue, 2024-01-02 at 20:24 +0200, Juha-Pekka Heikkila wrote: > Aux ccs framebuffers don't work on Xe driver hence disable them > from plane capabilities until they are fixed. Flat ccs framebuffers > work and they are left enabled. Here is separated plane capabilities > check on i915 so it can

Re: ✗ Fi.CI.IGT: failure for Resolve suspend-resume racing with GuC destroy-context-worker (rev13)

2024-01-09 Thread Matt Roper
On Thu, Jan 04, 2024 at 05:39:16PM +, Teres Alexis, Alan Previn wrote: > On Thu, 2024-01-04 at 10:57 +, Patchwork wrote: > > Patch Details > > Series: Resolve suspend-resume racing with GuC destroy-context-worker > > (rev13) > > URL:https://patchwork.freedesktop.org/series/121916/ > >

Re: [PATCH 3/5] drm/ttm: replace busy placement with flags v5

2024-01-09 Thread Zack Rusin
On Tue, Jan 9, 2024 at 2:47 AM Christian König wrote: > > From: Somalapuram Amaranath > > Instead of a list of separate busy placement add flags which indicate > that a placement should only be used when there is room or if we need to > evict. > > v2: add missing TTM_PL_FLAG_IDLE for i915 > v3:

Re: [PATCH 1/3] drm/nouveau: include drm/drm_edid.h only where needed

2024-01-09 Thread Danilo Krummrich
On 1/9/24 10:59, Jani Nikula wrote: On Mon, 08 Jan 2024, Danilo Krummrich wrote: On 1/4/24 21:16, Jani Nikula wrote: Including drm_edid.h from nouveau_connector.h causes the rebuild of 15 files when drm_edid.h is modified, while there are only a few files that actually need to include

Re: [PATCH] drm/i915/guc: Avoid circular locking issue on busyness flush

2024-01-09 Thread Daniel Vetter
On Tue, Dec 19, 2023 at 11:59:57AM -0800, john.c.harri...@intel.com wrote: > From: John Harrison > > Avoid the following lockdep complaint: > <4> [298.856498] == > <4> [298.856500] WARNING: possible circular locking dependency detected > <4>

✓ Fi.CI.BAT: success for Allow PSR mode changes without full modeset

2024-01-09 Thread Patchwork
== Series Details == Series: Allow PSR mode changes without full modeset URL : https://patchwork.freedesktop.org/series/128360/ State : success == Summary == CI Bug Log - changes from CI_DRM_14100 -> Patchwork_128360v1 Summary ---

✗ Fi.CI.CHECKPATCH: warning for Allow PSR mode changes without full modeset

2024-01-09 Thread Patchwork
== Series Details == Series: Allow PSR mode changes without full modeset URL : https://patchwork.freedesktop.org/series/128360/ State : warning == Summary == Error: dim checkpatch failed 6dc8ca5bdd65 drm/i915/display: No need for full modeset due to psr d034710b2ab1 drm/i915/psr: CAN_PSR and

RE: [PATCH 7/7] drm/i915/psr: Disable early transport by default

2024-01-09 Thread Kahola, Mika
> -Original Message- > From: Intel-gfx On Behalf Of Jouni > Högander > Sent: Monday, December 18, 2023 7:50 PM > To: intel-gfx@lists.freedesktop.org > Subject: [PATCH 7/7] drm/i915/psr: Disable early transport by default > > Early transport validation is currently incomplete. Due to

RE: [PATCH 6/7] drm/i915/psr: Enable psr2 early transport as possible

2024-01-09 Thread Kahola, Mika
> -Original Message- > From: Intel-gfx On Behalf Of Jouni > Högander > Sent: Monday, December 18, 2023 7:50 PM > To: intel-gfx@lists.freedesktop.org > Subject: [PATCH 6/7] drm/i915/psr: Enable psr2 early transport as possible > > Check source and sink support for psr2 early transport

RE: [PATCH 5/7] drm/i915/psr: Configure PIPE_SRCSZ_ERLY_TPT for psr2 early transport

2024-01-09 Thread Kahola, Mika
> -Original Message- > From: Intel-gfx On Behalf Of Jouni > Högander > Sent: Monday, December 18, 2023 7:50 PM > To: intel-gfx@lists.freedesktop.org > Subject: [PATCH 5/7] drm/i915/psr: Configure PIPE_SRCSZ_ERLY_TPT for psr2 > early transport > > There is a new register used to

✓ Fi.CI.BAT: success for drm/i915: Cursor vblank evasion (rev2)

2024-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Cursor vblank evasion (rev2) URL : https://patchwork.freedesktop.org/series/127744/ State : success == Summary == CI Bug Log - changes from CI_DRM_14100 -> Patchwork_127744v2 Summary ---

✗ Fi.CI.SPARSE: warning for drm/i915: Cursor vblank evasion (rev2)

2024-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Cursor vblank evasion (rev2) URL : https://patchwork.freedesktop.org/series/127744/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

RE: [PATCH 4/7] drm/i915/psr: Calculate and configure CUR_POS_ERLY_TPT

2024-01-09 Thread Kahola, Mika
> -Original Message- > From: Intel-gfx On Behalf Of Jouni > Högander > Sent: Monday, December 18, 2023 7:50 PM > To: intel-gfx@lists.freedesktop.org > Subject: [PATCH 4/7] drm/i915/psr: Calculate and configure CUR_POS_ERLY_TPT > > New register CUR_POS_ERLY_TPT related to early transport

Re: [PATCH 9/9] Revert "drm/i915/xe2lpd: Treat cursor plane as regular plane for DDB allocation"

2024-01-09 Thread Lisovskiy, Stanislav
On Fri, Dec 15, 2023 at 01:15:16PM +0200, Ville Syrjälä wrote: > On Wed, Dec 13, 2023 at 05:29:12PM +0200, Ville Syrjälä wrote: > > On Wed, Dec 13, 2023 at 05:15:06PM +0200, Ville Syrjälä wrote: > > > On Wed, Dec 13, 2023 at 01:28:15PM +0200, Lisovskiy, Stanislav wrote: > > > > On Wed, Dec 13,

[PATCH 1/2] drm/i915/display: No need for full modeset due to psr

2024-01-09 Thread Jouni Högander
There is no specific reason to force full modeset if psr is enabled. Signed-off-by: Jouni Högander Tested-by: Paz Zcharya --- drivers/gpu/drm/i915/display/intel_display.c | 7 --- drivers/gpu/drm/i915/display/intel_dp.c | 7 --- 2 files changed, 14 deletions(-) diff --git

[PATCH 2/2] drm/i915/psr: CAN_PSR and CAN_PANEL_REPLAY can be now local defines

2024-01-09 Thread Jouni Högander
CAN_PSR and CAN_PANEL_REPLAY are not used outside intel_psr.c anymore. Make them as intel_psr.c local defines. Signed-off-by: Jouni Högander --- drivers/gpu/drm/i915/display/intel_psr.c | 6 ++ drivers/gpu/drm/i915/display/intel_psr.h | 6 -- 2 files changed, 6 insertions(+), 6

[PATCH 0/2] Allow PSR mode changes without full modeset

2024-01-09 Thread Jouni Högander
Currently PSR mode changes are triggering full modeset. This patch set allows changes to PSR mode using fast modeset. Jouni Högander (2): drm/i915/display: No need for full modeset due to psr drm/i915/psr: CAN_PSR and CAN_PANEL_REPLAY can be now local defines

Re: [PATCH 1/3] drm/nouveau: include drm/drm_edid.h only where needed

2024-01-09 Thread Jani Nikula
On Mon, 08 Jan 2024, Danilo Krummrich wrote: > On 1/4/24 21:16, Jani Nikula wrote: >> Including drm_edid.h from nouveau_connector.h causes the rebuild of 15 >> files when drm_edid.h is modified, while there are only a few files that >> actually need to include drm_edid.h. >> >> Cc: Karol Herbst

RE: [PATCH] drm/i915/display/dp: 128/132b DP-capable with SST

2024-01-09 Thread Jani Nikula
On Tue, 09 Jan 2024, Jani Nikula wrote: > On Tue, 09 Jan 2024, "Murthy, Arun R" wrote: >>> -Original Message- >>> From: Nikula, Jani >>> Sent: Monday, January 8, 2024 7:01 PM >>> To: Murthy, Arun R ; >>> intel-gfx@lists.freedesktop.org; >>> Deak, Imre >>> Cc: Murthy, Arun R >>>

RE: [PATCH] drm/i915/display/dp: 128/132b DP-capable with SST

2024-01-09 Thread Jani Nikula
On Tue, 09 Jan 2024, "Murthy, Arun R" wrote: >> -Original Message- >> From: Nikula, Jani >> Sent: Monday, January 8, 2024 7:01 PM >> To: Murthy, Arun R ; >> intel-gfx@lists.freedesktop.org; >> Deak, Imre >> Cc: Murthy, Arun R >> Subject: Re: [PATCH] drm/i915/display/dp: 128/132b

Re: [PATCH 2/5] drm/ttm: return ENOSPC from ttm_bo_mem_space

2024-01-09 Thread Thomas Hellström
On Tue, 2024-01-09 at 08:47 +0100, Christian König wrote: > Only convert it to ENOMEM in ttm_bo_validate. > Could we have a more elaborate commit description here, (why is this change needed)? > Signed-off-by: Christian König > --- >  drivers/gpu/drm/ttm/ttm_bo.c | 5 - >  1 file changed, 4

Re: [PATCH v2 1/3] drm/i915/gt: Support fixed CCS mode

2024-01-09 Thread Tvrtko Ursulin
On 08/01/2024 15:13, Joonas Lahtinen wrote: Quoting Tvrtko Ursulin (2024-01-05 12:39:31) On 04/01/2024 21:23, Andi Shyti wrote: +void intel_gt_apply_ccs_mode(struct intel_gt *gt) +{ + mutex_lock(>ccs.mutex); + __intel_gt_apply_ccs_mode(gt); + mutex_unlock(>ccs.mutex); +} + +void

Re: Rework TTMs busy handling

2024-01-09 Thread Christian König
Am 09.01.24 um 09:14 schrieb Thomas Hellström: Hi, Christian On Tue, 2024-01-09 at 08:47 +0100, Christian König wrote: Hi guys, I'm trying to make this functionality a bit more useful for years now since we multiple reports that behavior of drivers can be suboptimal when multiple placements

✓ Fi.CI.BAT: success for series starting with [1/5] drm/vmwgfx: remove vmw_vram_gmr_placement

2024-01-09 Thread Patchwork
== Series Details == Series: series starting with [1/5] drm/vmwgfx: remove vmw_vram_gmr_placement URL : https://patchwork.freedesktop.org/series/128355/ State : success == Summary == CI Bug Log - changes from CI_DRM_14099 -> Patchwork_128355v1

✗ Fi.CI.SPARSE: warning for series starting with [1/5] drm/vmwgfx: remove vmw_vram_gmr_placement

2024-01-09 Thread Patchwork
== Series Details == Series: series starting with [1/5] drm/vmwgfx: remove vmw_vram_gmr_placement URL : https://patchwork.freedesktop.org/series/128355/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

✗ Fi.CI.CHECKPATCH: warning for series starting with [1/5] drm/vmwgfx: remove vmw_vram_gmr_placement

2024-01-09 Thread Patchwork
== Series Details == Series: series starting with [1/5] drm/vmwgfx: remove vmw_vram_gmr_placement URL : https://patchwork.freedesktop.org/series/128355/ State : warning == Summary == Error: dim checkpatch failed 3215387b42ab drm/vmwgfx: remove vmw_vram_gmr_placement -:70:

Re: Rework TTMs busy handling

2024-01-09 Thread Thomas Hellström
Hi, Christian On Tue, 2024-01-09 at 08:47 +0100, Christian König wrote: > Hi guys, > > I'm trying to make this functionality a bit more useful for years now > since we multiple reports that behavior of drivers can be suboptimal > when multiple placements be given. > > So basically instead of