Re: [Intel-gfx] [PATCH 2/2] drm/i915/hdcp: Move common message filling function to its own file

Hi Suraj, kernel test robot noticed the following build errors: [auto build test ERROR on drm-tip/drm-tip] url: https://github.com/intel-lab-lkp/linux/commits/Suraj-Kandpal/drm-i915-hdcp-Move-checks-for-gsc-health-status/20230920-143839 base: git://anongit.freedesktop.org/drm/drm-tip drm

[Intel-gfx] [PATCH] drm/i915: Zap some empty lines

From: Tvrtko Ursulin Recent refactoring left an unsightly block of empty lines. Remove them. Signed-off-by: Tvrtko Ursulin Cc: Dnyaneshwar Bhadane Cc: Anusha Srivatsa Cc: Radhakrishna Sripada --- drivers/gpu/drm/i915/i915_drv.h | 7 --- 1 file changed, 7 deletions(-) diff --git

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Lift runtime-pm acquire callbacks out of intel_wakeref.mutex

On Mon, 18 Sep 2023, Nirmoy Das wrote: > From: Chris Wilson > > When runtime pm is first woken, it will synchronously call the > registered callbacks for the device. These callbacks > may pull in their own forest of locks, which we do not want to > conflate with the intel_wakeref.mutex. A second

Re: [Intel-gfx] [PATCH 7/7] drm/i915: Enable GGTT updates with binder in MTL

Hi Nirmoy, https://gfxspecs.intel.com/Predator/Home/Index/52885?dstFilter=MTL=Filter I can't find Wa_13010847436 and Wa_14019519902 in the above link, where can I check these WA? Br, G.G. On 9/18/23 8:02 PM, Nirmoy Das wrote: MTL can hang because of a HW bug while parallel reading/writing

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Refactor i915 HDCP for XE (rev2)

== Series Details == Series: Refactor i915 HDCP for XE (rev2) URL : https://patchwork.freedesktop.org/series/123955/ State : warning == Summary == Error: dim checkpatch failed 86d0a82a07c2 drm/i915/hdcp: Move checks for gsc health status c896414f7fe9 drm/i915/hdcp: Move common message filling

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Refactor i915 HDCP for XE (rev2)

== Series Details == Series: Refactor i915 HDCP for XE (rev2) URL : https://patchwork.freedesktop.org/series/123955/ 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:117:1:

Re: [Intel-gfx] [PATCH 2/2] drm/i915/hdcp: Move common message filling function to its own file

> Subject: Re: [Intel-gfx] [PATCH 2/2] drm/i915/hdcp: Move common message > filling function to its own file > > On Wed, 20 Sep 2023, Suraj Kandpal wrote: > > diff --git a/drivers/gpu/drm/i915/display/intel_hdcp_gsc.h > > b/drivers/gpu/drm/i915/display/intel_hdcp_gsc.h > > index

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Zap some empty lines

== Series Details == Series: drm/i915: Zap some empty lines URL : https://patchwork.freedesktop.org/series/123963/ State : success == Summary == CI Bug Log - changes from CI_DRM_13653 -> Patchwork_123963v1 Summary --- **SUCCESS**

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Check lane count when determining FEC support

On Wed, 13 Sep 2023, Ville Syrjala wrote: > From: Ville Syrjälä > > ICL doesn't support FEC with a x1 DP link. Make sure > we don't try to enable FEC in such cases. The question is, should we rather require x2 link for FEC? I suppose x1 link with DSC+FEC is an unlikely scenario with our

Re: [Intel-gfx] [PATCH v3 17/25] drm/i915/dp_mst: Fix PBN calculation with FEC overhead

On Wed, Sep 20, 2023 at 01:58:43PM +0300, Ville Syrjälä wrote: > On Thu, Sep 14, 2023 at 10:26:51PM +0300, Imre Deak wrote: > > On 8b/10b MST links the PBN value for DSC streams must be calculated > > accounting for the FEC overhead. The same applies to 8b/10b non-DSC > > streams if there is

[Intel-gfx] [PATCH 1/2] drm/i915/hdcp: Move checks for gsc health status

Move checks for gsc components required for HDCP 2.2 to work into intel_hdcp_gsc.c. This will also help with XE refactor on HDCP's side. Signed-off-by: Suraj Kandpal --- drivers/gpu/drm/i915/display/intel_hdcp.c | 8 +--- drivers/gpu/drm/i915/display/intel_hdcp_gsc.c | 14

[Intel-gfx] [PATCH 0/2] Refactor i915 HDCP for XE

This patch series contains some refactors for i915 side of things which will help with a cleaner code and maximum reuse of code for XE going forward. Signed-off-by: Suraj Kandpal Suraj Kandpal (2): drm/i915/hdcp: Move checks for gsc health status drm/i915/hdcp: Move common message filling

[Intel-gfx] [PATCH 2/2] drm/i915/hdcp: Move common message filling function to its own file

Create a new file intel_hdcp_gsc_message that contain functions which fill the hdcp messages we send to gsc cs this refactor will help us reuse code for Xe later on Signed-off-by: Suraj Kandpal --- drivers/gpu/drm/i915/Makefile | 1 +

[Intel-gfx] [PATCH 2/2] drm/i915/hdcp: Move common message filling function to its own file

Create a new file intel_hdcp_gsc_message that contain functions which fill the hdcp messages we send to gsc cs this refactor will help us reuse code for Xe later on --v2 -add the missed file for proper build Signed-off-by: Suraj Kandpal --- drivers/gpu/drm/i915/Makefile | 1 +

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Update RC6 mask for mtl_drpc (rev2)

== Series Details == Series: drm/i915/gt: Update RC6 mask for mtl_drpc (rev2) URL : https://patchwork.freedesktop.org/series/123775/ State : success == Summary == CI Bug Log - changes from CI_DRM_13653 -> Patchwork_123775v2 Summary ---

[Intel-gfx] [PATCH] drm/i915: Remove unnecessary memory quiescing for aux inval

i915 already does memory quiesce before signaling breadcrumb so remove extra memory quiescing for aux invalidation which can cause unnecessary side effects. Fixes: 78a6ccd65fa3 ("drm/i915/gt: Ensure memory quiesced before invalidation") Cc: Jonathan Cavitt Cc: Andi Shyti Cc: # v5.8+ Cc:

Re: [Intel-gfx] [PATCH] drm/i915: Fix aux invalidation with proper pipe_control flag

Sent out https://patchwork.freedesktop.org/series/123975/ to replace this one as this not really fixing the issue. Thanks, Nirmoy On 9/19/2023 2:19 PM, Tapani Pälli wrote: On 19.9.2023 15.11, Andi Shyti wrote: Hi Nirmoy, On Tue, Sep 19, 2023 at 01:47:16PM +0200, Nirmoy Das wrote: The

Re: [Intel-gfx] [PATCH v3 18/25] drm/i915/dp_mst: Add atomic state for all streams on pre-tgl platforms

On Wed, Sep 20, 2023 at 01:59:53PM +0300, Ville Syrjälä wrote: > On Wed, Sep 20, 2023 at 12:11:58PM +0300, Lisovskiy, Stanislav wrote: > > On Thu, Sep 14, 2023 at 10:26:52PM +0300, Imre Deak wrote: > > > If an MST stream is modeset, its state must be checked along all the > > > other streams on

Re: [Intel-gfx] [PATCH v3 17/25] drm/i915/dp_mst: Fix PBN calculation with FEC overhead

On Thu, Sep 14, 2023 at 10:26:51PM +0300, Imre Deak wrote: > On 8b/10b MST links the PBN value for DSC streams must be calculated > accounting for the FEC overhead. The same applies to 8b/10b non-DSC > streams if there is another DSC stream on the same link. Fix up the PBN > calculation

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Require FEC for DSC on DP-MST

On Wed, 13 Sep 2023, Ville Syrjala wrote: > From: Ville Syrjälä > > The current check just asserts that we need FEC to use DSC > with (non-eDP) DP-SST. But MST also needs FEC for DSC. Just > check for !eDP instead to cover all the cases correctly. 128b/132b won't have crtc->fec_enable set, as

[Intel-gfx] ✓ Fi.CI.BAT: success for Refactor i915 HDCP for XE (rev2)

== Series Details == Series: Refactor i915 HDCP for XE (rev2) URL : https://patchwork.freedesktop.org/series/123955/ State : success == Summary == CI Bug Log - changes from CI_DRM_13653 -> Patchwork_123955v2 Summary --- **SUCCESS**

[Intel-gfx] [PATCH v3 2/2] drm/i915/hdcp: Move common message filling function to its own file

Create a new file intel_hdcp_gsc_message that contain functions which fill the hdcp messages we send to gsc cs this refactor will help us reuse code for Xe later on --v2 -add the missed file for proper build --v3 -use forward declarations instead of #includes [Jani] Signed-off-by: Suraj Kandpal

Re: [Intel-gfx] [PATCH v3 17/25] drm/i915/dp_mst: Fix PBN calculation with FEC overhead

On Thu, Sep 14, 2023 at 10:26:51PM +0300, Imre Deak wrote: > On 8b/10b MST links the PBN value for DSC streams must be calculated > accounting for the FEC overhead. The same applies to 8b/10b non-DSC > streams if there is another DSC stream on the same link. Fix up the PBN > calculation

Re: [Intel-gfx] [PATCH v3 18/25] drm/i915/dp_mst: Add atomic state for all streams on pre-tgl platforms

On Wed, Sep 20, 2023 at 12:11:58PM +0300, Lisovskiy, Stanislav wrote: > On Thu, Sep 14, 2023 at 10:26:52PM +0300, Imre Deak wrote: > > If an MST stream is modeset, its state must be checked along all the > > other streams on the same MST link, for instance to resolve a BW > > overallocation of a

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Check lane count when determining FEC support

On Wed, Sep 20, 2023 at 12:23:34PM +0300, Jani Nikula wrote: > On Wed, 13 Sep 2023, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > ICL doesn't support FEC with a x1 DP link. Make sure > > we don't try to enable FEC in such cases. > > The question is, should we rather require x2 link for

Re: [Intel-gfx] [PATCH] i915: Limit the length of an sg list to the requested length

On 19.09.2023 21:48, Matthew Wilcox (Oracle) wrote: The folio conversion changed the behaviour of shmem_sg_alloc_table() to put the entire length of the last folio into the sg list, even if the sg list should have been shorter. gen8_ggtt_insert_entries() relied on the list being the right

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

Hi Shekhar, kernel test robot noticed the following build errors: [auto build test ERROR on drm-tip/drm-tip] url: https://github.com/intel-lab-lkp/linux/commits/Shekhar-Chauhan/drm-i915-Add-Wa_18028616096/20230920-121417 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip patch link

Re: [Intel-gfx] [PATCH 2/2] drm/i915/hdcp: Move common message filling function to its own file

On Wed, 20 Sep 2023, Suraj Kandpal wrote: > diff --git a/drivers/gpu/drm/i915/display/intel_hdcp_gsc.h > b/drivers/gpu/drm/i915/display/intel_hdcp_gsc.h > index eba2057c5a9e..e53dbe8d9048 100644 > --- a/drivers/gpu/drm/i915/display/intel_hdcp_gsc.h > +++

Re: [Intel-gfx] [PATCH v3 18/25] drm/i915/dp_mst: Add atomic state for all streams on pre-tgl platforms

On Thu, Sep 14, 2023 at 10:26:52PM +0300, Imre Deak wrote: > If an MST stream is modeset, its state must be checked along all the > other streams on the same MST link, for instance to resolve a BW > overallocation of a non-sink MST port or to make sure that the FEC is > enabled/disabled the same

Re: [Intel-gfx] [PATCH 2/2] drm/i915/hdcp: Move common message filling function to its own file

Hi Suraj, kernel test robot noticed the following build errors: [auto build test ERROR on drm-tip/drm-tip] url: https://github.com/intel-lab-lkp/linux/commits/Suraj-Kandpal/drm-i915-hdcp-Move-checks-for-gsc-health-status/20230920-143839 base: git://anongit.freedesktop.org/drm/drm-tip drm

Re: [Intel-gfx] [PATCH] drm/i915: Zap some empty lines

> -Original Message- > From: dri-devel On Behalf Of Tvrtko > Ursulin > Sent: Wednesday, September 20, 2023 2:27 PM > To: Intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Cc: Srivatsa, Anusha ; Bhadane, Dnyaneshwar > ; Sripada, Radhakrishna > ; Ursulin, Tvrtko >

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Require FEC for DSC on DP-MST

On Wed, Sep 20, 2023 at 12:20:51PM +0300, Jani Nikula wrote: > On Wed, 13 Sep 2023, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > The current check just asserts that we need FEC to use DSC > > with (non-eDP) DP-SST. But MST also needs FEC for DSC. Just > > check for !eDP instead to cover

[Intel-gfx] [PATCH] drm/i915: add a note about fec_enable with 128b/132b

Add a note that fec_enable actually means FEC is to be enabled explicitly. 128b/132b always has FEC enabled, the driver doesn't need to enable it separately, and fec_enable will be false. Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/display/intel_display_types.h | 7

Re: [Intel-gfx] [PATCH] MAINTAINERS: drm/ci: add entries for xfail files

Hi, On Tue, Sep 19, 2023 at 03:22:49PM -0300, Helen Koike wrote: > DRM CI keeps track of which tests are failing, flaking or being skipped > by the ci in the expectations files. Add entries for those files to the > corresponding driver maintainer, so they can be notified when they > change. > >

[Intel-gfx] [PATCH v2] drm/i915/gt: Update RC6 mask for mtl_drpc

It is seen that for RC6 status register is sometimes setting unused bits without affecting functionality. So updated the mask with used bits. As mtl_drpc is debug fs function removing MISSING_CASE from default case as it doesn't make sense to panic (panic_on_warn=1) the system if register is

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Refactor i915 HDCP for XE (rev3)

== Series Details == Series: Refactor i915 HDCP for XE (rev3) URL : https://patchwork.freedesktop.org/series/123955/ State : warning == Summary == Error: dim checkpatch failed 004d02f4d754 drm/i915/hdcp: Move checks for gsc health status 39677cfad370 drm/i915/hdcp: Move common message filling

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Refactor i915 HDCP for XE (rev3)

== Series Details == Series: Refactor i915 HDCP for XE (rev3) URL : https://patchwork.freedesktop.org/series/123955/ 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:117:1:

Re: [Intel-gfx] [PATCH] drm/i915: add a note about fec_enable with 128b/132b

On Wed, Sep 20, 2023 at 02:29:01PM +0300, Jani Nikula wrote: > Add a note that fec_enable actually means FEC is to be enabled > explicitly. 128b/132b always has FEC enabled, the driver doesn't need to > enable it separately, and fec_enable will be false. > > Cc: Ville Syrjälä > Signed-off-by:

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Remove unnecessary memory quiescing for aux inval

== Series Details == Series: drm/i915: Remove unnecessary memory quiescing for aux inval URL : https://patchwork.freedesktop.org/series/123975/ State : success == Summary == CI Bug Log - changes from CI_DRM_13655 -> Patchwork_123975v1

[Intel-gfx] ✓ Fi.CI.BAT: success for Refactor i915 HDCP for XE (rev3)

== Series Details == Series: Refactor i915 HDCP for XE (rev3) URL : https://patchwork.freedesktop.org/series/123955/ State : success == Summary == CI Bug Log - changes from CI_DRM_13655 -> Patchwork_123955v3 Summary --- **SUCCESS**

Re: [Intel-gfx] [PATCH v2] drm: Update file owner during use

On 28/08/2023 20:58, Rob Clark wrote: On Wed, Jun 21, 2023 at 2:48 AM Tvrtko Ursulin wrote: From: Tvrtko Ursulin With the typical model where the display server opens the file descriptor and then hands it over to the client(*), we were showing stale data in debugfs. Fix it by updating

Re: [Intel-gfx] [PATCH v3 18/25] drm/i915/dp_mst: Add atomic state for all streams on pre-tgl platforms

On Wed, Sep 20, 2023 at 03:38:05PM +0300, Imre Deak wrote: > On Wed, Sep 20, 2023 at 02:25:14PM +0300, Lisovskiy, Stanislav wrote: > > On Wed, Sep 20, 2023 at 01:59:53PM +0300, Ville Syrjälä wrote: > > > On Wed, Sep 20, 2023 at 12:11:58PM +0300, Lisovskiy, Stanislav wrote: > > > > On Thu, Sep 14,

Re: [Intel-gfx] [PATCH 5/5] drm/i915: Implement fdinfo memory stats printing

On 24/08/2023 12:35, Upadhyay, Tejas wrote: -Original Message- From: Intel-gfx On Behalf Of Tvrtko Ursulin Sent: Friday, July 7, 2023 6:32 PM To: Intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org Subject: [Intel-gfx] [PATCH 5/5] drm/i915: Implement fdinfo memory stats

Re: [Intel-gfx] [PATCH] drm/i915: Do not disable preemption for resets

On 13/09/2023 18:04, Valentin Schneider wrote: On Wed, 13 Sept 2023 at 18:48, Sebastian Andrzej Siewior wrote: On 2023-07-05 10:30:25 [+0100], Tvrtko Ursulin wrote: From: Tvrtko Ursulin Commit ade8a0f59844 ("drm/i915: Make all GPU resets atomic") added a preempt disable section over the

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Check lane count when determining FEC support

On Wed, 20 Sep 2023, Ville Syrjälä wrote: > On Wed, Sep 20, 2023 at 12:23:34PM +0300, Jani Nikula wrote: >> On Wed, 13 Sep 2023, Ville Syrjala wrote: >> > From: Ville Syrjälä >> > >> > ICL doesn't support FEC with a x1 DP link. Make sure >> > we don't try to enable FEC in such cases. >> >> The

Re: [Intel-gfx] [PATCH v3 18/25] drm/i915/dp_mst: Add atomic state for all streams on pre-tgl platforms

On Wed, Sep 20, 2023 at 02:25:14PM +0300, Lisovskiy, Stanislav wrote: > On Wed, Sep 20, 2023 at 01:59:53PM +0300, Ville Syrjälä wrote: > > On Wed, Sep 20, 2023 at 12:11:58PM +0300, Lisovskiy, Stanislav wrote: > > > On Thu, Sep 14, 2023 at 10:26:52PM +0300, Imre Deak wrote: > > > > If an MST stream

Re: [Intel-gfx] [PATCH] drm/i915: Remove unnecessary memory quiescing for aux inval

I tested this first against tests that were failing for Mesa and it fixes all of the regressed cases (TGL LP). Also did a full run of all KHR-GLES31* and KHR-GLES32* test groups in the Khronos CTS suite, no regressions observed. Tested-by: Tapani Pälli On 20.9.2023 14.11, Nirmoy Das wrote:

Re: [Intel-gfx] [PATCH v2] drm: Update file owner during use

Am 20.09.23 um 15:21 schrieb Tvrtko Ursulin: On 28/08/2023 20:58, Rob Clark wrote: On Wed, Jun 21, 2023 at 2:48 AM Tvrtko Ursulin wrote: From: Tvrtko Ursulin With the typical model where the display server opens the file descriptor and then hands it over to the client(*), we were

Re: [Intel-gfx] [PATCH 00/10] drm/i915/spi: spi access for discrete graphics

> > > > No SPI controllers are directly visible to userspace, some SPI devices > > > > are selectively exposed but that needs to be explicitly requested and is > > > > generally discouraged. > > > > What are the options here? Explicitly request exception is the one. > > > Any other way to add

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t] tests/i915_pm_freq_api: Set min/max to expected values

On 9/19/2023 12:32 AM, Vinay Belgaumkar wrote: A prior(rps) test leaves the system in a bad state causing failures in the basic test. Set min/max to expected values before running it. Test will restore values at the end. Link: https://gitlab.freedesktop.org/drm/intel/-/issues/8670

Re: [Intel-gfx] [PATCH 5/5] drm/i915: Implement fdinfo memory stats printing

On Wed, Sep 20, 2023 at 7:35 AM Tvrtko Ursulin wrote: > > > On 24/08/2023 12:35, Upadhyay, Tejas wrote: > >> -Original Message- > >> From: Intel-gfx On Behalf Of > >> Tvrtko > >> Ursulin > >> Sent: Friday, July 7, 2023 6:32 PM > >> To: Intel-gfx@lists.freedesktop.org;

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/hdcp: Move common message filling function to its own file

On Wed, 20 Sep 2023, Suraj Kandpal wrote: > diff --git a/drivers/gpu/drm/i915/display/intel_hdcp_gsc_message.h > b/drivers/gpu/drm/i915/display/intel_hdcp_gsc_message.h > new file mode 100644 > index ..1096dd36823f > --- /dev/null > +++

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

Quoting Shekhar Chauhan (2023-09-20 01:05:47-03:00) >Drop UGM per set fragment threshold to 3 > >BSpec: 54833 >Signed-off-by: Shekhar Chauhan >--- > drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 + > drivers/gpu/drm/i915/gt/intel_workarounds.c | 3 +++ > 2 files changed, 4 insertions(+) > >diff

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_pm_freq_api: Set min/max to expected values

On Mon, Sep 18, 2023 at 12:02:59PM -0700, Vinay Belgaumkar wrote: > A prior(rps) test leaves the system in a bad state causing failures > in the basic test. Why? What was the freq immediately before the failure that made the machine to be busted and not accept the new freq request? Maybe we

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_pm_freq_api: Set min/max to expected values

On 9/20/2023 7:07 AM, Rodrigo Vivi wrote: On Mon, Sep 18, 2023 at 12:02:59PM -0700, Vinay Belgaumkar wrote: A prior(rps) test leaves the system in a bad state causing failures in the basic test. Why? What was the freq immediately before the failure that made the machine to be busted and not

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: add a note about fec_enable with 128b/132b

== Series Details == Series: drm/i915: add a note about fec_enable with 128b/132b URL : https://patchwork.freedesktop.org/series/123977/ State : success == Summary == CI Bug Log - changes from CI_DRM_13656 -> Patchwork_123977v1 Summary

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Do not disable preemption for resets (rev2)

== Series Details == Series: drm/i915: Do not disable preemption for resets (rev2) URL : https://patchwork.freedesktop.org/series/120218/ State : warning == Summary == Error: dim checkpatch failed 8595cf4574a7 drm/i915: Do not disable preemption for resets -:14: ERROR:GIT_COMMIT_ID: Please

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915/hdcp: Move common message filling function to its own file

> Subject: Re: [PATCH v3 2/2] drm/i915/hdcp: Move common message filling > function to its own file > > On Wed, 20 Sep 2023, Suraj Kandpal wrote: > > diff --git a/drivers/gpu/drm/i915/display/intel_hdcp_gsc_message.h > > b/drivers/gpu/drm/i915/display/intel_hdcp_gsc_message.h > > new file mode

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,01/22] drm/i915/xelpdp: Add XE_LPDP_FEATURES (rev2)

== Series Details == Series: series starting with [CI,01/22] drm/i915/xelpdp: Add XE_LPDP_FEATURES (rev2) URL : https://patchwork.freedesktop.org/series/123938/ State : warning == Summary == Error: dim checkpatch failed 90dd23f88c0a drm/i915/xelpdp: Add XE_LPDP_FEATURES 6330ee8e7216

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [CI,01/22] drm/i915/xelpdp: Add XE_LPDP_FEATURES (rev2)

== Series Details == Series: series starting with [CI,01/22] drm/i915/xelpdp: Add XE_LPDP_FEATURES (rev2) URL : https://patchwork.freedesktop.org/series/123938/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [CI,01/22] drm/i915/xelpdp: Add XE_LPDP_FEATURES (rev2)

== Series Details == Series: series starting with [CI,01/22] drm/i915/xelpdp: Add XE_LPDP_FEATURES (rev2) URL : https://patchwork.freedesktop.org/series/123938/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13656 -> Patchwork_123938v2

Re: [Intel-gfx] [PATCH 1/1] drm/i915/uapi: Enable L3 Bank Count Querying

On Thu, Sep 14, 2023 at 11:32:49AM -0700, Jonathan Cavitt wrote: > Extend the query ioctl to allow querying the count of the available L3 > Banks on a given engine. Why do you need this? Who is using? Where's the pull request for the UMDs and IGTs? > > Signed-off-by: Jonathan Cavitt > ---

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Lift runtime-pm acquire callbacks out of intel_wakeref.mutex

Hi Jani, On 9/20/2023 11:03 AM, Jani Nikula wrote: On Mon, 18 Sep 2023, Nirmoy Das wrote: From: Chris Wilson When runtime pm is first woken, it will synchronously call the registered callbacks for the device. These callbacks may pull in their own forest of locks, which we do not want to

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Do not disable preemption for resets (rev2)

== Series Details == Series: drm/i915: Do not disable preemption for resets (rev2) URL : https://patchwork.freedesktop.org/series/120218/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13656 -> Patchwork_120218v2 Summary

Re: [Intel-gfx] [PATCH 00/10] drm/i915/spi: spi access for discrete graphics

On Wed, Sep 20, 2023 at 01:52:07PM +, Usyskin, Alexander wrote: > I've tried to register spi controller with a spi-mem ops, but I can't find a > way to connect to mtd subsystem. > I took spi-intel as example, which connects to spi-nor but it relies on JDEC > ID of flash to configure itself.

[Intel-gfx] ✗ Fi.CI.BUILD: failure for Refactor i915 HDCP for XE

== Series Details == Series: Refactor i915 HDCP for XE URL : https://patchwork.freedesktop.org/series/123955/ State : failure == Summary == Error: make failed CALLscripts/checksyscalls.sh DESCEND objtool INSTALL libsubcmd_headers CC [M]

Re: [Intel-gfx] [PATCH] drm/i915: Zap some empty lines

On Wed, 20 Sep 2023, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Recent refactoring left an unsightly block of empty lines. Remove them. > > Signed-off-by: Tvrtko Ursulin > Cc: Dnyaneshwar Bhadane > Cc: Anusha Srivatsa > Cc: Radhakrishna Sripada One of the harder patches I've reviewed

[Intel-gfx] ✗ Fi.CI.IGT: failure for MAINTAINERS: drm/ci: add entries for xfail files

== Series Details == Series: MAINTAINERS: drm/ci: add entries for xfail files URL : https://patchwork.freedesktop.org/series/123932/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13652_full -> Patchwork_123932v1_full

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: Zap some empty lines

== Series Details == Series: drm/i915: Zap some empty lines URL : https://patchwork.freedesktop.org/series/123963/ 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 Update GGTT with MI_UPDATE_GTT on MTL (rev8)

== Series Details == Series: Update GGTT with MI_UPDATE_GTT on MTL (rev8) URL : https://patchwork.freedesktop.org/series/123329/ State : warning == Summary == Error: dim checkpatch failed a1b7a590c8ed drm/i915: Lift runtime-pm acquire callbacks out of intel_wakeref.mutex -:57:

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Update GGTT with MI_UPDATE_GTT on MTL (rev8)

== Series Details == Series: Update GGTT with MI_UPDATE_GTT on MTL (rev8) URL : https://patchwork.freedesktop.org/series/123329/ 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 00/12] drm/i915: VRR, LRR, and M/N stuff

On Wed, Sep 20, 2023 at 11:54:41AM -0700, Manasi Navare wrote: > Hi Ville, > > After the comments have been addressed, I have completed reviewing the > patches. Is there anything > else blocking this from getting merged? Could we get this merged if > everything looks good? Series pushed to

Re: [Intel-gfx] [PATCH 00/10] drm/i915/spi: spi access for discrete graphics

> > On Wed, Sep 20, 2023 at 01:52:07PM +, Usyskin, Alexander wrote: > > > I've tried to register spi controller with a spi-mem ops, but I can't find > > a way > to connect to mtd subsystem. > > I took spi-intel as example, which connects to spi-nor but it relies on > > JDEC ID > of flash

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: add a note about fec_enable with 128b/132b

== Series Details == Series: drm/i915: add a note about fec_enable with 128b/132b URL : https://patchwork.freedesktop.org/series/123977/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13656_full -> Patchwork_123977v1_full

[Intel-gfx] [PATCH] drm/i915/gem: Allow users to disable waitboost

Provide a bit to disable waitboost while waiting on a gem object. Waitboost results in increased power consumption by requesting RP0 while waiting for the request to complete. Add a bit in the gem_wait() IOCTL where this can be disabled. This is related to the libva API change here - Link:

Re: [Intel-gfx] [PATCH] drm/edid: Fixup h/vsync_end instead of h/vtotal

On Thu, 14 Sep 2023, Ville Syrjala wrote: > From: Ville Syrjälä > > There are some weird EDIDs floating around that have the sync > pulse extending beyond the end of the blanking period. > > On the currently problemtic machine (HP Omni 120) EDID reports > the following mode: > "1600x900": 60

Re: [Intel-gfx] [PATCH v2 12/12] drm/i915: Implement transcoder LRR for TGL+

Hi Ville, Quick question here on the use case and the trigger for the LRR case which is within VRR range. Could this perhaps be used if we had a virtual mode say 40Hz that now falls in the VRR range (30 -120Hz) that is exposed through the connector mode list and then if we do a modest to 40Hz

[Intel-gfx] [PATCH 1/4] drm/i915/vlv_dsi: Add DMI quirk for wrong panel modeline in BIOS on Asus TF103C (v3)

Vtotal is wrong in the BIOS supplied modeline for the DSI panel on the Asus TF103C leading to the last line of the display being shown as the first line. Original: "1280x800": 60 67700 1280 1312 1328 1376 800 808 812 820 0x8 0xa Fixed:"1280x800": 60 67700 1280 1312 1328 1376 800 808 812 816

[Intel-gfx] [PATCH 4/4] drm/i915/dsi: Add some debug logging to mipi_exec_i2c (v2)

Add some debug logging to mipi_exec_i2c, to make debugging various issues seen with it easier. Changes in v2: - Drop unnecessary __func__ drm_dbg_kms() argument Signed-off-by: Hans de Goede --- drivers/gpu/drm/i915/display/intel_dsi_vbt.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

[Intel-gfx] [PATCH 3/4] drm/i915/vlv_dsi: Add DMI quirk for backlight control issues on Lenovo Yoga Tab 3 (v2)

On the Lenovo Yoga Tab 3 Pro YT3-X90F there are 2 issues with the backlight on/off MIPI sequences: 1. The backlight on sequence has an I2C MIPI sequence element which uses bus 0, but there is a bogus I2cSerialBus resource under the GPU in the DSDT which causes i2c_acpi_find_adapter() to

[Intel-gfx] [PATCH 0/4] drm/i915/vlv_dsi: Add quirks for x86 android tablets (v3)

Hi All, Some vlv/chv tablets ship with Android as factory OS. The factory OS BSP style kernel on these tablets does not use the normal x86 hw autodetection instead it hardcodes a whole bunch of things including using panel drivers instead of relying on VBT MIPI sequences to turn the

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/perf: Remove gtt_offset from stream->oa_buffer.head/.tail (rev2)

== Series Details == Series: drm/i915/perf: Remove gtt_offset from stream->oa_buffer.head/.tail (rev2) URL : https://patchwork.freedesktop.org/series/123949/ State : success == Summary == CI Bug Log - changes from CI_DRM_13658 -> Patchwork_123949v2

Re: [Intel-gfx] [PATCH v2] drm/i915/irq: Clear GFX_MSTR_IRQ as part of IRQ reset

On Wed, Sep 20, 2023 at 05:13:36PM -0300, Gustavo Sousa wrote: > Quoting Ville Syrjälä (2023-09-20 17:00:07-03:00) > >On Wed, Sep 20, 2023 at 04:53:52PM -0300, Gustavo Sousa wrote: > >> Starting with Xe_LP+, GFX_MSTR_IRQ contains status bits that have W1C > >> behavior. If we do not properly reset

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_pm_freq_api: Set min/max to expected values

On Wed, Sep 20, 2023 at 09:18:07AM -0700, Belgaumkar, Vinay wrote: > > On 9/20/2023 7:07 AM, Rodrigo Vivi wrote: > > On Mon, Sep 18, 2023 at 12:02:59PM -0700, Vinay Belgaumkar wrote: > > > A prior(rps) test leaves the system in a bad state causing failures > > > in the basic test. > > Why? > > >

Re: [Intel-gfx] [PATCH v2 12/12] drm/i915: Implement transcoder LRR for TGL+

On Wed, Sep 20, 2023 at 12:40 PM Ville Syrjälä wrote: > > On Wed, Sep 20, 2023 at 11:47:05AM -0700, Manasi Navare wrote: > > Hi Ville, > > > > Quick question here on the use case and the trigger for the LRR case > > which is within VRR range. > > Could this perhaps be used if we had a virtual

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for i915: Limit the length of an sg list to the requested length

On 20.09.2023 14:00, Patchwork wrote: *Patch Details* *Series:* i915: Limit the length of an sg list to the requested length *URL:* https://patchwork.freedesktop.org/series/123940/ *State:*failure *Details:*

[Intel-gfx] ✓ Fi.CI.BAT: success for Update GGTT with MI_UPDATE_GTT on MTL (rev8)

== Series Details == Series: Update GGTT with MI_UPDATE_GTT on MTL (rev8) URL : https://patchwork.freedesktop.org/series/123329/ State : success == Summary == CI Bug Log - changes from CI_DRM_13658 -> Patchwork_123329v8 Summary ---

Re: [Intel-gfx] [PATCH v2] drm/i915: refactor deprecated strncpy

On Tue, Sep 19, 2023 at 04:45:31AM +, Justin Stitt wrote: > `strncpy` is deprecated for use on NUL-terminated destination strings [1]. > > We should prefer more robust and less ambiguous string interfaces. > > A suitable replacement is `strscpy` [2] due to the fact that it > guarantees

[Intel-gfx] [PATCH v12 2/4] drm/i915: Reserve some kernel space per vm

Reserve two pages in each vm for kernel space to use for things such as workarounds. Signed-off-by: Jonathan Cavitt Suggested-by: Chris Wilson --- drivers/gpu/drm/i915/gt/gen8_ppgtt.c | 7 +++ drivers/gpu/drm/i915/gt/intel_gtt.h | 1 + 2 files changed, 8 insertions(+) diff --git

[Intel-gfx] [PATCH v12 1/4] drm/i915: Enable NULL PTE support for vm scratch

Enable NULL PTE support for vm scratch pages. The use of NULL PTEs in teh vm scratch pages requires us to change how the i915 gem_contexts live selftest perform vm_isolation: instead of checking the scratch pages are isolated and don't affect each other, we check that all changes to the scratch

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

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 + 2 files changed, 8

[Intel-gfx] [PATCH v12 3/4] drm/i915: Add WABB blit for Wa_16018031267 / Wa_16018063123

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 | 4 +

[Intel-gfx] [PATCH v2 1/2] drm/edid: Fixup h/vsync_end instead of h/vtotal

From: Ville Syrjälä There are some weird EDIDs floating around that have the sync pulse extending beyond the end of the blanking period. On the currently problemtic machine (HP Omni 120) EDID reports the following mode: "1600x900": 60 108000 1600 1780 1860 1800 900 910 913 1000 0x40 0x5 which

[Intel-gfx] [PATCH v2 2/2] drm/i915/bios: Fixup h/vsync_end instead of h/vtotal

From: Ville Syrjälä We have the same h/vsync_end vs. h/vtotal quirk in the VBT parser that was also present in EDID parser. Adjust the VBT parser the same way as was done for hte EDID parser to fixup h/vsync_end instead of h/vtotal. While I'm not currently aware of any machines that need this

[Intel-gfx] [PATCH v12 0/4] Apply Wa_16018031267 / Wa_16018063123

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] drm/i915/display/dp: Change DSC vs lower bpp priority

Previously driver would lower bpp before trying DSC. Monitors capable of acheiving highest mode with either DSC or lower bpp would have bpp dropped instead of attempting DSC at higher bpp. Changed the order in which driver attempts DSC and lower bpp. Attempt DSC before trying lower bpp without

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/vlv_dsi: Add quirks for x86 android tablets (v3)

== Series Details == Series: drm/i915/vlv_dsi: Add quirks for x86 android tablets (v3) URL : https://patchwork.freedesktop.org/series/124004/ State : success == Summary == CI Bug Log - changes from CI_DRM_13659 -> Patchwork_124004v1

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/irq: Clear GFX_MSTR_IRQ as part of IRQ reset (rev2)

== Series Details == Series: drm/i915/irq: Clear GFX_MSTR_IRQ as part of IRQ reset (rev2) URL : https://patchwork.freedesktop.org/series/123928/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13659 -> Patchwork_123928v2

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/vlv_dsi: Add quirks for x86 android tablets (v3)

== Series Details == Series: drm/i915/vlv_dsi: Add quirks for x86 android tablets (v3) URL : https://patchwork.freedesktop.org/series/124004/ 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

== Series Details == Series: Apply Wa_16018031267 / Wa_16018063123 URL : https://patchwork.freedesktop.org/series/124011/ State : warning == Summary == Error: dim checkpatch failed 9cb3de2b89cf drm/i915: Enable NULL PTE support for vm scratch -:8: WARNING:TYPO_SPELLING: 'teh' may be

  1   2   >