[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Enabling content-type setting for HDMI displays. (rev5)

2018-04-20 Thread Patchwork
== Series Details == Series: Enabling content-type setting for HDMI displays. (rev5) URL : https://patchwork.freedesktop.org/series/41876/ State : warning == Summary == $ dim checkpatch origin/drm-tip a7139e8d18c4 drm: content-type property for HDMI connector -:115: CHECK:PARENTHESIS_ALIGNMENT

[Intel-gfx] ✗ Fi.CI.BAT: failure for Enabling content-type setting for HDMI displays. (rev5)

2018-04-20 Thread Patchwork
== Series Details == Series: Enabling content-type setting for HDMI displays. (rev5) URL : https://patchwork.freedesktop.org/series/41876/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4070 -> Patchwork_8758 = == Summary - FAILURE == Serious unknown changes coming with P

Re: [Intel-gfx] [PATCH 02/22] drm/i915/icl: Enable Sampler DFR

2018-04-20 Thread Sagar Arun Kamble
On 4/13/2018 9:30 PM, Oscar Mateo wrote: Sampler Dynamic Frequency Rebalancing (DFR) aims to reduce Sampler power by dynamically changing its clock frequency in low-throughput conditions. This patches enables it by default on Gen11. v2: Wrong operation to clear the bit (Praveen) v3: Rebased on

Re: [Intel-gfx] [PATCH] drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers

2018-04-20 Thread Daniel Vetter
On Thu, Apr 19, 2018 at 03:43:19PM +0200, Hans de Goede wrote: > Hi, > > On 05-04-18 15:26, Daniel Vetter wrote: > > On Thu, Apr 5, 2018 at 1:47 PM, Hans de Goede wrote: > > > Hi, > > > > > > > > > On 05-04-18 09:14, Daniel Vetter wrote: > > > > > > > > On Fri, Mar 30, 2018 at 02:27:15PM +0200

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm: Drop DRM_CONTROL_ALLOW from ioctls

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/4] drm: Drop DRM_CONTROL_ALLOW from ioctls URL : https://patchwork.freedesktop.org/series/42012/ State : warning == Summary == $ dim checkpatch origin/drm-tip 6f5ce675325d drm: Drop DRM_CONTROL_ALLOW from ioctls -:8: ERROR:GIT_COMMIT_ID: Ple

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/4] drm: Drop DRM_CONTROL_ALLOW from ioctls

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/4] drm: Drop DRM_CONTROL_ALLOW from ioctls URL : https://patchwork.freedesktop.org/series/42012/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4070 -> Patchwork_8759 = == Summary - SUCCESS == No regressions found. E

Re: [Intel-gfx] [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake

2018-04-20 Thread Jani Nikula
On Fri, 20 Apr 2018, Ian W MORRISON wrote: > I've performed backport testing and some additional analysis as follows: What testing did you do beyond booting? Did you run igt? BR, Jani. -- Jani Nikula, Intel Open Source Technology Center ___ Intel-gf

Re: [Intel-gfx] [PATCH v5 1/2] drm: content-type property for HDMI connector

2018-04-20 Thread Daniel Vetter
On Thu, Apr 19, 2018 at 03:38:53PM +0300, StanLis wrote: > From: Stanislav Lisovskiy > > Added content_type property to drm_connector_state > in order to properly handle external HDMI TV content-type setting. > > v2: > * Moved helper function which attaches content type property >to the drm

Re: [Intel-gfx] [PATCH] drm/i915: Wait for vblank after register read

2018-04-20 Thread Jani Nikula
On Fri, 20 Apr 2018, Mika Kahola wrote: > On Thu, 2018-04-19 at 17:09 +0300, Jani Nikula wrote: >> On Wed, 18 Apr 2018, Mika Kahola wrote: >> > >> > When reading out CRC's we  wait for a vblank on >> > intel_dp_sink_crc_start() >> > function. When we start reading out CRC's in intel_dp_sink_crc(

Re: [Intel-gfx] [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake

2018-04-20 Thread Ian W MORRISON
On 20 April 2018 at 17:50, Jani Nikula wrote: > On Fri, 20 Apr 2018, Ian W MORRISON wrote: >> I've performed backport testing and some additional analysis as follows: > > What testing did you do beyond booting? Did you run igt? > > BR, > Jani. I did some basic testing including running a 4K vide

Re: [Intel-gfx] [PATCH v5 1/2] drm: content-type property for HDMI connector

2018-04-20 Thread Lisovskiy, Stanislav
From: Daniel Vetter [daniel.vet...@ffwll.ch] on behalf of Daniel Vetter [dan...@ffwll.ch] > The property documentation to tie all the bits together (property, helper, > internals) is missing. It should be in > https://dri.freedesktop.org/docs/drm/gpu/drm

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/4] drm: Drop DRM_CONTROL_ALLOW from ioctls

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/4] drm: Drop DRM_CONTROL_ALLOW from ioctls URL : https://patchwork.freedesktop.org/series/42012/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4070_full -> Patchwork_8759_full = == Summary - WARNING == Minor unknown ch

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Enabling content-type setting for HDMI displays. (rev5)

2018-04-20 Thread Patchwork
== Series Details == Series: Enabling content-type setting for HDMI displays. (rev5) URL : https://patchwork.freedesktop.org/series/41876/ State : warning == Summary == $ dim checkpatch origin/drm-tip c2999e17c3c9 drm: content-type property for HDMI connector -:115: CHECK:PARENTHESIS_ALIGNMENT

Re: [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Enable display WA#1183 from its correct spot

2018-04-20 Thread Imre Deak
On Thu, Apr 19, 2018 at 11:07:31PM +, Patchwork wrote: > == Series Details == > > Series: drm/i915: Enable display WA#1183 from its correct spot > URL : https://patchwork.freedesktop.org/series/41983/ > State : success Thanks for the review, pushed it to -dinq. > > == Summary == > > = CI

[Intel-gfx] ✗ Fi.CI.BAT: failure for Enabling content-type setting for HDMI displays. (rev5)

2018-04-20 Thread Patchwork
== Series Details == Series: Enabling content-type setting for HDMI displays. (rev5) URL : https://patchwork.freedesktop.org/series/41876/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4070 -> Patchwork_8760 = == Summary - FAILURE == Serious unknown changes coming with P

Re: [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Remove skl dc6 enable/disable functions

2018-04-20 Thread Imre Deak
On Tue, Apr 17, 2018 at 01:28:48PM +, Patchwork wrote: > == Series Details == > > Series: drm/i915: Remove skl dc6 enable/disable functions > URL : https://patchwork.freedesktop.org/series/41803/ > State : success Thanks for the patch, pushed it to -dinq. > > == Summary == > > = CI Bug L

[Intel-gfx] [GVT-g] [ANNOUNCE] 2018-Q1 release of KVMGT (Intel GVT-g for KVM)

2018-04-20 Thread Xu, Terrence
Hi all, We are pleased to announce an update of Intel GVT-g for KVM. Intel GVT-g for KVM (a.k.a. KVMGT) is a full GPU virtualization solution with mediated pass-through, starting from 5th generation Intel Core(TM) processors with Intel processor graphics. A virtual GPU instance is maintain

[Intel-gfx] [PATCH] drm/i915/psr : Add psr1 live status

2018-04-20 Thread vathsala nagaraju
From: Vathsala Nagaraju Prints live state of psr1.Extending the existing PSR2 live state function to cover psr1. Tested on KBL with psr2 and psr1 panel. Cc: Rodrigo Vivi Cc: Dhinakaran Pandiyan Signed-off-by: Vathsala Nagaraju --- drivers/gpu/drm/i915/i915_debugfs.c | 68 ++

[Intel-gfx] [GVT-g] [ANNOUNCE] 2018-Q1 release of XenGT (Intel GVT-g for Xen)

2018-04-20 Thread Xu, Terrence
Hi all, We are pleased to announce an update of Intel GVT-g for Xen. Intel GVT-g is a full GPU virtualization solution with mediated pass-through, starting from 4th generation Intel Core(TM) processors with Intel processor graphics. A virtual GPU instance is maintained for each VM, with par

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/psr : Add psr1 live status

2018-04-20 Thread Patchwork
== Series Details == Series: drm/i915/psr : Add psr1 live status URL : https://patchwork.freedesktop.org/series/42021/ State : warning == Summary == $ dim checkpatch origin/drm-tip ea1d0148dd9e drm/i915/psr : Add psr1 live status -:103: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open

[Intel-gfx] [PATCH] drm/i915: Use ktime on wait_for

2018-04-20 Thread Mika Kuoppala
We use jiffies to determine when wait expires. However Imre did find out that jiffies can and will do a >1 increments on certain situations [1]. When this happens in a wait_for loop, we return timeout errorneously much earlier than what the real wallclock would say. We can't afford our waits to ti

[Intel-gfx] [PATCH] drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers v2

2018-04-20 Thread Hans de Goede
Before this commit the WaSkipStolenMemoryFirstPage workaround code was skipping the first 4k by passing 4096 as start of the address range passed to drm_mm_init(). This means that calling drm_mm_reserve_node() to try and reserve the firmware framebuffer so that we can inherit it would always fail,

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/psr : Add psr1 live status

2018-04-20 Thread Patchwork
== Series Details == Series: drm/i915/psr : Add psr1 live status URL : https://patchwork.freedesktop.org/series/42021/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072 -> Patchwork_8761 = == Summary - WARNING == Minor unknown changes coming with Patchwork_8761 need to

Re: [Intel-gfx] [PATCH] drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers

2018-04-20 Thread Hans de Goede
Hi, On 20-04-18 09:27, Daniel Vetter wrote: On Thu, Apr 19, 2018 at 03:43:19PM +0200, Hans de Goede wrote: Hi, On 05-04-18 15:26, Daniel Vetter wrote: On Thu, Apr 5, 2018 at 1:47 PM, Hans de Goede wrote: Hi, On 05-04-18 09:14, Daniel Vetter wrote: On Fri, Mar 30, 2018 at 02:27:15PM +020

Re: [Intel-gfx] [PATCH] drm/i915: Use ktime on wait_for

2018-04-20 Thread Sagar Arun Kamble
On 4/20/2018 3:24 PM, Mika Kuoppala wrote: We use jiffies to determine when wait expires. However Imre did find out that jiffies can and will do a >1 increments on certain situations [1]. When this happens in a wait_for loop, we return timeout errorneously much earlier than what the real wallcl

Re: [Intel-gfx] [PATCH] drm/i915: Use ktime on wait_for

2018-04-20 Thread Chris Wilson
Quoting Mika Kuoppala (2018-04-20 10:54:26) > We use jiffies to determine when wait expires. However > Imre did find out that jiffies can and will do a >1 > increments on certain situations [1]. When this happens > in a wait_for loop, we return timeout errorneously > much earlier than what the real

Re: [Intel-gfx] [PATCH v2] drm/i915: Fix drm:intel_enable_lvds ERROR message in kernel log

2018-04-20 Thread Jani Nikula
On Thu, 19 Apr 2018, Ville Syrjala wrote: > From: Florent Flament > > Fix `[drm:intel_enable_lvds] *ERROR* timed out waiting for panel to > power on` in kernel log at boot time. > > Toshiba Satellite Z930 laptops needs between 1 and 2 seconds to power > on its screen during Intel i915 DRM initial

Re: [Intel-gfx] [PATCH] drm/i915: Use ktime on wait_for

2018-04-20 Thread Chris Wilson
Quoting Sagar Arun Kamble (2018-04-20 11:23:50) > > > On 4/20/2018 3:24 PM, Mika Kuoppala wrote: > > We use jiffies to determine when wait expires. However > > Imre did find out that jiffies can and will do a >1 > > increments on certain situations [1]. When this happens > > in a wait_for loop, w

[Intel-gfx] ✓ Fi.CI.BAT: success for Enabling content-type setting for HDMI displays. (rev5)

2018-04-20 Thread Patchwork
== Series Details == Series: Enabling content-type setting for HDMI displays. (rev5) URL : https://patchwork.freedesktop.org/series/41876/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4070 -> Patchwork_8760 = == Summary - SUCCESS == No regressions found. External URL

Re: [Intel-gfx] [PATCH] drm/i915: Wait for vblank after register read

2018-04-20 Thread Mika Kahola
On Fri, 2018-04-20 at 11:22 +0300, Jani Nikula wrote: > On Fri, 20 Apr 2018, Mika Kahola wrote: > > > > On Thu, 2018-04-19 at 17:09 +0300, Jani Nikula wrote: > > > > > > On Wed, 18 Apr 2018, Mika Kahola wrote: > > > > > > > > > > > > When reading out CRC's we  wait for a vblank on > > > > int

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/psr : Add psr1 live status

2018-04-20 Thread Patchwork
== Series Details == Series: drm/i915/psr : Add psr1 live status URL : https://patchwork.freedesktop.org/series/42021/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072 -> Patchwork_8761 = == Summary - SUCCESS == No regressions found. External URL: https://patchwork

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Use ktime on wait_for

2018-04-20 Thread Patchwork
== Series Details == Series: drm/i915: Use ktime on wait_for URL : https://patchwork.freedesktop.org/series/42023/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072 -> Patchwork_8762 = == Summary - SUCCESS == No regressions found. External URL: https://patchwork.fre

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers (rev3)

2018-04-20 Thread Patchwork
== Series Details == Series: drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers (rev3) URL : https://patchwork.freedesktop.org/series/40929/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072 -> Patchwork_8763 = == Summary - SUCCESS == No regr

Re: [Intel-gfx] [PATCH] drm/i915: Use ktime on wait_for

2018-04-20 Thread Sagar Arun Kamble
On 4/20/2018 4:09 PM, Chris Wilson wrote: Quoting Sagar Arun Kamble (2018-04-20 11:23:50) On 4/20/2018 3:24 PM, Mika Kuoppala wrote: We use jiffies to determine when wait expires. However Imre did find out that jiffies can and will do a >1 increments on certain situations [1]. When this happ

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/psr : Add psr1 live status

2018-04-20 Thread Patchwork
== Series Details == Series: drm/i915/psr : Add psr1 live status URL : https://patchwork.freedesktop.org/series/42021/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072_full -> Patchwork_8761_full = == Summary - WARNING == Minor unknown changes coming with Patchwork_876

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Use ktime on wait_for

2018-04-20 Thread Patchwork
== Series Details == Series: drm/i915: Use ktime on wait_for URL : https://patchwork.freedesktop.org/series/42023/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072_full -> Patchwork_8762_full = == Summary - WARNING == Minor unknown changes coming with Patchwork_8762_fu

Re: [Intel-gfx] [PATCH] drm/i915: Use ktime on wait_for

2018-04-20 Thread Mika Kuoppala
Chris Wilson writes: > Quoting Sagar Arun Kamble (2018-04-20 11:23:50) >> >> >> On 4/20/2018 3:24 PM, Mika Kuoppala wrote: >> > We use jiffies to determine when wait expires. However >> > Imre did find out that jiffies can and will do a >1 >> > increments on certain situations [1]. When this ha

Re: [Intel-gfx] [PATCH] drm/i915: Use ktime on wait_for

2018-04-20 Thread Imre Deak
On Fri, Apr 20, 2018 at 11:27:55AM +0100, Chris Wilson wrote: > Quoting Mika Kuoppala (2018-04-20 10:54:26) > > We use jiffies to determine when wait expires. However > > Imre did find out that jiffies can and will do a >1 > > increments on certain situations [1]. When this happens > > in a wait_fo

Re: [Intel-gfx] [PATCH] drm/i915: Add NV12 support to intel_framebuffer_init

2018-04-20 Thread kbuild test robot
Hi Chandra, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on v4.17-rc1 next-20180420] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com

[Intel-gfx] [PATCH v11 06/11] drm: Add DRM client cap for aspect-ratio

2018-04-20 Thread Nautiyal, Ankit K
From: Ankit Nautiyal To enable aspect-ratio support in DRM, blindly exposing the aspect ratio information along with mode, can break things in existing user-spaces which have no intention or support to use this aspect ratio information. To avoid this, a new drm client cap is required to enable a

[Intel-gfx] [PATCH v11 09/11] drm: Expose modes with aspect ratio, only if requested

2018-04-20 Thread Nautiyal, Ankit K
From: Ankit Nautiyal We parse the EDID and add all the modes in the connector's modelist. This adds CEA modes with aspect ratio information too, regardless of whether user space requested this information or not. This patch prunes the modes with aspect-ratio information, from a connector's model

[Intel-gfx] [PATCH 4/4] drm/i915: Split i915_gem_timeline into individual timelines

2018-04-20 Thread Chris Wilson
We need to move to a more flexible timeline that doesn't assume one fence context per engine, and so allow for a single timeline to be used across a combination of engines. This means that preallocating a fence context per engine is now a hindrance, and so we want to introduce the singular timeline

[Intel-gfx] ✓ Fi.CI.IGT: success for Enabling content-type setting for HDMI displays. (rev5)

2018-04-20 Thread Patchwork
== Series Details == Series: Enabling content-type setting for HDMI displays. (rev5) URL : https://patchwork.freedesktop.org/series/41876/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4070_full -> Patchwork_8760_full = == Summary - WARNING == Minor unknown changes comin

[Intel-gfx] [PATCH v11 03/11] drm/edid: Fix cea mode aspect ratio handling

2018-04-20 Thread Nautiyal, Ankit K
From: Ville Syrjälä commit 6dffd431e229 ("drm: Add aspect ratio parsing in DRM layer") cause us to not send out any VICs in the AVI infoframes. That commit was since reverted, but if and when we add aspect ratio handing back we need to be more careful. Let's handle this by considering the aspect

[Intel-gfx] [PATCH v11 08/11] drm: Handle aspect ratio info in legacy and atomic modeset paths

2018-04-20 Thread Nautiyal, Ankit K
From: Ankit Nautiyal If the user-space does not support aspect-ratio, and requests for a modeset with mode having aspect ratio bits set, then the given user-mode must be rejected. Secondly, while preparing a user-mode from kernel mode, the aspect-ratio info must not be given, if aspect-ratio is n

[Intel-gfx] [PATCH v11 04/11] drm/edid: Don't send bogus aspect ratios in AVI infoframes

2018-04-20 Thread Nautiyal, Ankit K
From: Ville Syrjälä If the user mode would specify an aspect ratio other than 4:3 or 16:9 we now silently ignore it. Maybe a better apporoach is to return an error? Let's try that. Also we must be careful that we don't try to send illegal picture aspect in the infoframe as it's only capable of s

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers (rev3)

2018-04-20 Thread Patchwork
== Series Details == Series: drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers (rev3) URL : https://patchwork.freedesktop.org/series/40929/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4072_full -> Patchwork_8763_full = == Summary - FAILURE ==

[Intel-gfx] [PATCH 2/2] drm/i915: Add compiler barrier to wait_for

2018-04-20 Thread Mika Kuoppala
We need to be careful to not let compiler evaluate the expiration and the operation on it's terms. Document and enforce that COND will be evaluated before checking timeout expiration. Suggested-by: Chris Wilson Cc: Chris Wilson Signed-off-by: Mika Kuoppala --- drivers/gpu/drm/i915/intel_drv.h

[Intel-gfx] [PATCH v11 01/11] drm/modes: Introduce drm_mode_match()

2018-04-20 Thread Nautiyal, Ankit K
From: Ville Syrjälä Make mode matching less confusing by allowing the caller to specify which parts of the modes should match via some flags. Signed-off-by: Ville Syrjälä Reviewed-by: Shashank Sharma --- drivers/gpu/drm/drm_modes.c | 134 ++-- include/d

[Intel-gfx] [PATCH v11 05/11] video/hdmi: Reject illegal picture aspect ratios

2018-04-20 Thread Nautiyal, Ankit K
From: Ville Syrjälä AVI infoframe can only carry none, 4:3, or 16:9 picture aspect ratios. Return an error if the user asked for something different. Cc: Shashank Sharma Cc: "Lin, Jia" Cc: Akashdeep Sharma Cc: Jim Bride Cc: Jose Abreu Cc: Daniel Vetter Cc: Emil Velikov Cc: Thierry Reding

[Intel-gfx] [PATCH 1/2] drm/i915: Use ktime on wait_for

2018-04-20 Thread Mika Kuoppala
We use jiffies to determine when wait expires. However Imre did find out that jiffies can and will do a >1 increments on certain situations [1]. When this happens in a wait_for loop, we return timeout errorneously much earlier than what the real wallclock would say. We can't afford our waits to ti

[Intel-gfx] [PATCH 3/4] drm/i915: Move timeline from GTT to ring

2018-04-20 Thread Chris Wilson
In the future, we want to move a request between engines. To achieve this, we first realise that we have two timelines in effect here. The first runs through the GTT is required for ordering vma access, which is tracked currently by engine. The second is implied by sequential execution of commands

Re: [Intel-gfx] [PATCH v11 06/11] drm: Add DRM client cap for aspect-ratio

2018-04-20 Thread Ville Syrjälä
On Fri, Apr 20, 2018 at 07:01:46PM +0530, Nautiyal, Ankit K wrote: > From: Ankit Nautiyal > > To enable aspect-ratio support in DRM, blindly exposing the aspect > ratio information along with mode, can break things in existing > user-spaces which have no intention or support to use this aspect >

[Intel-gfx] [PATCH v11 02/11] drm/edid: Use drm_mode_match_no_clocks_no_stereo() for consistentcy

2018-04-20 Thread Nautiyal, Ankit K
From: Ville Syrjälä Use drm_mode_equal_no_clocks_no_stereo() in drm_match_hdmi_mode_clock_tolerance() for consistency as we also use it in drm_match_hdmi_mode() and the cea mode matching functions. This doesn't actually change anything since the input mode comes from detailed timings and we matc

Re: [Intel-gfx] [PATCH] drm/i915/icl: Adjust BSD2 semantics to mean any second VCS instance

2018-04-20 Thread Bloomfield, Jon
> -Original Message- > From: Tvrtko Ursulin > Sent: Wednesday, April 18, 2018 2:34 AM > To: Intel-gfx@lists.freedesktop.org > Cc: tursu...@ursulin.net; Ursulin, Tvrtko ; Chris > Wilson ; Bloomfield, Jon > ; Ye, Tony > Subject: [PATCH] drm/i915/icl: Adjust BSD2 semantics to mean any second

[Intel-gfx] [PATCH v11 07/11] drm: Add helper functions to handle aspect-ratio flag bits

2018-04-20 Thread Nautiyal, Ankit K
From: Ankit Nautiyal This patch adds helper functions for determining if aspect-ratio is expected in user-mode and for allowing/disallowing the aspect-ratio, if its not expected. Signed-off-by: Ankit Nautiyal --- drivers/gpu/drm/drm_modes.c | 47 + i

[Intel-gfx] [PATCH v11 10/11] drm: Add aspect ratio parsing in DRM layer

2018-04-20 Thread Nautiyal, Ankit K
From: "Sharma, Shashank" Current DRM layer functions don't parse aspect ratio information while converting a user mode->kernel mode or vice versa. This causes modeset to pick mode with wrong aspect ratio, eventually causing failures in HDMI compliance test cases, due to wrong VIC. This patch add

[Intel-gfx] [PATCH 1/4] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-20 Thread Chris Wilson
In commit 9b6586ae9f6b ("drm/i915: Keep a global seqno per-engine"), we moved from a global inflight counter to per-engine counters in the hope that will be easy to run concurrently in future. However, with the advent of the desire to move requests between engines, we do need a global counter to pr

[Intel-gfx] [PATCH 2/4] drm/i915: Retire requests along rings

2018-04-20 Thread Chris Wilson
In the next patch, rings are the central timeline as requests may jump between engines. Therefore in the future as we retire in order along the engine timeline, we may retire out-of-order within a ring (as the ring now occurs along multiple engines), leading to much hilarity in miscomputing the pos

[Intel-gfx] [PATCH v11 11/11] drm: Add and handle new aspect ratios in DRM layer

2018-04-20 Thread Nautiyal, Ankit K
From: "Sharma, Shashank" HDMI 2.0/CEA-861-F introduces two new aspect ratios: - 64:27 - 256:135 This patch: - Adds new DRM flags for to represent these new aspect ratios. - Adds new cases to handle these aspect ratios while converting from user->kernel mode or vise versa. This patch was once

[Intel-gfx] [PATCH v11 00/11] Aspect ratio support in DRM layer

2018-04-20 Thread Nautiyal, Ankit K
From: Ankit Nautiyal This patch series is a re-attempt to enable aspect ratio support in DRM layer. Currently the aspect ratio information gets lost in translation during a user->kernel mode or vice versa. The old patch series (https://pw-emeril.freedesktop.org/series/10850/) had 4 patches, out

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Add compiler barrier to wait_for

2018-04-20 Thread Chris Wilson
Quoting Mika Kuoppala (2018-04-20 14:45:50) > We need to be careful to not let compiler evaluate > the expiration and the operation on it's terms. > > Document and enforce that COND will be evaluated > before checking timeout expiration. > > Suggested-by: Chris Wilson > Cc: Chris Wilson > Signe

Re: [Intel-gfx] [PATCH] drm/i915: Add NV12 support to intel_framebuffer_init

2018-04-20 Thread kbuild test robot
Hi Chandra, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on v4.17-rc1 next-20180420] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com

Re: [Intel-gfx] [PATCH v11 07/11] drm: Add helper functions to handle aspect-ratio flag bits

2018-04-20 Thread Ville Syrjälä
On Fri, Apr 20, 2018 at 07:01:47PM +0530, Nautiyal, Ankit K wrote: > From: Ankit Nautiyal > > This patch adds helper functions for determining if aspect-ratio is > expected in user-mode and for allowing/disallowing the aspect-ratio, > if its not expected. > > Signed-off-by: Ankit Nautiyal > ---

Re: [Intel-gfx] [PATCH v11 09/11] drm: Expose modes with aspect ratio, only if requested

2018-04-20 Thread Ville Syrjälä
On Fri, Apr 20, 2018 at 07:01:49PM +0530, Nautiyal, Ankit K wrote: > From: Ankit Nautiyal > > We parse the EDID and add all the modes in the connector's modelist. > This adds CEA modes with aspect ratio information too, regardless of > whether user space requested this information or not. > > Th

Re: [Intel-gfx] [PATCH v5 1/2] drm: content-type property for HDMI connector

2018-04-20 Thread Daniel Vetter
On Fri, Apr 20, 2018 at 08:31:56AM +, Lisovskiy, Stanislav wrote: > > > From: Daniel Vetter [daniel.vet...@ffwll.ch] on behalf of Daniel Vetter > [dan...@ffwll.ch] > > > The property documentation to tie all the bits together (property, helper, > > in

[Intel-gfx] [PATCH v12 1/3] drm/i915: Implement WaProgramMgsrForCorrectSliceSpecificMmioReads

2018-04-20 Thread Yunwei Zhang
WaProgramMgsrForCorrectSliceSpecificMmioReads dictate that before any MMIO read into Slice/Subslice specific registers, MCR packet control register(0xFDC) needs to be programmed to point to any enabled slice/subslice pair. Otherwise, incorrect value will be returned. However, that means each subse

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: Use ktime on wait_for

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Use ktime on wait_for URL : https://patchwork.freedesktop.org/series/42035/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072 -> Patchwork_8764 = == Summary - SUCCESS == No regressions found. External

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/4] drm/i915: Stop tracking timeline->inflight_seqnos URL : https://patchwork.freedesktop.org/series/42033/ State : warning == Summary == $ dim checkpatch origin/drm-tip a9beb7851c63 drm/i915: Stop tracking timeline->inflight_seqnos -:14: ER

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [1/4] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/4] drm/i915: Stop tracking timeline->inflight_seqnos URL : https://patchwork.freedesktop.org/series/42033/ State : warning == Summary == $ dim sparse origin/drm-tip Commit: drm/i915: Stop tracking timeline->inflight_seqnos -O:drivers/gpu/dr

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/4] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/4] drm/i915: Stop tracking timeline->inflight_seqnos URL : https://patchwork.freedesktop.org/series/42033/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072 -> Patchwork_8765 = == Summary - SUCCESS == No regressions

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Aspect ratio support in DRM layer

2018-04-20 Thread Patchwork
== Series Details == Series: Aspect ratio support in DRM layer URL : https://patchwork.freedesktop.org/series/42030/ State : warning == Summary == $ dim checkpatch origin/drm-tip 004a2f4d2e45 drm/modes: Introduce drm_mode_match() -:39: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open p

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers (rev3)

2018-04-20 Thread Hans de Goede
Hi, On 04/20/2018 03:52 PM, Patchwork wrote: == Series Details == Series: drm/i915: Do NOT skip the first 4k of stolen memory for pre-allocated buffers (rev3) URL : https://patchwork.freedesktop.org/series/40929/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4072_full ->

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/2] drm/i915: Use ktime on wait_for

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Use ktime on wait_for URL : https://patchwork.freedesktop.org/series/42035/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4072_full -> Patchwork_8764_full = == Summary - FAILURE == Serious unknown changes

Re: [Intel-gfx] [PATCH] drm/i915/icl: Adjust BSD2 semantics to mean any second VCS instance

2018-04-20 Thread Tvrtko Ursulin
On 20/04/2018 15:19, Bloomfield, Jon wrote: -Original Message- From: Tvrtko Ursulin Sent: Wednesday, April 18, 2018 2:34 AM To: Intel-gfx@lists.freedesktop.org Cc: tursu...@ursulin.net; Ursulin, Tvrtko ; Chris Wilson ; Bloomfield, Jon ; Ye, Tony Subject: [PATCH] drm/i915/icl: Adjust BS

[Intel-gfx] ✓ Fi.CI.BAT: success for Aspect ratio support in DRM layer

2018-04-20 Thread Patchwork
== Series Details == Series: Aspect ratio support in DRM layer URL : https://patchwork.freedesktop.org/series/42030/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072 -> Patchwork_8766 = == Summary - SUCCESS == No regressions found. External URL: https://patchwork.f

Re: [Intel-gfx] [PATCH] drm/i915/icl: Adjust BSD2 semantics to mean any second VCS instance

2018-04-20 Thread Bloomfield, Jon
> -Original Message- > From: Tvrtko Ursulin > Sent: Friday, April 20, 2018 9:56 AM > To: Bloomfield, Jon ; Tvrtko Ursulin > ; Intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] drm/i915/icl: Adjust BSD2 semantics to mean > any second VCS instance > > > On 20/04/2018 15:19

Re: [Intel-gfx] [PATCH] drm/i915/psr : Add psr1 live status

2018-04-20 Thread Souza, Jose
On Fri, 2018-04-20 at 15:06 +0530, vathsala nagaraju wrote: > From: Vathsala Nagaraju > > Prints live state of psr1.Extending the existing > PSR2 live state function to cover psr1. > > Tested on KBL with psr2 and psr1 panel. > > Cc: Rodrigo Vivi > Cc: Dhinakaran Pandiyan > > Signed-off-by: V

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/4] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/4] drm/i915: Stop tracking timeline->inflight_seqnos URL : https://patchwork.freedesktop.org/series/42033/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4072_full -> Patchwork_8765_full = == Summary - WARNING == Minor

Re: [Intel-gfx] [PATCH] drm/i915/psr : Add psr1 live status

2018-04-20 Thread Rodrigo Vivi
On Fri, Apr 20, 2018 at 03:06:03PM +0530, vathsala nagaraju wrote: > From: Vathsala Nagaraju > > Prints live state of psr1.Extending the existing > PSR2 live state function to cover psr1. > > Tested on KBL with psr2 and psr1 panel. Does it really work? I mean... I heard DK complaining that any

Re: [Intel-gfx] [v2] drm/i915/fbdev: Enable late fbdev initial configuration

2018-04-20 Thread Paul Menzel
Dear Jose, On 04/19/18 01:41, Souza, Jose wrote: If the initial fbdev configuration(intel_fbdev_initial_config()) runs and Nit: Space before (. there still no sink connected it will cause drm_fb_helper_initial_config() to return 0 as no error happened(but internally the return is -EAGAIN).

Re: [Intel-gfx] [PATCH 3/4] drm/vmwgfx: Drop DRM_CONTROL_ALLOW

2018-04-20 Thread Thomas Hellstrom
On 04/20/2018 08:51 AM, Daniel Vetter wrote: Control nodes are no more! Signed-off-by: Daniel Vetter Cc: VMware Graphics Cc: Sinclair Yeh Cc: Thomas Hellstrom --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/dr

Re: [Intel-gfx] [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake

2018-04-20 Thread Rodrigo Vivi
On Tue, Apr 17, 2018 at 12:02:52PM +0300, Jani Nikula wrote: > On Mon, 16 Apr 2018, "Srivatsa, Anusha" wrote: > >>-Original Message- > >>From: Jani Nikula [mailto:jani.nik...@linux.intel.com] > >>Sent: Wednesday, April 11, 2018 5:27 AM > >>To: Ian W MORRISON > >>Cc: Vivi, Rodrigo ; Srivat

[Intel-gfx] ✗ Fi.CI.IGT: failure for Aspect ratio support in DRM layer

2018-04-20 Thread Patchwork
== Series Details == Series: Aspect ratio support in DRM layer URL : https://patchwork.freedesktop.org/series/42030/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4072_full -> Patchwork_8766_full = == Summary - FAILURE == Serious unknown changes coming with Patchwork_876

Re: [Intel-gfx] [PATCH] drm/i915: Wait for vblank after register read

2018-04-20 Thread Rodrigo Vivi
On Thu, Apr 19, 2018 at 10:03:05AM +0300, Mika Kahola wrote: > On Thu, 2018-04-19 at 09:11 +0300, Lofstedt, Marta wrote: > > For the PW results:  > > https://patchwork.freedesktop.org/series/41877/ > > > > it didn't fix the CRC mismatch on: > >  https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8

Re: [Intel-gfx] [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake

2018-04-20 Thread Srivatsa, Anusha
>-Original Message- >From: Vivi, Rodrigo >Sent: Friday, April 20, 2018 11:04 AM >To: Jani Nikula >Cc: Srivatsa, Anusha ; Ian W MORRISON >; airl...@linux.ie; Greg KH >; intel-gfx@lists.freedesktop.org; linux- >ker...@vger.kernel.org; sta...@vger.kernel.org; dri- >de...@lists.freedesktop.o

[Intel-gfx] [PATCH 1/2] agp/intel-gtt: Drop the code for gen > 1

2018-04-20 Thread Adam Jackson
The gen2+ code has been unreachable since: commit ebb7c78d358b2ea45c7d997423e6feb42e5ce4ef Author: Daniel Vetter Date: Wed Jan 27 14:38:00 2016 +0100 agp/intel-gtt: Only register fake agp driver for gen1 Signed-off-by: Adam Jackson --- drivers/char/agp/intel-gtt.c | 631

[Intel-gfx] [PATCH 2/2] drm/i915: Only build INTEL_GTT on 32-bit x86

2018-04-20 Thread Adam Jackson
The i810 was attached to Pentium III motherboards, no 64-bit CPU is ever going to have one. Signed-off-by: Adam Jackson --- drivers/gpu/drm/i915/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig index dfd95889f

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] agp/intel-gtt: Drop the code for gen > 1

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/2] agp/intel-gtt: Drop the code for gen > 1 URL : https://patchwork.freedesktop.org/series/42054/ State : warning == Summary == $ dim checkpatch origin/drm-tip 4c8f8499a696 agp/intel-gtt: Drop the code for gen > 1 -:8: ERROR:GIT_COMMIT_ID: P

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] agp/intel-gtt: Drop the code for gen > 1

2018-04-20 Thread Patchwork
== Series Details == Series: series starting with [1/2] agp/intel-gtt: Drop the code for gen > 1 URL : https://patchwork.freedesktop.org/series/42054/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4072 -> Patchwork_8767 = == Summary - FAILURE == Serious unknown changes c

Re: [Intel-gfx] [PATCH] drm/i915: Wait for vblank after register read

2018-04-20 Thread Dhinakaran Pandiyan
On Fri, 2018-04-20 at 11:15 -0700, Rodrigo Vivi wrote: > On Thu, Apr 19, 2018 at 10:03:05AM +0300, Mika Kahola wrote: > > On Thu, 2018-04-19 at 09:11 +0300, Lofstedt, Marta wrote: > > > For the PW results: > > > https://patchwork.freedesktop.org/series/41877/ > > > > > > it didn't fix the CRC mis

[Intel-gfx] [PATCH 16/22] drm/i915/icl: Wa_2006665173

2018-04-20 Thread Oscar Mateo
Disable blend embellishment in RCC. v2: Rebased on top of the WA refactoring Cc: Mika Kuoppala Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_reg.h | 18 +++--- drivers/gpu/drm/i915/intel_workarounds.c | 5 + 2 files changed, 16 insertions(+), 7 deletions(-)

[Intel-gfx] [PATCH 15/22] drm/i915/icl: WaEnableStateCacheRedirectToCS

2018-04-20 Thread Oscar Mateo
Redirects the state cache to the CS Command buffer section for performance reasons. v2: Rebased v3: Rebased on top of the WA refactoring Cc: Mika Kuoppala Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_reg.h | 1 + drivers/gpu/drm/i915/intel_workarounds.c | 4 2 files c

[Intel-gfx] [PATCH 13/22] drm/i915/icl: WaForwardProgressSoftReset

2018-04-20 Thread Oscar Mateo
Avoids a hang during soft reset. v2: Rebased on top of the WA refactoring Cc: Mika Kuoppala Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_reg.h | 5 + drivers/gpu/drm/i915/intel_workarounds.c | 8 2 files changed, 13 insertions(+) diff --git a/drivers/gpu/drm/

[Intel-gfx] [PATCH 06/22] drm/i915/icl: WaDisableCleanEvicts

2018-04-20 Thread Oscar Mateo
Avoids an undefined LLC behavior. BSpec: 9613 v2: Renamed to Wa_1405733216 v3: Spaces around '<<' and fix surrounding code v4: Rebased on top of the WA refactoring Cc: Mika Kuoppala Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_reg.h | 5 +++-- drivers/gpu/drm/i915/intel_w

[Intel-gfx] [PATCH 12/22] drm/i915/icl: Wa_1406838659

2018-04-20 Thread Oscar Mateo
Disable CGPSF unit clock gating to prevent an issue. v2: Rebased on top of the WA refactoring Cc: Mika Kuoppala Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_reg.h | 13 - drivers/gpu/drm/i915/intel_workarounds.c | 6 ++ 2 files changed, 14 insertions(+), 5

[Intel-gfx] [PATCH 21/22] drm/i915/icl: WaAllowUmdWriteTRTTRootTable

2018-04-20 Thread Oscar Mateo
Required for TR-TT (Tiled Resource Translation Table) support. Do Linux UMDs make use of this? This change has been security reviewed and the whitelisting approved. Virtualization of other OSes could certainly use it. v2: For whatever reason, this ended up in KBL (??!!) v3: Rebased on top of the

[Intel-gfx] [PATCH 05/22] drm/i915/icl: WaModifyGamTlbPartitioning

2018-04-20 Thread Oscar Mateo
Adjust default GAM TLB partitioning for performance reasons. v2: Only touch the bits that we really need v3: Rebased on top of the WA refactoring Cc: Mika Kuoppala Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_reg.h | 5 + drivers/gpu/drm/i915/intel_workarounds.c | 5 ++

[Intel-gfx] [PATCH v4 00/22] Workarounds for Icelake

2018-04-20 Thread Oscar Mateo
List of GT workarounds for Icelake that we have been carrying in internal. Can we get eyes on these please? Oscar Mateo (22): drm/i915/icl: Introduce initial Icelake Workarounds drm/i915/icl: Enable Sampler DFR drm/i915/icl: WaGAPZPriorityScheme drm/i915/icl: WaL3BankAddressHashing drm/i

[Intel-gfx] [PATCH 10/22] drm/i915/icl: Wa_1406680159 and Wa_2201832410

2018-04-20 Thread Oscar Mateo
Disable GWL clock gating to prevent two different issues that might cause hangs. Please notice that one of the issues is pre-production only. v2: Rebased on top of the WA refactoring Cc: Mika Kuoppala Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/intel_workarounds.c | 6 ++ 1 file c

  1   2   >