[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Use vblank worker to unpin old legacy cursor fb safely

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915: Use vblank worker to unpin old legacy cursor fb safely URL : https://patchwork.freedesktop.org/series/123125/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13583_full -> Patchwork_123125v1_full

[Intel-gfx] [PATCH] drm/i915: Add Wa_14015150844

2023-08-31 Thread Shekhar Chauhan
Disables Atomic-chaining of Typed Writes. BSpec: 54040 Signed-off-by: Shekhar Chauhan --- drivers/gpu/drm/i915/gt/intel_gt_regs.h | 2 ++ drivers/gpu/drm/i915/gt/intel_workarounds.c | 8 2 files changed, 10 insertions(+) diff --git a/drivers/gpu/drm/i915/gt/intel_gt_regs.h

[Intel-gfx] ✓ Fi.CI.BAT: success for WQ_UNBOUND warning since recent workqueue refactoring

2023-08-31 Thread Patchwork
== Series Details == Series: WQ_UNBOUND warning since recent workqueue refactoring URL : https://patchwork.freedesktop.org/series/123134/ State : success == Summary == CI Bug Log - changes from CI_DRM_13583 -> Patchwork_123134v1 Summary

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for WQ_UNBOUND warning since recent workqueue refactoring

2023-08-31 Thread Patchwork
== Series Details == Series: WQ_UNBOUND warning since recent workqueue refactoring URL : https://patchwork.freedesktop.org/series/123134/ State : warning == Summary == Error: dim checkpatch failed f4ee63f6c28e WQ_UNBOUND warning since recent workqueue refactoring -:15:

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gvt: Use list_for_each_entry() helper

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915/gvt: Use list_for_each_entry() helper URL : https://patchwork.freedesktop.org/series/123133/ State : success == Summary == CI Bug Log - changes from CI_DRM_13583 -> Patchwork_123133v1 Summary ---

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: Update GUC_KLV_0_KEY definition

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915/guc: Update GUC_KLV_0_KEY definition URL : https://patchwork.freedesktop.org/series/123130/ State : success == Summary == CI Bug Log - changes from CI_DRM_13583 -> Patchwork_123130v1 Summary ---

[Intel-gfx] ✗ Fi.CI.BUILD: failure for drm/i915/gvt: KVM: KVMGT fixes and page-track cleanups (rev11)

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915/gvt: KVM: KVMGT fixes and page-track cleanups (rev11) URL : https://patchwork.freedesktop.org/series/112196/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/112196/revisions/11/mbox/ not applied Applying:

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/guc: Update GUC_KLV_0_KEY definition

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915/guc: Update GUC_KLV_0_KEY definition URL : https://patchwork.freedesktop.org/series/123130/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Update GUC_KLV_0_KEY definition

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915/guc: Update GUC_KLV_0_KEY definition URL : https://patchwork.freedesktop.org/series/123130/ State : warning == Summary == Error: dim checkpatch failed aadae3354910 drm/i915/guc: Update GUC_KLV_0_KEY definition -:14: WARNING:COMMIT_LOG_LONG_LINE: Possible

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Use vblank worker to unpin old legacy cursor fb safely

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915: Use vblank worker to unpin old legacy cursor fb safely URL : https://patchwork.freedesktop.org/series/123125/ State : success == Summary == CI Bug Log - changes from CI_DRM_13583 -> Patchwork_123125v1

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Add Wa_14015150844 (rev2)

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915: Add Wa_14015150844 (rev2) URL : https://patchwork.freedesktop.org/series/123074/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13583 -> Patchwork_123074v2 Summary ---

[Intel-gfx] ✗ Fi.CI.BAT: failure for Apply Wa_16018031267 / Wa_16018063123

2023-08-31 Thread Patchwork
== Series Details == Series: Apply Wa_16018031267 / Wa_16018063123 URL : https://patchwork.freedesktop.org/series/123117/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13583 -> Patchwork_123117v1 Summary ---

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Apply Wa_16018031267 / Wa_16018063123

2023-08-31 Thread Patchwork
== Series Details == Series: Apply Wa_16018031267 / Wa_16018063123 URL : https://patchwork.freedesktop.org/series/123117/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Apply Wa_16018031267 / Wa_16018063123

2023-08-31 Thread Patchwork
== Series Details == Series: Apply Wa_16018031267 / Wa_16018063123 URL : https://patchwork.freedesktop.org/series/123117/ State : warning == Summary == Error: dim checkpatch failed aa56c62d03ba drm/i915: Add WABB blit for Wa_16018031267 / Wa_16018063123 -:10: WARNING:BAD_SIGN_OFF:

Re: [Intel-gfx] [PATCH] drm/i915/gt: Fix reservation address in ggtt_reserve_guc_top

2023-08-31 Thread Ceraolo Spurio, Daniele
On 8/25/2023 7:33 AM, Javier Pello wrote: There is an assertion in ggtt_reserve_guc_top that the global GTT is of size at least GUC_GGTT_TOP, which is not the case on a 32-bit platform; see commit 562d55d991b39ce376c492df2f7890fd6a541ffc ("drm/i915/bdw: Only use 2g GGTT for 32b platforms").

Re: [Intel-gfx] [PATCH v5] drm/i915: Avoid circular locking dependency when flush delayed work on gt reset

2023-08-31 Thread John Harrison
On 8/31/2023 07:00, Andi Shyti wrote: Hi, diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c index a0e3ef1c65d2..600388c849f7 100644 --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c +++

Re: [Intel-gfx] [PATCH 0/4] drm/amd/display: stop using drm_edid_override_connector_update()

2023-08-31 Thread Alex Hung
On 2023-08-30 01:29, Jani Nikula wrote: On Tue, 29 Aug 2023, Alex Hung wrote: On 2023-08-29 11:03, Jani Nikula wrote: On Tue, 29 Aug 2023, Jani Nikula wrote: On Tue, 29 Aug 2023, Alex Deucher wrote: On Tue, Aug 29, 2023 at 6:48 AM Jani Nikula wrote: On Wed, 23 Aug 2023, Jani Nikula

Re: [Intel-gfx] WQ_UNBOUND warning since recent workqueue refactoring

2023-08-31 Thread Heiner Kallweit
On 30.08.2023 20:56, Imre Deak wrote: > On Wed, Aug 30, 2023 at 07:51:13AM -1000, Tejun Heo wrote: > Hi, > >> Hello, >> >> (cc'ing i915 folks) >> >> On Wed, Aug 30, 2023 at 04:57:42PM +0200, Heiner Kallweit wrote: >>> Recently I started to see the following warning on linux-next and presumably

Re: [Intel-gfx] [PATCH 4/6] drm/cec: add drm_dp_cec_attach() as the non-edid version of set edid

2023-08-31 Thread Hans Verkuil
On 24/08/2023 15:46, Jani Nikula wrote: > Connectors have source physical address available in display > info. There's no need to parse the EDID again for this. Add > drm_dp_cec_attach() to do this. > > Seems like the set_edid/unset_edid naming is a bit specific now that > there's no need to pass

Re: [Intel-gfx] [PATCH 0/6] drm, cec and edid updates

2023-08-31 Thread Hans Verkuil
On 31/08/2023 20:51, Jani Nikula wrote: > On Thu, 24 Aug 2023, Jani Nikula wrote: >> Avoid accessing the raw edid directly. Pre-parse the source physical >> address during normal EDID parsing and use that for CEC. >> >> Jani Nikula (6): >> drm/edid: add drm_edid_is_digital() >>

Re: [Intel-gfx] [PATCH 3/6] drm/edid: parse source physical address

2023-08-31 Thread Hans Verkuil
On 24/08/2023 15:46, Jani Nikula wrote: > CEC needs the source physical address. Parsing it is trivial with the > existing EDID CEA DB infrastructure. > > Default to CEC_PHYS_ADDR_INVALID (0x) instead of 0 to cater for > easier CEC usage. > > Cc: Hans Verkuil > Cc:

[Intel-gfx] [PATCH -next] drm/i915/gvt: Use list_for_each_entry() helper

2023-08-31 Thread Jinjie Ruan
Convert list_for_each() to list_for_each_entry() so that the pos list_head pointer and list_entry() call are no longer needed, which can reduce a few lines of code. No functional changed. Signed-off-by: Jinjie Ruan --- drivers/gpu/drm/i915/gvt/dmabuf.c | 8 ++-- 1 file changed, 2

Re: [Intel-gfx] [PATCH] drm/i915/guc: fix compile issue of guc_klvs_abi.h

2023-08-31 Thread Linyu Yuan
On 8/29/2023 5:42 AM, Michal Wajdeczko wrote: On 25.08.2023 07:50, Linyu Yuan wrote: On 8/25/2023 1:37 PM, Jani Nikula wrote: On Fri, 25 Aug 2023, Linyu Yuan wrote: GCC report GUC_KLV_0_KEY and GUC_KLV_0_LEN is not constant when do preprocessing. Please paste the actual compiler warning.

Re: [Intel-gfx] [PATCH 6/6] media: cec: core: add note about *_from_edid() function usage in drm

2023-08-31 Thread Hans Verkuil
On 24/08/2023 15:46, Jani Nikula wrote: > In the drm subsystem, the source physical address is, in most cases, > available without having to parse the EDID again. Add notes about > preferring to use the pre-parsed address instead. > > Cc: Hans Verkuil > Cc: linux-me...@vger.kernel.org >

Re: [Intel-gfx] WQ_UNBOUND warning since recent workqueue refactoring

2023-08-31 Thread Heiner Kallweit
On 30.08.2023 20:56, Imre Deak wrote: > On Wed, Aug 30, 2023 at 07:51:13AM -1000, Tejun Heo wrote: > Hi, > >> Hello, >> >> (cc'ing i915 folks) >> >> On Wed, Aug 30, 2023 at 04:57:42PM +0200, Heiner Kallweit wrote: >>> Recently I started to see the following warning on linux-next and presumably

Re: [Intel-gfx] [PATCH] drm/i915/guc: fix compile issue of guc_klvs_abi.h

2023-08-31 Thread Linyu Yuan
On 8/29/2023 5:42 AM, Michal Wajdeczko wrote: On 25.08.2023 07:50, Linyu Yuan wrote: On 8/25/2023 1:37 PM, Jani Nikula wrote: On Fri, 25 Aug 2023, Linyu Yuan wrote: GCC report GUC_KLV_0_KEY and GUC_KLV_0_LEN is not constant when do preprocessing. Please paste the actual compiler warning.

Re: [Intel-gfx] [PATCH v4 16/29] KVM: x86: Reject memslot MOVE operations if KVMGT is attached

2023-08-31 Thread Like Xu
On 31/8/2023 4:50 am, Sean Christopherson wrote: On Wed, Aug 30, 2023, Like Xu wrote: On 2023/7/29 09:35, Sean Christopherson wrote: Disallow moving memslots if the VM has external page-track users, i.e. if KVMGT is being used to expose a virtual GPU to the guest, as KVMGT doesn't correctly

Re: [Intel-gfx] [PATCH 0/4] drm/amd/display: stop using drm_edid_override_connector_update()

2023-08-31 Thread Wu, Hersen
[AMD Official Use Only - General] + Charlie -Original Message- From: Jani Nikula Sent: Tuesday, August 29, 2023 6:49 AM To: Hung, Alex ; dri-de...@lists.freedesktop.org; amd-...@lists.freedesktop.org Cc: Li, Sun peng (Leo) ; David Airlie ; intel-gfx@lists.freedesktop.org; Siqueira,

Re: [Intel-gfx] [PATCH v2] drm/cec: add drm_dp_cec_attach() as the non-edid version of set edid

2023-08-31 Thread Hans Verkuil
Hi Jani, Sorry, I missed the v2. On 25/08/2023 15:01, Jani Nikula wrote: > Connectors have source physical address available in display > info. There's no need to parse the EDID again for this. Add > drm_dp_cec_attach() to do this. > > Seems like the set_edid/unset_edid naming is a bit specific

Re: [Intel-gfx] [PATCH v2] media: cec: core: add note about *_from_edid() function usage in drm

2023-08-31 Thread Hans Verkuil
On 31/08/2023 12:51, Jani Nikula wrote: > In the drm subsystem, the source physical address is, in most cases, > available without having to parse the EDID again. Add notes about > preferring to use the pre-parsed address instead. > > Cc: Hans Verkuil > Cc: linux-me...@vger.kernel.org >

Re: [Intel-gfx] [PATCH 5/6] drm/i915/cec: switch to setting physical address directly

2023-08-31 Thread Hans Verkuil
On 24/08/2023 15:46, Jani Nikula wrote: > Avoid parsing the EDID again for source physical address. Also gets rids > of a few remaining raw EDID usages. > > Cc: Hans Verkuil > Cc: linux-me...@vger.kernel.org > Signed-off-by: Jani Nikula Reviewed-by: Hans Verkuil Regards, Hans > ---

Re: [Intel-gfx] [PATCH v4 16/29] KVM: x86: Reject memslot MOVE operations if KVMGT is attached

2023-08-31 Thread Like Xu
On 2023/7/29 09:35, Sean Christopherson wrote: Disallow moving memslots if the VM has external page-track users, i.e. if KVMGT is being used to expose a virtual GPU to the guest, as KVMGT doesn't correctly handle moving memory regions. Note, this is potential ABI breakage! E.g. userspace could

Re: [Intel-gfx] [PATCH 0/4] drm/amd/display: stop using drm_edid_override_connector_update()

2023-08-31 Thread Wu, Hersen
[AMD Official Use Only - General] + Charlie Wang -Original Message- From: Alex Deucher Sent: Tuesday, August 29, 2023 11:44 AM To: Jani Nikula Cc: Hung, Alex ; dri-de...@lists.freedesktop.org; amd-...@lists.freedesktop.org; Li, Sun peng (Leo) ; intel-gfx@lists.freedesktop.org;

[Intel-gfx] [PULL] drm-intel-next-fixes

2023-08-31 Thread Rodrigo Vivi
Hi Dave and Daniel, Only a single patch towards -rc1. I noticed that you already sent this week's PR, but sending this just in case. Otherwise I believe it could wait for the regular fixes cycle. Here goes drm-intel-next-fixes-2023-08-31: - Mark requests for GuC virtual engines to avoid

[Intel-gfx] [PATCH] drm/i915/guc: Update GUC_KLV_0_KEY definition

2023-08-31 Thread Michal Wajdeczko
While building ARCH=x86 with GCC 7.5.0 we get compilation errors: CC drivers/gpu/drm/i915/gt/uc/intel_guc_submission.o In file included from :0:0: In function ‘__guc_context_policy_add_priority.isra.47’, inlined from ‘__guc_context_set_prio.isra.48’ at

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/cx0: Check and increase msgbus timeout threshold (rev3)

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915/cx0: Check and increase msgbus timeout threshold (rev3) URL : https://patchwork.freedesktop.org/series/122982/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13579_full -> Patchwork_122982v3_full

Re: [Intel-gfx] [PATCH v4 1/2] drm/i915: Add WABB blit for Wa_16018031267 / Wa_16018063123

2023-08-31 Thread Nirmoy Das
On 8/31/2023 7:40 PM, Matt Roper wrote: On Thu, Aug 31, 2023 at 08:09:54AM -0700, Jonathan Cavitt wrote: From: Nirmoy Das Apply WABB blit for Wa_16018031267 / Wa_16018063123. Additionally, update the lrc selftest to exercise the new WABB changes. Co-developed-by: Nirmoy Das Drive-by

Re: [Intel-gfx] [PATCH 0/6] drm, cec and edid updates

2023-08-31 Thread Jani Nikula
On Thu, 24 Aug 2023, Jani Nikula wrote: > Avoid accessing the raw edid directly. Pre-parse the source physical > address during normal EDID parsing and use that for CEC. > > Jani Nikula (6): > drm/edid: add drm_edid_is_digital() > drm/i915/display: use drm_edid_is_digital() > drm/edid:

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm, cec and edid updates (rev3)

2023-08-31 Thread Patchwork
== Series Details == Series: drm, cec and edid updates (rev3) URL : https://patchwork.freedesktop.org/series/122841/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13579_full -> Patchwork_122841v3_full Summary ---

Re: [Intel-gfx] [PATCH 1/1] drm/i915/gt: Wait longer for tasks in migrate selftest

2023-08-31 Thread Andi Shyti
Hi Jonathan, On Mon, Aug 28, 2023 at 12:28:52PM -0700, Jonathan Cavitt wrote: > The thread_global_copy subtest of the live migrate selftest creates a > large number of threads and waits 10ms for them all to start. This is > not enough time to wait for the threaded tasks to start, as some may >

Re: [Intel-gfx] [PATCH v4 1/2] drm/i915: Add WABB blit for Wa_16018031267 / Wa_16018063123

2023-08-31 Thread Matt Roper
On Thu, Aug 31, 2023 at 08:09:54AM -0700, Jonathan Cavitt wrote: > From: Nirmoy Das > > Apply WABB blit for Wa_16018031267 / Wa_16018063123. > Additionally, update the lrc selftest to exercise the new > WABB changes. > > Co-developed-by: Nirmoy Das Drive-by comment: since Nirmoy is already

Re: [Intel-gfx] [PATCH 2/6] drm/i915/display: use drm_edid_is_digital()

2023-08-31 Thread Ville Syrjälä
On Thu, Aug 24, 2023 at 04:46:03PM +0300, Jani Nikula wrote: > Reduce the use of struct edid and drm_edid_raw(). > > Signed-off-by: Jani Nikula > --- > drivers/gpu/drm/i915/display/intel_crt.c | 11 --- > drivers/gpu/drm/i915/display/intel_hdmi.c | 9 - >

Re: [Intel-gfx] [PATCH 1/6] drm/edid: add drm_edid_is_digital()

2023-08-31 Thread Ville Syrjälä
On Thu, Aug 24, 2023 at 04:46:02PM +0300, Jani Nikula wrote: > Checking edid->input & DRM_EDID_INPUT_DIGITAL is common enough to > deserve a helper that also lets us abstract the raw EDID a bit better. > > Signed-off-by: Jani Nikula Reviewed-by: Ville Syrjälä > --- >

Re: [Intel-gfx] [PATCH] drm/i915: Add Wa_14015150844

2023-08-31 Thread Matt Roper
On Thu, Aug 31, 2023 at 09:29:25PM +0530, Shekhar Chauhan wrote: > Disables Atomic-chaining of Typed Writes. > > BSpec: 54040 > Signed-off-by: Shekhar Chauhan > --- > drivers/gpu/drm/i915/gt/intel_gt_regs.h | 2 ++ > drivers/gpu/drm/i915/gt/intel_workarounds.c | 8 > 2 files

Re: [Intel-gfx] [PATCH v5 4/9] drm/i915: Eliminate IS_MTL_GRAPHICS_STEP

2023-08-31 Thread Matt Roper
On Thu, Aug 31, 2023 at 07:16:55PM +0300, Jani Nikula wrote: > On Mon, 21 Aug 2023, Matt Roper wrote: > > diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c > > b/drivers/gpu/drm/i915/display/skl_universal_plane.c > > index a408ec2d3958..4566c95da1ca 100644 > > ---

[Intel-gfx] [PATCH] drm/i915: Use vblank worker to unpin old legacy cursor fb safely

2023-08-31 Thread Ville Syrjala
From: Ville Syrjälä The cursor hardware only does sync updates, and thus the hardware will be scanning out from the old fb until the next start of vblank. So in order to make the legacy cursor fastpath actually safe we should not unpin the old fb until we're sure the hardware has ceased

Re: [Intel-gfx] [PATCH v5 4/9] drm/i915: Eliminate IS_MTL_GRAPHICS_STEP

2023-08-31 Thread Jani Nikula
On Mon, 21 Aug 2023, Matt Roper wrote: > diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c > b/drivers/gpu/drm/i915/display/skl_universal_plane.c > index a408ec2d3958..4566c95da1ca 100644 > --- a/drivers/gpu/drm/i915/display/skl_universal_plane.c > +++

Re: [Intel-gfx] [PATCH] drm/i915: add minimal i915_gem_object_frontbuffer.h

2023-08-31 Thread Jani Nikula
On Wed, 30 Aug 2023, "Hogander, Jouni" wrote: > On Wed, 2023-08-30 at 11:51 +0300, Jani Nikula wrote: >> Split out frontbuffer related declarations and static inlines from >> gem/i915_gem_object.h into new gem/i915_gem_object_frontbuffer.h. >> >> The main goal is to reduce header

Re: [Intel-gfx] [PATCH v4 16/29] KVM: x86: Reject memslot MOVE operations if KVMGT is attached

2023-08-31 Thread Sean Christopherson
On Thu, Aug 31, 2023, Like Xu wrote: > On 31/8/2023 4:50 am, Sean Christopherson wrote: > > On Wed, Aug 30, 2023, Like Xu wrote: > > > On 2023/7/29 09:35, Sean Christopherson wrote: > > > > Disallow moving memslots if the VM has external page-track users, i.e. > > > > if > > > > KVMGT is being

[Intel-gfx] [PATCH] drm/i915: Add Wa_14015150844

2023-08-31 Thread Shekhar Chauhan
Disables Atomic-chaining of Typed Writes. BSpec: 54040 Signed-off-by: Shekhar Chauhan --- drivers/gpu/drm/i915/gt/intel_gt_regs.h | 2 ++ drivers/gpu/drm/i915/gt/intel_workarounds.c | 8 2 files changed, 10 insertions(+) diff --git a/drivers/gpu/drm/i915/gt/intel_gt_regs.h

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_pm_freq_api: Test s2idle instead of S3

2023-08-31 Thread Gupta, Anshuman
> -Original Message- > From: Belgaumkar, Vinay > Sent: Thursday, August 31, 2023 5:09 AM > To: intel-gfx@lists.freedesktop.org; igt-...@lists.freedesktop.org > Cc: Belgaumkar, Vinay ; Gupta, Anshuman > > Subject: [PATCH i-g-t] tests/i915_pm_freq_api: Test s2idle instead of S3 > >

[Intel-gfx] [PATCH v4 0/2] Apply Wa_16018031267 / Wa_16018063123

2023-08-31 Thread Jonathan Cavitt
Apply Wa_16018031267 / Wa_16018063123. This necessitates submitting a fastcolor blit as WABB and setting the copy engine arbitration to round-robin mode. v2: - Rename old platform check in second patch to match declaration in first patch. - Refactor second patch name to match first patch. v3:

[Intel-gfx] [PATCH v4 1/2] drm/i915: Add WABB blit for Wa_16018031267 / Wa_16018063123

2023-08-31 Thread Jonathan Cavitt
From: Nirmoy Das Apply WABB blit for Wa_16018031267 / Wa_16018063123. Additionally, update the lrc selftest to exercise the new WABB changes. Co-developed-by: Nirmoy Das Signed-off-by: Jonathan Cavitt --- drivers/gpu/drm/i915/gt/intel_engine_regs.h | 3 + drivers/gpu/drm/i915/gt/intel_gt.h

[Intel-gfx] [PATCH v4 2/2] drm/i915: Set copy engine arbitration for Wa_16018031267 / Wa_16018063123

2023-08-31 Thread Jonathan Cavitt
From: Nirmoy Das Set copy engine arbitration into round robin mode for part of Wa_16018031267 / Wa_16018063123 mitigation. Signed-off-by: Nirmoy Das Signed-off-by: Jonathan Cavitt --- drivers/gpu/drm/i915/gt/intel_engine_regs.h | 3 +++ drivers/gpu/drm/i915/gt/intel_workarounds.c | 5 +

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/cx0: Check and increase msgbus timeout threshold (rev3)

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915/cx0: Check and increase msgbus timeout threshold (rev3) URL : https://patchwork.freedesktop.org/series/122982/ State : success == Summary == CI Bug Log - changes from CI_DRM_13579 -> Patchwork_122982v3

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/cx0: Check and increase msgbus timeout threshold (rev3)

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915/cx0: Check and increase msgbus timeout threshold (rev3) URL : https://patchwork.freedesktop.org/series/122982/ State : warning == Summary == Error: dim checkpatch failed e8aaebe5ef58 drm/i915/cx0: Check and increase msgbus timeout threshold -:110:

Re: [Intel-gfx] [PATCH 1/1] drm/i915/gt: Wait longer for tasks in migrate selftest

2023-08-31 Thread Andi Shyti
Hi Jonathan, On Mon, Aug 28, 2023 at 12:28:52PM -0700, Jonathan Cavitt wrote: > The thread_global_copy subtest of the live migrate selftest creates a > large number of threads and waits 10ms for them all to start. This is > not enough time to wait for the threaded tasks to start, as some may >

[Intel-gfx] ✓ Fi.CI.BAT: success for drm, cec and edid updates (rev3)

2023-08-31 Thread Patchwork
== Series Details == Series: drm, cec and edid updates (rev3) URL : https://patchwork.freedesktop.org/series/122841/ State : success == Summary == CI Bug Log - changes from CI_DRM_13579 -> Patchwork_122841v3 Summary --- **SUCCESS**

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm, cec and edid updates (rev3)

2023-08-31 Thread Patchwork
== Series Details == Series: drm, cec and edid updates (rev3) URL : https://patchwork.freedesktop.org/series/122841/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Populate connector->ddc always (rev2)

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915: Populate connector->ddc always (rev2) URL : https://patchwork.freedesktop.org/series/123006/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13579 -> Patchwork_123006v2 Summary ---

Re: [Intel-gfx] [PATCH v5] drm/i915: Avoid circular locking dependency when flush delayed work on gt reset

2023-08-31 Thread Andi Shyti
Hi, > > > > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c > > > > b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c > > > > index a0e3ef1c65d2..600388c849f7 100644 > > > > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c > > > > +++

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: Populate connector->ddc always (rev2)

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915: Populate connector->ddc always (rev2) URL : https://patchwork.freedesktop.org/series/123006/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Populate connector->ddc always (rev2)

2023-08-31 Thread Patchwork
== Series Details == Series: drm/i915: Populate connector->ddc always (rev2) URL : https://patchwork.freedesktop.org/series/123006/ State : warning == Summary == Error: dim checkpatch failed 47a1e16097f5 drm: Reorder drm_sysfs_connector_remove() vs. drm_debugfs_connector_remove()

[Intel-gfx] ✓ Fi.CI.IGT: success for Add DSC PPS readout (rev12)

2023-08-31 Thread Patchwork
== Series Details == Series: Add DSC PPS readout (rev12) URL : https://patchwork.freedesktop.org/series/120456/ State : success == Summary == CI Bug Log - changes from CI_DRM_13569_full -> Patchwork_120456v12_full Summary ---

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t] tests/i915_pm_freq_api: Test s2idle instead of S3

2023-08-31 Thread Riana Tauro
On 8/31/2023 5:08 AM, Vinay Belgaumkar wrote: Test skips whenever S3 is not supported, use s2idle instead, which is widely enabled. Cc: Anshuman Gupta Signed-off-by: Vinay Belgaumkar Looks good to me Reviewed-by: Riana Tauro --- tests/i915/i915_pm_freq_api.c | 2 +- 1 file changed, 1

Re: [Intel-gfx] [PATCH v2] drm/i915/cx0: Check and increase msgbus timeout threshold

2023-08-31 Thread Kahola, Mika
> -Original Message- > From: Sousa, Gustavo > Sent: Wednesday, August 30, 2023 3:15 PM > To: intel-gfx@lists.freedesktop.org > Cc: Sripada, Radhakrishna ; Kahola, Mika > ; Nikula, Jani > > Subject: [PATCH v2] drm/i915/cx0: Check and increase msgbus timeout threshold > > We have

Re: [Intel-gfx] [PATCH 6/6] media: cec: core: add note about *_from_edid() function usage in drm

2023-08-31 Thread Jani Nikula
On Wed, 30 Aug 2023, Hans Verkuil wrote: > On 24/08/2023 15:46, Jani Nikula wrote: >> In the drm subsystem, the source physical address is, in most cases, >> available without having to parse the EDID again. Add notes about >> preferring to use the pre-parsed address instead. >> >> Cc: Hans

[Intel-gfx] [PATCH v2] media: cec: core: add note about *_from_edid() function usage in drm

2023-08-31 Thread Jani Nikula
In the drm subsystem, the source physical address is, in most cases, available without having to parse the EDID again. Add notes about preferring to use the pre-parsed address instead. Cc: Hans Verkuil Cc: linux-me...@vger.kernel.org Signed-off-by: Jani Nikula --- v2: rephrase comments, in

[Intel-gfx] [PATCH v2 03/12] drm/i915: Call the DDC bus i2c adapter "ddc"

2023-08-31 Thread Ville Syrjala
From: Ville Syrjälä Rename the various names we've used for the DDC bus i2c adapter ("i2c", "adapter", etc.) to just "ddc". This differentiates it from the various other i2c busses we might have (DSI panel stuff, DVO control bus, etc.). v2: Don't add a bogus drm_get_edid() call (Jani)

Re: [Intel-gfx] [PATCH 11/12] drm/i915/hdmi: Remove old i2c symlink

2023-08-31 Thread Jani Nikula
On Tue, 29 Aug 2023, Ville Syrjala wrote: > From: Ville Syrjälä > > Remove the i915 specific i2c-N symlink from HDMI connectors. > This was added to sort of mirror the DP connectors that alreayd > had their aux ch based i2c adapter sitting beneath them in the > sysfs hierarchy. But now that we

Re: [Intel-gfx] [PATCH 10/12] drm/i915/hdmi: Nuke hdmi->ddc_bus

2023-08-31 Thread Jani Nikula
On Tue, 29 Aug 2023, Ville Syrjala wrote: > From: Ville Syrjälä > > Remove the mostly redundant hdmi->ddc_bus. The only thing that needs > it anymore is get_encoder_by_ddc_bus(), but that can be replaced with > a slight detour through attached_connector+intel_gmbus_get_adapter(). > >

Re: [Intel-gfx] [PATCH] drm/i915/cx0: Check and increase msgbus timeout threshold

2023-08-31 Thread Kahola, Mika
> -Original Message- > From: Sousa, Gustavo > Sent: Wednesday, August 30, 2023 3:19 PM > To: Kahola, Mika ; Sripada, Radhakrishna > ; intel- > g...@lists.freedesktop.org > Subject: RE: [Intel-gfx] [PATCH] drm/i915/cx0: Check and increase msgbus > timeout threshold > > Quoting Gustavo

Re: [Intel-gfx] [PATCH 1/2] drm/i915/dp: Drop redundant AUX power get/put in intel_dp_force()

2023-08-31 Thread Imre Deak
On Wed, Aug 30, 2023 at 05:29:09PM -0400, Rodrigo Vivi wrote: > On Wed, Aug 30, 2023 at 05:04:20PM +0300, Imre Deak wrote: > > intel_dp_force() takes the AUX power reference as required by the DP AUX > > transactions in intel_dp_set_edid(). However the low level AUX handler > > you mean