Re: [Intel-gfx] [PATCH 0/4] Prep series - CDCLK code churn

2022-10-21 Thread Balasubramani Vivekanandan
On 20.10.2022 17:20, Anusha Srivatsa wrote: > No functional changes. The series is more of a prep series > for the mid_cdclk_config series: > https://patchwork.freedesktop.org/series/109694/ > > Main change: > - Change usage of "crawler" and "squasher". Use crawling and > squashing instead. > -

Re: [Intel-gfx] [PATCH 4/4] drm/i915/display: Move squash_ctl register programming to its own function

2022-10-21 Thread Balasubramani Vivekanandan
On 20.10.2022 17:20, Anusha Srivatsa wrote: > No functional change. Introduce dg2_cdclk_squash_programming and > move squash_ctl register programming bits to this. > > Cc: Balasubramani Vivekanandan > Cc: Ville Syrjälä > Signed-off-by: Anusha Srivatsa > --- > drivers/gpu/drm/i915/display/intel

Re: [Intel-gfx] [PATCH 2/4] drm/i915/display: Introduce HAS_CDCLK_SQUASH macro

2022-10-21 Thread Balasubramani Vivekanandan
On 20.10.2022 17:20, Anusha Srivatsa wrote: > Driver had discrepancy in how cdclk squash and crawl support > were checked. Like crawl, add squash as a 1 bit feature flag > to the display section of DG2. > > Cc: Balasubramani Vivekanandan > Cc: Ville Syrjälä > Signed-off-by: Anusha Srivatsa > --

Re: [Intel-gfx] [PATCH v3 0/1] Remove Connector iterator macro

2022-10-21 Thread Shankar, Uma
> -Original Message- > From: Intel-gfx On Behalf Of Suraj > Kandpal > Sent: Thursday, October 20, 2022 4:17 PM > To: intel-gfx@lists.freedesktop.org > Cc: Nikula, Jani > Subject: [Intel-gfx] [PATCH v3 0/1] Remove Connector iterator macro > > Remove connector iterator macro (for_each_c

Re: [Intel-gfx] [PATCH] drm/i915: use i915_sg_dma_sizes() for internal backend

2022-10-21 Thread Tvrtko Ursulin
On 20/10/2022 16:10, Matthew Auld wrote: We rely on page_sizes.sg in setup_scratch_page() reporting the correct value if the underlying sgl is not contiguous, however in get_pages_internal() we are only looking at the layout of the created pages when calculating the sg_page_sizes, and not the f

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/psr: Remove inappropriate DSC slice alignment warning

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915/psr: Remove inappropriate DSC slice alignment warning URL : https://patchwork.freedesktop.org/series/109983/ State : success == Summary == CI Bug Log - changes from CI_DRM_12270 -> Patchwork_109983v1 Su

Re: [Intel-gfx] [PATCH] drm/i915: stop abusing swiotlb_max_segment

2022-10-21 Thread Tvrtko Ursulin
On 20/10/2022 12:03, Christoph Hellwig wrote: From: Robert Beckett swiotlb_max_segment used to return either the maximum size that swiotlb could bounce, or for Xen PV PAGE_SIZE even if swiotlb could bounce buffer larger mappings. This made i915 on Xen PV work as it bypasses the coherency asp

Re: [Intel-gfx] [PATCH v2 1/7] drm/i915/pxp: Make gt and pxp init/fini aware of PXP-owning-GT

2022-10-21 Thread Teres Alexis, Alan Previn
On Mon, 2022-10-17 at 10:03 -0700, Alan Previn Teres Alexis wrote: > > > > On Thu, 2022-10-13 at 13:48 -0700, Ceraolo Spurio, Daniele wrote: > > > > > > > > On 10/5/2022 9:38 PM, Alan Previn wrote: > > > > > > In preparation for future MTL-PXP feature support, PXP control > > > > > > context sh

Re: [Intel-gfx] [PATCH] drm/i915/psr: Remove inappropriate DSC slice alignment warning

2022-10-21 Thread Kahola, Mika
> -Original Message- > From: Hogander, Jouni > Sent: Friday, October 21, 2022 8:49 AM > To: Intel-gfx@lists.freedesktop.org > Cc: Hogander, Jouni ; Souza, Jose > ; Kahola, Mika ; Gupta, > Anshuman > Subject: [PATCH] drm/i915/psr: Remove inappropriate DSC slice alignment > warning > > Sel

Re: [Intel-gfx] [PATCH v2 0/4] drm/i915/tgl+: Fix race conditions during DKL PHY accesses

2022-10-21 Thread Jani Nikula
On Thu, 20 Oct 2022, Imre Deak wrote: > This is v2 of [1] addressing the review comments from Jani. Did not do detailed review, Acked-by: Jani Nikula > > [1] > https://lore.kernel.org/intel-gfx/y1barftah%2fl+x...@ideak-desk.fi.intel.com/T/#t > > Cc: Jani Nikula > Cc: Ville Syrjälä > > Imre

Re: [Intel-gfx] [PATCH 3/4] drm/i915/display: Move chunks of code out of bxt_set_cdclk()

2022-10-21 Thread Jani Nikula
On Thu, 20 Oct 2022, Anusha Srivatsa wrote: > No functional change. Moving segments out to simplify > bxt_set_cdlck() > > Cc: Balasubramani Vivekanandan > Cc: Ville Syrjälä > Signed-off-by: Anusha Srivatsa > --- > drivers/gpu/drm/i915/display/intel_cdclk.c | 40 ++ > 1 file

[Intel-gfx] [PATCH v4] overflow: Introduce overflows_type() and castable_to_type()

2022-10-21 Thread Gwan-gyeong Mun
From: Kees Cook Implement a robust overflows_type() macro to test if a variable or constant value would overflow another variable or type. This can be used as a constant expression for static_assert() (which requires a constant expression[1][2]) when used on constant values. This must be construc

Re: [Intel-gfx] [PATCH 4/4] drm/i915/display: Move squash_ctl register programming to its own function

2022-10-21 Thread Jani Nikula
On Thu, 20 Oct 2022, Anusha Srivatsa wrote: > No functional change. Introduce dg2_cdclk_squash_programming and > move squash_ctl register programming bits to this. > > Cc: Balasubramani Vivekanandan > Cc: Ville Syrjälä > Signed-off-by: Anusha Srivatsa > --- > drivers/gpu/drm/i915/display/intel

Re: [Intel-gfx] [PATCH 2/2] drm/i915/pmu: Connect engine busyness stats from GuC to pmu

2022-10-21 Thread Tvrtko Ursulin
On 27/10/2021 01:48, Umesh Nerlige Ramappa wrote: [snip] +static void guc_timestamp_ping(struct work_struct *wrk) +{ + struct intel_guc *guc = container_of(wrk, typeof(*guc), +timestamp.work.work); + struct intel_uc *uc = container_of(gu

Re: [Intel-gfx] [PATCH 0/4] Prep series - CDCLK code churn

2022-10-21 Thread Jani Nikula
On Fri, 21 Oct 2022, Balasubramani Vivekanandan wrote: > On 20.10.2022 17:20, Anusha Srivatsa wrote: >> No functional changes. The series is more of a prep series >> for the mid_cdclk_config series: >> https://patchwork.freedesktop.org/series/109694/ >> >> Main change: >> - Change usage of "cra

Re: [Intel-gfx] [PATCH 2/4] drm/i915/display: Introduce HAS_CDCLK_SQUASH macro

2022-10-21 Thread Jani Nikula
On Fri, 21 Oct 2022, Balasubramani Vivekanandan wrote: > On 20.10.2022 17:20, Anusha Srivatsa wrote: >> Driver had discrepancy in how cdclk squash and crawl support >> were checked. Like crawl, add squash as a 1 bit feature flag >> to the display section of DG2. >> >> Cc: Balasubramani Vivekanan

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: stop abusing swiotlb_max_segment (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: stop abusing swiotlb_max_segment (rev2) URL : https://patchwork.freedesktop.org/series/109946/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

Re: [Intel-gfx] [PATCH] drm/i915: use i915_sg_dma_sizes() for internal backend

2022-10-21 Thread Andrzej Hajda
Hi Matthew, On 20.10.2022 17:10, Matthew Auld wrote: We rely on page_sizes.sg in setup_scratch_page() reporting the correct value if the underlying sgl is not contiguous, however in get_pages_internal() we are only looking at the layout of the created pages when calculating the sg_page_sizes, an

Re: [Intel-gfx] [PATCH] drm/i915: Activate DRRS after state readout

2022-10-21 Thread Jani Nikula
On Thu, 20 Oct 2022, Ville Syrjala wrote: > From: Ville Syrjälä > > On BDW+ we have just the one set of DP M/N registers. The > values we write into said registers depends on whether we > want DRRS to be in high or low gear. This causes issues > for the state checker which currently has to assume

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: stop abusing swiotlb_max_segment (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: stop abusing swiotlb_max_segment (rev2) URL : https://patchwork.freedesktop.org/series/109946/ State : success == Summary == CI Bug Log - changes from CI_DRM_12271 -> Patchwork_109946v2 Summary ---

Re: [Intel-gfx] [PATCH v7 0/9] dyndbg: drm.debug adaptation

2022-10-21 Thread Jani Nikula
On Thu, 20 Oct 2022, Ville Syrjälä wrote: > On Sat, Sep 24, 2022 at 03:02:34PM +0200, Greg KH wrote: >> On Sun, Sep 11, 2022 at 11:28:43PM -0600, Jim Cromie wrote: >> > hi Greg, Dan, Jason, DRM-folk, >> > >> > heres follow-up to V6: >> > rebased on driver-core/driver-core-next for -v6 applied b

Re: [Intel-gfx] Random submitter change in Freedesktop Patchwork

2022-10-21 Thread Saarinen, Jani
Hi, > -Original Message- > From: Intel-gfx On Behalf Of Ville > Syrjälä > Sent: torstai 20. lokakuuta 2022 19.21 > To: Jani Nikula > Cc: Sarvela, Tomi P ; > intel-gfx@lists.freedesktop.org; > dri-de...@lists.freedesktop.org; Vivi, Rodrigo > Subject: Re: [Intel-gfx] Random submitter ch

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/dg1: Remove require_force_probe protection (rev3)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915/dg1: Remove require_force_probe protection (rev3) URL : https://patchwork.freedesktop.org/series/100601/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12270_full -> Patchwork_100601v3_full ==

[Intel-gfx] [PATCH v2 1/2] drm/i915: Encapsulate lmem rpm stuff in intel_runtime_pm

2022-10-21 Thread Anshuman Gupta
Runtime pm is not really per GT, therefore it make sense to move lmem_userfault_list, lmem_userfault_lock and userfault_wakeref from intel_gt to intel_runtime_pm structure, which is embedded to i915. No functional change. Signed-off-by: Anshuman Gupta --- drivers/gpu/drm/i915/gem/i915_gem_mman.

[Intel-gfx] [PATCH v2 2/2] drm/i915/dgfx: Grab wakeref at i915_ttm_unmap_virtual

2022-10-21 Thread Anshuman Gupta
We had already grabbed the rpm wakeref at obj destruction path, but it also required to grab the wakeref when object moves. When i915_gem_object_release_mmap_offset() gets called by i915_ttm_move_notify(), it will release the mmap offset without grabbing the wakeref. We want to avoid that therefore

Re: [Intel-gfx] [PATCH] drm/i915/hdmi: SPD infoframe updated for discrete

2022-10-21 Thread Jani Nikula
On Wed, 19 Oct 2022, "Taylor, Clinton A" wrote: > Replace internal with discrete on dgfx platforms Please use the imperative mood in the commit subject, i.e. "update" not "updated". > > Signed-off-by: Taylor, Clinton A > --- > drivers/gpu/drm/i915/display/intel_hdmi.c | 6 +- > 1 file chan

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/psr: Remove inappropriate DSC slice alignment warning

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915/psr: Remove inappropriate DSC slice alignment warning URL : https://patchwork.freedesktop.org/series/109983/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12270_full -> Patchwork_109983v1_full ==

Re: [Intel-gfx] [PATCH v2 1/2] drm/i915: call i915_request_await_object from _i915_vma_move_to_active

2022-10-21 Thread Tvrtko Ursulin
On 19/10/2022 22:59, Andrzej Hajda wrote: Since almost all calls to i915_vma_move_to_active are prepended with i915_request_await_object, let's call the latter from _i915_vma_move_to_active by default and add flag allowing bypassing it. Adjust all callers accordingly. The patch should not intro

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for overflow: Introduce overflows_type() and castable_to_type() (rev3)

2022-10-21 Thread Patchwork
== Series Details == Series: overflow: Introduce overflows_type() and castable_to_type() (rev3) URL : https://patchwork.freedesktop.org/series/109076/ State : warning == Summary == Error: dim checkpatch failed 89eeed59f1ab overflow: Introduce overflows_type() and castable_to_type() -:88: CHECK

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for overflow: Introduce overflows_type() and castable_to_type() (rev3)

2022-10-21 Thread Patchwork
== Series Details == Series: overflow: Introduce overflows_type() and castable_to_type() (rev3) URL : https://patchwork.freedesktop.org/series/109076/ 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: success for overflow: Introduce overflows_type() and castable_to_type() (rev3)

2022-10-21 Thread Patchwork
== Series Details == Series: overflow: Introduce overflows_type() and castable_to_type() (rev3) URL : https://patchwork.freedesktop.org/series/109076/ State : success == Summary == CI Bug Log - changes from CI_DRM_12273 -> Patchwork_109076v3

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Activate DRRS after state readout (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: Activate DRRS after state readout (rev2) URL : https://patchwork.freedesktop.org/series/109950/ State : success == Summary == CI Bug Log - changes from CI_DRM_12273 -> Patchwork_109950v2 Summary --

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: refactor i915_vma_move_to_active (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: refactor i915_vma_move_to_active (rev2) URL : https://patchwork.freedesktop.org/series/109910/ State : success == Summary == CI Bug Log - changes from CI_DRM_12273 -> Patchwork_109910v2 Summary ---

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: stop abusing swiotlb_max_segment (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: stop abusing swiotlb_max_segment (rev2) URL : https://patchwork.freedesktop.org/series/109946/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12271_full -> Patchwork_109946v2_full Summa

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Clean up crtc state flag checks (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: Clean up crtc state flag checks (rev2) URL : https://patchwork.freedesktop.org/series/109949/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12273 -> Patchwork_109949v2 Summary ---

Re: [Intel-gfx] [PATCH 2/2] drm/i915/selftests: add igt_vma_move_to_active_unlocked

2022-10-21 Thread Andi Shyti
Hi Andrzej, [...] > +static inline int __must_check > +igt_vma_move_to_active_unlocked(struct i915_vma *vma, struct i915_request > *rq, > + unsigned int flags) > +{ > + int err; > + > + i915_vma_lock(vma); > + err = _i915_vma_move_to_active(vma, rq, &rq->f

Re: [Intel-gfx] [PATCH 1/2] drm/i915: add wait and lock to i915_vma_move_to_active

2022-10-21 Thread Andi Shyti
Hi Andrzej, (at first I r-b'ed this patch, but then I wanted to think on some more "simplification" (if it really simplifies things). Please read the review in patch 2 first ) > diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c > b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c

Re: [Intel-gfx] [PATCH v4] overflow: Introduce overflows_type() and castable_to_type()

2022-10-21 Thread Nathan Chancellor
Hi Gwan-gyeong, On Fri, Oct 21, 2022 at 11:33:33AM +0300, Gwan-gyeong Mun wrote: > From: Kees Cook > > Implement a robust overflows_type() macro to test if a variable or > constant value would overflow another variable or type. This can be > used as a constant expression for static_assert() (whi

Re: [Intel-gfx] [PATCH 3/5] drm/i915/mtl: Modify CAGF functions for MTL

2022-10-21 Thread Dixit, Ashutosh
On Wed, 19 Oct 2022 16:37:19 -0700, Ashutosh Dixit wrote: > > From: Badal Nilawar > > Update CAGF functions for MTL to get actual resolved frequency of 3D and > SAMedia. > > v2: Update MTL_MIRROR_TARGET_WP1 position/formatting (MattR) > Move MTL branches in cagf functions to top (MattR) >

Re: [Intel-gfx] [PATCH 1/5] drm/i915/rps: Prefer REG_FIELD_GET in intel_rps_get_cagf

2022-10-21 Thread Rodrigo Vivi
On Wed, Oct 19, 2022 at 04:37:17PM -0700, Ashutosh Dixit wrote: > Instead of masks/shifts settle on REG_FIELD_GET as the standard way to > extract reg fields. This allows future patches touching this code to also > consistently use REG_FIELD_GET and friends. > > Suggested-by: Rodrigo Vivi > Signe

Re: [Intel-gfx] [PATCH 2/5] drm/i915: Use GEN12_RPSTAT register for GT freq

2022-10-21 Thread Rodrigo Vivi
On Wed, Oct 19, 2022 at 04:37:18PM -0700, Ashutosh Dixit wrote: > From: Don Hiatt > > On GEN12+ use GEN12_RPSTAT register to get actual resolved GT > freq. GEN12_RPSTAT does not require a forcewake and will return 0 freq if > GT is in RC6. > > v2: > - Fixed review comments(Ashutosh) > - Adde

Re: [Intel-gfx] [PATCH 3/5] drm/i915/mtl: Modify CAGF functions for MTL

2022-10-21 Thread Rodrigo Vivi
On Fri, Oct 21, 2022 at 09:02:03AM -0700, Dixit, Ashutosh wrote: > On Wed, 19 Oct 2022 16:37:19 -0700, Ashutosh Dixit wrote: > > > > From: Badal Nilawar > > > > Update CAGF functions for MTL to get actual resolved frequency of 3D and > > SAMedia. > > > > v2: Update MTL_MIRROR_TARGET_WP1 position/f

Re: [Intel-gfx] [PATCH 4/5] drm/i915/gt: Use RC6 residency types as arguments to residency functions

2022-10-21 Thread Rodrigo Vivi
On Wed, Oct 19, 2022 at 04:37:20PM -0700, Ashutosh Dixit wrote: > Previously RC6 residency functions directly accepted RC6 residency register > MMIO offsets (there are four RC6 residency registers). This worked but > required an assumption on the residency register layout so was not future > proof.

[Intel-gfx] [PATCH v2 0/4] drm/i915: Clean up crtc state flag checks

2022-10-21 Thread Ville Syrjala
From: Ville Syrjälä Some cleanups for checking whether the crtc was flagged for modesets/fastsets/color update. v2: Handle {connectors,active}_changed, and convert one more fastset check in fbc code Ville Syrjälä (4): drm/i915: Introduce intel_crtc_needs_fastset() drm/i915: Remove some

[Intel-gfx] [PATCH v2 1/4] drm/i915: Introduce intel_crtc_needs_fastset()

2022-10-21 Thread Ville Syrjala
From: Ville Syrjälä Replace the somewhat obscure crtc_state.update_pipe checks with a more descriptive thing. Also nicely matches the intel_crtc_needs_modeset() counterpart for full modesets. v2: Handle one more case in the fbc code Reviewed-by: Jani Nikula #v1 Signed-off-by: Ville Syrjälä --

[Intel-gfx] [PATCH v2 2/4] drm/i915: Remove some local 'mode_changed' bools

2022-10-21 Thread Ville Syrjala
From: Ville Syrjälä These 'mode_changed' booleans aren't very helpful. Just replace them with direct intel_crtc_needs_modeset() calls which is more descriptive. Reviewed-by: Jani Nikula Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_display.c | 14 ++ 1 file c

[Intel-gfx] [PATCH v2 3/4] drm/i915: Don't flag both full modeset and fastset at the same time

2022-10-21 Thread Ville Syrjala
From: Ville Syrjälä Be consistent in whether we flag a full modeset or a fastset for the pipe. intel_modeset_all_pipes() would seem to be the only codepath not getting this right. The other case is when we flag the fastset initially, currently we just clear the mode_changed flag and set the upda

[Intel-gfx] [PATCH v2 4/4] drm/i915: Introduce intel_crtc_needs_color_update()

2022-10-21 Thread Ville Syrjala
From: Ville Syrjälä Add a common helper to answer the question "do we need to update color management stuff?". Reviewed-by: Jani Nikula Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/hsw_ips.c | 6 ++ drivers/gpu/drm/i915/display/intel_crtc.c | 3 +-- drivers

Re: [Intel-gfx] [PATCH v2 7/7] drm/i915/pxp: Make intel_pxp power management implicitly sort PXP-owning-GT

2022-10-21 Thread Teres Alexis, Alan Previn
with the removal of pxptee_iface_owner and the separation of a global-pxp check vs a per-gt pxp (is enabled) check, this patch will get dropped as the latter change would appear in earlier in the series On Wed, 2022-10-05 at 21:38 -0700, Alan Previn wrote: > pxptee_iface_owner

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Fix a potential UAF at device unload

2022-10-21 Thread Ville Syrjälä
On Fri, Sep 23, 2022 at 09:35:14AM +0200, Nirmoy Das wrote: > i915_gem_drain_freed_objects() might not be enough to > free all the objects and RCU delayed work might get > scheduled after the i915 device struct gets freed. > > Call i915_gem_drain_workqueue() to catch all RCU delayed work. shard-s

Re: [Intel-gfx] [PATCH 5/5] drm/i915/mtl: C6 residency and C state type for MTL SAMedia

2022-10-21 Thread Rodrigo Vivi
On Wed, Oct 19, 2022 at 04:37:21PM -0700, Ashutosh Dixit wrote: > From: Badal Nilawar > > Add support for C6 residency and C state type for MTL SAMedia. Also add > mtl_drpc. I believe this patch deserves a slip between the actual support and the debugfs, but I'm late to the review, so feel free

Re: [Intel-gfx] [PATCH v3 6/6] freezer, sched: Rewrite core freezer logic

2022-10-21 Thread Ville Syrjälä
On Mon, Aug 22, 2022 at 01:18:22PM +0200, Peter Zijlstra wrote: > +#ifdef CONFIG_LOCKDEP > + /* > + * It's dangerous to freeze with locks held; there be dragons there. > + */ > + if (!(state & __TASK_FREEZABLE_UNSAFE)) > + WARN_ON_ONCE(debug_locks && p->lockdep_depth);

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Clean up crtc state flag checks (rev3)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: Clean up crtc state flag checks (rev3) URL : https://patchwork.freedesktop.org/series/109949/ State : success == Summary == CI Bug Log - changes from CI_DRM_12275 -> Patchwork_109949v3 Summary ---

[Intel-gfx] [PATCH v3 0/6] drm/i915/pxp: Prepare intel_pxp entry points for MTL

2022-10-21 Thread Alan Previn
MTL has two tiles that is represented by the intel_gt structure in the i915 code. The PXP feature has a control-structure that contains the PXP context and this hangs of the intel_gt structure. In MTL, the standalone media tile (i.e. not the root tile) contains the VDBOX and KCR engine which is wh

[Intel-gfx] [PATCH v3 2/6] drm/i915/pxp: Make intel_pxp_is_enabled implicitly sort PXP-owning-GT

2022-10-21 Thread Alan Previn
Make intel_pxp_is_enabled a global check and implicitly find the PXP-owning-GT. PXP feature support is a device-config flag. In preparation for MTL PXP control-context shall reside on of the two GT's. That said, update intel_pxp_is_enabled to take in i915 as its input and internally find the right

[Intel-gfx] [PATCH v3 1/6] drm/i915/pxp: Make gt and pxp init/fini aware of PXP-owning-GT

2022-10-21 Thread Alan Previn
In preparation for future MTL-PXP feature support, PXP control context should only valid on the correct gt tile. Depending on the device-info this depends on which tile owns the VEBOX and KCR. PXP is still a global feature though (despite its control-context located in the owning GT structure). Add

[Intel-gfx] [PATCH v3 3/6] drm/i915/pxp: Make intel_pxp_is_active implicitly sort PXP-owning-GT

2022-10-21 Thread Alan Previn
Make intel_pxp_is_active a global check and implicitly find the PXP-owning-GT. As per prior two patches, callers of this function shall now pass in i915 since PXP is a global GPU feature. Make intel_pxp_is_active implicitly find the right gt so it's transparent for global view callers (like displa

[Intel-gfx] [PATCH v3 4/6] drm/i915/pxp: Make PXP tee component bind/unbind aware of PXP-owning-GT

2022-10-21 Thread Alan Previn
Ensure i915_pxp_tee_component_bind / unbind implicitly sorts out getting the correct per-GT PXP control-context from the PXP-owning-GT when establishing or ending connection. Thus, replace _i915_to_pxp_gt with intel_pxp_get_owning_gt (also takes in i915). Signed-off-by: Alan Previn --- drivers/g

[Intel-gfx] [PATCH v3 6/6] drm/i915/pxp: Make intel_pxp_key_check implicitly sort PXP-owning-GT

2022-10-21 Thread Alan Previn
Make intel_pxp_key_check implicitly find the PXP-owning-GT. Callers of this function shall now pass in i915 since PXP is a global GPU feature. Make intel_pxp_key_check implicitly find the right gt to verify pxp session key establishment count so it's transparent to the callers. Signed-off-by: Alan

[Intel-gfx] [PATCH v3 5/6] drm/i915/pxp: Make intel_pxp_start implicitly sort PXP-owning-GT

2022-10-21 Thread Alan Previn
Make intel_pxp_is_start implicitly find the PXP-owning-GT. Callers of this function shall now pass in i915 since PXP is a global GPU feature. Make intel_pxp_start implicitly find the right gt to start PXP arb session so it's transparent to the callers. Signed-off-by: Alan Previn --- drivers/gpu/

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for i915: CAGF and RC6 changes for MTL (rev9)

2022-10-21 Thread Patchwork
== Series Details == Series: i915: CAGF and RC6 changes for MTL (rev9) URL : https://patchwork.freedesktop.org/series/108156/ 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: success for i915: CAGF and RC6 changes for MTL (rev9)

2022-10-21 Thread Patchwork
== Series Details == Series: i915: CAGF and RC6 changes for MTL (rev9) URL : https://patchwork.freedesktop.org/series/108156/ State : success == Summary == CI Bug Log - changes from CI_DRM_12275 -> Patchwork_108156v9 Summary --- **SU

Re: [Intel-gfx] [PATCH v3] drm/i915/slpc: Optmize waitboost for SLPC

2022-10-21 Thread Belgaumkar, Vinay
On 10/20/2022 4:36 PM, Dixit, Ashutosh wrote: On Thu, 20 Oct 2022 13:16:00 -0700, Belgaumkar, Vinay wrote: On 10/20/2022 11:33 AM, Dixit, Ashutosh wrote: On Wed, 19 Oct 2022 17:29:44 -0700, Vinay Belgaumkar wrote: Hi Vinay, Waitboost (when SLPC is enabled) results in a H2G message. This can

Re: [Intel-gfx] [PATCH v2 1/2] drm/i915: use intel_uncore_rmw when appropriate

2022-10-21 Thread Andi Shyti
Hi Andrzej, On Wed, Oct 19, 2022 at 04:38:17PM +0200, Andrzej Hajda wrote: > This patch replaces all occurences of the form > intel_uncore_write(reg, intel_uncore_read(reg) OP val) > with intel_uncore_rmw. > > Signed-off-by: Andrzej Hajda Reviewed-by: Andi Shyti There are some lines that go o

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/gt: use intel_uncore_rmw when appropriate

2022-10-21 Thread Andi Shyti
Hi Andrzej, On Wed, Oct 19, 2022 at 04:38:18PM +0200, Andrzej Hajda wrote: > This patch replaces all occurences of the form > intel_uncore_write(reg, intel_uncore_read(reg) OP val) > with intel_uncore_rmw. > > Signed-off-by: Andrzej Hajda Reviewed-by: Andi Shyti Thanks, Andi

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/pxp: Prepare intel_pxp entry points for MTL (rev3)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915/pxp: Prepare intel_pxp entry points for MTL (rev3) URL : https://patchwork.freedesktop.org/series/109429/ State : success == Summary == CI Bug Log - changes from CI_DRM_12275 -> Patchwork_109429v3 Summa

Re: [Intel-gfx] [PATCH v3] drm/i915/slpc: Optmize waitboost for SLPC

2022-10-21 Thread Dixit, Ashutosh
On Fri, 21 Oct 2022 11:24:42 -0700, Belgaumkar, Vinay wrote: > > > On 10/20/2022 4:36 PM, Dixit, Ashutosh wrote: > > On Thu, 20 Oct 2022 13:16:00 -0700, Belgaumkar, Vinay wrote: > >> On 10/20/2022 11:33 AM, Dixit, Ashutosh wrote: > >>> On Wed, 19 Oct 2022 17:29:44 -0700, Vinay Belgaumkar wrote: > >

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/guc: Add compute reglist for guc err capture

2022-10-21 Thread John Harrison
On 10/19/2022 00:29, Alan Previn wrote: We missed this at initial upstream because at that time none of the GuC enabled platforms had a compute engine. Add this now. Signed-off-by: Alan Previn Reviewed-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c | 4 1 file

Re: [Intel-gfx] [PATCH v3 1/2] drm/i915/guc: Add error-capture init warnings when needed

2022-10-21 Thread John Harrison
On 10/19/2022 00:29, Alan Previn wrote: If GuC is being used and we initialized GuC-error-capture, we need to be warning if we don't provide an error-capture register list in the firmware ADS, for valid GT engines. A warning makes sense as this would impact debugability without realizing why a re

[Intel-gfx] ✗ Fi.CI.IGT: failure for overflow: Introduce overflows_type() and castable_to_type() (rev3)

2022-10-21 Thread Patchwork
== Series Details == Series: overflow: Introduce overflows_type() and castable_to_type() (rev3) URL : https://patchwork.freedesktop.org/series/109076/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12273_full -> Patchwork_109076v3_full ==

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Add DG2 OA support (rev9)

2022-10-21 Thread Patchwork
== Series Details == Series: Add DG2 OA support (rev9) URL : https://patchwork.freedesktop.org/series/107584/ State : warning == Summary == Error: dim checkpatch failed 54115be03fa0 drm/i915/perf: Fix OA filtering logic for GuC mode 404934d8ea75 drm/i915/perf: Add 32-bit OAG and OAR formats fo

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Add DG2 OA support (rev9)

2022-10-21 Thread Patchwork
== Series Details == Series: Add DG2 OA support (rev9) URL : https://patchwork.freedesktop.org/series/107584/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

Re: [Intel-gfx] [PATCH 2/4] drm/i915/display: Introduce HAS_CDCLK_SQUASH macro

2022-10-21 Thread Srivatsa, Anusha
> -Original Message- > From: Jani Nikula > Sent: Friday, October 21, 2022 1:47 AM > To: Vivekanandan, Balasubramani > ; Srivatsa, Anusha > ; intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH 2/4] drm/i915/display: Introduce > HAS_CDCLK_SQUASH macro > > On Fri, 21 Oct 202

Re: [Intel-gfx] [PATCH 4/4] drm/i915/display: Move squash_ctl register programming to its own function

2022-10-21 Thread Srivatsa, Anusha
> -Original Message- > From: Jani Nikula > Sent: Friday, October 21, 2022 1:41 AM > To: Srivatsa, Anusha ; intel- > g...@lists.freedesktop.org > Cc: Vivekanandan, Balasubramani > > Subject: Re: [Intel-gfx] [PATCH 4/4] drm/i915/display: Move squash_ctl > register programming to its own f

Re: [Intel-gfx] [PATCH 3/4] drm/i915/display: Move chunks of code out of bxt_set_cdclk()

2022-10-21 Thread Srivatsa, Anusha
> -Original Message- > From: Jani Nikula > Sent: Friday, October 21, 2022 1:32 AM > To: Srivatsa, Anusha ; intel- > g...@lists.freedesktop.org > Cc: Vivekanandan, Balasubramani > > Subject: Re: [Intel-gfx] [PATCH 3/4] drm/i915/display: Move chunks of code > out of bxt_set_cdclk() > > O

Re: [Intel-gfx] [PATCH 4/4] drm/i915/display: Move squash_ctl register programming to its own function

2022-10-21 Thread Srivatsa, Anusha
> -Original Message- > From: Vivekanandan, Balasubramani > > Sent: Friday, October 21, 2022 12:08 AM > To: Srivatsa, Anusha ; intel- > g...@lists.freedesktop.org > Cc: Ville Syrjälä > Subject: Re: [PATCH 4/4] drm/i915/display: Move squash_ctl register > programming to its own function >

[Intel-gfx] [PATCH v2] drm/i915/hdmi: SPD infoframe update for discrete

2022-10-21 Thread Taylor, Clinton A
Replace internal with discrete of dgfx platforms. v2: commit title reword (Jani) Cc: Jani Nikula Signed-off-by: Taylor, Clinton A --- drivers/gpu/drm/i915/display/intel_hdmi.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_hdmi.c b/d

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Activate DRRS after state readout (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: Activate DRRS after state readout (rev2) URL : https://patchwork.freedesktop.org/series/109950/ State : success == Summary == CI Bug Log - changes from CI_DRM_12273_full -> Patchwork_109950v2_full Summ

[Intel-gfx] [PATCH 0/4] Prep series - CDCLK code churn

2022-10-21 Thread Anusha Srivatsa
No functional changes. The series is more of a prep series for the mid_cdclk_config series: https://patchwork.freedesktop.org/series/109694/ Main change: - Change usage of "crawler" and "squasher". Use crawling and squashing instead. - Handle both squash and crawl similar in terms of checking if

[Intel-gfx] [PATCH 2/4] drm/i915/display: Introduce HAS_CDCLK_SQUASH macro

2022-10-21 Thread Anusha Srivatsa
Driver had discrepancy in how cdclk squash and crawl support were checked. Like crawl, add squash as a 1 bit feature flag to the display section of DG2. Cc: Balasubramani Vivekanandan Cc: Ville Syrjälä Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +

[Intel-gfx] [PATCH 3/4] drm/i915/display: Move chunks of code out of bxt_set_cdclk()

2022-10-21 Thread Anusha Srivatsa
No functional change. Moving segments out to simplify bxt_set_cdlck() v2: s/bxt_cdclk_pll/bxt_cdclk_pll_update (Jani) Cc: Jani Nikula Cc: Balasubramani Vivekanandan Cc: Ville Syrjälä Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/display/intel_cdclk.c | 40 ++ 1

[Intel-gfx] [PATCH 4/4] drm/i915/display: Move squash_ctl register programming to its own function

2022-10-21 Thread Anusha Srivatsa
No functional change. Introduce dg2_cdclk_squash_program and move squash_ctl register programming bits to this. v2: s/dg2_cdclk_squash_programming/dg2_cdclk_squash_program (Jani) Cc: Jani Nikula Cc: Balasubramani Vivekanandan Cc: Ville Syrjälä Signed-off-by: Anusha Srivatsa --- drivers/gpu/d

[Intel-gfx] [PATCH 1/4] drm/i915/display: Change terminology for cdclk actions

2022-10-21 Thread Anusha Srivatsa
No functional changes. Changing terminolgy in some print statements. s/has_cdclk_squasher/has_cdclk_squash, s/crawler/crawl and s/squasher/squash. Cc: Balasubramani Vivekanandan Cc: Ville Syrjälä Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/display/intel_cdclk.c | 16 ---

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/hdmi: SPD infoframe updated for discrete (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915/hdmi: SPD infoframe updated for discrete (rev2) URL : https://patchwork.freedesktop.org/series/109908/ State : success == Summary == CI Bug Log - changes from CI_DRM_12277 -> Patchwork_109908v2 Summary

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Prep series - CDCLK code churn (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: Prep series - CDCLK code churn (rev2) URL : https://patchwork.freedesktop.org/series/109974/ State : warning == Summary == Error: dim checkpatch failed 750f09e493d8 drm/i915/display: Change terminology for cdclk actions 7e35fba6b197 drm/i915/display: Introduce HAS_

Re: [Intel-gfx] [PATCH v3] drm/i915/slpc: Optmize waitboost for SLPC

2022-10-21 Thread Belgaumkar, Vinay
On 10/21/2022 11:40 AM, Dixit, Ashutosh wrote: On Fri, 21 Oct 2022 11:24:42 -0700, Belgaumkar, Vinay wrote: On 10/20/2022 4:36 PM, Dixit, Ashutosh wrote: On Thu, 20 Oct 2022 13:16:00 -0700, Belgaumkar, Vinay wrote: On 10/20/2022 11:33 AM, Dixit, Ashutosh wrote: On Wed, 19 Oct 2022 17:29:44

[Intel-gfx] ✗ Fi.CI.BAT: failure for Prep series - CDCLK code churn (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: Prep series - CDCLK code churn (rev2) URL : https://patchwork.freedesktop.org/series/109974/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12277 -> Patchwork_109974v2 Summary --- **FAILUR

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: refactor i915_vma_move_to_active (rev2)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: refactor i915_vma_move_to_active (rev2) URL : https://patchwork.freedesktop.org/series/109910/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12273_full -> Patchwork_109910v2_full Summa

[Intel-gfx] [PATCH 1/1] drm/i915/pxp: Separate PXP FW interface structures for both v42 and 43

2022-10-21 Thread Alan Previn
Previously, we only used PXP FW interface version-42 structures for PXP arbitration session on ADL/TGL products and version-43 for HuC authentication on DG2. With MTL, we'll need both these capabilities side by side with these older platform feature support. That said, let's create separate files t

[Intel-gfx] ✗ Fi.CI.BUILD: failure for series starting with [1/1] drm/i915/pxp: Separate PXP FW interface structures for both v42 and 43

2022-10-21 Thread Patchwork
== Series Details == Series: series starting with [1/1] drm/i915/pxp: Separate PXP FW interface structures for both v42 and 43 URL : https://patchwork.freedesktop.org/series/110013/ State : failure == Summary == Error: patch https://patchwork.freedesktop.org/api/1.0/series/110013/revisions/1

[Intel-gfx] [PATCH v3 0/7] drm/i915: prepare for uC loading on MTL

2022-10-21 Thread Daniele Ceraolo Spurio
The introduction of the media GT brings a few changes for GuC/HuC. The main difference between the 2 GTs is that only the media one has the HuC, while both have the GuC. Also, the fact that both GTs use the same G-unit and GGTT means we now have parallel interrupt/communication paths. Lastly, WOPCM

[Intel-gfx] [PATCH v3 6/7] drm/i915/guc: define media GT GuC send regs

2022-10-21 Thread Daniele Ceraolo Spurio
The media GT shares the G-unit with the root GT, so a second set of communication registers is required for the media GuC. Signed-off-by: Daniele Ceraolo Spurio Cc: John Harrison Cc: Alan Previn Reviewed-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_guc.c | 14 ++ dri

[Intel-gfx] [PATCH v3 5/7] drm/i915/mtl: Handle wopcm per-GT and limit calculations.

2022-10-21 Thread Daniele Ceraolo Spurio
From: Aravind Iddamsetty With MTL standalone media architecture the wopcm layout has changed with separate partitioning in WOPCM for GCD/GT GuC and SA Media GuC. The size of WOPCM is 4MB with lower 2MB for SA Media and upper 2MB for GCD/GT. +=+===> ++ <== WOPCM TOP

[Intel-gfx] [PATCH v3 2/7] drm/i915/uc: fetch uc firmwares for each GT

2022-10-21 Thread Daniele Ceraolo Spurio
The FW binaries are independently loaded on each GT. On MTL, the memory is shared so we could potentially re-use a single allocation, but on discrete multi-gt platforms we are going to need independent copies, so it is easier to do the same on MTL as well, given that the amount of duplicated memory

[Intel-gfx] [PATCH v3 3/7] drm/i915/uc: use different ggtt pin offsets for uc loads

2022-10-21 Thread Daniele Ceraolo Spurio
Our current FW loading process is the same for all FWs: - Pin FW to GGTT at the start of the ggtt->uc_fw node - Load the FW - Unpin This worked because we didn't have a case where 2 FWs would be loaded on the same GGTT at the same time. On MTL, however, this can happen if both GTs are reset at th

[Intel-gfx] [PATCH v3 1/7] drm/i915/huc: only load HuC on GTs that have VCS engines

2022-10-21 Thread Daniele Ceraolo Spurio
On MTL the primary GT doesn't have any media capabilities, so no video engines and no HuC. We must therefore skip the HuC fetch and load on that specific case. Given that other multi-GT platforms might have HuC on the primary GT, we can't just check for that and it is easier to instead check for th

[Intel-gfx] [PATCH v3 4/7] drm/i915/guc: Add GuC deprivilege feature to MTL

2022-10-21 Thread Daniele Ceraolo Spurio
From: Stuart Summers MTL supports GuC deprivilege. Add the feature flag to this platform. Signed-off-by: Stuart Summers Cc: Radhakrishna Sripada Cc: John Harrison Cc: Alan Previn Reviewed-by: John Harrison --- drivers/gpu/drm/i915/i915_pci.c | 1 + 1 file changed, 1 insertion(+) diff --gi

[Intel-gfx] [PATCH v3 7/7] drm/i915/guc: handle interrupts from media GuC

2022-10-21 Thread Daniele Ceraolo Spurio
The render and media GuCs share the same interrupt enable register, so we can no longer disable interrupts when we disable communication for one of the GuCs as this would impact the other GuC. Instead, we keep the interrupts always enabled in HW and use a variable in the GuC structure to determine

Re: [Intel-gfx] [PATCH 2/2] drm/i915/pmu: Connect engine busyness stats from GuC to pmu

2022-10-21 Thread Umesh Nerlige Ramappa
On Fri, Oct 21, 2022 at 09:42:53AM +0100, Tvrtko Ursulin wrote: On 27/10/2021 01:48, Umesh Nerlige Ramappa wrote: [snip] +static void guc_timestamp_ping(struct work_struct *wrk) +{ + struct intel_guc *guc = container_of(wrk, typeof(*guc), +tim

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: prepare for uC loading on MTL (rev3)

2022-10-21 Thread Patchwork
== Series Details == Series: drm/i915: prepare for uC loading on MTL (rev3) URL : https://patchwork.freedesktop.org/series/108925/ State : warning == Summary == Error: dim checkpatch failed 23e8682ebaac drm/i915/huc: only load HuC on GTs that have VCS engines -:44: WARNING:AVOID_BUG: Do not cr

  1   2   >