Re: [Intel-gfx] [PATCH 00/68] Broadwell 48b addressing and prelocations (no relocs)

2014-08-22 Thread Chris Wilson
On Thu, Aug 21, 2014 at 08:11:23PM -0700, Ben Widawsky wrote: The primary goal of these patches is to introduce what I've started calling, prelocations on Broadwell. A prelocation is like a relocation, except not. When a GPU client specifies a prelocation, it is instructing the kernel where in

Re: [Intel-gfx] [REGRESSION BISECTED] backlight control stops workin with 3.14 and later

2014-08-22 Thread Jani Nikula
On Fri, 22 Aug 2014, Bertrik Sikken bert...@sikken.nl wrote: On 19-8-2014 3:29, Jani Nikula wrote: On Tue, 19 Aug 2014, Bertrik Sikken bert...@sikken.nl wrote: On Sun, 17 Aug 2014, Bertrik Sikken bert...@sikken.nl wrote: On 15-8-2014 3:43, Jani Nikula wrote: On Thu, 14 Aug 2014, Bertrik

Re: [Intel-gfx] [PATCH 00/68] Broadwell 48b addressing and prelocations (no relocs)

2014-08-22 Thread Kenneth Graunke
On Friday, August 22, 2014 07:30:37 AM Chris Wilson wrote: On Thu, Aug 21, 2014 at 08:11:23PM -0700, Ben Widawsky wrote: The primary goal of these patches is to introduce what I've started calling, prelocations on Broadwell. A prelocation is like a relocation, except not. When a GPU client

[Intel-gfx] [PATCH] drm/i915: Add 180 degree primary plane rotation support

2014-08-22 Thread sonika . jindal
From: Sonika Jindal sonika.jin...@intel.com Primary planes support 180 degree rotation. Expose the feature through rotation drm property. v2: Calculating linear/tiled offsets based on pipe source width and height. Added 180 degree rotation support in ironlake_update_plane. v3: Checking if CRTC

Re: [Intel-gfx] [PATCH 00/68] Broadwell 48b addressing and prelocations (no relocs)

2014-08-22 Thread Chris Wilson
On Thu, Aug 21, 2014 at 11:59:04PM -0700, Kenneth Graunke wrote: On Friday, August 22, 2014 07:30:37 AM Chris Wilson wrote: On Thu, Aug 21, 2014 at 08:11:23PM -0700, Ben Widawsky wrote: The primary goal of these patches is to introduce what I've started calling, prelocations on Broadwell.

Re: [Intel-gfx] [PATCH 2/2] drm/i915: improve assert_panel_unlocked

2014-08-22 Thread Ville Syrjälä
On Thu, Aug 21, 2014 at 12:01:07PM -0300, Paulo Zanoni wrote: 2014-08-21 11:56 GMT-03:00 Ville Syrjälä ville.syrj...@linux.intel.com: On Thu, Aug 21, 2014 at 03:06:26PM +0300, Jani Nikula wrote: Fix assert_panel_unlocked for vlv/chv, and improve it a bit for non-LVDS. Also don't pretend it

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Add 180 degree primary plane rotation support

2014-08-22 Thread Ville Syrjälä
On Fri, Aug 22, 2014 at 09:29:56AM +0530, Jindal, Sonika wrote: On 8/21/2014 7:07 PM, Ville Syrjälä wrote: On Thu, Aug 21, 2014 at 05:14:35PM +0530, Jindal, Sonika wrote: On 8/21/2014 2:03 PM, Ville Syrjälä wrote: On Thu, Aug 21, 2014 at 11:45:41AM +0530, sonika.jin...@intel.com

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Add 180 degree primary plane rotation support

2014-08-22 Thread Jindal, Sonika
On 8/22/2014 1:44 PM, Ville Syrjälä wrote: On Fri, Aug 22, 2014 at 09:29:56AM +0530, Jindal, Sonika wrote: On 8/21/2014 7:07 PM, Ville Syrjälä wrote: On Thu, Aug 21, 2014 at 05:14:35PM +0530, Jindal, Sonika wrote: On 8/21/2014 2:03 PM, Ville Syrjälä wrote: On Thu, Aug 21, 2014 at

[Intel-gfx] [PATCH] drm/i915: Add 180 degree primary plane rotation support

2014-08-22 Thread sonika . jindal
From: Sonika Jindal sonika.jin...@intel.com Primary planes support 180 degree rotation. Expose the feature through rotation drm property. v2: Calculating linear/tiled offsets based on pipe source width and height. Added 180 degree rotation support in ironlake_update_plane. v3: Checking if CRTC

Re: [Intel-gfx] [PATCH] drm/i915/dp: Backlight PWM enable before BL Enable assert

2014-08-22 Thread Ville Syrjälä
On Thu, Aug 21, 2014 at 10:23:43AM -0700, Clint Taylor wrote: On 08/20/2014 04:23 AM, Ville Syrjälä wrote: On Mon, Aug 18, 2014 at 01:48:35PM -0700, clinton.a.tay...@intel.com wrote: From: Clint Taylor clinton.a.tay...@intel.com Backlight on delay uses PWM enable time to seperate PWM to

Re: [Intel-gfx] [REGRESSION BISECTED] backlight control stops workin with 3.14 and later

2014-08-22 Thread Bertrik Sikken
On Fri, 22 Aug 2014, Bertrik Sikken bert...@sikken.nl wrote: On 19-8-2014 3:29, Jani Nikula wrote: On Tue, 19 Aug 2014, Bertrik Sikken bert...@sikken.nl wrote: On Sun, 17 Aug 2014, Bertrik Sikken bert...@sikken.nl wrote: On 15-8-2014 3:43, Jani Nikula wrote: On Thu, 14 Aug 2014, Bertrik

Re: [Intel-gfx] [PATCH 1/2] drm/i915/bdw: Apply workarounds using the golden render state

2014-08-22 Thread Mika Kuoppala
Ville Syrjälä ville.syrj...@linux.intel.com writes: On Wed, Aug 20, 2014 at 03:19:17PM +0100, Arun Siluvery wrote: Workarounds for bdw are currently applied in init_clock_gating() but they are lost following a gpu reset. Some of the WA registers are part of register state context and they are

[Intel-gfx] [PATCH v2 2/2] drm/i915: improve assert_panel_unlocked

2014-08-22 Thread Jani Nikula
Fix assert_panel_unlocked for vlv/chv, and improve it a bit for non-LVDS. Also don't pretend it works for DDI. There's still work to do to get this right for eDP on PCH platforms, but this is a start. v2: WARN_ON(HAS_DDI) Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by:

[Intel-gfx] [PATCH] drm/i915: don't check for i830 in vlv specific code

2014-08-22 Thread Jani Nikula
Signed-off-by: Jani Nikula jani.nik...@intel.com --- drivers/gpu/drm/i915/intel_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 51b4cd29f932..83eabd758ed9 100644 ---

Re: [Intel-gfx] [PATCH 1/2] drm/i915/bdw: Apply workarounds using the golden render state

2014-08-22 Thread Siluvery, Arun
On 22/08/2014 12:06, Mika Kuoppala wrote: Ville Syrjälä ville.syrj...@linux.intel.com writes: On Wed, Aug 20, 2014 at 03:19:17PM +0100, Arun Siluvery wrote: Workarounds for bdw are currently applied in init_clock_gating() but they are lost following a gpu reset. Some of the WA registers are

Re: [Intel-gfx] [Mesa-dev] [PATCH] i965: First step toward prelocation

2014-08-22 Thread Alex Deucher
On Thu, Aug 21, 2014 at 11:12 PM, Ben Widawsky benjamin.widaw...@intel.com wrote: This was a quick proof of concept to show the new API for prelocating buffers. What are prelocated buffers? Alex It needs way more testing, to not ifdef the no-relocs, and to do a libdrm ABI dep bump. ---

[Intel-gfx] WARNING in 3.17-rc1 for i915

2014-08-22 Thread Larry Finger
My Toshiba A50 with graphics adapter described by 00:02.0 VGA compatible controller [0300]: Intel Corporation 4th Gen Core Processor Integrated Graphics Controller [8086:0416] (rev 06) gets the following warning on 3.17-rc1: [ 1271.563533] [ cut here ] [ 1271.563568]

Re: [Intel-gfx] [PATCH] drm/i915: don't check for i830 in vlv specific code

2014-08-22 Thread Ville Syrjälä
On Fri, Aug 22, 2014 at 03:06:35PM +0300, Jani Nikula wrote: Signed-off-by: Jani Nikula jani.nik...@intel.com --- drivers/gpu/drm/i915/intel_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_display.c

Re: [Intel-gfx] [PATCH] drm/i915/dp: Backlight PWM enable before BL Enable assert

2014-08-22 Thread Jani Nikula
+Art On Thu, 21 Aug 2014, Clint Taylor clinton.a.tay...@intel.com wrote: There is also a need to add this delay when turning off the PWM enable bit through intel_panel_disable_backlight(). If the PWM enable bit is turned off while the PWM signal is high then the signal remains high. If

Re: [Intel-gfx] [PATCH 00/68] Broadwell 48b addressing and prelocations (no relocs)

2014-08-22 Thread Daniel Vetter
On Fri, Aug 22, 2014 at 9:03 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: If a GPU client uses only prelocations, the relocation process can be entirely skipped. This sounds like a big win initially, Close to zero if the client uses existing interfaces. -Chris Chris, I don't

Re: [Intel-gfx] [PATCH 00/68] Broadwell 48b addressing and prelocations (no relocs)

2014-08-22 Thread Chris Wilson
On Fri, Aug 22, 2014 at 03:30:12PM +0200, Daniel Vetter wrote: On Fri, Aug 22, 2014 at 9:03 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: If a GPU client uses only prelocations, the relocation process can be entirely skipped. This sounds like a big win initially, Close to

[Intel-gfx] [PATCH] drm/i915: Differentiate between LLC or snooped for the user

2014-08-22 Thread Chris Wilson
Rather than describing an object as either snooped or LLC, we can do better as we should know what machine we are running on! Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk --- drivers/gpu/drm/i915/i915_debugfs.c | 4 ++-- drivers/gpu/drm/i915/i915_drv.h | 4 +++-

[Intel-gfx] [PATCH v2 10.1/14] drm/i915: Reset power sequencer pipe tracking when disp2d is off

2014-08-22 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com The power sequencer loses its state when the disp2d power well is down. Clear the dev_priv-pps_pipe tracking so that the power sequencer state gets reinitialized the next time it's needed. v2: Fix the pps_mutex vs. power_domain mutex deadlock by

Re: [Intel-gfx] [PATCH] drm/i915/dp: Backlight PWM enable before BL Enable assert

2014-08-22 Thread Runyan, Arthur J
I'll check it out. I haven't heard any complaint about this before, but it may be one of those things that started back on i745 and never got documented. -Original Message- From: Jani Nikula [mailto:jani.nik...@linux.intel.com] Sent: Friday, August 22, 2014 6:07 AM To: Taylor, Clinton

Re: [Intel-gfx] [Mesa-dev] [PATCH] i965: First step toward prelocation

2014-08-22 Thread Ben Widawsky
On Fri, Aug 22, 2014 at 08:15:28AM -0400, Alex Deucher wrote: On Thu, Aug 21, 2014 at 11:12 PM, Ben Widawsky benjamin.widaw...@intel.com wrote: This was a quick proof of concept to show the new API for prelocating buffers. What are prelocated buffers?

[Intel-gfx] [PATCH 0/2] Apply BDW workarounds using LRIs in render init fn

2014-08-22 Thread Arun Siluvery
Workarounds for BDW are applied using LRIs during render ring initialization. Only those WA registers that are part of register state are initialized in this fn, remaining are still in its current place init_clock_gating() which are not affected by a gpu reset. I can send another patch where they

[Intel-gfx] [PATCH 2/2] drm/i915/bdw: Export workaround data to debugfs

2014-08-22 Thread Arun Siluvery
The workarounds that are applied are exported to a debugfs file; this is used to verify their state after the test case (reset or suspend/resume etc). This patch is only required to support i-g-t. Signed-off-by: Arun Siluvery arun.siluv...@linux.intel.com --- drivers/gpu/drm/i915/i915_debugfs.c

[Intel-gfx] [PATCH 1/2] drm/i915/bdw: Apply workarounds in render ring init function

2014-08-22 Thread Arun Siluvery
For BDW workarounds are currently initialized in init_clock_gating() but they are lost during reset, suspend/resume etc; this patch moves the WAs that are part of register state context to render ring init fn otherwise default context ends up with incorrect values as they don't get initialized

[Intel-gfx] [PATCH] igt/gem_workarounds: igt to test workaround registers

2014-08-22 Thread Arun Siluvery
Some of the workarounds are lost followed by a gpu reset, suspend/resume; this patch adds a test which compares register state before and after the test scenario. This test currently verifies only bdw workarounds. v2: address patch cleanup comments (ThomasW) Add binary to ignore list and use

Re: [Intel-gfx] [PATCH 00/68] Broadwell 48b addressing and prelocations (no relocs)

2014-08-22 Thread Daniel Vetter
On Fri, Aug 22, 2014 at 3:38 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Fri, Aug 22, 2014 at 03:30:12PM +0200, Daniel Vetter wrote: On Fri, Aug 22, 2014 at 9:03 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: If a GPU client uses only prelocations, the relocation process can

Re: [Intel-gfx] [PATCH 00/68] Broadwell 48b addressing and prelocations (no relocs)

2014-08-22 Thread Daniel Vetter
On Fri, Aug 22, 2014 at 3:38 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Fri, Aug 22, 2014 at 03:30:12PM +0200, Daniel Vetter wrote: On Fri, Aug 22, 2014 at 9:03 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: If a GPU client uses only prelocations, the relocation process can

[Intel-gfx] Updated drm-intel-testing

2014-08-22 Thread Daniel Vetter
Hi all, New -testing cycle with cool stuff: - basic code for execlist, which is the fancy new cmd submission on gen8. Still disabled by default (Ben, Oscar Mateo, Thomas Daniel et al) - remove the useless usage of console_lock for I915_FBDEV=n (Chris) - clean up relations between ctx and ppgtt