Re: [Intel-gfx] [PATCH] drm/i915: Engine relative MMIO

Hi John, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on next-20190222] [cannot apply to v5.0-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: extract AUX mask assignment to separate function

== Series Details == Series: drm/i915: extract AUX mask assignment to separate function URL : https://patchwork.freedesktop.org/series/57119/ State : success == Summary == CI Bug Log - changes from CI_DRM_5655_full -> Patchwork_12291_full S

Re: [Intel-gfx] [PATCH 2/4] drm/i915: Disable PSR2 while getting pipe CRC

On Sat, 2019-02-23 at 02:48 +, Souza, Jose wrote: > On Fri, 2019-02-22 at 18:13 -0800, Dhinakaran Pandiyan wrote: > > On Wed, 2019-02-13 at 18:02 -0800, José Roberto de Souza wrote: > > > As stated in CRC_CTL spec, after PSR entry state CRC will not be > > > calculated anymore what is not a pro

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/3] drm/i915/icl: move MG pll hw_state readout

== Series Details == Series: series starting with [1/3] drm/i915/icl: move MG pll hw_state readout URL : https://patchwork.freedesktop.org/series/57116/ State : success == Summary == CI Bug Log - changes from CI_DRM_5655_full -> Patchwork_12289_full

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915: refactor transcoders reporting on error state

== Series Details == Series: series starting with [1/2] drm/i915: refactor transcoders reporting on error state URL : https://patchwork.freedesktop.org/series/57113/ State : success == Summary == CI Bug Log - changes from CI_DRM_5655_full -> Patchwork_12288_full ==

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Call MG_DP_MODE() macro with the right parameters order

== Series Details == Series: drm/i915: Call MG_DP_MODE() macro with the right parameters order URL : https://patchwork.freedesktop.org/series/57107/ State : success == Summary == CI Bug Log - changes from CI_DRM_5654_full -> Patchwork_12285_full

Re: [Intel-gfx] [PATCH 2/4] drm/i915: Disable PSR2 while getting pipe CRC

On Fri, 2019-02-22 at 18:13 -0800, Dhinakaran Pandiyan wrote: > On Wed, 2019-02-13 at 18:02 -0800, José Roberto de Souza wrote: > > As stated in CRC_CTL spec, after PSR entry state CRC will not be > > calculated anymore what is not a problem as IGT tests do some > > screen > > change and then reque

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: remove unused bits from Panel Power Sequence State

== Series Details == Series: series starting with [1/2] drm/i915: remove unused bits from Panel Power Sequence State URL : https://patchwork.freedesktop.org/series/57122/ State : success == Summary == CI Bug Log - changes from CI_DRM_5655 -> Patchwork_12293 ===

Re: [Intel-gfx] [PATCH 2/4] drm/i915: Disable PSR2 while getting pipe CRC

On Wed, 2019-02-13 at 18:02 -0800, José Roberto de Souza wrote: > As stated in CRC_CTL spec, after PSR entry state CRC will not be > calculated anymore what is not a problem as IGT tests do some screen > change and then request the pipe CRC right after the change so PSR > will go to idle state and

Re: [Intel-gfx] [PATCH 1/4] drm/i915/psr: Only lookup for enabled CRTCs when forcing a fastset

On Fri, 2019-02-22 at 16:33 -0800, Souza, Jose wrote: > On Fri, 2019-02-22 at 16:27 -0800, Pandiyan, Dhinakaran wrote: > > On Wed, 2019-02-13 at 18:02 -0800, José Roberto de Souza wrote: > > > Forcing a specific CRTC to the eDP connector was causing the > > > intel_psr_fastset_force() to mark mode_

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/4] drm/i915: Disable LP1+ watermarks on Lenovo Thinkpad T431s

== Series Details == Series: series starting with [1/4] drm/i915: Disable LP1+ watermarks on Lenovo Thinkpad T431s URL : https://patchwork.freedesktop.org/series/57101/ State : success == Summary == CI Bug Log - changes from CI_DRM_5654_full -> Patchwork_12284_full ===

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/pmu: Always sample an active ringbuffer (rev2)

== Series Details == Series: drm/i915/pmu: Always sample an active ringbuffer (rev2) URL : https://patchwork.freedesktop.org/series/57090/ State : success == Summary == CI Bug Log - changes from CI_DRM_5655 -> Patchwork_12292 Summary --

[Intel-gfx] [PATCH i-g-t] i915/gem_ctx_switch: Use minimum qlen over all engines and measure switches

Not all engines are created equal, and our weighting ends up favouring the many faster xCS rings at the expense of RCS. Our qlen estimation also failed to factor in the context switch overhead, which is a significant factor for nop batches. So we oversubscribe the number of batches submitted to RCS

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: extract AUX mask assignment to separate function

== Series Details == Series: drm/i915: extract AUX mask assignment to separate function URL : https://patchwork.freedesktop.org/series/57119/ State : success == Summary == CI Bug Log - changes from CI_DRM_5655 -> Patchwork_12291 Summary ---

[Intel-gfx] [PATCH 2/2] drm/i915: don't check internal state in PP_STATUS

Instead of checking the bits that give the internal machine state we can simply rely on the information from the other bits: 1) on or off, 2) transitioning or not. Bit 31 has the "Panel Power On Status" Bits 29:28 has the "Power Sequence Progress" So, wait_panel_on() only needs to wait for bit 31

[Intel-gfx] [PATCH 1/2] drm/i915: remove unused bits from Panel Power Sequence State

No change in behavior. Just removing the unused bits since it makes it easier to compare them on new platforms and one of them was wrong (PP_SEQUENCE_STATE_ON_S1_0 vs the supposedly correct name PP_SEQUENCE_STATE_ON_S1_1) Cc: Rodrigo Vivi Signed-off-by: Lucas De Marchi --- drivers/gpu/drm/i915/

Re: [Intel-gfx] [PATCH v15 04/16] misc/mei/hdcp: Initiate Wired HDCP2.2 Tx Session

Hi Ramalingam, Thank you for the patch! Yet something to improve: [auto build test ERROR on char-misc/char-misc-testing] [also build test ERROR on v5.0-rc4 next-20190222] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https

Re: [Intel-gfx] [PATCH 1/4] drm/i915/psr: Only lookup for enabled CRTCs when forcing a fastset

On Fri, 2019-02-22 at 16:27 -0800, Pandiyan, Dhinakaran wrote: > On Wed, 2019-02-13 at 18:02 -0800, José Roberto de Souza wrote: > > Forcing a specific CRTC to the eDP connector was causing the > > intel_psr_fastset_force() to mark mode_chaged in the wrong and > > disabled CRTC causing no update in

Re: [Intel-gfx] [PATCH 1/4] drm/i915/psr: Only lookup for enabled CRTCs when forcing a fastset

On Wed, 2019-02-13 at 18:02 -0800, José Roberto de Souza wrote: > Forcing a specific CRTC to the eDP connector was causing the > intel_psr_fastset_force() to mark mode_chaged in the wrong and > disabled CRTC causing no update in the PSR state. > > Looks like our internal state track do not clear o

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Engine relative MMIO

== Series Details == Series: drm/i915: Engine relative MMIO URL : https://patchwork.freedesktop.org/series/57117/ State : failure == Summary == CALLscripts/checksyscalls.sh DESCEND objtool CHK include/generated/compile.h CC [M] drivers/gpu/drm/i915/gvt/mmio_context.o drivers/gp

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/3] drm/i915/icl: move MG pll hw_state readout

== Series Details == Series: series starting with [1/3] drm/i915/icl: move MG pll hw_state readout URL : https://patchwork.freedesktop.org/series/57116/ State : success == Summary == CI Bug Log - changes from CI_DRM_5655 -> Patchwork_12289

Re: [Intel-gfx] [PATCH] drm/i915: extract AUX mask assignment to separate function

On Fri, 2019-02-22 at 15:59 -0800, Lucas De Marchi wrote: > No change in behavior, this only allows to more easily follow the > flow > of gen8_de_irq_handler without the mask assignments for each > platform. > This also re-organizes the branches a little bit, so the one-off case > for CNL_WITH_PORT

[Intel-gfx] [PATCH] drm/i915/pmu: Always sample an active ringbuffer

As we no longer have a precise indication of requests queued to an engine, make no presumptions and just sample the ring registers to see if the engine is busy. v2: Report busy while the ring is idling on a semaphore/event. v3: Give the struct a name! v4: Always 0 outside the powerwell; trusting t

[Intel-gfx] [PATCH] drm/i915: extract AUX mask assignment to separate function

No change in behavior, this only allows to more easily follow the flow of gen8_de_irq_handler without the mask assignments for each platform. This also re-organizes the branches a little bit, so the one-off case for CNL_WITH_PORT_F is separate from the generic gen >= 11. Cc: Ville Syrjälä Cc: Jos

Re: [Intel-gfx] [PATCH] drm/i915: Engine relative MMIO

Quoting john.c.harri...@intel.com (2019-02-22 23:49:18) > From: John Harrison > > With virtual engines, it is no longer possible to know which specific > physical engine a given request will be executed on at the time that > request is generated. This means that the request itself must be engine

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/pmu: Always sample an active ringbuffer

== Series Details == Series: drm/i915/pmu: Always sample an active ringbuffer URL : https://patchwork.freedesktop.org/series/57090/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5654_full -> Patchwork_12283_full Summary ---

[Intel-gfx] [PATCH] drm/i915: Engine relative MMIO

From: John Harrison With virtual engines, it is no longer possible to know which specific physical engine a given request will be executed on at the time that request is generated. This means that the request itself must be engine agnostic - any direct register writes must be relative to the engi

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: refactor transcoders reporting on error state

== Series Details == Series: series starting with [1/2] drm/i915: refactor transcoders reporting on error state URL : https://patchwork.freedesktop.org/series/57113/ State : success == Summary == CI Bug Log - changes from CI_DRM_5655 -> Patchwork_12288

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [1/2] drm/i915: refactor transcoders reporting on error state

== Series Details == Series: series starting with [1/2] drm/i915: refactor transcoders reporting on error state URL : https://patchwork.freedesktop.org/series/57113/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: refactor transcoders reporti

[Intel-gfx] [PATCH 1/3] drm/i915/icl: move MG pll hw_state readout

Let the MG plls have their own hooks since it shares very little with other PLL types. It's also better so the platform info contains the info if the PLL is for MG PHY rather than relying on the PLL ids. Signed-off-by: Lucas De Marchi --- drivers/gpu/drm/i915/intel_dpll_mgr.c | 122 +

[Intel-gfx] [PATCH 3/3] drm/i915/icl: decouple dpll ids from type

Use the first 3 bits of dpll_info.platform_flags to mark the type of the PLL instead of relying on the IDs. This is more future proof for allowing the same set of functions to be reused, even if the IDs change. The warning about PLL id not corresponding to a combo phy in intel_display was removed

[Intel-gfx] [PATCH 2/3] drm/i915: introduce platform_flags to use with dpll hooks

Define platform flags taking the high 16 bits of the under-utilized flags in dpll_info. This makes clear separation on what flags are used by the core and what flags can be freely defined for use in the context of the platforms sharing a set of hooks. The initial motivation for this is the Ice Lak

[Intel-gfx] [PATCH 1/2] drm/i915: refactor transcoders reporting on error state

Instead of keeping track of the number of transcoders, loop through all the interesting ones and check if there is a correspondent offset. Cc: Rodrigo Vivi Signed-off-by: Lucas De Marchi --- drivers/gpu/drm/i915/intel_display.c | 21 - 1 file changed, 12 insertions(+), 9 del

[Intel-gfx] [PATCH 2/2] drm/i915: allow platforms without eDP transcoder

Define a HAS_TRANSCODER_EDP() macro that checks if we have defined an offset for this transcoder. This allows platforms to be defined without eDP transcoder. Cc: Mika Kahola Cc: Imre Deak Cc: Rodrigo Vivi Signed-off-by: Lucas De Marchi --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/g

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Yet another if/else sort of newer to older platforms.

== Series Details == Series: drm/i915: Yet another if/else sort of newer to older platforms. URL : https://patchwork.freedesktop.org/series/57112/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5655 -> Patchwork_12287 Summar

Re: [Intel-gfx] [PATCH] drm/i915: Call MG_DP_MODE() macro with the right parameters order

On Fri, Feb 22, 2019 at 12:24 PM José Roberto de Souza wrote: > > The commit that this patch fixes changed the order of the parameters > of MG_DP_MODE() but din't update the callers, breaking type-c on ICL. ugh... > > Fixes: 58106b7d816e ("drm/i915: Make MG PHY macros semantically consistent")

[Intel-gfx] [PATCH] drm/i915: Yet another if/else sort of newer to older platforms.

No functional change. Just a reorg to match the preferred behavior. When rebasing internal branch on top of latest sort I noticed few more cases that needs to get reordered. Let's do in a bundle this time and hoping there's no other missing places. Cc: Ville Syrjälä Cc: Chris Wilson Cc: Lucas

[Intel-gfx] ✗ Fi.CI.IGT: failure for RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev8)

== Series Details == Series: RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev8) URL : https://patchwork.freedesktop.org/series/57039/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5653_full -> IGTPW_2488_full Sum

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915: Add support for retrying hotplug

== Series Details == Series: series starting with [1/2] drm/i915: Add support for retrying hotplug URL : https://patchwork.freedesktop.org/series/57108/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5654 -> Patchwork_12286

Re: [Intel-gfx] [PATCH] drm/i915: Remove unused HAS_PCH_CNP_LP

On Thu, Feb 21, 2019 at 03:08:00PM -0800, Lucas De Marchi wrote: > On Thu, Feb 21, 2019 at 01:17:16PM -0800, Rodrigo Vivi wrote: > > Other than LPT, no other PCH needed to differentiate between > > LP and HP. So let's remove this before we spread this mistake > > to future platforms. > > > > Cc: A

Re: [Intel-gfx] [PATCH] drm/i915: Sort ctx workarounds init from newer to older platforms.

On Fri, Feb 22, 2019 at 10:32:25AM +, Chris Wilson wrote: > Quoting Rodrigo Vivi (2019-02-21 23:14:52) > > No functional change. Just a reorg to match the preferred > > behavior. > > > > Cc: Tvrtko Ursulin > > Cc: Ville Syrjälä > > Cc: Lucas De Marchi > > Signed-off-by: Rodrigo Vivi > > --

Re: [Intel-gfx] [PATCH v2] drm/i915: Sort newer to older platforms.

On Thu, Feb 21, 2019 at 11:48:58PM +0200, Ville Syrjälä wrote: > On Thu, Feb 21, 2019 at 01:44:30PM -0800, Rodrigo Vivi wrote: > > No functional change. Just a reorg to match the preferred > > behavior. > > > > v2: missing else (Ville) > > > > Cc: Ville Syrjälä > > Cc: Lucas De Marchi > > Signe

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [1/2] drm/i915: Add support for retrying hotplug

== Series Details == Series: series starting with [1/2] drm/i915: Add support for retrying hotplug URL : https://patchwork.freedesktop.org/series/57108/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: Add support for retrying hotplug -drivers/

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Call MG_DP_MODE() macro with the right parameters order

== Series Details == Series: drm/i915: Call MG_DP_MODE() macro with the right parameters order URL : https://patchwork.freedesktop.org/series/57107/ State : success == Summary == CI Bug Log - changes from CI_DRM_5654 -> Patchwork_12285 Summ

[Intel-gfx] [PATCH 1/2] drm/i915: Add support for retrying hotplug

From: Imre Deak There is some scenarios that we are aware that sink probe can fail, so lets add the infrastructure to let hotplug() hook to request another probe after some time. Cc: Ville Syrjälä Signed-off-by: José Roberto de Souza Signed-off-by: Jani Nikula Signed-off-by: Imre Deak --- d

[Intel-gfx] [PATCH 2/2] drm/i915/icl: Probe again type-c connectors that failed

Unpowered type-c dongles can take some time to boot and be responsible, causing the probe to fail and sink never be detected without further actions from userspace. It was not a issue for older platforms because there was a hardware bridge between DDI/DP ports and type-c controller adding a implic

[Intel-gfx] ✗ Fi.CI.IGT: failure for RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev7)

== Series Details == Series: RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev7) URL : https://patchwork.freedesktop.org/series/57039/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5653_full -> IGTPW_2486_full Sum

[Intel-gfx] [PATCH] drm/i915: Call MG_DP_MODE() macro with the right parameters order

The commit that this patch fixes changed the order of the parameters of MG_DP_MODE() but din't update the callers, breaking type-c on ICL. Fixes: 58106b7d816e ("drm/i915: Make MG PHY macros semantically consistent") Cc: Clint Taylor Cc: Imre Deak Cc: Jani Nikula Cc: Aditya Swarup Cc: Manasi na

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/4] drm/i915: Disable LP1+ watermarks on Lenovo Thinkpad T431s

== Series Details == Series: series starting with [1/4] drm/i915: Disable LP1+ watermarks on Lenovo Thinkpad T431s URL : https://patchwork.freedesktop.org/series/57101/ State : success == Summary == CI Bug Log - changes from CI_DRM_5654 -> Patchwork_12284 =

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Sort ctx workarounds init from newer to older platforms.

== Series Details == Series: drm/i915: Sort ctx workarounds init from newer to older platforms. URL : https://patchwork.freedesktop.org/series/57056/ State : success == Summary == CI Bug Log - changes from CI_DRM_5651_full -> Patchwork_12281_full ===

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915: Disable LP1+ watermarks on Lenovo Thinkpad T431s

== Series Details == Series: series starting with [1/4] drm/i915: Disable LP1+ watermarks on Lenovo Thinkpad T431s URL : https://patchwork.freedesktop.org/series/57101/ State : warning == Summary == $ dim checkpatch origin/drm-tip 4545b0056a9b drm/i915: Disable LP1+ watermarks on Lenovo Think

[Intel-gfx] [PATCH 2/4] drm/i915: Generalize pci quirks

From: Ville Syrjälä Add support for multiple independent pci quirk tables. I want to reuse the quirk table approach for some watermark quirks but I prefer to keep the details in one place rather than spreading them all over. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_drv.h

[Intel-gfx] [PATCH 3/4] drm/i915: Use intel_apply_pci_quirks() to apply ILK+ wm quirks

From: Ville Syrjälä Use the newly introduced intel_apply_pci_quirks() to clean up the way we apply the ilk+ watermark quirks. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_pm.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/

[Intel-gfx] [PATCH 1/4] drm/i915: Disable LP1+ watermarks on Lenovo Thinkpad T431s

From: Ville Syrjälä Lenovo Thinkpad T431s (ivb) apparently can't handle LP1+ watermarks being enabled. It underruns severly enough that the screen is unusable. The latency values and watemarks look as expected. And sadly updating the BIOS to the latest version (BIOS GHET41WW (1.26 ) 11/21/2018) d

[Intel-gfx] [PATCH 4/4] drm/i915: Restrict SNB LP3+ disable to Thinkpad X220 tablet

From: Ville Syrjälä The only machine we know for sure to require the LP3+ disable is the Lenovo Thinkpad X220 tablet. Originally in commit 03981c6ebec4 ("drm/i915: Disable LP3 watermarks on all SNB machines") I disabled LP3+ watermarks on all SNB machines, partially for safety, and partially sinc

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Sort newer to older platforms. (rev2)

== Series Details == Series: drm/i915: Sort newer to older platforms. (rev2) URL : https://patchwork.freedesktop.org/series/57051/ State : success == Summary == CI Bug Log - changes from CI_DRM_5651_full -> Patchwork_12280_full Summary

[Intel-gfx] ✗ Fi.CI.IGT: failure for RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev5)

== Series Details == Series: RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev5) URL : https://patchwork.freedesktop.org/series/57039/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5651_full -> IGTPW_2484_full Sum

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] lib: Add --skip-crc-compare option

On Wed, Feb 20, 2019 at 04:34:21PM -0800, Matt Roper wrote: > When using --interactive-debug, it's sometimes desirable to ignore CRC > mismatches and let the test proceed as if they passed so that the > on-screen outcome can be inspected. Let's add a debug option to allow > this. > > Cc: igt-...@

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/pmu: Always sample an active ringbuffer

== Series Details == Series: drm/i915/pmu: Always sample an active ringbuffer URL : https://patchwork.freedesktop.org/series/57090/ State : success == Summary == CI Bug Log - changes from CI_DRM_5654 -> Patchwork_12283 Summary --- **

[Intel-gfx] ✗ Fi.CI.IGT: failure for RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev4)

== Series Details == Series: RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev4) URL : https://patchwork.freedesktop.org/series/57039/ State : failure == Summary == CI Bug Log - changes from IGT_4851_full -> IGTPW_2483_full Summar

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Remove unused HAS_PCH_CNP_LP

== Series Details == Series: drm/i915: Remove unused HAS_PCH_CNP_LP URL : https://patchwork.freedesktop.org/series/57048/ State : success == Summary == CI Bug Log - changes from CI_DRM_5650_full -> Patchwork_12279_full Summary --- **

Re: [Intel-gfx] [PATCH v5] drm/i915/pmu: Always sample an active ringbuffer

Chris Wilson writes: > As we no longer have a precise indication of requests queued to an > engine, make no presumptions and just sample the ring registers to see > if the engine is busy. > > v2: Report busy while the ring is idling on a semaphore/event. > v3: Give the struct a name! > v4: Always

[Intel-gfx] [PATCH i-g-t] i915/gem_ctx_isolation: Sanitycheck nonpriv access

Verify that our list of nonpriv registers exist and are writable. Signed-off-by: Chris Wilson Cc: Dale B Stimson Cc: Tvrtko Ursulin Cc: Joonas Lahtinen --- tests/i915/gem_ctx_isolation.c | 164 +++-- 1 file changed, 135 insertions(+), 29 deletions(-) diff --git a/

Re: [Intel-gfx] [PATCH 08/25] drm/i915/pmu: Always sample an active ringbuffer

Chris Wilson writes: > Quoting Mika Kuoppala (2019-02-22 12:10:33) >> Chris Wilson writes: >> >> > As we no longer have a precise indication of requests queued to an >> > engine, make no presumptions and just sample the ring registers to see >> > if the engine is busy. >> > >> > v2: Report busy

[Intel-gfx] ✓ Fi.CI.BAT: success for RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev8)

== Series Details == Series: RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev8) URL : https://patchwork.freedesktop.org/series/57039/ State : success == Summary == CI Bug Log - changes from CI_DRM_5653 -> IGTPW_2488 Summary -

[Intel-gfx] [PATCH v5] drm/i915/pmu: Always sample an active ringbuffer

As we no longer have a precise indication of requests queued to an engine, make no presumptions and just sample the ring registers to see if the engine is busy. v2: Report busy while the ring is idling on a semaphore/event. v3: Give the struct a name! v4: Always 0 outside the powerwell; trusting t

Re: [Intel-gfx] [PATCH 08/25] drm/i915/pmu: Always sample an active ringbuffer

Quoting Mika Kuoppala (2019-02-22 12:10:33) > Chris Wilson writes: > > > As we no longer have a precise indication of requests queued to an > > engine, make no presumptions and just sample the ring registers to see > > if the engine is busy. > > > > v2: Report busy while the ring is idling on a s

Re: [Intel-gfx] [PATCH 08/25] drm/i915/pmu: Always sample an active ringbuffer

Chris Wilson writes: > As we no longer have a precise indication of requests queued to an > engine, make no presumptions and just sample the ring registers to see > if the engine is busy. > > v2: Report busy while the ring is idling on a semaphore/event. > v3: Give the struct a name! > v4: Always

[Intel-gfx] ✗ Fi.CI.IGT: failure for RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev3)

== Series Details == Series: RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev3) URL : https://patchwork.freedesktop.org/series/57039/ State : failure == Summary == CI Bug Log - changes from IGT_4851_full -> IGTPW_2481_full Summar

[Intel-gfx] [PATCH i-g-t] RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access

Verify that our list of nonpriv registers exist and are writable. Signed-off-by: Chris Wilson --- tests/i915/gem_ctx_isolation.c| 164 +- tests/intel-ci/fast-feedback.testlist | 5 + 2 files changed, 140 insertions(+), 29 deletions(-) diff --git a/tests/i915/ge

[Intel-gfx] ✓ Fi.CI.BAT: success for RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev7)

== Series Details == Series: RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev7) URL : https://patchwork.freedesktop.org/series/57039/ State : success == Summary == CI Bug Log - changes from CI_DRM_5653 -> IGTPW_2486 Summary -

Re: [Intel-gfx] [PATCH v2] drm/i915/guc: Flush the residual log capture irq on disabling

Quoting Mika Kuoppala (2019-02-22 11:19:31) > Chris Wilson writes: > > > As we disable the log capture events, flush any residual interrupt > > before we flush and disable the worker. > > > > v2: Mika pointed out that it wasn't the worker re-queueing itself, but a > > rogue irq. > > > > Bugzilla:

[Intel-gfx] [PATCH i-g-t] RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access

Verify that our list of nonpriv registers exist and are writable. Signed-off-by: Chris Wilson --- tests/i915/gem_ctx_isolation.c| 151 +- tests/intel-ci/fast-feedback.testlist | 5 + 2 files changed, 131 insertions(+), 25 deletions(-) diff --git a/tests/i915/ge

Re: [Intel-gfx] [PATCH v2] drm/i915/guc: Flush the residual log capture irq on disabling

Chris Wilson writes: > As we disable the log capture events, flush any residual interrupt > before we flush and disable the worker. > > v2: Mika pointed out that it wasn't the worker re-queueing itself, but a > rogue irq. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109716 > Signed-

[Intel-gfx] [PATCH i-g-t] RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access

Verify that our list of nonpriv registers exist and are writable. Signed-off-by: Chris Wilson --- tests/i915/gem_ctx_isolation.c| 146 +- tests/intel-ci/fast-feedback.testlist | 5 + 2 files changed, 126 insertions(+), 25 deletions(-) diff --git a/tests/i915/ge

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm & vgaarb: handle vgacon removal in vgaarb.

== Series Details == Series: drm & vgaarb: handle vgacon removal in vgaarb. URL : https://patchwork.freedesktop.org/series/57086/ State : failure == Summary == CALLscripts/checksyscalls.sh DESCEND objtool CHK include/generated/compile.h CC [M] drivers/gpu/drm/i915/i915_drv.o dr

[Intel-gfx] [PATCH v5 2/3] drm/fb-helper: call vga_remove_vgacon automatically.

Add vga_remove_vgacon() call to drm_fb_helper_remove_conflicting_pci_framebuffers(). Signed-off-by: Gerd Hoffmann Reviewed-by: Daniel Vetter --- include/drm/drm_fb_helper.h | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/include/drm/drm_fb_helper.h b/include/

[Intel-gfx] [PATCH v5 1/3] drm: move i915_kick_out_vgacon to vgaarb

Also rename it to vga_remove_vgacon and add kerneldoc text. Signed-off-by: Gerd Hoffmann Reviewed-by: Daniel Vetter --- include/linux/vgaarb.h | 2 ++ drivers/gpu/drm/i915/i915_drv.c | 35 +- drivers/gpu/vga/vgaarb.c| 48

[Intel-gfx] [PATCH v5 0/3] drm & vgaarb: handle vgacon removal in vgaarb.

Gerd Hoffmann (3): drm: move i915_kick_out_vgacon to vgaarb drm/fb-helper: call vga_remove_vgacon automatically. drm/qxl: remove conflicting framebuffers earlier include/drm/drm_fb_helper.h | 14 +--- include/linux/vgaarb.h | 2 ++ drivers/gpu/drm/i915/i915_drv.c | 3

[Intel-gfx] [PATCH v5 3/3] drm/qxl: remove conflicting framebuffers earlier

Add error checking while being at it. Signed-off-by: Gerd Hoffmann Reviewed-by: Daniel Vetter --- drivers/gpu/drm/qxl/qxl_drv.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/qxl/qxl_drv.c b/drivers/gpu/drm/qxl/qxl_drv.c index bb81e310eb6d..578d867a81d5

[Intel-gfx] ✗ Fi.CI.IGT: failure for RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev2)

== Series Details == Series: RFT i915/gem_ctx_isolation: Sanitycheck nonpriv access (rev2) URL : https://patchwork.freedesktop.org/series/57039/ State : failure == Summary == CI Bug Log - changes from IGT_4851_full -> IGTPW_2477_full Summar

Re: [Intel-gfx] [PATCH] drm/i915: Sort ctx workarounds init from newer to older platforms.

Quoting Rodrigo Vivi (2019-02-21 23:14:52) > No functional change. Just a reorg to match the preferred > behavior. > > Cc: Tvrtko Ursulin > Cc: Ville Syrjälä > Cc: Lucas De Marchi > Signed-off-by: Rodrigo Vivi > --- > drivers/gpu/drm/i915/intel_workarounds.c | 36 > 1

[Intel-gfx] ✓ Fi.CI.IGT: success for Make DRM DSC helpers more generally usable

== Series Details == Series: Make DRM DSC helpers more generally usable URL : https://patchwork.freedesktop.org/series/57044/ State : success == Summary == CI Bug Log - changes from CI_DRM_5650_full -> Patchwork_12278_full Summary ---

[Intel-gfx] [PULL] drm-misc-fixes

Hi dave/Daniel, Sorry for being a bit late, hopefully still in time for v5.0 :) drm-misc-fixes-2019-02-22: drm-misc-fixes for v5.0: - Block fb changes for async atomic updates to prevent a use after free. - Fix ID mismatch error on load in bochs. - Fix memory leak when drm_setup fails. - Fixes ar

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/dp: Preliminary support for DP YCbCr4:2:0 outputs (rev2)

== Series Details == Series: drm/i915/dp: Preliminary support for DP YCbCr4:2:0 outputs (rev2) URL : https://patchwork.freedesktop.org/series/56059/ State : success == Summary == CI Bug Log - changes from CI_DRM_5650_full -> Patchwork_12277_full