[Intel-gfx] ✗ Fi.CI.BAT: failure for Remove the VFIO_IOMMU_NOTIFY_DMA_UNMAP notifier (rev2)

2022-06-07 Thread Patchwork
== Series Details == Series: Remove the VFIO_IOMMU_NOTIFY_DMA_UNMAP notifier (rev2) URL : https://patchwork.freedesktop.org/series/104793/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11732 -> Patchwork_104793v2 Summary

Re: [Intel-gfx] [PATCH v2 2/2] vfio: Replace the iommu notifier with a device list

2022-06-07 Thread Christoph Hellwig
Looks good: Reviewed-by: Christoph Hellwig

Re: [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: More PVC+DG2 workarounds

2022-06-07 Thread Vudum, Lakshminarayana
No Idea about that Matt. I have never did that. Lakshmi. -Original Message- From: Roper, Matthew D Sent: Tuesday, June 7, 2022 9:43 PM To: intel-gfx@lists.freedesktop.org Cc: Vudum, Lakshminarayana Subject: Re: ✗ Fi.CI.SPARSE: warning for drm/i915: More PVC+DG2 workarounds On Wed,

Re: [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: More PVC+DG2 workarounds

2022-06-07 Thread Matt Roper
On Wed, Jun 08, 2022 at 02:38:12AM +, Patchwork wrote: > == Series Details == > > Series: drm/i915: More PVC+DG2 workarounds > URL : https://patchwork.freedesktop.org/series/104866/ > State : warning > > == Summary == > > Error: dim sparse failed > Sparse version: v0.6.2 > Fast mode used,

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/xehp: Correct steering initialization

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/xehp: Correct steering initialization URL : https://patchwork.freedesktop.org/series/104842/ State : success == Summary == CI Bug Log - changes from CI_DRM_11732 -> Patchwork_104842v1 Summary ---

Re: [Intel-gfx] [PATCH v2 2/2] vfio: Replace the iommu notifier with a device list

2022-06-07 Thread Tian, Kevin
> From: Jason Gunthorpe > Sent: Wednesday, June 8, 2022 7:02 AM > > Instead of bouncing the function call to the driver op through a blocking > notifier just have the iommu layer call it directly. > > Register each device that is being attached to the iommu with the lower > driver which then

Re: [Intel-gfx] [PATCH v2 1/2] vfio: Replace the DMA unmapping notifier with a callback

2022-06-07 Thread Tian, Kevin
> From: Jason Gunthorpe > Sent: Wednesday, June 8, 2022 7:02 AM > > Instead of having drivers register the notifier with explicit code just > have them provide a dma_unmap callback op in their driver ops and rely on > the core code to wire it up. > > Suggested-by: Christoph Hellwig >

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: More PVC+DG2 workarounds

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915: More PVC+DG2 workarounds URL : https://patchwork.freedesktop.org/series/104866/ 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] ✓ Fi.CI.IGT: success for drm/i915/dg2: Correct DSS check for Wa_1308578152

2022-06-07 Thread Matt Roper
On Tue, Jun 07, 2022 at 10:59:40PM +, Patchwork wrote: > == Series Details == > > Series: drm/i915/dg2: Correct DSS check for Wa_1308578152 > URL : https://patchwork.freedesktop.org/series/104825/ > State : success > > == Summary == > > CI Bug Log - changes from CI_DRM_11731_full ->

[Intel-gfx] [PATCH] drm/i915: More PVC+DG2 workarounds

2022-06-07 Thread Matt Roper
A new PVC+DG2 workaround has appeared recently: - Wa_16015675438 And a couple existing DG2 workarounds have been extended to PVC: - Wa_14015795083 - Wa_18018781329 Note that Wa_16015675438 asks us to program a register that is in the 0x2xxx range typically associated with the RCS engine, even

[Intel-gfx] linux-next: manual merge of the drm-intel tree with Linus' tree

2022-06-07 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-intel tree got a conflict in: drivers/gpu/drm/i915/gt/intel_rps.c between commit: 56758cc45955 ("drm/i915/rps: Centralize computation of freq caps") from Linus' tree and commit: ee421bb4cb95 ("drm/i915/pcode: Extend pcode functions for

[Intel-gfx] linux-next: manual merge of the drm-intel tree with Linus' tree

2022-06-07 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-intel tree got a conflict in: drivers/gpu/drm/i915/display/intel_dmc_regs.h between commit: 2518f226c60d ("Merge tag 'drm-next-2022-05-25' of git://anongit.freedesktop.org/drm/drm") from Linus' tree and commit: 21c47196aec3 ("drm/i915/dmc:

[Intel-gfx] ✗ Fi.CI.BUILD: failure for drm/i915/guc: Use drm_err instead of pr_err

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/guc: Use drm_err instead of pr_err URL : https://patchwork.freedesktop.org/series/104859/ State : failure == Summary == Error: make failed CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh DESCEND objtool CHK

Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: Use non-blocking H2G for waitboost

2022-06-07 Thread John Harrison
On 6/7/2022 15:29, Dixit, Ashutosh wrote: On Sat, 14 May 2022 23:05:06 -0700, Vinay Belgaumkar wrote: SLPC min/max frequency updates require H2G calls. We are seeing timeouts when GuC channel is backed up and it is unable to respond in a timely fashion causing warnings and affecting CI. This

Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: Use non-blocking H2G for waitboost

2022-06-07 Thread John Harrison
On 6/7/2022 16:02, John Harrison wrote: On 5/16/2022 00:59, Jani Nikula wrote: On Sat, 14 May 2022, Vinay Belgaumkar wrote: SLPC min/max frequency updates require H2G calls. We are seeing timeouts when GuC channel is backed up and it is unable to respond in a timely fashion causing warnings

Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: Use non-blocking H2G for waitboost

2022-06-07 Thread John Harrison
On 5/16/2022 00:59, Jani Nikula wrote: On Sat, 14 May 2022, Vinay Belgaumkar wrote: SLPC min/max frequency updates require H2G calls. We are seeing timeouts when GuC channel is backed up and it is unable to respond in a timely fashion causing warnings and affecting CI. This is seen when

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/dg2: Correct DSS check for Wa_1308578152

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/dg2: Correct DSS check for Wa_1308578152 URL : https://patchwork.freedesktop.org/series/104825/ State : success == Summary == CI Bug Log - changes from CI_DRM_11731_full -> Patchwork_104825v1_full

Re: [Intel-gfx] [PATCH 32/33] drm/i915/guc: Implement GuC priority management

2022-06-07 Thread John Harrison
On 6/7/2022 06:58, Tvrtko Ursulin wrote: A blast from the past.. On 27/07/2021 01:23, Matthew Brost wrote: Implement a simple static mapping algorithm of the i915 priority levels (int, -1k to 1k exposed to user) to the 4 GuC levels. Mapping is as follows: i915 level < 0  -> GuC low

Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: Use non-blocking H2G for waitboost

2022-06-07 Thread Dixit, Ashutosh
On Sat, 14 May 2022 23:05:06 -0700, Vinay Belgaumkar wrote: > > SLPC min/max frequency updates require H2G calls. We are seeing > timeouts when GuC channel is backed up and it is unable to respond > in a timely fashion causing warnings and affecting CI. > > This is seen when waitboosting happens

Re: [Intel-gfx] [PATCH] drm/i915/guc: Use drm_err instead of pr_err

2022-06-07 Thread Dixit, Ashutosh
On Tue, 07 Jun 2022 15:23:17 -0700, John Harrison wrote: > > On 6/7/2022 15:07, Dixit, Ashutosh wrote: > > On Tue, 07 Jun 2022 14:51:03 -0700, john.c.harri...@intel.com wrote: > >> From: John Harrison > >> > >> Don't use pr_err in places where we have access to a struct_drm. > > Seem to be many

Re: [Intel-gfx] [PATCH] drm/i915/guc: Use drm_err instead of pr_err

2022-06-07 Thread John Harrison
On 6/7/2022 15:07, Dixit, Ashutosh wrote: On Tue, 07 Jun 2022 14:51:03 -0700, john.c.harri...@intel.com wrote: From: John Harrison Don't use pr_err in places where we have access to a struct_drm. Seem to be many more pr_err's in selftests. Is there a reason why drm_err's cannot be used in

Re: [Intel-gfx] [PATCH] drm/i915/guc: Use drm_err instead of pr_err

2022-06-07 Thread Dixit, Ashutosh
On Tue, 07 Jun 2022 14:51:03 -0700, john.c.harri...@intel.com wrote: > > From: John Harrison > > Don't use pr_err in places where we have access to a struct_drm. Seem to be many more pr_err's in selftests. Is there a reason why drm_err's cannot be used in selftests (especially those using an

[Intel-gfx] [PATCH] drm/i915/guc: Use drm_err instead of pr_err

2022-06-07 Thread John . C . Harrison
From: John Harrison Don't use pr_err in places where we have access to a struct_drm. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 10 ++--- drivers/gpu/drm/i915/gt/uc/selftest_guc.c | 38 +-- .../drm/i915/gt/uc/selftest_guc_multi_lrc.c

Re: [Intel-gfx] [RFC v3 1/3] drm/doc/rfc: VM_BIND feature design document

2022-06-07 Thread Niranjana Vishwanathapura
On Tue, Jun 07, 2022 at 11:18:11AM -0700, Niranjana Vishwanathapura wrote: On Tue, Jun 07, 2022 at 12:12:03PM -0500, Jason Ekstrand wrote: On Fri, Jun 3, 2022 at 6:52 PM Niranjana Vishwanathapura wrote: On Fri, Jun 03, 2022 at 10:20:25AM +0300, Lionel Landwerlin wrote: > On

Re: [Intel-gfx] [RFC v3 3/3] drm/doc/rfc: VM_BIND uapi definition

2022-06-07 Thread Niranjana Vishwanathapura
On Tue, Jun 07, 2022 at 11:42:08AM +0100, Tvrtko Ursulin wrote: On 03/06/2022 07:53, Niranjana Vishwanathapura wrote: On Wed, Jun 01, 2022 at 10:08:35PM -0700, Niranjana Vishwanathapura wrote: On Wed, Jun 01, 2022 at 11:27:17AM +0200, Daniel Vetter wrote: On Wed, 1 Jun 2022 at 11:03, Dave

[Intel-gfx] [PATCH v6 i-g-t 2/3] tests/i915/query: Add descriptions to existing tests

2022-06-07 Thread John . C . Harrison
From: John Harrison None of the query tests had a description. So make some up. Signed-off-by: John Harrison Reviewed-by: Kamil Konieczny --- tests/i915/i915_query.c | 12 1 file changed, 12 insertions(+) diff --git a/tests/i915/i915_query.c b/tests/i915/i915_query.c index

[Intel-gfx] [PATCH v6 i-g-t 3/3] tests/i915/query: Query, parse and validate the hwconfig table

2022-06-07 Thread John . C . Harrison
From: Rodrigo Vivi Newer platforms have an embedded table giving details about that platform's hardware configuration. This table can be retrieved from the KMD via the existing query API. So add a test for it as both an example of how to fetch the table and to validate the contents as much as is

[Intel-gfx] [PATCH v6 i-g-t 0/3] Update DRM UAPI and add test for new hw info query

2022-06-07 Thread John . C . Harrison
From: John Harrison Various UMDs require hardware configuration information about the current platform. A new interface has been added to the KMD to return this information. So, add a test for the new interface. Also, update to the latest DRM UAPI header file that contains the new query enums.

[Intel-gfx] [PATCH v6 i-g-t 1/3] include/drm-uapi: Update to latest i915_drm.h

2022-06-07 Thread John . C . Harrison
From: John Harrison Update to the latest master version of the DRM UAPI header file from git://anongit.freedesktop.org/git/drm/drm: c4955d9cd2fc Merge tag 'drm-intel-next-fixes-2022-05-24' of git://anongit.freedesktop.org/drm/drm-intel into drm-next Signed-off-by: John Harrison

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/dg2: Add Wa_14015795083 (rev2)

2022-06-07 Thread Matt Roper
On Tue, Jun 07, 2022 at 05:46:25PM +, Patchwork wrote: > == Series Details == > > Series: drm/i915/dg2: Add Wa_14015795083 (rev2) > URL : https://patchwork.freedesktop.org/series/104760/ > State : failure > > == Summary == > > CI Bug Log - changes from CI_DRM_11730_full ->

Re: [Intel-gfx] [RFC v3 3/3] drm/doc/rfc: VM_BIND uapi definition

2022-06-07 Thread Niranjana Vishwanathapura
On Tue, Jun 07, 2022 at 11:27:14AM +0100, Tvrtko Ursulin wrote: On 17/05/2022 19:32, Niranjana Vishwanathapura wrote: VM_BIND and related uapi definitions v2: Ensure proper kernel-doc formatting with cross references. Also add new uapi and documentation as per review comments from

Re: [Intel-gfx] [PATCH v1 06/13] drm/probe-helper: make .get_modes() optional, add default action

2022-06-07 Thread Ville Syrjälä
On Tue, Jun 07, 2022 at 02:14:54PM +0300, Jani Nikula wrote: > On Thu, 02 Jun 2022, Ville Syrjälä wrote: > > On Tue, May 24, 2022 at 01:39:28PM +0300, Jani Nikula wrote: > >> Add default action when .get_modes() not set. This also defines what a > >> .get_modes() hook should do. > >> > >> Cc:

[Intel-gfx] [RESEND RFC 18/18] drm/display/dp_mst: Move all payload info into the atomic state

2022-06-07 Thread Lyude Paul
Now that we've finally gotten rid of the non-atomic MST users leftover in the kernel, we can finally get rid of all of the legacy payload code we have and move as much as possible into the MST atomic state structs. The main purpose of this is to make the MST code a lot less confusing to work on,

Re: [Intel-gfx] [igt-dev] [PATCH v4 i-g-t 1/2] include/drm-uapi: Update to latest i915_drm.h

2022-06-07 Thread Dixit, Ashutosh
On Fri, 03 Jun 2022 08:55:45 -0700, john.c.harri...@intel.com wrote: > > From: John Harrison > > Update to the latest master version of the DRM UAPI header file from > git://anongit.freedesktop.org/git/drm/drm: > c4955d9cd2fc Merge tag 'drm-intel-next-fixes-2022-05-24' of >

[Intel-gfx] [RESEND RFC 13/18] drm/display/dp_mst: Add helpers for serializing SST <-> MST transitions

2022-06-07 Thread Lyude Paul
There's another kind of situation where we could potentially race with nonblocking modesets and MST, especially if we were to only use the locking provided by atomic modesetting: * Display 1 begins as enabled on DP-1 in SST mode * Display 1 switches to MST mode, exposes one sink in MST mode *

[Intel-gfx] [RESEND RFC 09/18] drm/display/dp_mst: Don't open code modeset checks for releasing time slots

2022-06-07 Thread Lyude Paul
I'm not sure why, but at the time I originally wrote the find/release time slot helpers I thought we should avoid keeping modeset tracking out of the MST helpers. In retrospect though there's no actual good reason to do this, and the logic has ended up being identical across all the drivers using

[Intel-gfx] [RESEND RFC 08/18] drm/display/dp_mst: Add nonblocking helpers for DP MST

2022-06-07 Thread Lyude Paul
As Daniel Vetter pointed out, if we only use the atomic modesetting locks with MST it's technically possible for a driver with non-blocking modesets to race when it comes to MST displays - as we make the mistake of not doing our own CRTC commit tracking in the topology_state object. This could

[Intel-gfx] [RESEND RFC 04/18] drm/display/dp_mst: Call them time slots, not VCPI slots

2022-06-07 Thread Lyude Paul
VCPI is only sort of the correct term here, originally the majority of this code simply referred to timeslots vaguely as "slots" - and since I started working on it and adding atomic functionality, the name "VCPI slots" has been used to represent time slots. Now that we actually have consistent

Re: [Intel-gfx] [CI 1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct

2022-06-07 Thread Jani Nikula
On Tue, 07 Jun 2022, Jani Nikula wrote: > Add a dedicated file for the local functions around struct > tasklet_struct. Far from ideal, but better placed in a dedicated file > than i915_gem.h. > > Signed-off-by: Jani Nikula > Acked-by: Tvrtko Ursulin Series pushed to drm-intel-next. BR, Jani.

Re: [Intel-gfx] [PATCH] drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp()

2022-06-07 Thread Jani Nikula
On Tue, 07 Jun 2022, Luca Coelho wrote: > From: Luca Coelho > > The intel_dp_dsc_get_output_bpp() function outputs two lines of > unconditional logs, which was okay when it was called only once. But > now, we also call this function from intel_dp_mode_valid(), which is > in turn called for

[Intel-gfx] [RFC 04/18] drm/display/dp_mst: Call them time slots, not VCPI slots

2022-06-07 Thread Lyude Paul
VCPI is only sort of the correct term here, originally the majority of this code simply referred to timeslots vaguely as "slots" - and since I started working on it and adding atomic functionality, the name "VCPI slots" has been used to represent time slots. Now that we actually have consistent

[Intel-gfx] [PATCH] drm/i915/xehp: Correct steering initialization

2022-06-07 Thread Matt Roper
Another mistake during the conversion to DSS bitmaps: after retrieving the DSS ID intel_sseu_find_first_xehp_dss() we forgot to modulo it down to obtain which ID within the current gslice it is. Fixes: b87d39019651 ("drm/i915/sseu: Disassociate internal subslice mask representation from uapi")

Re: [Intel-gfx] [CI] drm/i915/client: only include what's needed

2022-06-07 Thread Jani Nikula
On Tue, 07 Jun 2022, Jani Nikula wrote: > Only the uapi header is required. > > Signed-off-by: Jani Nikula > Acked-by: Tvrtko Ursulin Pushed to drm-intel-gt-next. BR, Jani. > --- > drivers/gpu/drm/i915/i915_drm_client.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Intel-gfx] [RFC v3 1/3] drm/doc/rfc: VM_BIND feature design document

2022-06-07 Thread Niranjana Vishwanathapura
On Tue, Jun 07, 2022 at 12:12:03PM -0500, Jason Ekstrand wrote: On Fri, Jun 3, 2022 at 6:52 PM Niranjana Vishwanathapura wrote: On Fri, Jun 03, 2022 at 10:20:25AM +0300, Lionel Landwerlin wrote: > On 02/06/2022 23:35, Jason Ekstrand wrote: > > On Thu, Jun 2, 2022 at

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dg2: Correct DSS check for Wa_1308578152

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/dg2: Correct DSS check for Wa_1308578152 URL : https://patchwork.freedesktop.org/series/104825/ State : success == Summary == CI Bug Log - changes from CI_DRM_11731 -> Patchwork_104825v1 Summary

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/dg2: Add Wa_14015795083 (rev2)

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/dg2: Add Wa_14015795083 (rev2) URL : https://patchwork.freedesktop.org/series/104760/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11730_full -> Patchwork_104760v2_full Summary

Re: [Intel-gfx] [RFC v3 1/3] drm/doc/rfc: VM_BIND feature design document

2022-06-07 Thread Jason Ekstrand
On Fri, Jun 3, 2022 at 6:52 PM Niranjana Vishwanathapura < niranjana.vishwanathap...@intel.com> wrote: > On Fri, Jun 03, 2022 at 10:20:25AM +0300, Lionel Landwerlin wrote: > > On 02/06/2022 23:35, Jason Ekstrand wrote: > > > > On Thu, Jun 2, 2022 at 3:11 PM Niranjana Vishwanathapura > >

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [CI,1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct

2022-06-07 Thread Patchwork
== Series Details == Series: series starting with [CI,1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct URL : https://patchwork.freedesktop.org/series/104812/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11730_full -> Patchwork_104812v1_full

Re: [Intel-gfx] [PATCH] drm/i915/dg2: Correct DSS check for Wa_1308578152

2022-06-07 Thread Balasubramani Vivekanandan
On 07.06.2022 08:47, Matt Roper wrote: > When converting our DSS masks to bitmaps, we fumbled the condition used > to check whether any DSS are present in the first gslice. Since > intel_sseu_find_first_xehp_dss() returns a 0-based number, we need a >= > condition rather than >. > > Fixes:

[Intel-gfx] [PATCH] drm/i915/dg2: Correct DSS check for Wa_1308578152

2022-06-07 Thread Matt Roper
When converting our DSS masks to bitmaps, we fumbled the condition used to check whether any DSS are present in the first gslice. Since intel_sseu_find_first_xehp_dss() returns a 0-based number, we need a >= condition rather than >. Fixes: b87d39019651 ("drm/i915/sseu: Disassociate internal

Re: [Intel-gfx] [PATCH v2] drm/i915/dg2: Add Wa_14015795083

2022-06-07 Thread Matt Roper
On Tue, Jun 07, 2022 at 04:15:42PM +0530, Anshuman Gupta wrote: > i915 must disable Render DOP clock gating globally. > > v2: > - Addressed cosmetic review comments. > > Bspec: 52621 > Cc: Matt Roper > Cc: Badal Nilawar > Signed-off-by: Anshuman Gupta Reviewed-by: Matt Roper > --- >

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/client: only include what's needed

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/client: only include what's needed URL : https://patchwork.freedesktop.org/series/104809/ State : success == Summary == CI Bug Log - changes from CI_DRM_11730_full -> Patchwork_104809v1_full Summary

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp()

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp() URL : https://patchwork.freedesktop.org/series/104801/ State : success == Summary == CI Bug Log - changes from CI_DRM_11730_full -> Patchwork_104801v1_full

Re: [Intel-gfx] [PATCH 32/33] drm/i915/guc: Implement GuC priority management

2022-06-07 Thread Tvrtko Ursulin
A blast from the past.. On 27/07/2021 01:23, Matthew Brost wrote: Implement a simple static mapping algorithm of the i915 priority levels (int, -1k to 1k exposed to user) to the 4 GuC levels. Mapping is as follows: i915 level < 0 -> GuC low level (3) i915 level == 0 ->

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dg2: Add Wa_14015795083 (rev2)

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/dg2: Add Wa_14015795083 (rev2) URL : https://patchwork.freedesktop.org/series/104760/ State : success == Summary == CI Bug Log - changes from CI_DRM_11730 -> Patchwork_104760v2 Summary ---

Re: [Intel-gfx] [PATCH 1/2] drm/i915/opregion: add function to check if headless sku

2022-06-07 Thread Souza, Jose
On Tue, 2022-06-07 at 13:05 +, Hogander, Jouni wrote: > On Tue, 2022-06-07 at 10:36 +0300, Jani Nikula wrote: > > On Mon, 06 Jun 2022, "Souza, Jose" wrote: > > > On Mon, 2022-06-06 at 11:16 +0300, Jani Nikula wrote: > > > > On Mon, 06 Jun 2022, "Hogander, Jouni" > > > > wrote: > > > > > On

Re: [Intel-gfx] [PATCH 1/2] drm/i915/opregion: add function to check if headless sku

2022-06-07 Thread Hogander, Jouni
On Tue, 2022-06-07 at 10:36 +0300, Jani Nikula wrote: > On Mon, 06 Jun 2022, "Souza, Jose" wrote: > > On Mon, 2022-06-06 at 11:16 +0300, Jani Nikula wrote: > > > On Mon, 06 Jun 2022, "Hogander, Jouni" > > > wrote: > > > > On Fri, 2022-06-03 at 16:32 +, Souza, Jose wrote: > > > > > On Fri,

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct

2022-06-07 Thread Patchwork
== Series Details == Series: series starting with [CI,1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct URL : https://patchwork.freedesktop.org/series/104812/ State : success == Summary == CI Bug Log - changes from CI_DRM_11730 -> Patchwork_104812v1

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct

2022-06-07 Thread Patchwork
== Series Details == Series: series starting with [CI,1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct URL : https://patchwork.freedesktop.org/series/104812/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct

2022-06-07 Thread Patchwork
== Series Details == Series: series starting with [CI,1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct URL : https://patchwork.freedesktop.org/series/104812/ State : warning == Summary == Error: dim checkpatch failed 87f22c2bf223 drm/i915/tasklet: separate local hacks

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/client: only include what's needed

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/client: only include what's needed URL : https://patchwork.freedesktop.org/series/104809/ State : success == Summary == CI Bug Log - changes from CI_DRM_11730 -> Patchwork_104809v1 Summary ---

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/client: only include what's needed

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915/client: only include what's needed URL : https://patchwork.freedesktop.org/series/104809/ 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 drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp()

2022-06-07 Thread Patchwork
== Series Details == Series: drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp() URL : https://patchwork.freedesktop.org/series/104801/ State : success == Summary == CI Bug Log - changes from CI_DRM_11730 -> Patchwork_104801v1

Re: [Intel-gfx] [PATCH v1 11/13] drm/edid: add HF-EEODB support to EDID read and allocation

2022-06-07 Thread Jani Nikula
On Fri, 03 Jun 2022, Ville Syrjälä wrote: > On Tue, May 24, 2022 at 01:39:33PM +0300, Jani Nikula wrote: >> HDMI 2.1 section 10.3.6 defines an HDMI Forum EDID Extension Override >> Data Block, which may contain a different extension count than the base >> block claims. Add support for reading

Re: [Intel-gfx] [PATCH v1 06/13] drm/probe-helper: make .get_modes() optional, add default action

2022-06-07 Thread Jani Nikula
On Thu, 02 Jun 2022, Ville Syrjälä wrote: > On Tue, May 24, 2022 at 01:39:28PM +0300, Jani Nikula wrote: >> Add default action when .get_modes() not set. This also defines what a >> .get_modes() hook should do. >> >> Cc: David Airlie >> Cc: Daniel Vetter >> Signed-off-by: Jani Nikula >> ---

Re: [Intel-gfx] [PATCH] drm/i915/dgfx: Disable d3cold Correctly

2022-06-07 Thread Gupta, Anshuman
> -Original Message- > From: Luca Coelho > Sent: Monday, June 6, 2022 7:53 PM > To: Gupta, Anshuman ; intel- > g...@lists.freedesktop.org > Cc: Vivi, Rodrigo > Subject: Re: [Intel-gfx] [PATCH] drm/i915/dgfx: Disable d3cold Correctly > > On Mon, 2022-06-06 at 17:56 +0530, Anshuman

[Intel-gfx] [PATCH v2] drm/i915/dg2: Add Wa_14015795083

2022-06-07 Thread Anshuman Gupta
i915 must disable Render DOP clock gating globally. v2: - Addressed cosmetic review comments. Bspec: 52621 Cc: Matt Roper Cc: Badal Nilawar Signed-off-by: Anshuman Gupta --- drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 + drivers/gpu/drm/i915/gt/intel_workarounds.c | 3 +++ 2 files

Re: [Intel-gfx] [RFC v3 3/3] drm/doc/rfc: VM_BIND uapi definition

2022-06-07 Thread Tvrtko Ursulin
On 03/06/2022 07:53, Niranjana Vishwanathapura wrote: On Wed, Jun 01, 2022 at 10:08:35PM -0700, Niranjana Vishwanathapura wrote: On Wed, Jun 01, 2022 at 11:27:17AM +0200, Daniel Vetter wrote: On Wed, 1 Jun 2022 at 11:03, Dave Airlie wrote: On Tue, 24 May 2022 at 05:20, Niranjana

Re: [Intel-gfx] [RFC v3 3/3] drm/doc/rfc: VM_BIND uapi definition

2022-06-07 Thread Tvrtko Ursulin
On 17/05/2022 19:32, Niranjana Vishwanathapura wrote: VM_BIND and related uapi definitions v2: Ensure proper kernel-doc formatting with cross references. Also add new uapi and documentation as per review comments from Daniel. Signed-off-by: Niranjana Vishwanathapura ---

[Intel-gfx] [CI 3/3] drm/i915/utils: throw out unused stuff

2022-06-07 Thread Jani Nikula
Remove some of the unused helpers from i915_utils.h. Signed-off-by: Jani Nikula Acked-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_utils.h | 40 --- 1 file changed, 40 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_utils.h

[Intel-gfx] [CI 2/3] drm/i915/drv: drop intel_bios.h include

2022-06-07 Thread Jani Nikula
No longer needed after panel data was moved. Signed-off-by: Jani Nikula Acked-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_drv.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index c3854b8a014f..eba94fa76b18 100644 ---

[Intel-gfx] [CI 1/3] drm/i915/tasklet: separate local hacks around struct tasklet_struct

2022-06-07 Thread Jani Nikula
Add a dedicated file for the local functions around struct tasklet_struct. Far from ideal, but better placed in a dedicated file than i915_gem.h. Signed-off-by: Jani Nikula Acked-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/TODO.txt | 2 +- drivers/gpu/drm/i915/i915_gem.h | 33

[Intel-gfx] [CI] drm/i915/client: only include what's needed

2022-06-07 Thread Jani Nikula
Only the uapi header is required. Signed-off-by: Jani Nikula Acked-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_drm_client.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_drm_client.h b/drivers/gpu/drm/i915/i915_drm_client.h index

Re: [Intel-gfx] [PATCH] drm/i915/dg2: Catch and log more unexpected values in DG1_MSTR_TILE_INTR

2022-06-07 Thread Tvrtko Ursulin
On 06/06/2022 16:21, Matt Roper wrote: On Mon, Jun 06, 2022 at 12:55:20PM +0100, Tvrtko Ursulin wrote: On 27/05/2022 19:42, Matt Roper wrote: On Thu, May 26, 2022 at 11:18:17AM +0100, Tvrtko Ursulin wrote: On 25/05/2022 19:05, Matt Roper wrote: On Wed, May 25, 2022 at 05:03:13PM +0100,

Re: [Intel-gfx] [PATCH] drm/i915: Improve user experience and driver robustness under SIGINT or similar

2022-06-07 Thread Tvrtko Ursulin
On 27/05/2022 13:07, Andrzej Hajda wrote: On 27.05.2022 09:24, Tvrtko Ursulin wrote: From: Tvrtko Ursulin We have long standing customer complaints that pressing Ctrl-C (or to the effect of) causes engine resets with otherwise well behaving programs. Not only is logging engine resets

Re: [Intel-gfx] [PATCH 0/7] drm/i915: i915_drv.h & i915_gem.h header refactoring

2022-06-07 Thread Jani Nikula
On Mon, 06 Jun 2022, Tvrtko Ursulin wrote: > On 03/06/2022 14:05, Jani Nikula wrote: >> Turn i915_gem.h into a useful header that contains stuff the name >> implies, and clean up i915_drv.h a bit. >> >> Jani Nikula (7): >>drm/i915/tasklet: separate local hacks around struct tasklet_struct >>

Re: [Intel-gfx] [PATCH] drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp()

2022-06-07 Thread Luca Coelho
On Tue, 2022-06-07 at 11:05 +0300, Jani Nikula wrote: > On Tue, 07 Jun 2022, Luca Coelho wrote: > > From: Luca Coelho > > > > The intel_dp_dsc_get_output_bpp() function outputs two lines of > > unconditional logs, which was okay when it was called only once. But > > now, we also call this

Re: [Intel-gfx] [PATCH] drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp()

2022-06-07 Thread Jani Nikula
On Tue, 07 Jun 2022, Luca Coelho wrote: > From: Luca Coelho > > The intel_dp_dsc_get_output_bpp() function outputs two lines of > unconditional logs, which was okay when it was called only once. But > now, we also call this function from intel_dp_mode_valid(), which is > in turn called for

[Intel-gfx] [PATCH] drm/i915: remove noisy logs in intel_dp_dsc_get_output_bpp()

2022-06-07 Thread Luca Coelho
From: Luca Coelho The intel_dp_dsc_get_output_bpp() function outputs two lines of unconditional logs, which was okay when it was called only once. But now, we also call this function from intel_dp_mode_valid(), which is in turn called for every mode we need to validate. This causes a lot of

Re: [Intel-gfx] [PATCH 1/2] drm/i915/opregion: add function to check if headless sku

2022-06-07 Thread Jani Nikula
On Mon, 06 Jun 2022, "Souza, Jose" wrote: > On Mon, 2022-06-06 at 11:16 +0300, Jani Nikula wrote: >> On Mon, 06 Jun 2022, "Hogander, Jouni" wrote: >> > On Fri, 2022-06-03 at 16:32 +, Souza, Jose wrote: >> > > On Fri, 2022-06-03 at 13:14 +, Hogander, Jouni wrote: >> > > > On Fri,