[Intel-gfx] ✗ Fi.CI.IGT: failure for Second round of i915_reg.h splitting (rev2)

2022-01-25 Thread Patchwork
== Series Details == Series: Second round of i915_reg.h splitting (rev2) URL : https://patchwork.freedesktop.org/series/99079/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11129_full -> Patchwork_22094_full Summary ---

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/5] drm/i915: Skip dsc readout if the transcoder is disabled (rev2)

2022-01-25 Thread Patchwork
== Series Details == Series: series starting with [1/5] drm/i915: Skip dsc readout if the transcoder is disabled (rev2) URL : https://patchwork.freedesktop.org/series/99276/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11130 -> Patchwork_22096 ===

[Intel-gfx] [PATCH 0/3] Minor Fixes and Refactoring for HDMI PCON stuff

2022-01-25 Thread Ankit Nautiyal
Misc fixes and refactoring in HDMI2.1 PCON helper functions. Ankit Nautiyal (3): drm/i915_hdmi: Fix the definition of intel_hdmi_dsc_get_bpp drm/drm_edid: Add helper to get max FRL rate for an HDMI sink drm/i915/display: Simplify helpers for getting DSC slices and bpp drivers/gpu/drm/drm_e

[Intel-gfx] [PATCH 1/3] drm/i915_hdmi: Fix the definition of intel_hdmi_dsc_get_bpp

2022-01-25 Thread Ankit Nautiyal
Fix the data-type of the argument output_format to enum, for the function intel_hdmi_dsc_get_bpp. Fixes: 6e6cb758e035 ("drm/i915: Add helper functions for calculating DSC parameters for HDMI2.1") Cc: Ankit Nautiyal Cc: Uma Shankar Cc: Jani Nikula Cc: "Ville Syrj_l_" Cc: "Jos_ Roberto de Souza

[Intel-gfx] [PATCH 3/3] drm/i915/display: Simplify helpers for getting DSC slices and bpp

2022-01-25 Thread Ankit Nautiyal
Genralize the helper for getting DSC slice count and compressed bpp for HDMI sink supporting DSC. This patch: -Removes the assumption on the bpc and sends it as an argument for calculating compressed bpc. -Sends the resolution, and output format as parameters for which the DSC paremeters are to be

[Intel-gfx] [PATCH 2/3] drm/drm_edid: Add helper to get max FRL rate for an HDMI sink

2022-01-25 Thread Ankit Nautiyal
Move the common function for getting the max FRL rate for an HDMI sink, from intel_dp.c to drm/drm_edid. Signed-off-by: Ankit Nautiyal --- drivers/gpu/drm/drm_edid.c | 38 + drivers/gpu/drm/i915/display/intel_dp.c | 19 - include/drm/drm_edid.h

Re: [Intel-gfx] [PATCH 17/54] gpu: drm: replace cpumask_weight with cpumask_empty where appropriate

2022-01-25 Thread Tvrtko Ursulin
On 23/01/2022 18:38, Yury Norov wrote: i915_pmu_cpu_online() calls cpumask_weight() to check if any bit of a given cpumask is set. We can do it more efficiently with cpumask_empty() because cpumask_empty() stops traversing the cpumask as soon as it finds first set bit, while cpumask_weight() co

[Intel-gfx] [PATCH][next] drm/i915/guc: fix spelling mistake "notificaion" -> "notification"

2022-01-25 Thread Colin Ian King
There is a spelling mistake in a drm_err error message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/

Re: [Intel-gfx] [PATCH 1/3] drm/i915_hdmi: Fix the definition of intel_hdmi_dsc_get_bpp

2022-01-25 Thread Jani Nikula
On Tue, 25 Jan 2022, Ankit Nautiyal wrote: > Fix the data-type of the argument output_format to enum, for the > function intel_hdmi_dsc_get_bpp. The subject prefix should be "drm/i915/hdmi:". > > Fixes: 6e6cb758e035 ("drm/i915: Add helper functions for calculating DSC > parameters for HDMI2.1")

Re: [Intel-gfx] [PATCH 2/3] drm/drm_edid: Add helper to get max FRL rate for an HDMI sink

2022-01-25 Thread Jani Nikula
On Tue, 25 Jan 2022, Ankit Nautiyal wrote: > Move the common function for getting the max FRL rate for an HDMI sink, > from intel_dp.c to drm/drm_edid. The subject prefix should be "drm/edid:" But I'm not sure these functions belong in drm_edid.c though. If you see a function prefixed drm_hdmi_,

Re: [Intel-gfx] [PATCH 6/7] drm: Document fdinfo format specification

2022-01-25 Thread Tvrtko Ursulin
On 21/01/2022 11:50, Tvrtko Ursulin wrote: On 20/01/2022 16:44, Rob Clark wrote: [snip] If there is a tool somewhere that displays this info, that would be useful for testing my implementation. I have a patch to Intel specific intel_gpu_top (see https://patchwork.freedesktop.org/patch/46

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for Minor Fixes and Refactoring for HDMI PCON stuff

2022-01-25 Thread Patchwork
== Series Details == Series: Minor Fixes and Refactoring for HDMI PCON stuff URL : https://patchwork.freedesktop.org/series/99311/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

Re: [Intel-gfx] [PATCH 6/7] drm: Document fdinfo format specification

2022-01-25 Thread Christian König
Wow, nice. Certainly +1 from my side to have that generalized, documented and uniform among all drivers. Regards, Christian. Am 25.01.22 um 11:24 schrieb Tvrtko Ursulin: On 21/01/2022 11:50, Tvrtko Ursulin wrote: On 20/01/2022 16:44, Rob Clark wrote: [snip] If there is a tool somewhere

[Intel-gfx] ✓ Fi.CI.BAT: success for Minor Fixes and Refactoring for HDMI PCON stuff

2022-01-25 Thread Patchwork
== Series Details == Series: Minor Fixes and Refactoring for HDMI PCON stuff URL : https://patchwork.freedesktop.org/series/99311/ State : success == Summary == CI Bug Log - changes from CI_DRM_11132 -> Patchwork_22097 Summary --- **

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: fix spelling mistake "notificaion" -> "notification"

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/guc: fix spelling mistake "notificaion" -> "notification" URL : https://patchwork.freedesktop.org/series/99313/ State : warning == Summary == $ dim checkpatch origin/drm-tip 93a354db36b2 drm/i915/guc: fix spelling mistake "notificaion" -> "notification" -:

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: fix spelling mistake "notificaion" -> "notification"

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/guc: fix spelling mistake "notificaion" -> "notification" URL : https://patchwork.freedesktop.org/series/99313/ State : success == Summary == CI Bug Log - changes from CI_DRM_11132 -> Patchwork_22098 Su

Re: [Intel-gfx] [PATCH 4/4] drm/i915: Don't allocate extra ddb during async flip for DG2

2022-01-25 Thread Lisovskiy, Stanislav
On Mon, Jan 24, 2022 at 08:55:57PM +0200, Ville Syrjälä wrote: > On Mon, Jan 24, 2022 at 03:52:34PM +0200, Stanislav Lisovskiy wrote: > > In terms of async flip optimization we don't to allocate > > extra ddb space, so lets skip it. > > > > v2: - Extracted min ddb async flip check to separate func

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gvt: Convert list_for_each to entry variant

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/gvt: Convert list_for_each to entry variant URL : https://patchwork.freedesktop.org/series/99314/ State : success == Summary == CI Bug Log - changes from CI_DRM_11132 -> Patchwork_22099 Summary ---

Re: [Intel-gfx] [PATCH] drm/i915/selftests: Use less in contexts steal guc id test

2022-01-25 Thread Matthew Auld
On Sat, 22 Jan 2022 at 00:14, Matthew Brost wrote: > > Using more guc_ids in the stealing guc id test has no real benefit. > Tearing down lots of contexts all at the same time takes a bit of time > due to the H2G / G2H ping-pong with the GuC. On some slower platforms > this can cause timeous when

[Intel-gfx] [PATCH] drm/i915/selftests: fix error handling in guc_steal_ids

2022-01-25 Thread Matthew Auld
Ensure we drop the reference to the last rq, otherwise we are leaking the context etc. Also don't zero the previous valid context. Signed-off-by: Matthew Auld Cc: Matthew Brost --- drivers/gpu/drm/i915/gt/uc/selftest_guc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/d

Re: [Intel-gfx] [PATCH 1/3] drm/i915_hdmi: Fix the definition of intel_hdmi_dsc_get_bpp

2022-01-25 Thread Nautiyal, Ankit K
On 1/25/2022 3:15 PM, Jani Nikula wrote: On Tue, 25 Jan 2022, Ankit Nautiyal wrote: Fix the data-type of the argument output_format to enum, for the function intel_hdmi_dsc_get_bpp. The subject prefix should be "drm/i915/hdmi:". Thanks for pointing it out. Will fix this. Fixes: 6e6cb758e

Re: [Intel-gfx] [PATCH 1/4] drm/i915: Add request cancel low level trace point

2022-01-25 Thread Tvrtko Ursulin
On 24/01/2022 15:01, Matthew Brost wrote: Add request cancel trace point guarded by CONFIG_DRM_I915_LOW_LEVEL_TRACEPOINT. Okay-ish I guess (There is pr_notice with the only real caller, but I suppose you want it for selftests? Oh yes, why is missing from the commit message.), but I'd emit i

Re: [Intel-gfx] [PATCH 2/3] drm/drm_edid: Add helper to get max FRL rate for an HDMI sink

2022-01-25 Thread Nautiyal, Ankit K
On 1/25/2022 3:22 PM, Jani Nikula wrote: On Tue, 25 Jan 2022, Ankit Nautiyal wrote: Move the common function for getting the max FRL rate for an HDMI sink, from intel_dp.c to drm/drm_edid. The subject prefix should be "drm/edid:" But I'm not sure these functions belong in drm_edid.c though.

Re: [Intel-gfx] [PATCH v2] drm/edid: Support type 7 timings

2022-01-25 Thread Jani Nikula
On Tue, 25 Jan 2022, Jani Nikula wrote: > On Sun, 23 Jan 2022, Yaroslav Bolyukin wrote: >> Per VESA DisplayID Standard v2.0: Type VII Timing – Detailed Timing Data >> >> Definitions were already provided as type I, but not used >> >> Signed-off-by: Yaroslav Bolyukin > > Reviewed-by: Jani Nikula

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/selftests: fix error handling in guc_steal_ids

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/selftests: fix error handling in guc_steal_ids URL : https://patchwork.freedesktop.org/series/99318/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11132 -> Patchwork_22100 Summary -

[Intel-gfx] ✗ Fi.CI.IGT: failure for Minor Fixes and Refactoring for HDMI PCON stuff

2022-01-25 Thread Patchwork
== Series Details == Series: Minor Fixes and Refactoring for HDMI PCON stuff URL : https://patchwork.freedesktop.org/series/99311/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11132_full -> Patchwork_22097_full Summary ---

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/guc: fix spelling mistake "notificaion" -> "notification"

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/guc: fix spelling mistake "notificaion" -> "notification" URL : https://patchwork.freedesktop.org/series/99313/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11132_full -> Patchwork_22098_full ==

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/gvt: Convert list_for_each to entry variant

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/gvt: Convert list_for_each to entry variant URL : https://patchwork.freedesktop.org/series/99314/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11132_full -> Patchwork_22099_full Summar

Re: [Intel-gfx] [PATCH 5/5] drm/i915: split out gem/i915_gem_domain.h from i915_drv.h

2022-01-25 Thread Jani Nikula
On Mon, 24 Jan 2022, Maarten Lankhorst wrote: > All looks sane. > > Reviewed-by: Maarten Lankhorst Thanks... but I didn't realize this already conflicts between drm-intel-next and drm-intel-gt-next. I wonder what the best plan would be for making this happen. BR, Jani. -- Jani Nikula, Intel

Re: [Intel-gfx] [PATCH 3/4] drm/i915/execlists: Fix execlists request cancellation corner case

2022-01-25 Thread Tvrtko Ursulin
On 24/01/2022 15:01, Matthew Brost wrote: More than 1 request can be submitted to a single ELSP at a time if multiple requests are ready run to on the same context. When a request is canceled it is marked bad, an idle pulse is triggered to the engine (high priority kernel request), the execlist

Re: [Intel-gfx] [PATCH 1/4] drm/i915: Pass plane to watermark calculation functions

2022-01-25 Thread Ville Syrjälä
On Mon, Jan 24, 2022 at 11:06:50AM +0200, Stanislav Lisovskiy wrote: > Sometimes we might need to change the way we calculate > watermarks, based on which particular plane it is calculated > for. Thus it would be convenient to pass plane struct to those > functions. > > v2: Pass plane instead of p

Re: [Intel-gfx] [PATCH 3/4] drm/i915/execlists: Fix execlists request cancellation corner case

2022-01-25 Thread Matthew Brost
On Tue, Jan 25, 2022 at 03:27:31PM +, Tvrtko Ursulin wrote: > > On 24/01/2022 15:01, Matthew Brost wrote: > > More than 1 request can be submitted to a single ELSP at a time if > > multiple requests are ready run to on the same context. When a request > > is canceled it is marked bad, an idle

Re: [Intel-gfx] [PATCH 1/4] drm/i915: Add request cancel low level trace point

2022-01-25 Thread Matthew Brost
On Tue, Jan 25, 2022 at 12:27:43PM +, Tvrtko Ursulin wrote: > > On 24/01/2022 15:01, Matthew Brost wrote: > > Add request cancel trace point guarded by > > CONFIG_DRM_I915_LOW_LEVEL_TRACEPOINT. > > Okay-ish I guess (There is pr_notice with the only real caller, but I > suppose you want it for

[Intel-gfx] [PATCH 5/8] drm/i915/dp: rewrite DP 2.0 128b/132b link training based on errata

2022-01-25 Thread Jani Nikula
The DP 2.0 errata completely overhauls the 128b/132b link training, with no provisions for backward compatibility with the original DP 2.0 specification. The changes are too intrusive to consider reusing the same code for both 8b/10b and 128b/132b, mainly because the LTTPR channel equalisation is

[Intel-gfx] [PATCH 2/8] drm/dp: add 128b/132b link status helpers from DP 2.0 E11

2022-01-25 Thread Jani Nikula
The DP 2.0 errata redefines link training. There are some new status bits, and some of the old ones need to be checked independently. Add helpers to do this. Cc: Uma Shankar Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/dp/drm_dp.c| 63 ++

[Intel-gfx] [PATCH 6/8] drm/i915/dp: add 128b/132b support to link status checks

2022-01-25 Thread Jani Nikula
Abstract link status check to a function that takes 128b/132b and 8b/10b into account, and use it. Also dump link status on failures. Cc: Uma Shankar Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/display/intel_dp.c | 39 ++- .../drm/i915/display/in

[Intel-gfx] [PATCH 4/8] drm/i915/dp: move intel_dp_prepare_link_train() call

2022-01-25 Thread Jani Nikula
Call it from the higher level function, as it will be shared between two code paths. Cc: Uma Shankar Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/display/intel_dp_link_training.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i9

[Intel-gfx] [PATCH 3/8] drm/dp: add some new DPCD macros from DP 2.0 E11

2022-01-25 Thread Jani Nikula
Add some of the new additions from DP 2.0 E11. Cc: Uma Shankar Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- include/drm/dp/drm_dp_helper.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/drm/dp/drm_dp_helper.h b/include/drm/dp/drm_dp_helper.h index c499d735b992..69487bd8ed56

[Intel-gfx] [PATCH 0/8] drm/dp, drm/i915: 128b/132b updates

2022-01-25 Thread Jani Nikula
128b/132b updates to reflect DP 2.0 errata changes, plus some other DP updates. Jani Nikula (8): drm/dp: add drm_dp_128b132b_read_aux_rd_interval() drm/dp: add 128b/132b link status helpers from DP 2.0 E11 drm/dp: add some new DPCD macros from DP 2.0 E11 drm/i915/dp: move intel_dp_prepare_

[Intel-gfx] [PATCH 7/8] drm/i915/dp: give more time for CDS

2022-01-25 Thread Jani Nikula
Try to avoid the timeout during debugging. Cc: Uma Shankar Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/display/intel_dp_link_training.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c b/drivers/gpu/drm/i91

[Intel-gfx] [PATCH 1/8] drm/dp: add drm_dp_128b132b_read_aux_rd_interval()

2022-01-25 Thread Jani Nikula
The DP 2.0 errata changes DP_128B132B_TRAINING_AUX_RD_INTERVAL (DPCD 0x2216) completely. Add a new function to read that. Follow-up will need to clean up existing functions. v2: fix reversed interpretation of bit 7 meaning (Uma) Cc: Uma Shankar Cc: Ville Syrjälä Signed-off-by: Jani Nikula ---

[Intel-gfx] [PATCH 8/8] drm/i915/mst: update slot information for 128b/132b

2022-01-25 Thread Jani Nikula
128b/132b supports using 64 slots starting from 0, while 8b/10b reserves slot 0 for metadata. Commit d6c6a76f80a1 ("drm: Update MST First Link Slot Information Based on Encoding Format") added support for updating the topology state accordingly, and commit 41724ea273cd ("drm/amd/display: Add DP 2.

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/dp, drm/i915: 128b/132b updates

2022-01-25 Thread Patchwork
== Series Details == Series: drm/dp, drm/i915: 128b/132b updates URL : https://patchwork.freedesktop.org/series/99324/ State : warning == Summary == $ dim checkpatch origin/drm-tip b47aaf278ffa drm/dp: add drm_dp_128b132b_read_aux_rd_interval() e5a3a86f9a2a drm/dp: add 128b/132b link status he

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/dp, drm/i915: 128b/132b updates

2022-01-25 Thread Patchwork
== Series Details == Series: drm/dp, drm/i915: 128b/132b updates URL : https://patchwork.freedesktop.org/series/99324/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/dp, drm/i915: 128b/132b updates

2022-01-25 Thread Patchwork
== Series Details == Series: drm/dp, drm/i915: 128b/132b updates URL : https://patchwork.freedesktop.org/series/99324/ State : success == Summary == CI Bug Log - changes from CI_DRM_11133 -> Patchwork_22101 Summary --- **SUCCESS**

Re: [Intel-gfx] [PATCH 8/8] drm/i915/mst: update slot information for 128b/132b

2022-01-25 Thread Lyude Paul
Acked-by: Lyude Paul BTW - I made a ton of progress last week on getting all of this stuff moved into the atomic state :), mainly just trying to get amd hooked up with this now (and need to rebase): https://gitlab.freedesktop.org/lyudess/linux/-/commits/wip/mst-atomic-only-v1 So we soon won't n

[Intel-gfx] [PATCH] drm/i915: Fix header test for !CONFIG_X86

2022-01-25 Thread Lucas De Marchi
Architectures other than x86 have a stub implementation calling pr_err() and WARN_ON_ONCE(). The appropriate headers need to be included, otherwise the header-test target will fail with: HDRTEST drivers/gpu/drm/i915/i915_mm.h In file included from : ./drivers/gpu/drm/i915/i915_mm.h: In function

Re: [Intel-gfx] [PATCH] drm/i915/guc: Update guc shim control programming on newer platforms

2022-01-25 Thread Belgaumkar, Vinay
On 1/20/2022 2:24 PM, Daniele Ceraolo Spurio wrote: Starting from xehpsdv, bit 0 of of the GuC shim control register has been repurposed, while bit 2 is now reserved, so we need to avoid setting those for their old meaning on newer platforms. Cc: Vinay Belgaumkar Cc: Stuart Summers Signed-of

Re: [Intel-gfx] [PATCH][next] drm/i915/guc: fix spelling mistake "notificaion" -> "notification"

2022-01-25 Thread John Harrison
On 1/25/2022 01:13, Colin Ian King wrote: There is a spelling mistake in a drm_err error message. Fix it. Signed-off-by: Colin Ian King Reviewed-by: John Harrison However, note that this message is going to be deleted anyway. Or at least dropped down to an informational. Partly, there was c

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Fix header test for !CONFIG_X86

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915: Fix header test for !CONFIG_X86 URL : https://patchwork.freedesktop.org/series/99331/ State : warning == Summary == $ dim checkpatch origin/drm-tip e5830b0897b4 drm/i915: Fix header test for !CONFIG_X86 -:17: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapp

[Intel-gfx] [PATCH][next] drm/i915/guc: Use struct_size() helper in kmalloc()

2022-01-25 Thread Gustavo A. R. Silva
Make use of the struct_size() helper instead of an open-coded version, in order to avoid any potential type mistakes or integer overflows that, in the worst scenario, could lead to heap overflows. Also, address the following sparse warnings: drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c:792:23: warnin

Re: [Intel-gfx] [PATCH 17/54] gpu: drm: replace cpumask_weight with cpumask_empty where appropriate

2022-01-25 Thread Yury Norov
On Tue, Jan 25, 2022 at 1:28 AM Tvrtko Ursulin wrote: > > > On 23/01/2022 18:38, Yury Norov wrote: > > i915_pmu_cpu_online() calls cpumask_weight() to check if any bit of a > > given cpumask is set. We can do it more efficiently with cpumask_empty() > > because cpumask_empty() stops traversing the

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/dp, drm/i915: 128b/132b updates

2022-01-25 Thread Patchwork
== Series Details == Series: drm/dp, drm/i915: 128b/132b updates URL : https://patchwork.freedesktop.org/series/99324/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11133_full -> Patchwork_22101_full Summary --- **FA

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Fix header test for !CONFIG_X86

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915: Fix header test for !CONFIG_X86 URL : https://patchwork.freedesktop.org/series/99331/ State : success == Summary == CI Bug Log - changes from CI_DRM_11133 -> Patchwork_22102 Summary --- **SUCCES

[Intel-gfx] [PATCH v5 1/5] drm/i915: add needs_compact_pt flag

2022-01-25 Thread Robert Beckett
From: Ramalingam C Add a new platform flag, needs_compact_pt, to mark the requirement of compact pt layout support for the ppGTT when using 64K GTT pages. With this flag has_64k_pages will only indicate requirement of 64K GTT page sizes or larger for device local memory access. Suggested-by: Ma

[Intel-gfx] [PATCH v5 0/5] discrete card 64K page support

2022-01-25 Thread Robert Beckett
This series continues support for 64K pages for discrete cards. It supersedes the 64K patches from https://patchwork.freedesktop.org/series/95686/#rev4 Changes since that series: - set min alignment for DG2 to 2MB in i915_address_space_init - replace coloring with simpler 2MB VA alignment for lme

[Intel-gfx] [PATCH v5 2/5] drm/i915: enforce min GTT alignment for discrete cards

2022-01-25 Thread Robert Beckett
From: Matthew Auld For local-memory objects we need to align the GTT addresses to 64K, both for the ppgtt and ggtt. We need to support vm->min_alignment > 4K, depending on the vm itself and the type of object we are inserting. With this in mind update the GTT selftests to take this into account.

[Intel-gfx] [PATCH v5 4/5] drm/i915: add gtt misalignment test

2022-01-25 Thread Robert Beckett
add test to check handling of misaligned offsets and sizes v4: * remove spurious blank lines * explicitly cast intel_region_id to intel_memory_type in misaligned_pin Reported-by: kernel test robot Signed-off-by: Robert Beckett --- drivers/gpu/drm/i915/selftests/i915_gem_gtt.c |

[Intel-gfx] [PATCH v5 3/5] drm/i915: support 64K GTT pages for discrete cards

2022-01-25 Thread Robert Beckett
From: Matthew Auld discrete cards optimise 64K GTT pages for local-memory, since everything should be allocated at 64K granularity. We say goodbye to sparse entries, and instead get a compact 256B page-table for 64K pages, which should be more cache friendly. 4K pages for local-memory are no long

[Intel-gfx] [PATCH v5 5/5] drm/i915/uapi: document behaviour for DG2 64K support

2022-01-25 Thread Robert Beckett
From: Matthew Auld On discrete platforms like DG2, we need to support a minimum page size of 64K when dealing with device local-memory. This is quite tricky for various reasons, so try to document the new implicit uapi for this. v3: fix typos and less emphasis v2: Fixed suggestions on formatting

Re: [Intel-gfx] [PATCH] drm/i915: Fix header test for !CONFIG_X86

2022-01-25 Thread Lucas De Marchi
On Tue, Jan 25, 2022 at 10:31:42AM -0800, Lucas De Marchi wrote: Architectures other than x86 have a stub implementation calling pr_err() and WARN_ON_ONCE(). The appropriate headers need to be included, otherwise the header-test target will fail with: HDRTEST drivers/gpu/drm/i915/i915_mm.h In f

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Fix header test for !CONFIG_X86

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915: Fix header test for !CONFIG_X86 URL : https://patchwork.freedesktop.org/series/99331/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11133_full -> Patchwork_22102_full Summary ---

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: Use struct_size() helper in kmalloc()

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/guc: Use struct_size() helper in kmalloc() URL : https://patchwork.freedesktop.org/series/99332/ State : success == Summary == CI Bug Log - changes from CI_DRM_11134 -> Patchwork_22103 Summary ---

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for discrete card 64K page support (rev3)

2022-01-25 Thread Patchwork
== Series Details == Series: discrete card 64K page support (rev3) URL : https://patchwork.freedesktop.org/series/99119/ State : warning == Summary == $ dim checkpatch origin/drm-tip df51c5a24eef drm/i915: add needs_compact_pt flag c3d4f40f9033 drm/i915: enforce min GTT alignment for discrete

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for discrete card 64K page support (rev3)

2022-01-25 Thread Patchwork
== Series Details == Series: discrete card 64K page support (rev3) URL : https://patchwork.freedesktop.org/series/99119/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

[Intel-gfx] [CI] drm/i915/rpl-s: Add stepping info

2022-01-25 Thread Anusha Srivatsa
Add stepping-substepping info in accordance to BSpec changes. Though it looks weird, the revision ID for the newer stepping is indeed backwards and is in accordance to the spec. v2: Rearrange the platforms in logical order (Matt) Bspec: 53655 Cc: Matt Roper Signed-off-by: Anusha Srivatsa Review

[Intel-gfx] ✗ Fi.CI.BAT: failure for discrete card 64K page support (rev3)

2022-01-25 Thread Patchwork
== Series Details == Series: discrete card 64K page support (rev3) URL : https://patchwork.freedesktop.org/series/99119/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11134 -> Patchwork_22104 Summary --- **FAILURE**

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/rpl-s: Add stepping info (rev4)

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/rpl-s: Add stepping info (rev4) URL : https://patchwork.freedesktop.org/series/99162/ State : warning == Summary == $ dim checkpatch origin/drm-tip a2b150061f90 drm/i915/rpl-s: Add stepping info -:39: CHECK:LINE_SPACING: Please don't use multiple blank lin

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/rpl-s: Add stepping info (rev4)

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/rpl-s: Add stepping info (rev4) URL : https://patchwork.freedesktop.org/series/99162/ State : success == Summary == CI Bug Log - changes from CI_DRM_11135 -> Patchwork_22105 Summary --- **SUCCESS

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/guc: Use struct_size() helper in kmalloc()

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/guc: Use struct_size() helper in kmalloc() URL : https://patchwork.freedesktop.org/series/99332/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11134_full -> Patchwork_22103_full Summary

[Intel-gfx] [PATCH 0/3] drm/i915: Fix header test and log spam on !x86

2022-01-25 Thread Lucas De Marchi
Some minor fixes and changes to help porting i915 to arm64, or even anything !x86. Lucas De Marchi (3): drm: Stop spamming log with drm_cache message drm/i915: Do not spam log with missiing arch support drm/i915: Fix header test for !CONFIG_X86 drivers/gpu/drm/drm_cache.c| 3 --- drive

[Intel-gfx] [PATCH 1/3] drm: Stop spamming log with drm_cache message

2022-01-25 Thread Lucas De Marchi
Only x86 and in some cases PPC have support added in drm_cache.c for the clflush class of functions. However warning once is sufficient to taint the log instead of spamming it with "Architecture has no drm_cache.c support" every few millisecond. Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Thomas

[Intel-gfx] [PATCH 2/3] drm/i915: Do not spam log with missiing arch support

2022-01-25 Thread Lucas De Marchi
Following what was done in drm_cache.c, when the stub for remap_io_mapping() was added in commit 67c430bbaae1 ("drm/i915: Skip remap_io_mapping() for non-x86 platforms"), it was added a log message. However the WARN_ON_ONCE() should already be enough - we don't want to spam the log with those messa

[Intel-gfx] [PATCH 3/3] drm/i915: Fix header test for !CONFIG_X86

2022-01-25 Thread Lucas De Marchi
Architectures others than x86 have a stub implementation calling WARN_ON_ONCE(). The appropriate headers need to be included, otherwise the header-test target will fail with: HDRTEST drivers/gpu/drm/i915/i915_mm.h In file included from : ./drivers/gpu/drm/i915/i915_mm.h: In function ‘remap_io_ma

[Intel-gfx] i915 Updates: ADL-P DMC v2.16

2022-01-25 Thread Tolakanahalli Pradeep, Madhumitha
Hi Ben, Josh, Kyle, The following changes since commit eb8ea1b46893c42edbd516f971a93b4d097730ab: Merge branch 'v1.1.7' of https://github.com/irui- wang/linux_fw_vpu_v1.1.7 into main (2022-01-24 06:49:52 -0500) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-fir

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/rpl-s: Add stepping info (rev4)

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915/rpl-s: Add stepping info (rev4) URL : https://patchwork.freedesktop.org/series/99162/ State : success == Summary == CI Bug Log - changes from CI_DRM_11135_full -> Patchwork_22105_full Summary ---

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Fix header test and log spam on !x86

2022-01-25 Thread Patchwork
== Series Details == Series: drm/i915: Fix header test and log spam on !x86 URL : https://patchwork.freedesktop.org/series/99344/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11136 -> Patchwork_22106 Summary --- **F

Re: [Intel-gfx] [PATCH 5/8] drm/i915/dp: rewrite DP 2.0 128b/132b link training based on errata

2022-01-25 Thread Ville Syrjälä
On Tue, Jan 25, 2022 at 07:03:43PM +0200, Jani Nikula wrote: > The DP 2.0 errata completely overhauls the 128b/132b link training, with > no provisions for backward compatibility with the original DP 2.0 > specification. > > The changes are too intrusive to consider reusing the same code for both

Re: [Intel-gfx] [PATCH 5/5] drm/i915: split out gem/i915_gem_domain.h from i915_drv.h

2022-01-25 Thread Maarten Lankhorst
Op 25-01-2022 om 16:25 schreef Jani Nikula: > On Mon, 24 Jan 2022, Maarten Lankhorst > wrote: >> All looks sane. >> >> Reviewed-by: Maarten Lankhorst > Thanks... but I didn't realize this already conflicts between > drm-intel-next and drm-intel-gt-next. I wonder what the best plan would > be for

[Intel-gfx] [PATCH] drm/i915: Lock dpt_obj around set_cache_level, v2.

2022-01-25 Thread Maarten Lankhorst
set_cache_level may unbind the object, which will result in the below lockdep splat: <6> [184.578145] [IGT] kms_addfb_basic: starting subtest addfb25-framebuffer-vs-set-tiling <4> [184.578220] [ cut here ] <4> [184.578221] WARN_ON(debug_locks && !(lock_is_held(&(&((obj)->b

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/5] drm/i915: Skip dsc readout if the transcoder is disabled (rev3)

2022-01-25 Thread Patchwork
== Series Details == Series: series starting with [1/5] drm/i915: Skip dsc readout if the transcoder is disabled (rev3) URL : https://patchwork.freedesktop.org/series/99276/ State : success == Summary == CI Bug Log - changes from CI_DRM_11137 -> Patchwork_22107 ===