Re: [Intel-gfx] [PATCH 3/3] drm/i915/bdw: Use the GEN8 SRM when qeueing a flip

2014-04-08 Thread Chris Wilson
On Mon, Apr 07, 2014 at 11:20:14PM +0100, Damien Lespiau wrote: On Mon, Apr 07, 2014 at 01:59:17PM -0700, Ben Widawsky wrote: Cool. This explains the bad DERRMR values I was seeing in in error states. I'm honestly didn't check if we actually need an SRM for BDW still, but I'll assume you

Re: [Intel-gfx] [PATCH 3/6] drm/i915: Add support for stealing purgable stolen pages

2014-04-08 Thread Chris Wilson
On Tue, Apr 08, 2014 at 04:32:02AM +, Gupta, Sourab wrote: Hi Rodrigo, In this patch, while freeing the purgeable stolen object, the memory node also has to be freed, so as to make space for new object. We need to call drm_mm_remove_node while freeing obj. The below modification patch

Re: [Intel-gfx] [PATCH] drm/i915: Make vm eviction uninterruptible

2014-04-08 Thread Chris Wilson
On Mon, Apr 07, 2014 at 02:58:34PM -0700, Ben Widawsky wrote: Blocking important fixes for a test case is harmful to customers of our software. I won't argue past that. If you won't take it as is, add it to the JIRA task like you said. I'll carry this one around with my dynamic page table

Re: [Intel-gfx] [PATCH 3/6] drm/i915: Add support for stealing purgable stolen pages

2014-04-08 Thread Gupta, Sourab
On Tue, 2014-04-08 at 06:45 +, Chris Wilson wrote: On Tue, Apr 08, 2014 at 04:32:02AM +, Gupta, Sourab wrote: Hi Rodrigo, In this patch, while freeing the purgeable stolen object, the memory node also has to be freed, so as to make space for new object. We need to call

Re: [Intel-gfx] [PATCH] drm/i915: Make vm eviction uninterruptible

2014-04-08 Thread Daniel Vetter
On Mon, Apr 7, 2014 at 11:58 PM, Ben Widawsky b...@bwidawsk.net wrote: Blocking important fixes for a test case is harmful to customers of our software. I won't argue past that. If you won't take it as is, add it to the JIRA task like you said. I'll carry this one around with my dynamic page

Re: [Intel-gfx] [PATCH] drm/i915: support address only i2c-over-aux transactions

2014-04-08 Thread Jani Nikula
On Mon, 07 Apr 2014, Daniel Vetter dan...@ffwll.ch wrote: On Mon, Apr 7, 2014 at 4:35 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Mon, Apr 7, 2014 at 5:37 AM, Jani Nikula jani.nik...@intel.com wrote: To support bare address requests used by the drm dp helpers. Signed-off-by: Jani Nikula

Re: [Intel-gfx] [PATCH 3/3] drm/i915/bdw: Use the GEN8 SRM when qeueing a flip

2014-04-08 Thread Damien Lespiau
On Tue, Apr 08, 2014 at 07:24:23AM +0100, Chris Wilson wrote: On Mon, Apr 07, 2014 at 11:20:14PM +0100, Damien Lespiau wrote: On Mon, Apr 07, 2014 at 01:59:17PM -0700, Ben Widawsky wrote: Cool. This explains the bad DERRMR values I was seeing in in error states. I'm honestly didn't check

Re: [Intel-gfx] i915 DVI resolution regression (3.13.7+)

2014-04-08 Thread Jani Nikula
On Tue, 08 Apr 2014, Daniel J Blueman dan...@quora.org wrote: Ville et al, It looks like commit e3ea8fa6beaf55fee64bf816f3b8a80ad733b2c2 (or another commit in 3.13.7) broke modes which require DVI-D dual-link, eg 2560x1440 with my panel. I don't see these modelines in 3.13.7 or later (eg

Re: [Intel-gfx] i915 DVI resolution regression (3.13.7+)

2014-04-08 Thread Daniel J Blueman
On 8 April 2014 15:14, Jani Nikula jani.nik...@linux.intel.com wrote: On Tue, 08 Apr 2014, Daniel J Blueman dan...@quora.org wrote: Ville et al, It looks like commit e3ea8fa6beaf55fee64bf816f3b8a80ad733b2c2 (or another commit in 3.13.7) broke modes which require DVI-D dual-link, eg 2560x1440

Re: [Intel-gfx] i915 DVI resolution regression (3.13.7+)

2014-04-08 Thread Daniel Vetter
On Tue, Apr 8, 2014 at 9:32 AM, Daniel J Blueman dan...@quora.org wrote: I am using a dual-link DVI-D to DVI-D cable to this monitor, since I previously couldn't get 2560x1440 via HDMI. If it isn't this commit, then it may be another commit in 3.13.7, albeit it feels less likely. Before we

Re: [Intel-gfx] [PATCH] drm/i915: support address only i2c-over-aux transactions

2014-04-08 Thread Daniel Vetter
On Tue, Apr 8, 2014 at 8:58 AM, Jani Nikula jani.nik...@intel.com wrote: Before the conversion to dp aux helpers there was a switch case [1] that ended up in msg_bytes = 3 for address only start/stop messages (MODE_I2C_START or MODE_I2C_STOP bit set [2]). With Alex's series in, but without my

[Intel-gfx] Fujitsu S6010 still woes (partially)

2014-04-08 Thread Thomas Richter
Hi Daniel, dear intel-experts, again I had the chance to test the latest intel-drm-nightly of the 3.14.0 kernel on the Siemens S6010 with its dreadful nso2501 DVO. Unfortunately, there are still a couple of issues here, and I also want to report on some progress and some workarounds. 1) Panning

Re: [Intel-gfx] [PATCH] drm/i915: Rename GEN8_PIPE_FLIP_DONE to PRIMARY_FLIP_DONE

2014-04-08 Thread Ville Syrjälä
On Tue, Apr 08, 2014 at 01:22:44AM +0100, Damien Lespiau wrote: It is now clear that this interrupt is for the primary plane and not something global to the pipe. It also matches what the spec calls it. Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä

[Intel-gfx] [PATCH] drm/i915: Remove misleading debug message

2014-04-08 Thread Damien Lespiau
haswell_write_eld() is also used on broadwell, so let's not explicitely mention Haswell. The rest of the function has plenty of debug output which will print the function name, so we know where we are anyway. Signed-off-by: Damien Lespiau damien.lesp...@intel.com ---

Re: [Intel-gfx] Fujitsu S6010 still woes (partially)

2014-04-08 Thread Ville Syrjälä
On Tue, Apr 08, 2014 at 11:48:14AM +0200, Thomas Richter wrote: Hi Daniel, dear intel-experts, again I had the chance to test the latest intel-drm-nightly of the 3.14.0 kernel on the Siemens S6010 with its dreadful nso2501 DVO. Unfortunately, there are still a couple of issues here, and I

Re: [Intel-gfx] Fujitsu S6010 still woes (partially)

2014-04-08 Thread Daniel Vetter
On Tue, Apr 8, 2014 at 11:48 AM, Thomas Richter rich...@rus.uni-stuttgart.de wrote: 3) Suspend to RAM: Whether with or without the quirk, s2ram is non-functioning, but *almost* functioning. The problem on the S6010 is again the ns2501. Unfortunately, I do not know which of the intel functions

Re: [Intel-gfx] Fujitsu S6010 still woes (partially)

2014-04-08 Thread Thomas Richter
Am 08.04.2014 13:52, schrieb Daniel Vetter: On Tue, Apr 8, 2014 at 11:48 AM, Thomas Richter Hm, my X30 also locks up here on resume. What hack do you apply to make the ns2501 driver get through resume? I don't care about black screen, but I just wonder whether my X30 has the same issue - atm it

Re: [Intel-gfx] [PATCH] drm/i915: Remove misleading debug message

2014-04-08 Thread Jani Nikula
On Tue, 08 Apr 2014, Damien Lespiau damien.lesp...@intel.com wrote: haswell_write_eld() is also used on broadwell, so let's not explicitely mention Haswell. The rest of the function has plenty of debug output which will print the function name, so we know where we are anyway. Signed-off-by:

Re: [Intel-gfx] Fujitsu S6010 still woes (partially)

2014-04-08 Thread Thomas Richter
Am 08.04.2014 13:37, schrieb Ville Syrjälä: On Tue, Apr 08, 2014 at 11:48:14AM +0200, Thomas Richter wrote: I saw the watermark issue on my S6010 too. I have no good explanation for it since low value in the register means the watermark is actually high. I know )-: So it's a mystery why

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Bring UP Power Wells before disabling RC6.

2014-04-08 Thread Ville Syrjälä
On Mon, Apr 07, 2014 at 02:36:20PM -0700, Ben Widawsky wrote: On Mon, Apr 07, 2014 at 05:01:46PM -0300, Rodrigo Vivi wrote: From: Deepak S deepa...@intel.com We need do forcewake before Disabling RC6, This is what the BIOS expects while going into suspend. v2: updated commit

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Bring UP Power Wells before disabling RC6.

2014-04-08 Thread S, Deepak
On 4/8/2014 6:13 PM, Ville Syrjälä wrote: On Mon, Apr 07, 2014 at 02:36:20PM -0700, Ben Widawsky wrote: On Mon, Apr 07, 2014 at 05:01:46PM -0300, Rodrigo Vivi wrote: From: Deepak S deepa...@intel.com We need do forcewake before Disabling RC6, This is what the BIOS expects while going into

Re: [Intel-gfx] [PATCH] drm/i915: support address only i2c-over-aux transactions

2014-04-08 Thread Alex Deucher
On Tue, Apr 8, 2014 at 4:03 AM, Daniel Vetter dan...@ffwll.ch wrote: On Tue, Apr 8, 2014 at 8:58 AM, Jani Nikula jani.nik...@intel.com wrote: Before the conversion to dp aux helpers there was a switch case [1] that ended up in msg_bytes = 3 for address only start/stop messages (MODE_I2C_START

Re: [Intel-gfx] [PATCH] drm/i915: support address only i2c-over-aux transactions

2014-04-08 Thread Alex Deucher
On Tue, Apr 8, 2014 at 9:09 AM, Christian König deathsim...@vodafone.de wrote: Am 08.04.2014 15:04, schrieb Alex Deucher: On Tue, Apr 8, 2014 at 4:03 AM, Daniel Vetter dan...@ffwll.ch wrote: On Tue, Apr 8, 2014 at 8:58 AM, Jani Nikula jani.nik...@intel.com wrote: Before the conversion to

Re: [Intel-gfx] [PATCH] drm/i915: support address only i2c-over-aux transactions

2014-04-08 Thread Christian König
Am 08.04.2014 15:04, schrieb Alex Deucher: On Tue, Apr 8, 2014 at 4:03 AM, Daniel Vetter dan...@ffwll.ch wrote: On Tue, Apr 8, 2014 at 8:58 AM, Jani Nikula jani.nik...@intel.com wrote: Before the conversion to dp aux helpers there was a switch case [1] that ended up in msg_bytes = 3 for

Re: [Intel-gfx] Fujitsu S6010 still woes (partially)

2014-04-08 Thread Ville Syrjälä
On Tue, Apr 08, 2014 at 02:17:10PM +0200, Thomas Richter wrote: Am 08.04.2014 13:37, schrieb Ville Syrjälä: On Tue, Apr 08, 2014 at 11:48:14AM +0200, Thomas Richter wrote: I saw the watermark issue on my S6010 too. I have no good explanation for it since low value in the register means

Re: [Intel-gfx] [PATCH 4/6] drm/i915: add flag to prevent dmesg spam on context banning

2014-04-08 Thread Mika Kuoppala
Rodrigo Vivi rodrigo.v...@gmail.com writes: From: Mika Kuoppala mika.kuopp...@linux.intel.com Piglit runner and QA are both looking at the dmesg for DRM_ERRORs with test cases. Add flag to stop_rings debugfs interface to prevent DRM_ERROR output when default context banning is being

Re: [Intel-gfx] Fujitsu S6010 still woes (partially)

2014-04-08 Thread Daniel Vetter
On Tue, Apr 8, 2014 at 2:05 PM, Thomas Richter rich...@rus.uni-stuttgart.de wrote: Am 08.04.2014 13:52, schrieb Daniel Vetter: On Tue, Apr 8, 2014 at 11:48 AM, Thomas Richter Hm, my X30 also locks up here on resume. What hack do you apply to make the ns2501 driver get through resume? I don't

Re: [Intel-gfx] [PATCH v4 3/3] drm/i915: New drm crtc property for varying the size of borders

2014-04-08 Thread Ville Syrjälä
On Wed, Mar 26, 2014 at 09:25:12AM +0530, akash.g...@intel.com wrote: From: Akash Goel akash.g...@intel.com This patch adds a new drm crtc property for varying the size of the horizontal vertical borers of the output/display window. This will control the output of Panel fitter. v2: Added

[Intel-gfx] [PATCH 07/15] drm/i915: vlv: check port power domain instead of only D0 for eDP VDD on

2014-04-08 Thread Imre Deak
Some platforms need additional power domains to be on in addition to the device D0 state to access the panel registers. Suggested by Daniel. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=76987 Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/intel_dp.c | 6 +-

[Intel-gfx] [PATCH 08/15] drm/i915: vlv: setup RPS min/max frequencies once during init time

2014-04-08 Thread Imre Deak
When enabling RPM on VLV, GT power save enabling becomes relatively frequent, so optimize it a bit. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/intel_pm.c | 66 + 1 file changed, 41 insertions(+), 25 deletions(-) diff --git

[Intel-gfx] [PATCH 03/15] drm/i915: vlv: add RC6 residency counters

2014-04-08 Thread Imre Deak
Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_debugfs.c | 5 + drivers/gpu/drm/i915/i915_reg.h | 3 +++ 2 files changed, 8 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c index 1e83ae4..02f1b39 100644 ---

Re: [Intel-gfx] [PATCH v4 3/3] drm/i915: New drm crtc property for varying the size of borders

2014-04-08 Thread Damien Lespiau
On Tue, Apr 08, 2014 at 05:31:06PM +0100, Damien Lespiau wrote: On Mon, Apr 07, 2014 at 04:06:34AM +, Goel, Akash wrote: Hi Ville, Please could you review this patch. You need a pass of checkpatch.pl in here. Not sure what happened with the coding style. Ignore this, I was looking

[Intel-gfx] [PATCH 06/15] drm/i915: get init power domain for gpu error state capture

2014-04-08 Thread Imre Deak
Since the state capture happens from a deferred work, we may drop the last power domain/RPM reference since the error got triggered (from an interrupt handler for example). I hit this by writing to the i915_wedged debugfs file. Signed-off-by: Imre Deak imre.d...@intel.com ---

[Intel-gfx] [PATCH 13/15] drm/i915: vlv: add gunit s0ix save/restore helpers

2014-04-08 Thread Imre Deak
Needed by the next patch adding RPM suspend/resume support to VLV. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_drv.c | 192 drivers/gpu/drm/i915/i915_drv.h | 62 + 2 files changed, 254 insertions(+) diff --git

[Intel-gfx] [PATCH 15/15] drm/i915: vlv: enable RPM

2014-04-08 Thread Imre Deak
Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_drv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index d94e10a..84d4298 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++

[Intel-gfx] [PATCH 1/5] dmr/i915: Displayport - Add a function to set the training pattern

2014-04-08 Thread Todd Previte
Adds a function to set the training pattern for Displayport. This is functionality required to establish more fine-grained control over the Displayport interface, both for operational reliability and compliance testing. Signed-off-by: Todd Previte tprev...@gmail.com ---

[Intel-gfx] [PATCH 0/5] RFC: Displayport Link Policy Maker Update

2014-04-08 Thread Todd Previte
This patch set lays the initial groundwork for updating the Displayport link policy maker in the i915 driver. These first 5 patches add modular functions that have been split out from the larger, monolithic ones present in the driver. The purpose of this work is the following: 1)

[Intel-gfx] [PATCH 2/5] drm/i915: Displayport - Add function to check link status

2014-04-08 Thread Todd Previte
Adds a function to check the link status across all lanes for Displayport. This is functionality required to establish more fine-grained control over the Displayport interface, both for operational reliability and compliance testing. Signed-off-by: Todd Previte tprev...@gmail.com ---

[Intel-gfx] [PATCH 4/5] drm/i915: Displayport - Add function for executing a single iteration of clock recovery

2014-04-08 Thread Todd Previte
Adds a function to execute a single iteration of the clock recovery sequence for Displayport. This is functionality required to establish more fine-grained control over the Displayport interface, both for operational reliability and compliance testing. Signed-off-by: Todd Previte

[Intel-gfx] [PATCH 3/5] drm/i915: Displayport - Add function to enable/disable scrambling on the main link

2014-04-08 Thread Todd Previte
Adds a function to enable and disable scrambling directly for the main link. This is functionality required to establish more fine-grained control over the Displayport interface, both for operational reliability and compliance testing. Signed-off-by: Todd Previte tprev...@gmail.com ---

[Intel-gfx] [PATCH 5/5] drm/i915: Displayport - Add function to execute a single iteration of channel equalization

2014-04-08 Thread Todd Previte
Adds a function to execute a single iteration of the channel equalization sequence for Displayport. This is functionality required to establish more fine-grained control over the Displayport interface, both for operational reliability and compliance testing. Signed-off-by: Todd Previte

Re: [Intel-gfx] [PATCH 09/24] drm/i915: Keep vblank interrupts enabled while enabling/disabling planes

2014-04-08 Thread Ville Syrjälä
On Mon, Apr 07, 2014 at 06:21:08PM -0300, Paulo Zanoni wrote: 2014-03-07 13:32 GMT-03:00 ville.syrj...@linux.intel.com: From: Ville Syrjälä ville.syrj...@linux.intel.com We may have use for vblank interrupts during plane enabling/disabling, so don't call drm_vblank_off() until planes are

Re: [Intel-gfx] [PATCH v4 3/3] drm/i915: New drm crtc property for varying the size of borders

2014-04-08 Thread Damien Lespiau
On Mon, Apr 07, 2014 at 04:06:34AM +, Goel, Akash wrote: Hi Ville, Please could you review this patch. You need a pass of checkpatch.pl in here. Not sure what happened with the coding style. -- Damien ___ Intel-gfx mailing list

Re: [Intel-gfx] [PATCH 08/24] drm/i915: Shuffle wait_for_vblank out of primary_enable/disable funcs

2014-04-08 Thread Ville Syrjälä
On Mon, Apr 07, 2014 at 05:27:41PM -0300, Paulo Zanoni wrote: 2014-03-07 13:32 GMT-03:00 ville.syrj...@linux.intel.com: From: Ville Syrjälä ville.syrj...@linux.intel.com Rather than have a wait_for_vblank() in the primary plane enable/disable funcs, move the wait_for_vblank() to happen

[Intel-gfx] [PATCH 01/15] drm/i915: vlv: clean up GTLC wake control/status register macros

2014-04-08 Thread Imre Deak
These will be needed by the upcoming VLV RPM helpers. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_gem.c | 5 +++-- drivers/gpu/drm/i915/i915_reg.h | 10 -- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c

[Intel-gfx] [PATCH 05/15] drm/i915: take init power domain for sysfs/debugfs entries where needed

2014-04-08 Thread Imre Deak
For simplicity take the init power domain, which puts the device into D0 and turns on all display power wells. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_debugfs.c | 17 +++-- drivers/gpu/drm/i915/i915_sysfs.c | 4 2 files changed, 19

[Intel-gfx] [PATCH 00/15] vlv: add support for RPM

2014-04-08 Thread Imre Deak
This adds the required helpers for saving/restoring HW context when going to D3 (and possibly to S0ix afterwards) state on VLV and then enables RPM. Since we depend on the RC6 power context mechanism to save some state this also touches generic RPM parts, so that RPM is only enabled after the

[Intel-gfx] [PATCH 02/15] drm/i915: vlv: clear master interrupt flag when disabling interrupts

2014-04-08 Thread Imre Deak
Not clearing this flag causes spurious interrupts at least in D3 state, so before enabling RPM we need to fix this. We were already setting this flag when enabling interrupts, only clearing it was missing. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_irq.c | 2 ++ 1

[Intel-gfx] [PATCH 09/15] drm/i915: vlv: factor out vlv_force_gfx_clock

2014-04-08 Thread Imre Deak
This will be needed by the VLV RPM helpers too. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_drv.c | 32 drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_pm.c | 16 ++-- 3 files changed, 35

[Intel-gfx] [PATCH 04/15] drm/i915: fix rc6 status debug print

2014-04-08 Thread Imre Deak
The parsing was incorrect for ILK and VLV. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/intel_pm.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c index a0527a1..869a4e3 100644

[Intel-gfx] [PATCH 11/15] drm/i915: vlv: disable RPM if RC6 is not enabled

2014-04-08 Thread Imre Deak
On VLV we depend on RC6 saving the GT render and media HW context before going to D3 state, so disable RPM if RC6 is not enabled. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/intel_drv.h | 2 ++ drivers/gpu/drm/i915/intel_pm.c | 29 - 2

[Intel-gfx] [PATCH 10/15] drm/i915: disable runtime PM until delayed RPS/RC6 enabling completes

2014-04-08 Thread Imre Deak
At least on some platforms we depend on RC6 being enabled for RPM, so disable RPM until the delayed RC6 enabling completes. For simplicity don't differentiate between platforms, those that don't have this dependency will enable RC6 only rarely during driver init and system resume. Signed-off-by:

[Intel-gfx] [PATCH 12/15] drm/i915: add various missing GTI/Gunit register definitions

2014-04-08 Thread Imre Deak
Needed by the VLV S0ix context save/restore helpers. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_reg.h | 43 - 1 file changed, 42 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_reg.h

[Intel-gfx] [PATCH 14/15] drm/i915: vlv: add runtime PM support

2014-04-08 Thread Imre Deak
Add runtime PM support for VLV, but leave it disabled. The next patch enables it. The suspend/resume sequence used is based on [1] and [2]. In practice we depend on the GT RC6 mechanism to save the HW context depending on the render and media power wells. By the time we run the runtime suspend

Re: [Intel-gfx] [PATCH 1/5] dmr/i915: Displayport - Add a function to set the training pattern

2014-04-08 Thread Paulo Zanoni
Hi 2014-04-08 14:47 GMT-03:00 Todd Previte tprev...@gmail.com: Adds a function to set the training pattern for Displayport. This is functionality required to establish more fine-grained control over the Displayport interface, both for operational reliability and compliance testing. Just a

Re: [Intel-gfx] [PATCH] drm/mm: Don't WARN if drm_mm_reserve_node

2014-04-08 Thread Ben Widawsky
On Mon, Apr 07, 2014 at 10:13:13PM -0700, Jesse Barnes wrote: On Mon, 7 Apr 2014 23:25:20 +0200 Daniel Vetter daniel.vet...@ffwll.ch wrote: Jesse's BIOS fb reconstruction code actually relies on the -ENOSPC return value to detect overlapping framebuffers (which the bios uses always when

Re: [Intel-gfx] Fujitsu S6010 still woes (partially)

2014-04-08 Thread Thomas Richter
On 08.04.2014 18:10, Daniel Vetter wrote: On Tue, Apr 8, 2014 at 2:05 PM, Thomas Richter Also, from the linux suspend mechanism, /usr/lib/pm-utils/sleep.d/99video is just useless or breaks more than it helps. I just removed it. It tries some weird workarounds that are not beneficial, and the

[Intel-gfx] [PATCH] dri3: Print log message

2014-04-08 Thread Adel Gadllah
This is based on the not yet merged dri3 branch, it simply adds a log message to avoid user confusion. Adel Gadllah (1): dri3: Print log message src/uxa/intel_dri3.c | 2 +- src/uxa/intel_driver.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) -- 1.9.0

[Intel-gfx] [PATCH] dri3: Print log message

2014-04-08 Thread Adel Gadllah
Whe currently only print a direct rendering: DRI2 Enabled which seems to confuse people so add a DRI3 enabled message as well when successfully initalizing dri3. Signed-off-by: Adel Gadllah adel.gadl...@gmail.com Reviewed-by: Keith Packard kei...@keithp.com --- src/uxa/intel_dri3.c | 2 +-

[Intel-gfx] [PATCH] tests/gen7_forcewake_mt: Don't set the GGTT bit in SRM command

2014-04-08 Thread bradley . d . volkin
From: Brad Volkin bradley.d.vol...@intel.com The command parser in newer kernels will reject it and setting this bit is not required for the actual test case. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=76670 Signed-off-by: Brad Volkin bradley.d.vol...@intel.com ---

Re: [Intel-gfx] [PATCH] tests/gem_exec_parse: Test for OACONTROL tracking

2014-04-08 Thread Volkin, Bradley D
Hi Daniel, we've merged the kernel change for this but not the test. I'm assuming we still want the test case. Brad On Thu, Mar 27, 2014 at 11:44:45AM -0700, Volkin, Bradley D wrote: From: Brad Volkin bradley.d.vol...@intel.com Signed-off-by: Brad Volkin bradley.d.vol...@intel.com ---

[Intel-gfx] [PATCH v2 5/15] drm/i915: take init power domain for sysfs/debugfs entries where needed

2014-04-08 Thread Imre Deak
v2: - make it actually compile, I managed to send the wrong version as v1 somehow Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_debugfs.c | 18 -- drivers/gpu/drm/i915/i915_sysfs.c | 4 2 files changed, 20 insertions(+), 2 deletions(-) diff

[Intel-gfx] [PATCH 1/3] drm/crtc: Add property for aspect ratio

2014-04-08 Thread Vandana Kannan
Added a property to enable user space to set aspect ratio. This patch contains declaration of the property and code to create the property. Signed-off-by: Vandana Kannan vandana.kan...@intel.com Cc: dri-de...@lists.freedesktop.org --- drivers/gpu/drm/drm_crtc.c | 31

[Intel-gfx] [PATCH 2/3] drm/edid: Check for user aspect ratio input

2014-04-08 Thread Vandana Kannan
In case user has specified an input for aspect ratio through the property, then the user space value for PAR would take preference over the value from CEA mode list. Signed-off-by: Vandana Kannan vandana.kan...@intel.com Cc: dri-de...@lists.freedesktop.org --- drivers/gpu/drm/drm_edid.c | 9

[Intel-gfx] [PATCH 3/3] drm/i915: Add aspect ratio property for HDMI

2014-04-08 Thread Vandana Kannan
Create and attach the drm property to set aspect ratio. If there is no user specified value, then PAR_NONE/Automatic option is set by default. User can select aspect ratio 4:3 or 16:9. The aspect ratio selected by user would come into effect with a mode set. Signed-off-by: Vandana Kannan

[Intel-gfx] [PATCH] drm/i915: Add more registers to the whitelist for mesa

2014-04-08 Thread bradley . d . volkin
From: Brad Volkin bradley.d.vol...@intel.com These are additional registers needed for performance monitoring and ARB_draw_indirect extensions in mesa. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=76719 Cc: Kenneth Graunke kenn...@whitecape.org Signed-off-by: Brad Volkin

[Intel-gfx] [PATCH 5/5] drm/i915:Use the coarse mechanism based on drm fd to dispatch the BSD command on BDW GT3

2014-04-08 Thread Zhao Yakui
The BDW GT3 has two independent BSD rings, which can be used to process the video commands. To be simpler, it is transparent to user-space driver/middleware. Instead the kernel driver will decide which ring is to dispatch the BSD video command. As every BSD ring is powerful, it is enough to

[Intel-gfx] [PATCH 3/5] drm/i915:Handle the irq interrupt for the second BSD ring

2014-04-08 Thread Zhao Yakui
Signed-off-by: Zhao Yakui yakui.z...@intel.com --- drivers/gpu/drm/i915/i915_irq.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c index bdda3b5..d5b1dd3 100644 --- a/drivers/gpu/drm/i915/i915_irq.c +++

[Intel-gfx] [PATCH 0/5] drm/i915: Add the support of dual BSD rings on BDW GT3

2014-04-08 Thread Zhao Yakui
This is the patch set that tries to add the support of dual BSD rings on BDW GT3. Based on hardware spec, the BDW GT3 has two independent BSD rings, which can be used to process the video commands. To be simpler, it is transparent to user-space driver/middleware. In such case the kernel driver

[Intel-gfx] [PATCH 2/5] drm/i915:Initialize the second BSD ring on BDW GT3 machine

2014-04-08 Thread Zhao Yakui
Based on the hardware spec, the BDW GT3 machine has two independent BSD ring that can be used to dispatch the video commands. So just initialize it. Signed-off-by: Zhao Yakui yakui.z...@intel.com --- drivers/gpu/drm/i915/i915_drv.c |4 +-- drivers/gpu/drm/i915/i915_drv.h |

[Intel-gfx] [PATCH 4/5] drm/i915:Add the VCS2 switch in Intel_ring_setup_status_page for Gen7 to remove the switch check warning

2014-04-08 Thread Zhao Yakui
The Gen7 doesn't have the second BSD ring. But it will complain the switch check warning message during compilation. So just add it to remove the switch check warning. Signed-off-by: Zhao Yakui yakui.z...@intel.com --- drivers/gpu/drm/i915/intel_ringbuffer.c |1 + 1 file changed, 1

[Intel-gfx] [PATCH 1/5] drm/i915: Split the BDW device definition to prepare for dual BSD rings on BDW GT3

2014-04-08 Thread Zhao Yakui
Based on the hardware spec, the BDW GT3 has the different configuration with the BDW GT1/GT2. So split the BDW device info definition. This is to do the preparation for adding the Dual BSD rings on BDW GT3 machine. Signed-off-by: Zhao Yakui yakui.z...@intel.com ---

Re: [Intel-gfx] i915 DVI resolution regression (3.13.7+)

2014-04-08 Thread Daniel J Blueman
Ville et al, On 8 April 2014 16:02, Daniel Vetter daniel.vet...@ffwll.ch wrote: On Tue, Apr 8, 2014 at 9:32 AM, Daniel J Blueman dan...@quora.org wrote: I am using a dual-link DVI-D to DVI-D cable to this monitor, since I previously couldn't get 2560x1440 via HDMI. If it isn't this commit,

Re: [Intel-gfx] i915 DVI resolution regression (3.13.7+)

2014-04-08 Thread Dave Airlie
On Tue, Apr 8, 2014 at 5:32 PM, Daniel J Blueman dan...@quora.org wrote: On 8 April 2014 15:14, Jani Nikula jani.nik...@linux.intel.com wrote: On Tue, 08 Apr 2014, Daniel J Blueman dan...@quora.org wrote: Ville et al, It looks like commit e3ea8fa6beaf55fee64bf816f3b8a80ad733b2c2 (or another

Re: [Intel-gfx] [PATCH] drm/i915: Make vm eviction uninterruptible

2014-04-08 Thread Ben Widawsky
On Tue, Apr 08, 2014 at 07:50:39AM +0100, Chris Wilson wrote: On Mon, Apr 07, 2014 at 02:58:34PM -0700, Ben Widawsky wrote: Blocking important fixes for a test case is harmful to customers of our software. I won't argue past that. If you won't take it as is, add it to the JIRA task like you

Re: [Intel-gfx] [PATCH] drm/i915: Make vm eviction uninterruptible

2014-04-08 Thread Ben Widawsky
On Tue, Apr 08, 2014 at 08:53:15AM +0200, Daniel Vetter wrote: On Mon, Apr 7, 2014 at 11:58 PM, Ben Widawsky b...@bwidawsk.net wrote: Blocking important fixes for a test case is harmful to customers of our software. I won't argue past that. If you won't take it as is, add it to the JIRA

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Bring UP Power Wells before disabling RC6.

2014-04-08 Thread Ben Widawsky
On Tue, Apr 08, 2014 at 06:22:52PM +0530, S, Deepak wrote: On 4/8/2014 6:13 PM, Ville Syrjälä wrote: On Mon, Apr 07, 2014 at 02:36:20PM -0700, Ben Widawsky wrote: On Mon, Apr 07, 2014 at 05:01:46PM -0300, Rodrigo Vivi wrote: From: Deepak S deepa...@intel.com We need do forcewake before

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Bring UP Power Wells before disabling RC6.

2014-04-08 Thread S, Deepak
On 4/9/2014 9:43 AM, Ben Widawsky wrote: On Tue, Apr 08, 2014 at 06:22:52PM +0530, S, Deepak wrote: On 4/8/2014 6:13 PM, Ville Syrjälä wrote: On Mon, Apr 07, 2014 at 02:36:20PM -0700, Ben Widawsky wrote: On Mon, Apr 07, 2014 at 05:01:46PM -0300, Rodrigo Vivi wrote: From: Deepak S