[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/execlists: Leave resetting ring to intel_ring

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/execlists: Leave resetting ring to intel_ring URL : https://patchwork.freedesktop.org/series/72082/ State : success == Summary == CI Bug Log - changes from CI_DRM_7751_full -> Patchwork_16115_full Summa

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Align engine->uabi_class/instance with i915_drm.h

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915: Align engine->uabi_class/instance with i915_drm.h URL : https://patchwork.freedesktop.org/series/72078/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7751_full -> Patchwork_16114_full

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915/dsi: Ensure that the ACPI adapter lookup overrides the bus num

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/dsi: Ensure that the ACPI adapter lookup overrides the bus num URL : https://patchwork.freedesktop.org/series/72225/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compil

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dsi: Ensure that the ACPI adapter lookup overrides the bus num

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/dsi: Ensure that the ACPI adapter lookup overrides the bus num URL : https://patchwork.freedesktop.org/series/72225/ State : success == Summary == CI Bug Log - changes from CI_DRM_7770 -> Patchwork_16164

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915: eDP DPCD aux backlight fixes (rev7)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915: eDP DPCD aux backlight fixes (rev7) URL : https://patchwork.freedesktop.org/series/69914/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compile.h Kernel: arch/x86/boot/

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: eDP DPCD aux backlight fixes (rev7)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915: eDP DPCD aux backlight fixes (rev7) URL : https://patchwork.freedesktop.org/series/69914/ State : success == Summary == CI Bug Log - changes from CI_DRM_7769 -> Patchwork_16163 Summary --- **SUC

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list (rev2)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list (rev2) URL : https://patchwork.freedesktop.org/series/72221/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list (rev2)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list (rev2) URL : https://patchwork.freedesktop.org/series/72221/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7769 -> Patchwork_16162 =

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915: Add support for HDCP 1.4 over MST connectors (rev3)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915: Add support for HDCP 1.4 over MST connectors (rev3) URL : https://patchwork.freedesktop.org/series/70393/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compile.h Kernel

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Add support for HDCP 1.4 over MST connectors (rev3)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915: Add support for HDCP 1.4 over MST connectors (rev3) URL : https://patchwork.freedesktop.org/series/70393/ State : success == Summary == CI Bug Log - changes from CI_DRM_7769 -> Patchwork_16160 Summary

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list (rev2)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list (rev2) URL : https://patchwork.freedesktop.org/series/72221/ State : warning == Summary == $ dim checkpatch origin/drm-tip a9954bbab8d0 drm/i915/gem: Store mmap_offsets in an rbtree rather than

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Fix i915_error_state_store error defination

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915: Fix i915_error_state_store error defination URL : https://patchwork.freedesktop.org/series/72220/ State : failure == Summary == Applying: drm/i915: Fix i915_error_state_store error defination Using index info to reconstruct a base tree... M drivers/

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Add support for HDCP 1.4 over MST connectors (rev3)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915: Add support for HDCP 1.4 over MST connectors (rev3) URL : https://patchwork.freedesktop.org/series/70393/ State : warning == Summary == $ dim checkpatch origin/drm-tip 312b2607dec2 drm/i915: Fix sha_text population code -:57: WARNING:LINE_SPACING: Missing

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915/gt: Be paranoid and reset the GPU before release (rev2)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gt: Be paranoid and reset the GPU before release (rev2) URL : https://patchwork.freedesktop.org/series/72185/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compile.h Ker

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Be paranoid and reset the GPU before release (rev2)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gt: Be paranoid and reset the GPU before release (rev2) URL : https://patchwork.freedesktop.org/series/72185/ State : success == Summary == CI Bug Log - changes from CI_DRM_7769 -> Patchwork_16159 Summa

[Intel-gfx] ✗ Fi.CI.BUILD: warning for series starting with [1/2] drm/i915/gvt: Wean gvt off dev_priv->engine[]

2020-01-17 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/gvt: Wean gvt off dev_priv->engine[] URL : https://patchwork.freedesktop.org/series/72194/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compi

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915/gvt: Wean gvt off dev_priv->engine[]

2020-01-17 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/gvt: Wean gvt off dev_priv->engine[] URL : https://patchwork.freedesktop.org/series/72194/ State : success == Summary == CI Bug Log - changes from CI_DRM_7769 -> Patchwork_16158 ===

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915/gvt: Wean gvt off dev_priv->engine[]

2020-01-17 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/gvt: Wean gvt off dev_priv->engine[] URL : https://patchwork.freedesktop.org/series/72194/ State : warning == Summary == $ dim checkpatch origin/drm-tip ae3c3de52ba8 drm/i915/gvt: Wean gvt off dev_priv->engine[] -:681: WARNING:LO

[Intel-gfx] [PATCH] drm/i915/dsi: Ensure that the ACPI adapter lookup overrides the bus num

2020-01-17 Thread Vivek Kasireddy
Remove the i2c_bus_num >= 0 check from the adapter lookup function as this would prevent ACPI bus number override. This check was mainly there to return early if the bus number has already been found but we anyway return in the next line if the slave address does not match. Fixes: 8cbf89db2941 ("d

[Intel-gfx] ✓ Fi.CI.IGT: success for Misc GuC CT improvements - part II

2020-01-17 Thread Patchwork
== Series Details == Series: Misc GuC CT improvements - part II URL : https://patchwork.freedesktop.org/series/72071/ State : success == Summary == CI Bug Log - changes from CI_DRM_7748_full -> Patchwork_16112_full Summary --- **SUCC

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/debugfs: remove i915_dpcd file

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/debugfs: remove i915_dpcd file URL : https://patchwork.freedesktop.org/series/72192/ State : success == Summary == CI Bug Log - changes from CI_DRM_7768 -> Patchwork_16157 Summary --- **SUCCESS**

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915/debugfs: remove i915_dpcd file

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/debugfs: remove i915_dpcd file URL : https://patchwork.freedesktop.org/series/72192/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compile.h Kernel: arch/x86/boot/bzImag

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915/dp: debug log max vswing and pre-emphasis

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/dp: debug log max vswing and pre-emphasis URL : https://patchwork.freedesktop.org/series/72191/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compile.h Kernel: arch/x86/

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dp: debug log max vswing and pre-emphasis

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/dp: debug log max vswing and pre-emphasis URL : https://patchwork.freedesktop.org/series/72191/ State : success == Summary == CI Bug Log - changes from CI_DRM_7768 -> Patchwork_16156 Summary ---

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915/bios: stop using vbt.ddi_port_info directly

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/bios: stop using vbt.ddi_port_info directly URL : https://patchwork.freedesktop.org/series/72190/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compile.h Kernel: arch/x8

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/bios: stop using vbt.ddi_port_info directly

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/bios: stop using vbt.ddi_port_info directly URL : https://patchwork.freedesktop.org/series/72190/ State : success == Summary == CI Bug Log - changes from CI_DRM_7768 -> Patchwork_16155 Summary ---

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gt: Use the BIT when checking the flags, not the index

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gt: Use the BIT when checking the flags, not the index URL : https://patchwork.freedesktop.org/series/72060/ State : success == Summary == CI Bug Log - changes from CI_DRM_7748_full -> Patchwork_16111_full ==

[Intel-gfx] [PATCH v4] drm/i915: Don't use VBT for detecting DPCD backlight controls

2020-01-17 Thread Lyude Paul
Despite the fact that the VBT appears to have a field for specifying that a system is equipped with a panel that supports standard VESA backlight controls over the DP AUX channel, so far every system we've spotted DPCD backlight control support on doesn't actually set this field correctly and all h

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] i915/gem_mmap_offset: Exercise many, many mmap_offset

2020-01-17 Thread Patchwork
== Series Details == Series: series starting with [1/2] i915/gem_mmap_offset: Exercise many, many mmap_offset URL : https://patchwork.freedesktop.org/series/7/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7768 -> IGTPW_3940 ===

[Intel-gfx] [PATCH v2] drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list

2020-01-17 Thread Chris Wilson
Currently we create a new mmap_offset for every call to mmap_offset_ioctl. This exposes ourselves to an abusive client that may simply create new mmap_offsets ad infinitum, which will exhaust physical memory and the virtual address space. In addition to the exhaustion, a very long linear list of mm

[Intel-gfx] [PATCH i-g-t 1/2] i915/gem_mmap_offset: Exercise many, many mmap_offset

2020-01-17 Thread Chris Wilson
Just keep on generating a new mmap_offset for the same old buffer, but for different handles and so exercise the scaling of the obj->mmo lists. Signed-off-by: Chris Wilson Cc: Abdiel Janulgue --- tests/i915/gem_mmap_offset.c | 64 1 file changed, 64 insertio

[Intel-gfx] [PATCH i-g-t 2/2] i915/gem_mmap_offset: Relax isolation rules

2020-01-17 Thread Chris Wilson
It is legal for mmap_offset_ioctl to return the same offset for different clients, so long as that offset is only valid for use with the client's fd and no one elses. Signed-off-by: Chris Wilson --- tests/i915/gem_mmap_offset.c | 20 +++- 1 file changed, 11 insertions(+), 9 delet

[Intel-gfx] ✓ Fi.CI.IGT: success for drm: Clean up VBLANK callbacks in struct drm_driver (rev7)

2020-01-17 Thread Patchwork
== Series Details == Series: drm: Clean up VBLANK callbacks in struct drm_driver (rev7) URL : https://patchwork.freedesktop.org/series/71873/ State : success == Summary == CI Bug Log - changes from CI_DRM_7748_full -> Patchwork_16109_full S

Re: [Intel-gfx] [PATCH] drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list

2020-01-17 Thread Chris Wilson
Quoting Chris Wilson (2020-01-17 22:00:38) > Currently we create a new mmap_offset for every call to > mmap_offset_ioctl. This exposes ourselves to an abusive client that may > simply create new mmap_offsets ad infinitum, which will exhaust physical > memory and the virtual address space. In additi

[Intel-gfx] [PATCH] drm/i915/gem: Store mmap_offsets in an rbtree rather than a plain list

2020-01-17 Thread Chris Wilson
Currently we create a new mmap_offset for every call to mmap_offset_ioctl. This exposes ourselves to an abusive client that may simply create new mmap_offsets ad infinitum, which will exhaust physical memory and the virtual address space. In addition to the exhaustion, a very long linear list of mm

Re: [Intel-gfx] [PATCH] drm/i915: Set num_qgv_points to zero if we can't query the actual points

2020-01-17 Thread Matt Roper
On Thu, Jan 16, 2020 at 01:45:44PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > If the pcode fails to give us the goods on any of the QGV points just > declare that we have none so that we'll properly ignore the bw limits. Should we also try to sanitize that the actual point info return

Re: [Intel-gfx] [PATCH v4 8/9] drivers/perf: open access for CAP_SYS_PERFMON privileged process

2020-01-17 Thread Alexey Budankov
On 17.01.2020 13:51, Will Deacon wrote: > On Wed, Dec 18, 2019 at 12:30:29PM +0300, Alexey Budankov wrote: >> >> Open access to monitoring for CAP_SYS_PERFMON privileged processes. >> For backward compatibility reasons access to the monitoring remains open >> for CAP_SYS_ADMIN privileged processe

Re: [Intel-gfx] [PATCH 1/4] drm/mst: Don't do atomic checks over disabled managers

2020-01-17 Thread Mikita Lipski
Thanks for the catch! Makes sense to skip disabled managers there, but I wonder why it didn't cause a crash with amdgpu. Anyways looks good to me. Acked-by: Mikita Lipski On 1/16/20 8:58 PM, José Roberto de Souza wrote: When a main MST port is disconnected drivers should call drm_dp_mst_topol

[Intel-gfx] [PATCH] drm/i915: Fix i915_error_state_store error defination

2020-01-17 Thread Zhang Xiaoxu
Since commit 742379c0c4001 ("drm/i915: Start chopping up the GPU error capture"), function 'i915_error_state_store' was defined and used with only one parameter. But if no 'CONFIG_DRM_I915_CAPTURE_ERROR', this function was defined with two parameter. This may lead compile error. This patch fix it

Re: [Intel-gfx] [PATCH v4 8/9] drivers/perf: open access for CAP_SYS_PERFMON privileged process

2020-01-17 Thread Will Deacon
On Wed, Dec 18, 2019 at 12:30:29PM +0300, Alexey Budankov wrote: > > Open access to monitoring for CAP_SYS_PERFMON privileged processes. > For backward compatibility reasons access to the monitoring remains open > for CAP_SYS_ADMIN privileged processes but CAP_SYS_ADMIN usage for secure > monitori

Re: [Intel-gfx] [PATCH 0/4] drm/i915/display: conversion to new logging macros.

2020-01-17 Thread Wambui Karuga
On Fri, 17 Jan 2020, Jani Nikula wrote: On Thu, 16 Jan 2020, Wambui Karuga wrote: This series converts the printk based logging macros in drm/i915/display/intel_display.c to the new struct drm_device based logging macros. This change was split into four for manageability and due to the size

Re: [Intel-gfx] [PATCH 1/4] drm/mst: Don't do atomic checks over disabled managers

2020-01-17 Thread Alex Deucher
On Fri, Jan 17, 2020 at 3:21 PM Lyude Paul wrote: > > Reviewed-by: Lyude Paul > Applied. I'll send a PR shortly. Thanks! Alex > On Thu, 2020-01-16 at 17:58 -0800, José Roberto de Souza wrote: > > When a main MST port is disconnected drivers should call > > drm_dp_mst_topology_mgr_set_mst() d

Re: [Intel-gfx] [PATCH 2/4] drm/mst: Some style improvements in drm_dp_mst_topology_mgr_set_mst()

2020-01-17 Thread Lyude Paul
JFYI: I'm going to go ahead and push this patch by itself to drm-misc-next since it applies cleanly, the other patches in this series don't depend on this, and I'm about to send out a patch that modifies the code around these hunks anyway. On Thu, 2020-01-16 at 17:58 -0800, José Roberto de Souza w

[Intel-gfx] [PATCH i-g-t] i915/gem_mmap_offset: Exercise many, many mmap_offset

2020-01-17 Thread Chris Wilson
Just keep on generating a new mmap_offset for the same old buffer, but for different handles and so exercise the scaling of the obj->mmo lists. Signed-off-by: Chris Wilson Cc: Abdiel Janulgue --- tests/i915/gem_mmap_offset.c | 66 1 file changed, 66 insertio

Re: [Intel-gfx] [PATCH i-g-t] scripts/trace.pl: Update for preempt-to-busy, heartbeats and timeslicing

2020-01-17 Thread Ye, Tony
Failed on parsing the trace log: Died at ./trace.pl line 296. Regards, Tony On 1/17/2020 10:49 AM, Tvrtko Ursulin wrote: From: Tvrtko Ursulin ... Signed-off-by: Tvrtko Ursulin Cc: Tony Ye --- Tony could you please check if this works for you and even send me some raw trace files you mi

Re: [Intel-gfx] [PATCH i-g-t] scripts/trace.pl: Update for preempt-to-busy, heartbeats and timeslicing

2020-01-17 Thread Ye, Tony
Failed on parsing the trace log: Died at ./trace.pl line 296. Raw data attached. Regards, Tony On 1/17/2020 10:49 AM, Tvrtko Ursulin wrote: From: Tvrtko Ursulin ... Signed-off-by: Tvrtko Ursulin Cc: Tony Ye --- Tony could you please check if this works for you and even send me some

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [v2] drm/i915: Keep track of request among the scheduling lists (rev3)

2020-01-17 Thread Patchwork
== Series Details == Series: series starting with [v2] drm/i915: Keep track of request among the scheduling lists (rev3) URL : https://patchwork.freedesktop.org/series/72048/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7748_full -> Patchwork_16108_full =

Re: [Intel-gfx] [PATCH v3 4/5] drm/i915: Don't use VBT for detecting DPCD backlight controls

2020-01-17 Thread Lyude Paul
On Fri, 2020-01-17 at 13:36 +0200, Jani Nikula wrote: > On Thu, 16 Jan 2020, Lyude Paul wrote: > > Despite the fact that the VBT appears to have a field for specifying > > that a system is equipped with a panel that supports standard VESA > > backlight controls over the DP AUX channel, so far ever

Re: [Intel-gfx] [PATCH 2/4] drm/mst: Some style improvements in drm_dp_mst_topology_mgr_set_mst()

2020-01-17 Thread Lyude Paul
Reviewed-by: Lyude Paul On Thu, 2020-01-16 at 17:58 -0800, José Roberto de Souza wrote: > Removing this lose code block and removing unnecessary bracket. > > Cc: Lyude Paul > Signed-off-by: José Roberto de Souza > --- > drivers/gpu/drm/drm_dp_mst_topology.c | 14 ++ > 1 file chang

Re: [Intel-gfx] [PATCH 1/4] drm/mst: Don't do atomic checks over disabled managers

2020-01-17 Thread Lyude Paul
Reviewed-by: Lyude Paul On Thu, 2020-01-16 at 17:58 -0800, José Roberto de Souza wrote: > When a main MST port is disconnected drivers should call > drm_dp_mst_topology_mgr_set_mst() disabling the MST manager, this > function will set manager mst_primary to NULL and it will cause the > crash bell

[Intel-gfx] [PATCH v3 11/12] drm/i915: Factor out HDCP shim functions from dp for use by dp_mst

2020-01-17 Thread Sean Paul
From: Sean Paul These functions are all the same for dp and dp_mst, so move them into a dedicated file for both sst and mst to use. Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20191203173638.94919-11-s...@poorly.run #v1 Link: https://patchwork.freedesktop.org

[Intel-gfx] [PATCH v3 08/12] drm/i915: Don't fully disable HDCP on a port if multiple pipes are using it

2020-01-17 Thread Sean Paul
From: Sean Paul This patch is required for HDCP over MST. If a port is being used for multiple HDCP streams, we don't want to fully disable HDCP on a port if one of them is disabled. Instead, we just disable the HDCP signalling on that particular pipe and exit early. The last pipe to disable HDCP

[Intel-gfx] [PATCH v3 10/12] drm/i915: Use ddi_update_pipe in intel_dp_mst

2020-01-17 Thread Sean Paul
From: Sean Paul In order to act upon content_protection property changes, we'll need to implement the .update_pipe() hook. We can re-use intel_ddi_update_pipe for this Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20191203173638.94919-10-s...@poorly.run #v1 Link

[Intel-gfx] [PATCH v3 12/12] drm/i915: Add HDCP 1.4 support for MST connectors

2020-01-17 Thread Sean Paul
From: Sean Paul Now that all the groundwork has been laid, we can turn on HDCP 1.4 over MST. Everything except for toggling the HDCP signalling and HDCP 2.2 support is the same as the DP case, so we'll re-use those callbacks Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patc

[Intel-gfx] [PATCH v3 01/12] drm/i915: Fix sha_text population code

2020-01-17 Thread Sean Paul
From: Sean Paul This patch fixes a few bugs: 1- We weren't taking into account sha_leftovers when adding multiple ksvs to sha_text. As such, we were or'ing the end of ksv[j - 1] with the beginning of ksv[j] 2- In the sha_leftovers == 2 and sha_leftovers == 3 case, bstatus was being pla

[Intel-gfx] [PATCH v3 06/12] drm/i915: Factor out hdcp->value assignments

2020-01-17 Thread Sean Paul
From: Sean Paul This is a bit of housecleaning for a future patch. Instead of sprinkling hdcp->value assignments and prop_work scheduling everywhere, introduce a function to do it for us. Reviewed-by: Ramalingam C Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20

[Intel-gfx] [PATCH v3 09/12] drm/i915: Support DP MST in enc_to_dig_port() function

2020-01-17 Thread Sean Paul
From: Sean Paul Although DP_MST fake encoders are not subclassed from digital ports, they are associated with them. Support these encoders. Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20191203173638.94919-9-s...@poorly.run #v1 Link: https://patchwork.freedesk

[Intel-gfx] [PATCH v3 07/12] drm/i915: Protect workers against disappearing connectors

2020-01-17 Thread Sean Paul
From: Sean Paul This patch adds some protection against connectors being destroyed before the HDCP workers are finished. For check_work, we do a synchronous cancel after the connector is unregistered which will ensure that it is finished before destruction. In the case of prop_work, we can't do

[Intel-gfx] [PATCH v3 04/12] drm/i915: Intercept Aksv writes in the aux hooks

2020-01-17 Thread Sean Paul
From: Sean Paul Instead of hand rolling the transfer ourselves in the hdcp hook, inspect aux messages and add the aksv flag in the aux transfer hook. IIRC, this was the original implementation and folks wanted this hack to be isolated to the hdcp code, which makes sense. However in testing an L

[Intel-gfx] [PATCH v3 02/12] drm/i915: Clear the repeater bit on HDCP disable

2020-01-17 Thread Sean Paul
From: Sean Paul On HDCP disable, clear the repeater bit. This ensures if we connect a non-repeater sink after a repeater, the bit is in the state we expect. Fixes: ee5e5e7a5e0f (drm/i915: Add HDCP framework + base implementation) Cc: Chris Wilson Cc: Ramalingam C Cc: Daniel Vetter Cc: Sean Pa

[Intel-gfx] [PATCH v3 05/12] drm/i915: Use the cpu_transcoder in intel_hdcp to toggle HDCP signalling

2020-01-17 Thread Sean Paul
From: Sean Paul Instead of using intel_dig_port's encoder pipe to determine which transcoder to toggle signalling on, use the cpu_transcoder field already stored in intel_hdmi. This is particularly important for MST. Suggested-by: Ville Syrjälä Reviewed-by: Ramalingam C Signed-off-by: Sean Pa

[Intel-gfx] [PATCH v3 03/12] drm/i915: WARN if HDCP signalling is enabled upon disable

2020-01-17 Thread Sean Paul
From: Sean Paul HDCP signalling should not be left on, WARN if it is Cc: Ville Syrjälä Cc: Daniel Vetter Reviewed-by: Ramalingam C Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20191212190230.188505-4-s...@poorly.run #v2 Changes in v2: - Added to the set in

[Intel-gfx] [PATCH v3 00/12] drm/i915: Add support for HDCP 1.4 over MST connectors

2020-01-17 Thread Sean Paul
From: Sean Paul Hey all, Here's v3, which addresses all review comments in v2. Sean Sean Paul (12): drm/i915: Fix sha_text population code drm/i915: Clear the repeater bit on HDCP disable drm/i915: WARN if HDCP signalling is enabled upon disable drm/i915: Intercept Aksv writes in the au

Re: [Intel-gfx] [PATCH v13 4/5] drm/i915: Manipulate DBuf slices properly

2020-01-17 Thread Matt Roper
On Fri, Jan 17, 2020 at 11:50:25AM +0200, Stanislav Lisovskiy wrote: > Start manipulating DBuf slices as a mask, > but not as a total number, as current approach > doesn't give us full control on all combinations > of slices, which we might need(like enabling S2 > only can't enabled by setting enab

Re: [Intel-gfx] [PATCH v13 5/5] drm/i915: Correctly map DBUF slices to pipes

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 11:50:26AM +0200, Stanislav Lisovskiy wrote: > Added proper DBuf slice mapping to correspondent > pipes, depending on pipe configuration as stated > in BSpec. > > v2: > - Remove unneeded braces > - Stop using macro for DBuf assignments as > it seems to reduce

[Intel-gfx] [PATCH i-g-t] scripts/trace.pl: Update for preempt-to-busy, heartbeats and timeslicing

2020-01-17 Thread Tvrtko Ursulin
From: Tvrtko Ursulin ... Signed-off-by: Tvrtko Ursulin Cc: Tony Ye --- Tony could you please check if this works for you and even send me some raw trace files you might have so I see how it looks? P.S. I don't yet guarantee the results are 100% reliable and truthful. Or that it won't crash. -

Re: [Intel-gfx] [PATCH v13 4/5] drm/i915: Manipulate DBuf slices properly

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 11:50:25AM +0200, Stanislav Lisovskiy wrote: > Start manipulating DBuf slices as a mask, > but not as a total number, as current approach > doesn't give us full control on all combinations > of slices, which we might need(like enabling S2 > only can't enabled by setting enab

Re: [Intel-gfx] [PATCH v13 3/5] drm/i915: Introduce parameterized DBUF_CTL

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 11:50:24AM +0200, Stanislav Lisovskiy wrote: > Now start using parameterized DBUF_CTL instead > of hardcoded, this would allow shorter access > functions when reading or storing entire state. > > Tried to implement it in a MMIO_PIPE manner, however > DBUF_CTL1 address is hi

[Intel-gfx] ✗ Fi.CI.BUILD: warning for series starting with [v3,1/2] drm/i915/userptr: add user_size limit check

2020-01-17 Thread Patchwork
== Series Details == Series: series starting with [v3,1/2] drm/i915/userptr: add user_size limit check URL : https://patchwork.freedesktop.org/series/72186/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/com

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v3,1/2] drm/i915/userptr: add user_size limit check

2020-01-17 Thread Patchwork
== Series Details == Series: series starting with [v3,1/2] drm/i915/userptr: add user_size limit check URL : https://patchwork.freedesktop.org/series/72186/ State : success == Summary == CI Bug Log - changes from CI_DRM_7764 -> Patchwork_16154 =

[Intel-gfx] [PATCH i-g-t 5/5] HAX: run gem_render_copy tests in BAT

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński --- tests/intel-ci/fast-feedback.testlist | 38 +++ 1 file changed, 38 insertions(+) diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist index 37a92b4e7..6465725b9 100644 --- a/tests/intel-ci/fast-feedbac

[Intel-gfx] [PATCH i-g-t 1/5] lib/intel_bufops: Introduce buffer operations

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński Different GENs supports different tile surfaces. Older GENs have HW fences to allow X / Y surface tiling / detiling. Newer GENs have to tile / detile such surface in software. To make test developer life easier this code adds buffer operations (short buf_ops) to use ap

[Intel-gfx] [PATCH i-g-t 2/5] lib/intel_batchbuffer: Add CCS width/height functions for Intel igt_buf

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński igt_buf has some fields which can be interpreted differently across vendors (ccs structure). Patch adds functions which are aware of meaning of this field. Signed-off-by: Zbigniew Kempczyński Cc: Chris Wilson Cc: Katarzyna Dec Reviewed-by: Chris Wilson --- lib/int

[Intel-gfx] [PATCH i-g-t 3/5] lib/rendercopy_bufmgr: Add rendercopy buffer manager

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński This is middle layer between render copy tests and buffer operations (buf_ops). Render copy tests uses libdrm so adding wrapper to independent buf_ops was necessary. Signed-off-by: Zbigniew Kempczyński Cc: Chris Wilson Cc: Katarzyna Dec Reviewed-by: Chris Wilson --

[Intel-gfx] [PATCH i-g-t 4/5] i915/gem_render_copy.c: Simplify code by switch to rendercopy bufmgr

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński Switch to rendercopy bufmgr to simplify working with tiled surfaces. Signed-off-by: Zbigniew Kempczyński Cc: Chris Wilson Cc: Katarzyna Dec --- tests/i915/gem_render_copy.c | 422 ++- 1 file changed, 116 insertions(+), 306 deletions(

[Intel-gfx] [PATCH] drm/i915/gt: Be paranoid and reset the GPU before release

2020-01-17 Thread Chris Wilson
Just in the very unlikely case we have not stopped the GPU before we return the pages being used by the GPU to the system, force a reset. Signed-off-by: Chris Wilson Reviewed-by: Matthew Auld --- drivers/gpu/drm/i915/gt/intel_engine_cs.c | 16 1 file changed, 16 insertions(+)

Re: [Intel-gfx] [PATCH] drm/i915/gt: Be paranoid and reset the GPU before release

2020-01-17 Thread Matthew Auld
On Fri, 17 Jan 2020 at 12:59, Chris Wilson wrote: > > Just in the very unlikely case we have not stopped the GPU before we > return the pages being used by the GPU to the system, force a reset. > > Signed-off-by: Chris Wilson Reviewed-by: Matthew Auld

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Be paranoid and reset the GPU before release

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gt: Be paranoid and reset the GPU before release URL : https://patchwork.freedesktop.org/series/72185/ State : success == Summary == CI Bug Log - changes from CI_DRM_7764 -> Patchwork_16153 Summary

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915/gt: Be paranoid and reset the GPU before release

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gt: Be paranoid and reset the GPU before release URL : https://patchwork.freedesktop.org/series/72185/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compile.h Kernel: ar

[Intel-gfx] [PATCH i-g-t 3/5] lib/rendercopy_bufmgr: Add rendercopy buffer manager

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński This is middle layer between render copy tests and buffer operations (buf_ops). Render copy tests uses libdrm so adding wrapper to independent buf_ops was necessary. Signed-off-by: Zbigniew Kempczyński Cc: Chris Wilson Cc: Katarzyna Dec --- lib/Makefile.sources

[Intel-gfx] [PATCH i-g-t 1/5] lib/intel_bufops: Introduce buffer operations

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński Different GENs supports different tile surfaces. Older GENs have HW fences to allow X / Y surface tiling / detiling. Newer GENs have to tile / detile such surface in software. To make test developer life easier this code adds buffer operations (short buf_ops) to use ap

[Intel-gfx] [PATCH i-g-t 2/5] lib/intel_batchbuffer: Add CCS width/height functions for Intel igt_buf

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński igt_buf has some fields which can be interpreted differently across vendors (ccs structure). Patch adds functions which are aware of meaning of this field. Signed-off-by: Zbigniew Kempczyński Cc: Chris Wilson Cc: Katarzyna Dec --- lib/intel_batchbuffer.c | 47 +

[Intel-gfx] [PATCH i-g-t 4/5] i915/gem_render_copy.c: Simplify code by switch to rendercopy bufmgr

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński Switch to rendercopy bufmgr to simplify working with tiled surfaces. Signed-off-by: Zbigniew Kempczyński Cc: Chris Wilson Cc: Katarzyna Dec --- tests/i915/gem_render_copy.c | 423 ++- 1 file changed, 117 insertions(+), 306 deletions(

[Intel-gfx] [PATCH i-g-t 5/5] HAX: run gem_render_copy tests in BAT

2020-01-17 Thread Chris Wilson
From: Zbigniew Kempczyński --- tests/intel-ci/fast-feedback.testlist | 38 +++ 1 file changed, 38 insertions(+) diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist index 37a92b4e7..6465725b9 100644 --- a/tests/intel-ci/fast-feedbac

Re: [Intel-gfx] [PATCH] drm/i915: Check require bandwidth did not exceed LSPCON limitation

2020-01-17 Thread Lee, Shawn C
On Fri, Jan 17, 2020, Ville Syrjälä wrote: >On Fri, Jan 17, 2020 at 09:47:17PM +0800, Lee Shawn C wrote: >> While mode setting, driver would calculate mode rate based on >> resolution and bpp. And choose the best bpp that did not exceed DP >> bandwidtd. >> >> But LSPCON had more restriction du

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Report the currently active execlists request (rev2)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gt: Report the currently active execlists request (rev2) URL : https://patchwork.freedesktop.org/series/72179/ State : success == Summary == CI Bug Log - changes from CI_DRM_7762 -> Patchwork_16152 Summ

[Intel-gfx] ✗ Fi.CI.BUILD: warning for drm/i915/gt: Report the currently active execlists request (rev2)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gt: Report the currently active execlists request (rev2) URL : https://patchwork.freedesktop.org/series/72179/ State : warning == Summary == CALLscripts/checksyscalls.sh CALLscripts/atomic/check-atomics.sh CHK include/generated/compile.h Ke

Re: [Intel-gfx] ✗ Fi.CI.BUILD: failure for drm/i915: conversion to new drm logging macros.

2020-01-17 Thread Jani Nikula
On Thu, 16 Jan 2020, Patchwork wrote: > == Series Details == > > Series: drm/i915: conversion to new drm logging macros. > URL : https://patchwork.freedesktop.org/series/72096/ > State : failure > > == Summary == > > Applying: drm/i915/atomic: use new logging macros for debug > Applying: drm/i91

Re: [Intel-gfx] [PATCH 0/5] Use new logging macros in drm/i915

2020-01-17 Thread Jani Nikula
On Thu, 09 Jan 2020, Wambui Karuga wrote: > This patchset continues the conversion to using the new struct > drm_device based logging macros in drm/i915. Pushed to drm-intel-next-queued, thanks for the patches. BR, Jani. > > Wambui Karuga (5): > drm/i915: conversion to new logging macros in i

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gt: Report the currently active execlists request (rev2)

2020-01-17 Thread Patchwork
== Series Details == Series: drm/i915/gt: Report the currently active execlists request (rev2) URL : https://patchwork.freedesktop.org/series/72179/ State : warning == Summary == $ dim checkpatch origin/drm-tip df4d12c02eca drm/i915/gt: Report the currently active execlists request -:16: ERROR

Re: [Intel-gfx] [PATCH 8/9] drm/i915/bios: check port presence based on child device

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 05:28:23PM +0200, Jani Nikula wrote: > On Fri, 17 Jan 2020, Ville Syrjälä wrote: > > On Fri, Jan 17, 2020 at 05:12:38PM +0200, Ville Syrjälä wrote: > >> On Fri, Jan 17, 2020 at 04:29:28PM +0200, Jani Nikula wrote: > >> > Affects only two calls in output setup, and ddi init

[Intel-gfx] [PATCH 1/2] drm/i915/gvt: Wean gvt off dev_priv->engine[]

2020-01-17 Thread Chris Wilson
Stop trying to escape out of the gvt layer to find the engine that we initially setup for use with gvt. Record the engines during initialisation and use them henceforth. add/remove: 1/4 grow/shrink: 22/28 up/down: 341/-1410 (-1069) Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gvt/cmd_pa

[Intel-gfx] [PATCH 2/2] drm/i915/gvt: Wean gvt off using dev_priv

2020-01-17 Thread Chris Wilson
Teach gvt to use intel_gt directly as it currently assumes direct HW access. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gvt/aperture_gm.c | 77 +++-- drivers/gpu/drm/i915/gvt/cfg_space.c| 4 +- drivers/gpu/drm/i915/gvt/debugfs.c | 9 ++- drivers/gpu/drm/i915

Re: [Intel-gfx] [PATCH 8/9] drm/i915/bios: check port presence based on child device

2020-01-17 Thread Jani Nikula
On Fri, 17 Jan 2020, Ville Syrjälä wrote: > On Fri, Jan 17, 2020 at 05:12:38PM +0200, Ville Syrjälä wrote: >> On Fri, Jan 17, 2020 at 04:29:28PM +0200, Jani Nikula wrote: >> > Affects only two calls in output setup, and ddi init will check the >> > features in more fine grained way. >> > >> > Thi

Re: [Intel-gfx] [PATCH 8/9] drm/i915/bios: check port presence based on child device

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 05:13:25PM +0200, Ville Syrjälä wrote: > On Fri, Jan 17, 2020 at 05:12:38PM +0200, Ville Syrjälä wrote: > > On Fri, Jan 17, 2020 at 04:29:28PM +0200, Jani Nikula wrote: > > > Affects only two calls in output setup, and ddi init will check the > > > features in more fine grai

Re: [Intel-gfx] [PATCH 2/2] drm/i915/gvt: Wean gvt off dev_priv->engine[]

2020-01-17 Thread Jani Nikula
On Mon, 21 Oct 2019, Chris Wilson wrote: > Quoting Zhenyu Wang (2019-10-21 06:37:24) >> On 2019.10.16 19:39:02 +0100, Chris Wilson wrote: >> > Stop trying to escape out of the gvt layer to find the engine that we >> > initially setup for use with gvt. Record the engines during initialisation >> >

Re: [Intel-gfx] [PATCH 9/9] drm/i915: use intel_bios_is_port_present()

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 04:29:29PM +0200, Jani Nikula wrote: > Don't access i915->vbt.ddi_port_info[] directly. > > Signed-off-by: Jani Nikula Reviewed-by: Ville Syrjälä > --- > drivers/gpu/drm/i915/display/intel_combo_phy.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > dif

Re: [Intel-gfx] [PATCH 8/9] drm/i915/bios: check port presence based on child device

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 05:12:38PM +0200, Ville Syrjälä wrote: > On Fri, Jan 17, 2020 at 04:29:28PM +0200, Jani Nikula wrote: > > Affects only two calls in output setup, and ddi init will check the > > features in more fine grained way. > > > > This will make future changes easier. > > > > Signed

Re: [Intel-gfx] [PATCH 8/9] drm/i915/bios: check port presence based on child device

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 04:29:28PM +0200, Jani Nikula wrote: > Affects only two calls in output setup, and ddi init will check the > features in more fine grained way. > > This will make future changes easier. > > Signed-off-by: Jani Nikula > --- > drivers/gpu/drm/i915/display/intel_bios.c | 4

Re: [Intel-gfx] [PATCH 7/9] drm/i915/bios: add intel_bios_port_supports_*()

2020-01-17 Thread Ville Syrjälä
On Fri, Jan 17, 2020 at 04:29:27PM +0200, Jani Nikula wrote: > Don't access i915->vbt.ddi_port_info[] directly. 3-7 Reviewed-by: Ville Syrjälä > > Signed-off-by: Jani Nikula > --- > drivers/gpu/drm/i915/display/intel_bios.c | 26 +++ > drivers/gpu/drm/i915/display/intel_bi

  1   2   >