Re: [Intel-gfx] [PATCH] drm/i915: Adding dbuf support for skl nv12 format.

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6273 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH i-g-t] kms_atomic: Measure speed of some plane ioctls

2015-04-28 Thread Chris Wilson
On Tue, Apr 28, 2015 at 01:46:23PM +0100, Tvrtko Ursulin wrote: From: Tvrtko Ursulin tvrtko.ursu...@intel.com Measures DRM_IOCTL_MODE_SETCRTC and DRM_IOCTL_MODE_SETPLANE as proxy for drm_atomic_helper_update_plane if I got it right. Discovered some slow cursor updates (1.6ms) so needed

[Intel-gfx] [PATCH v3 i-g-t] kms_flip_tiling: New tiling tests, including Y/Yf

2015-04-28 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com New subtests to excercise flips from tiled to tiled and from linear to tiled frame buffers. These will catch display programming issues like not preserving the tiling mode in page flips or not re-programming the watermarks. v2: Cleanup crc object

Re: [Intel-gfx] [PATCH v4] drm/i915/chv: Implement WaDisableShadowRegForCpd

2015-04-28 Thread Jani Nikula
On Thu, 16 Apr 2015, deepa...@linux.intel.com wrote: From: Deepak S deepa...@linux.intel.com This WA is avoid problem between shadow vs wake FIFO unload problem during CPD/RC6 transactions on CHV. v2: Define individual bits GTFIFOCTL (Ville) v3: move WA to uncore_early_sanitize (ville)

Re: [Intel-gfx] [PATCH v2] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread Chris Wilson
On Tue, Apr 28, 2015 at 02:38:25PM +, Antoine, Peter wrote: So is the plan to push these patches and have follow-on work to cover the other paths? As this fixes the Bugzilla issue that has been raised. You've identified an issue, but I think your patch is incomplete. -Chris -- Chris

Re: [Intel-gfx] [PATCH 4/5] drm: Make HW_LOCK access functions optional.

2015-04-28 Thread Antoine, Peter
On Tue, 2015-04-28 at 16:08 +0300, Ville Syrjälä wrote: On Tue, Apr 28, 2015 at 11:29:06AM +, Antoine, Peter wrote: diff --git a/include/drm/drmP.h b/include/drm/drmP.h index 62c40777..367e42f 100644 --- a/include/drm/drmP.h +++ b/include/drm/drmP.h @@ -137,17

Re: [Intel-gfx] [PATCH v4 1/8] drm/i915/skl: Add support to load SKL CSR firmware.

2015-04-28 Thread Imre Deak
On to, 2015-04-16 at 14:22 +0530, Animesh Manna wrote: [...] + +#define I915_CSR_SKL i915/skl_dmc_ver1.bin Animesh, does this firmware loading patchset work with skl_dmc_ver4.bin? If so please re-send this patch with the above pointing to i915/skl_dmc_ver4.bin (since that is the published

Re: [Intel-gfx] [PATCH v2] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread Antoine, Peter
So is the plan to push these patches and have follow-on work to cover the other paths? As this fixes the Bugzilla issue that has been raised. Peter. -Original Message- From: Deepak S [mailto:deepa...@linux.intel.com] Sent: Tuesday, April 28, 2015 9:56 AM To: Chris Wilson; S, Deepak;

Re: [Intel-gfx] [PATCH-V2 1/2] drm/i915: add callback to enable/disable codec wakeup

2015-04-28 Thread Jani Nikula
On Tue, 28 Apr 2015, han...@intel.com wrote: From: Lu, Han han...@intel.com In SKL, HDMI/DP codec and PCH HD Audio Controller are in different power wells, so it's necessary to reset display audio codecs when power well on, otherwise display audio codecs will disappear when resume from low

Re: [Intel-gfx] [PATCH] drm/i915: Remove wait for for punit to updates freq.

2015-04-28 Thread Deepak S
On Monday 13 April 2015 05:40 PM, Ville Syrjälä wrote: On Mon, Apr 13, 2015 at 02:55:12PM +0300, Jani Nikula wrote: On Thu, 05 Mar 2015, deepa...@linux.intel.com wrote: From: Deepak S deepa...@linux.intel.com When GPU is idle on VLV, Request freq to punit should be good enough to get the

[Intel-gfx] atom D525 trouble

2015-04-28 Thread Patrick Welche
In ancient pre-KMS times, I was able to get a particular pineview-based (Atom D525) desktop computer monitor to run at decent resolutions. Recently, I have tried the 2.99.917 snapshot, and the best I can manage is 1024x768. Bad parts of the system are: - the monitor's EDID is invalid - it

[Intel-gfx] [PATCH i-g-t] igt_kms: Avoid NULL ptr deref when commiting disabled planes

2015-04-28 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com I think; commit a26f9f9ad0e679c7ce413a25d34f6914e1174151 Author: chandra konduru chandra.kond...@intel.com Date: Mon Mar 30 13:52:04 2015 -0700 i-g-t: Adding plane scaling test case introduced a condition where it attempts to

Re: [Intel-gfx] [PATCH] drm/i915: Remove wait for for punit to updates freq.

2015-04-28 Thread Deepak S
On Tuesday 28 April 2015 11:46 PM, Jesse Barnes wrote: Yeah I think this is fine (may need a rebase though, you can keep my r-b if you do that in case Jani doesn't want to deal with the merge conflicts). Reviewed-by: Jesse Barnes jbar...@virtuousgeek.org Sure Jesse, I will rebase the

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

2015-04-28 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-intel tree got a conflict in drivers/gpu/drm/i915/i915_drv.c between commit 5df0582bf036 (drm/i915/vlv: remove wait for previous GFX clk disable request) from Linus' tree and commit 85250ddff7a6 (drm/i915/chv: Remove Wait for a previous gfx force-off)

[Intel-gfx] [PATCH i-g-t] kms_atomic: Measure speed of some plane ioctls

2015-04-28 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com Measures DRM_IOCTL_MODE_SETCRTC and DRM_IOCTL_MODE_SETPLANE as proxy for drm_atomic_helper_update_plane if I got it right. Discovered some slow cursor updates (1.6ms) so needed something to test different kernel configs etc. v2: * Move to a test

[Intel-gfx] [PATCH i-g-t 03/10] intel_backlight: switch to INREG and OUTREG

2015-04-28 Thread Jani Nikula
Use INREG and OUTREG instead of using mmio directly. Signed-off-by: Jani Nikula jani.nik...@intel.com --- tools/intel_backlight.c | 20 +--- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git a/tools/intel_backlight.c b/tools/intel_backlight.c index

[Intel-gfx] [PATCH i-g-t 02/10] intel_reg: switch to INREG and OUTREG

2015-04-28 Thread Jani Nikula
Use INREG and OUTREG instead of using mmio directly. Signed-off-by: Jani Nikula jani.nik...@intel.com --- tools/intel_reg.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/tools/intel_reg.c b/tools/intel_reg.c index 975529d4555b..0f98266932e5 100644 ---

[Intel-gfx] [PATCH i-g-t 00/10] rename mmio global variable to igt_global_mmio

2015-04-28 Thread Jani Nikula
First reduce to use of the mmio global variable by switching to INREG and OUTREG, and finally rename mmio to igt_global_mmio. BR, Jani. Jani Nikula (10): lib: add 16 and 8 bit versions of INREG and OUTREG intel_reg: switch to INREG and OUTREG intel_backlight: switch to INREG and OUTREG

[Intel-gfx] [PATCH i-g-t 01/10] lib: add 16 and 8 bit versions of INREG and OUTREG

2015-04-28 Thread Jani Nikula
Signed-off-by: Jani Nikula jani.nik...@intel.com --- lib/intel_io.h | 4 lib/intel_mmio.c | 24 2 files changed, 28 insertions(+) diff --git a/lib/intel_io.h b/lib/intel_io.h index 04aa3fd496b4..1c3b4445cd5b 100644 --- a/lib/intel_io.h +++ b/lib/intel_io.h @@

[Intel-gfx] [PATCH i-g-t 06/10] intel_watermark: switch to INREG

2015-04-28 Thread Jani Nikula
Use INREG instead of using mmio directly. Signed-off-by: Jani Nikula jani.nik...@intel.com --- tools/intel_watermark.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/intel_watermark.c b/tools/intel_watermark.c index e5dee46db78f..0b7c5e5193ef 100644 ---

[Intel-gfx] [PATCH i-g-t 07/10] intel_display_poller: use INREG and OUTREG

2015-04-28 Thread Jani Nikula
Use INREG and OUTREG instead of using mmio directly. Signed-off-by: Jani Nikula jani.nik...@intel.com --- tools/intel_display_poller.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/intel_display_poller.c b/tools/intel_display_poller.c index

[Intel-gfx] [PATCH i-g-t 10/10] rename global mmio variable to igt_global_mmio

2015-04-28 Thread Jani Nikula
Global variable names should reflect the fact that they are indeed global, and at the very least they should not be as short as just mmio. Rename mmio to igt_global_mmio. Signed-off-by: Jani Nikula jani.nik...@intel.com --- lib/intel_io.h | 2 +- lib/intel_mmio.c | 44

[Intel-gfx] [PATCH i-g-t 04/10] intel_reg_checker: switch to INREG

2015-04-28 Thread Jani Nikula
Use INREG instead of using mmio directly. Signed-off-by: Jani Nikula jani.nik...@intel.com --- tools/intel_reg_checker.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/tools/intel_reg_checker.c b/tools/intel_reg_checker.c index 22d979613ea1..2d6da70c3afa 100644 ---

[Intel-gfx] [PATCH i-g-t 08/10] intel_vga_{read, write}: use INREG and OUTREG

2015-04-28 Thread Jani Nikula
Use INREG and OUTREG instead of using mmio directly. Signed-off-by: Jani Nikula jani.nik...@intel.com --- tools/intel_vga_read.c | 2 +- tools/intel_vga_write.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/intel_vga_read.c b/tools/intel_vga_read.c index

[Intel-gfx] [PATCH i-g-t 05/10] intel_reg_{read, write}: switch to INREG and OUTREG

2015-04-28 Thread Jani Nikula
Use INREG and OUTREG instead of using mmio directly. Signed-off-by: Jani Nikula jani.nik...@intel.com --- tools/intel_reg_read.c | 5 ++--- tools/intel_reg_write.c | 8 +++- 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/tools/intel_reg_read.c b/tools/intel_reg_read.c index

[Intel-gfx] [PATCH i-g-t 09/10] tests/gen7_forcewake_mt: use local mmio variable

2015-04-28 Thread Jani Nikula
igfx_get_mmio() uses the global mmio variable by accident. Use a local variable instead. The intention is to rename the global variable later on, so shadowing it here does not matter. Signed-off-by: Jani Nikula jani.nik...@intel.com --- tests/gen7_forcewake_mt.c | 1 + 1 file changed, 1

Re: [Intel-gfx] [PATCH] drm/i915: Remove incorrect restriction on 32bit offsets in ppGTT backend

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6275 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

[Intel-gfx] [PATCH i-g-t] igt_kms: Do not reset plane position on assigning a fb

2015-04-28 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com commit a26f9f9ad0e679c7ce413a25d34f6914e1174151 Author: chandra konduru chandra.kond...@intel.com Date: Mon Mar 30 13:52:04 2015 -0700 i-g-t: Adding plane scaling test case Started doing this and broke kms_rotation_crc.

Re: [Intel-gfx] [PATCH 4/5] drm: Make HW_LOCK access functions optional.

2015-04-28 Thread Ville Syrjälä
On Tue, Apr 28, 2015 at 11:29:06AM +, Antoine, Peter wrote: diff --git a/include/drm/drmP.h b/include/drm/drmP.h index 62c40777..367e42f 100644 --- a/include/drm/drmP.h +++ b/include/drm/drmP.h @@ -137,17 +137,18 @@ void drm_err(const char *format, ...); /*@{*/

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915: DP link training optimization

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6274 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH 4/5] drm: Make HW_LOCK access functions optional.

2015-04-28 Thread Antoine, Peter
reply at end. On Tue, 2015-04-28 at 13:40 +0300, Ville Syrjälä wrote: On Tue, Apr 28, 2015 at 05:52:20AM +, Antoine, Peter wrote: Hi, (replies inline) -Original Message- From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com] Sent: Monday, April 27, 2015 6:04 PM

[Intel-gfx] [PATCH i-g-t v2] man: add man page for intel_reg in reStructured text format

2015-04-28 Thread Jani Nikula
Produce the intel_reg man page from rst using rst2man. Also facilitate writing any man page in reStructured text, as long as rst2man is available. v2: configure check for rst2man, credits to Thomas Wood for that. Signed-off-by: Jani Nikula jani.nik...@intel.com --- README| 1 +

Re: [Intel-gfx] [PATCH 1/5] drm: Kernel Crash in drm_unlock

2015-04-28 Thread Dave Gordon
On 28/04/15 10:21, Dave Gordon wrote: On 24/04/15 06:52, Antoine, Peter wrote: I picked up this work due to the following Jira ticket created by the security team (on Android) and was asked to give it a second look and found a few more issues with the hw lock code.

[Intel-gfx] [PATCH] drm/i915/gtt: Allocate va range only if vma is not bound

2015-04-28 Thread Mika Kuoppala
When we have bound vma into an address space, the layout of page table structures is immutable. So we can be absolutely certain that if vma is already bound, there is no need to (re)allocate a virtual address range for it. v2: - add sanity checks and remove superfluous GLOBAL_BIND set - we

Re: [Intel-gfx] [PATCH] drm/i915/gtt: Allocate va range only if vma is not bound

2015-04-28 Thread Chris Wilson
On Tue, Apr 28, 2015 at 05:56:17PM +0300, Mika Kuoppala wrote: When we have bound vma into an address space, the layout of page table structures is immutable. So we can be absolutely certain that if vma is already bound, there is no need to (re)allocate a virtual address range for it. v2: -

Re: [Intel-gfx] [PATCH v3 04/15] drm/i915: GuC firmware loader

2015-04-28 Thread Dave Gordon
On 23/04/15 18:48, Dave Gordon wrote: On 17/04/15 22:21, yu@intel.com wrote: From: Alex Dai yu@intel.com Add GuC firmware loader. It uses the unified firmware loader to fetch firmware blob first, then load to hw in driver main thread. Issue: VIZ-4884 Signed-off-by: Alex Dai

Re: [Intel-gfx] [PATCH v3 14/15] drm/i915: Taking forcewake during GuC load.

2015-04-28 Thread Dave Gordon
On 17/04/15 22:21, yu@intel.com wrote: From: Sagar Kamble sagar.a.kam...@intel.com The firmware loader will use GuC DMA engine to move data from ggtt to WOPCM. Need to take forcewake before GuC loading. Issue: VIZ-4884 Change-Id: Ie422fc1e122933b161ff63cab23622197e6bba54

Re: [Intel-gfx] [PATCH v3 04/15] drm/i915: GuC firmware loader

2015-04-28 Thread Yu Dai
On 04/28/2015 08:12 AM, Dave Gordon wrote: On 23/04/15 18:48, Dave Gordon wrote: On 17/04/15 22:21, yu@intel.com wrote: From: Alex Dai yu@intel.com Add GuC firmware loader. It uses the unified firmware loader to fetch firmware blob first, then load to hw in driver main thread.

[Intel-gfx] [PATCH v3] drm/i915: Setup static bias for GPU

2015-04-28 Thread deepak . s
From: Deepak S deepa...@linux.intel.com Based on the spec, Setting up static BIAS for GPU to improve the rps performace. v2: rename reg defn to match spec. (Ville) v3: Updated bias setting for chv (Deepak) Signed-off-by: Deepak S deepa...@linux.intel.com --- drivers/gpu/drm/i915/i915_reg.h |

Re: [Intel-gfx] [PATCH/RFC] i915: fix screen flickering

2015-04-28 Thread Thomas Gummerer
Thomas Gummerer t.gumme...@gmail.com writes: Commit c9f038a1a592 (drm/i915: Don't assume primary cursor are always on for wm calculation (v4)) fixes a null pointer dereference. Setting the primary and cursor panes to false in ilk_compute_wm_parameters to false does however give the following

[Intel-gfx] [PATCH/RFC] i915: fix screen flickering

2015-04-28 Thread Thomas Gummerer
Commit c9f038a1a592 (drm/i915: Don't assume primary cursor are always on for wm calculation (v4)) fixes a null pointer dereference. Setting the primary and cursor panes to false in ilk_compute_wm_parameters to false does however give the following errors in the kernel log and causes the screen to

Re: [Intel-gfx] [PATCH] drm/i915: Remove wait for for punit to updates freq.

2015-04-28 Thread Deepak S
On Wednesday 29 April 2015 12:02 AM, Ville Syrjälä wrote: On Tue, Apr 28, 2015 at 11:16:29AM -0700, Jesse Barnes wrote: On 03/04/2015 08:08 PM, deepa...@linux.intel.com wrote: From: Deepak S deepa...@linux.intel.com When GPU is idle on VLV, Request freq to punit should be good enough to get

[Intel-gfx] [PATCH v4] drm/i915/chv: Set min freq to efficient frequency on chv

2015-04-28 Thread deepak . s
From: Deepak S deepa...@linux.intel.com After feedback from the hardware team, now we set the GPU min/idel freq to RPe. Punit is expecting us to operate GPU between Rpe Rp0. If we drop the frequency to RPn, punit is failing to change the input voltage to minimum :( Since Punit validates the rps

[Intel-gfx] [PATCH v2] drm/i915: Setup static bias for GPU

2015-04-28 Thread deepak . s
From: Deepak S deepa...@linux.intel.com Based on the spec, Setting up static BIAS for GPU to improve the rps performace. v2: rename reg defn to match spec. (Ville) Signed-off-by: Deepak S deepa...@linux.intel.com --- drivers/gpu/drm/i915/i915_reg.h | 5 + drivers/gpu/drm/i915/intel_pm.c |

Re: [Intel-gfx] [PATCH 4/5] drm/i915: Add a partial GGTT view type

2015-04-28 Thread Tvrtko Ursulin
On 04/27/2015 03:50 PM, Tvrtko Ursulin wrote: +for_each_sg_page(obj-pages-sgl, obj_sg_iter, obj-pages-nents, +view-params.partial.offset) +{ +if (st-nents = view-params.partial.size) +break; + +sg_set_page(sg, NULL, PAGE_SIZE, 0); +

Re: [Intel-gfx] [PATCH i-g-t] tests/drm_hw_lock: Tests for hw_lock fixes.

2015-04-28 Thread Antoine, Peter
Thanks for the review, new patch inbound. -Original Message- From: Thomas Wood [mailto:thomas.w...@intel.com] Sent: Monday, April 27, 2015 4:25 PM To: Antoine, Peter Cc: Intel Graphics Development; airl...@redhat.com; dri-de...@lists.freedesktop.org; Daniel Vetter Subject: Re:

Re: [Intel-gfx] [PATCH v2] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread Chris Wilson
On Tue, Apr 28, 2015 at 08:29:13AM +, S, Deepak wrote: Thanks Chirs for review, We moved Init_hw to initialize WA's before any BB submission. Init_hw calls init_clock_gating But you appreciate the same issue exists on other paths? -Chris -- Chris Wilson, Intel Open Source

Re: [Intel-gfx] [PATCH v2] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread Deepak S
Yes agreed, we need to make changes in other paths :) On Tuesday 28 April 2015 02:14 PM, Chris Wilson wrote: On Tue, Apr 28, 2015 at 08:29:13AM +, S, Deepak wrote: Thanks Chirs for review, We moved Init_hw to initialize WA's before any BB submission. Init_hw calls init_clock_gating

[Intel-gfx] [PATCH] drm/i915: Remove incorrect restriction on 32bit offsets in ppGTT backend

2015-04-28 Thread Chris Wilson
This is the wrong layer to apply an arbitrary restriction and the wrong error code (object too large!). If we do want to prevent large offsets being return to the user on 32bit systems (to hide bugs in userspace), you want to restrict the drm_mm range manager instead. This first tells userspace

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915: DP link training optimization

2015-04-28 Thread Mika Kahola
On Tue, 2015-04-28 at 13:19 +0530, Sivakumar Thulasimani wrote: On 4/28/2015 12:13 PM, Mika Kahola wrote: This patch adds DP link training optimization by reusing the previously trained values. v2: - rebase V3: - rebase Signed-off-by: Mika Kahola mika.kah...@intel.com ---

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915: DP link training optimization

2015-04-28 Thread Sivakumar Thulasimani
On 4/28/2015 1:44 PM, Mika Kahola wrote: On Tue, 2015-04-28 at 13:19 +0530, Sivakumar Thulasimani wrote: On 4/28/2015 12:13 PM, Mika Kahola wrote: This patch adds DP link training optimization by reusing the previously trained values. v2: - rebase V3: - rebase Signed-off-by: Mika Kahola

Re: [Intel-gfx] [PATCH v2] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread S, Deepak
Thanks Chirs for review, We moved Init_hw to initialize WA's before any BB submission. Init_hw calls init_clock_gating Thanks Deepak -Original Message- From: Chris Wilson [mailto:ch...@chris-wilson.co.uk] Sent: Tuesday, April 28, 2015 12:53 PM To: Antoine, Peter Cc:

Re: [Intel-gfx] [PATCH] drm/i915: Remove wait for for punit to updates freq.

2015-04-28 Thread Jesse Barnes
On 03/04/2015 08:08 PM, deepa...@linux.intel.com wrote: From: Deepak S deepa...@linux.intel.com When GPU is idle on VLV, Request freq to punit should be good enough to get the voltage back to VNN. Also, make sure gfx clock force applies before requesting the freq fot vlv. Bugzilla:

Re: [Intel-gfx] [PATCH] drm/i915: Remove wait for for punit to updates freq.

2015-04-28 Thread Ville Syrjälä
On Tue, Apr 28, 2015 at 11:16:29AM -0700, Jesse Barnes wrote: On 03/04/2015 08:08 PM, deepa...@linux.intel.com wrote: From: Deepak S deepa...@linux.intel.com When GPU is idle on VLV, Request freq to punit should be good enough to get the voltage back to VNN. Also, make sure gfx clock

Re: [Intel-gfx] [PATCH 2/5] drm/i915: Do not make assumptions on GGTT VMA sizes

2015-04-28 Thread Joonas Lahtinen
On ma, 2015-04-27 at 14:55 +0100, Tvrtko Ursulin wrote: Hi, On 04/24/2015 01:09 PM, Joonas Lahtinen wrote: GGTT VMA sizes might be smaller than the whole object size due to different GGTT views. v2: - Separate GGTT view constraint calculations from normal view constraint

Re: [Intel-gfx] [PATCH v2] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread Chris Wilson
On Tue, Apr 28, 2015 at 08:18:14AM +0100, Peter Antoine wrote: This patch fixed a timing issue that causes a GPU hang when a the system comes out of power saving. During pm_resume, We are submitting batchbuffers before enabling Interrupts this is causing us to miss the context switch

[Intel-gfx] [PATCH v2] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread Peter Antoine
This patch fixed a timing issue that causes a GPU hang when a the system comes out of power saving. During pm_resume, We are submitting batchbuffers before enabling Interrupts this is causing us to miss the context switch interrupt, and in consequence intel_execlists_handle_ctx_events is not

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915: DP link training optimization

2015-04-28 Thread Mika Kahola
On Tue, 2015-04-28 at 15:18 +0530, Sivakumar Thulasimani wrote: On 4/28/2015 3:12 PM, Mika Kahola wrote: On Tue, 2015-04-28 at 13:51 +0530, Sivakumar Thulasimani wrote: On 4/28/2015 1:44 PM, Mika Kahola wrote: On Tue, 2015-04-28 at 13:19 +0530, Sivakumar Thulasimani wrote: On 4/28/2015

[Intel-gfx] [PATCH-V2 2/2] ALSA:hda - reset display codec when power on

2015-04-28 Thread han . lu
From: Lu, Han han...@intel.com In SKL, HDMI/DP codec and PCH HD Audio Controller are in different power wells, so it's necessary to reset display audio codecs when power well on, otherwise display audio codecs will disappear when resume from low power state. The reset step when power on is:

[Intel-gfx] [PATCH-V2 1/2] drm/i915: add callback to enable/disable codec wakeup

2015-04-28 Thread han . lu
From: Lu, Han han...@intel.com In SKL, HDMI/DP codec and PCH HD Audio Controller are in different power wells, so it's necessary to reset display audio codecs when power well on, otherwise display audio codecs will disappear when resume from low power state. The reset step when power on is:

Re: [Intel-gfx] [PATCH 4/5] drm: Make HW_LOCK access functions optional.

2015-04-28 Thread Ville Syrjälä
On Tue, Apr 28, 2015 at 05:52:20AM +, Antoine, Peter wrote: Hi, (replies inline) -Original Message- From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com] Sent: Monday, April 27, 2015 6:04 PM To: Antoine, Peter Cc: intel-gfx@lists.freedesktop.org; airl...@redhat.com;

Re: [Intel-gfx] [PATCH] drm/i915: Silence compiler warning in dvo

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6272 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH v2 i-g-t] kms_atomic: Measure speed of some plane ioctls

2015-04-28 Thread Tvrtko Ursulin
On 04/27/2015 10:20 PM, Chris Wilson wrote: On Mon, Apr 27, 2015 at 04:34:05PM +0100, Tvrtko Ursulin wrote: +static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe, +igt_plane_t *plane) +{ + drmModeModeInfo *mode; + igt_display_t

Re: [Intel-gfx] [PATCH v2] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6271 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915: DP link training optimization

2015-04-28 Thread Mika Kahola
On Tue, 2015-04-28 at 13:51 +0530, Sivakumar Thulasimani wrote: On 4/28/2015 1:44 PM, Mika Kahola wrote: On Tue, 2015-04-28 at 13:19 +0530, Sivakumar Thulasimani wrote: On 4/28/2015 12:13 PM, Mika Kahola wrote: This patch adds DP link training optimization by reusing the previously

[Intel-gfx] [PATCH i-g-t v2] tests/drm_hw_lock: Tests for hw_lock fixes.

2015-04-28 Thread Peter Antoine
There are several issues with the hardware locks functions that stretch from kernel crashes to priority escalations. This new test will test the the fixes for these features. This test will cause a driver/kernel crash on un-patched kernels, the following patches should be applied to stop the

Re: [Intel-gfx] [PATCH i-g-t] tests/drm_hw_lock: Tests for hw_lock fixes.

2015-04-28 Thread ch...@chris-wilson.co.uk
On Tue, Apr 28, 2015 at 09:28:51AM +, Antoine, Peter wrote: Also, the PARAM should give legitimate code the opportunity to avoid the problems but avoiding these features completely. There are no legitimate users. -Chris -- Chris Wilson, Intel Open Source Technology Centre

Re: [Intel-gfx] [PATCH 1/5] drm: Kernel Crash in drm_unlock

2015-04-28 Thread ch...@chris-wilson.co.uk
On Tue, Apr 28, 2015 at 10:21:49AM +0100, Dave Gordon wrote: On 24/04/15 06:52, Antoine, Peter wrote: I picked up this work due to the following Jira ticket created by the security team (on Android) and was asked to give it a second look and found a few more issues with the hw lock code.

Re: [Intel-gfx] [PATCH 1/5] drm: Kernel Crash in drm_unlock

2015-04-28 Thread Dave Gordon
On 24/04/15 06:52, Antoine, Peter wrote: I picked up this work due to the following Jira ticket created by the security team (on Android) and was asked to give it a second look and found a few more issues with the hw lock code. https://jira01.devtools.intel.com/browse/GMINL-5388 I/O control

Re: [Intel-gfx] [PATCH v2 i-g-t] kms_atomic: Measure speed of some plane ioctls

2015-04-28 Thread Chris Wilson
On Tue, Apr 28, 2015 at 10:18:53AM +0100, Tvrtko Ursulin wrote: On 04/27/2015 10:20 PM, Chris Wilson wrote: Same comment for commit = COMMIT_LEGACY here, mainly now for consistency with before. Here we want to be able to choose between commit types if we want to keep two subtests. I just

Re: [Intel-gfx] [PATCH] drm/i915: Avoid GPU hang when coming out of S3 or S4

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6271 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH i-g-t] tests/drm_hw_lock: Tests for hw_lock fixes.

2015-04-28 Thread Antoine, Peter
On Mon, 2015-04-27 at 16:33 +0100, Chris Wilson wrote: On Mon, Apr 27, 2015 at 04:24:37PM +0100, Thomas Wood wrote: On 23 April 2015 at 15:07, Peter Antoine peter.anto...@intel.com wrote: There are several issues with the hardware locks functions that stretch from kernel crashes to

Re: [Intel-gfx] [PATCH] drm/i915: Avoid GPU hang when coming out of P3 or P4

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6271 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH] drm/i915: Avoid GPU hang when coming out of P3 or P4

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6271 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915: DP link training optimization

2015-04-28 Thread Sivakumar Thulasimani
On 4/28/2015 3:12 PM, Mika Kahola wrote: On Tue, 2015-04-28 at 13:51 +0530, Sivakumar Thulasimani wrote: On 4/28/2015 1:44 PM, Mika Kahola wrote: On Tue, 2015-04-28 at 13:19 +0530, Sivakumar Thulasimani wrote: On 4/28/2015 12:13 PM, Mika Kahola wrote: This patch adds DP link training

[Intel-gfx] [PATCH v3 1/2] drm/i915: DP link training optimization

2015-04-28 Thread Mika Kahola
This is a first of series patches that optimize DP link training. The first patch is for eDP only where we reuse the previously trained link training values from cache i.e. voltage swing and pre-emphasis levels. In case we are not able to train the link by reusing the known values, the link

[Intel-gfx] [PATCH v3 2/2] drm/i915: DP link training optimization

2015-04-28 Thread Mika Kahola
This patch adds DP link training optimization by reusing the previously trained values. v2: - rebase V3: - rebase Signed-off-by: Mika Kahola mika.kah...@intel.com --- drivers/gpu/drm/i915/intel_dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Intel-gfx] [PATCH v3] drm/i915: DP link training optimization

2015-04-28 Thread Mika Kahola
This is patch series optimizes DP link training by reusing the link parameter settings if DP link has bee previously trained. In case we are not able to train the link by reusing the known values, the link training parameters are set to zero and training is restarted. The first patch is for eDP

Re: [Intel-gfx] Very bad performance from intel chipset on Thinkpad T540p (2880x1620)

2015-04-28 Thread Dave Gordon
On 24/04/15 17:18, Marc MERLIN wrote: On Fri, Apr 24, 2015 at 04:53:53PM +0100, Chris Wilson wrote: Whether of not it tears depends upon your window manager. On bare X, using mplayer -vo xv or -vo gl, should not tear. Under a compositing window manager, it depends upon how it decides to update

Re: [Intel-gfx] [PATCH v3 2/2] drm/i915: DP link training optimization

2015-04-28 Thread Sivakumar Thulasimani
On 4/28/2015 12:13 PM, Mika Kahola wrote: This patch adds DP link training optimization by reusing the previously trained values. v2: - rebase V3: - rebase Signed-off-by: Mika Kahola mika.kah...@intel.com --- drivers/gpu/drm/i915/intel_dp.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [Intel-gfx] [PATCH] drm/i915/skl: Bypass debug message if scalers are not requested

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6262 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH-V2 2/2] ALSA:hda - reset display codec when power on

2015-04-28 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6277 -Summary- Platform Delta drm-intel-nightly Series Applied PNV