Re: [Intel-gfx] [PATCH v4 1/5] drm/dp: Add AUX channel infrastructure

2014-02-04 Thread Jani Nikula
On Tue, 21 Jan 2014, Thierry Reding thierry.red...@gmail.com wrote: This is a superset of the current i2c_dp_aux bus functionality and can be used to transfer native AUX in addition to I2C-over-AUX messages. Helpers are provided to read and write the DPCD, either blockwise or byte-wise. Many

Re: [Intel-gfx] [PATCH v4 4/5] drm/dp: Allow registering AUX channels as I2C busses

2014-02-04 Thread Jani Nikula
On Tue, 21 Jan 2014, Thierry Reding thierry.red...@gmail.com wrote: Implements an I2C-over-AUX I2C adapter on top of the generic drm_dp_aux infrastructure. It extracts the retry logic from existing drivers, which should help in porting those drivers to this new helper. Reviewed-by: Alex

Re: [Intel-gfx] [PATCH 02/13] drm/i915: Implement command buffer parsing logic

2014-02-04 Thread Daniel Vetter
On Mon, Feb 03, 2014 at 03:00:19PM -0800, Volkin, Bradley D wrote: Ping. Daniel or Chris, can one of you clarify this request? Thanks. I've been enjoying fosdem ... On Thu, Jan 30, 2014 at 10:05:27AM -0800, Volkin, Bradley D wrote: On Thu, Jan 30, 2014 at 03:07:15AM -0800, Daniel Vetter

Re: [Intel-gfx] [PATCH] intel: Merge i915_drm.h with cmd parser define

2014-02-04 Thread Daniel Vetter
On Thu, Jan 30, 2014 at 09:28:25AM -0800, Volkin, Bradley D wrote: On Thu, Jan 30, 2014 at 01:20:57AM -0800, Daniel Vetter wrote: On Wed, Jan 29, 2014 at 02:26:12PM -0800, Volkin, Bradley D wrote: On Wed, Jan 29, 2014 at 02:13:21PM -0800, Chris Wilson wrote: On Wed, Jan 29, 2014 at

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Adding VBT fields to support eDP DRRS feature

2014-02-04 Thread Daniel Vetter
On Mon, Feb 03, 2014 at 09:13:17AM +0530, Vandana Kannan wrote: Again, everywhere else in intel_bios.c we use panel_type, directly as it is in VBT, 0-based. Are you saying it's all wrong? panel_type can't be 1-based in this one instance, and 0-based in all other instances, right? This

Re: [Intel-gfx] [PATCH 08/28] drm/i915: Fix IVB GT2 WaDisableDopClockGating and WaDisablePSDDualDispatchEnable

2014-02-04 Thread Daniel Vetter
On Fri, Jan 31, 2014 at 08:05:37AM -0500, Rodrigo Vivi wrote: Both registers must be programmed for the Mode bit to be valid. DevIVB:GT2 ... So I also agree ;) Maybe you should improve the commit message now that we are sure, but anyway: I've added a little not to the commit message when

Re: [Intel-gfx] [PATCH] drm/i915: Add Baytrail PSR Support.

2014-02-04 Thread Daniel Vetter
On Sat, Feb 01, 2014 at 11:34:02AM +, Chris Wilson wrote: On Wed, Jan 29, 2014 at 08:21:41PM +0100, Daniel Vetter wrote: On Wed, Jan 29, 2014 at 12:55:35PM -0200, Rodrigo Vivi wrote: This patch adds PSR Support to Baytrail. Baytrail cannot easily detect screen updates and force

Re: [Intel-gfx] [PATCH] drm/i915: Add Baytrail PSR Support.

2014-02-04 Thread Daniel Vetter
On Thu, Jan 30, 2014 at 03:01:08PM -0200, Rodrigo Vivi wrote: On Thu, Jan 30, 2014 at 11:02 AM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Wed, Jan 29, 2014 at 01:50:06PM -0200, Rodrigo Vivi wrote: @@ -7501,6 +7501,9 @@ static void intel_crtc_update_cursor(struct drm_crtc *crtc,

Re: [Intel-gfx] [PATCH 0/5] Android build system clean-up

2014-02-04 Thread Damien Lespiau
On Fri, Jan 31, 2014 at 02:57:35PM +, rafael.barba...@intel.com wrote: From: Rafael Barbalho rafael.barba...@intel.com IGT in android still had some hang-ups from the initial porting, we were re-compiling the lib directory every time for each tool or test binary. It also could get its

Re: [Intel-gfx] [PATCH] drm/i915: Introduce mapping of user pages into video memory (userptr) ioctl

2014-02-04 Thread Daniel Vetter
On Mon, Feb 03, 2014 at 03:28:37PM +, Tvrtko Ursulin wrote: On 01/29/2014 08:34 PM, Daniel Vetter wrote: Actually I've found something else to complain about: On Tue, Jan 28, 2014 at 2:16 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: +#define I915_USERPTR_READ_ONLY 0x1 This

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of acthd based guilty batch search

2014-02-04 Thread Daniel Vetter
On Thu, Jan 30, 2014 at 07:04:44PM +0200, Mika Kuoppala wrote: As we seek the guilty batch using request and hangcheck score, this code is not needed anymore. v2: Rebase. Passing dev_priv instead of getting it from last_ring Signed-off-by: Mika Kuoppala mika.kuopp...@intel.com

Re: [Intel-gfx] [PATCH v6] drm/i915/vlv: WA to fix Voltage not getting dropped to Vmin when Gfx is power gated.

2014-02-04 Thread Daniel Vetter
On Fri, Jan 31, 2014 at 11:09:47PM +0530, S, Deepak wrote: On 1/31/2014 10:40 PM, Ville Syrjälä wrote: On Thu, Jan 30, 2014 at 11:08:16PM +0530, deepa...@intel.com wrote: From: Deepak S deepa...@intel.com When we enter RC6 and GFX Clocks are off, the voltage remains higher than Vmin. When

Re: [Intel-gfx] [PATCH intel-gpu-tools] gem_ring_sync_copy: Add a ring to ring synchronization test

2014-02-04 Thread Damien Lespiau
On Wed, Jan 29, 2014 at 04:17:37PM +, Damien Lespiau wrote: +static void run_test(data_t *data, enum ring r1, enum ring r2, enum test test) +{ + struct ring_ops *r1_ops = ops[r1]; + struct ring_ops *r2_ops = ops[r2]; + drm_intel_bo *a, *b, *c; + + a = bo_create(data,

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Treat using a purged buffer as a source of EFAULT

2014-02-04 Thread Daniel Vetter
On Fri, Jan 31, 2014 at 11:34:58AM +, Chris Wilson wrote: Since a purged buffer is one without any associated pages, attempting to use it should generate EFAULT rather than EINVAL, as it is not strictly an invalid parameter. Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Care to

Re: [Intel-gfx] [PATCH 00/11] Enabling 180 degree rotation for sprite and crtc planes

2014-02-04 Thread Daniel Vetter
On Mon, Feb 03, 2014 at 11:59:05AM +0530, Sagar Arun Kamble wrote: On Fri, 2014-01-31 at 22:38 +0200, Ville Syrjälä wrote: On Sat, Feb 01, 2014 at 12:40:36AM +0530, sagar.a.kam...@intel.com wrote: From: Sagar Kamble sagar.a.kam...@intel.com With these patches 180 degree rotation for

Re: [Intel-gfx] [PATCH] drm/i915: release mutex in i915_gem_init()'s error path

2014-02-04 Thread Daniel Vetter
On Fri, Jan 31, 2014 at 05:14:02PM +0200, Mika Kuoppala wrote: Found with smatch. Signed-off-by: Mika Kuoppala mika.kuopp...@intel.com Both smatch patches merged to dinq, thanks. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch

Re: [Intel-gfx] [PATCH 1/1] demos: Add intel_plane_rotate test to verify rotation of planes and crtc

2014-02-04 Thread Daniel Vetter
On Sat, Feb 01, 2014 at 12:43:48AM +0530, sagar.a.kam...@intel.com wrote: From: Sagar Kamble sagar.a.kam...@intel.com This test will verify the 180 degree rotation of sprite and crtc planes. It will allow user to control rotation separately for crtc and sprite planes. Signed-off-by: Sagar

Re: [Intel-gfx] [PATCH 1/1] demos: Add intel_plane_rotate test to verify rotation of planes and crtc

2014-02-04 Thread Chris Wilson
On Tue, Feb 04, 2014 at 12:18:24PM +0100, Daniel Vetter wrote: On Sat, Feb 01, 2014 at 12:43:48AM +0530, sagar.a.kam...@intel.com wrote: From: Sagar Kamble sagar.a.kam...@intel.com This test will verify the 180 degree rotation of sprite and crtc planes. It will allow user to control

Re: [Intel-gfx] [PATCH 1/1] demos: Add intel_plane_rotate test to verify rotation of planes and crtc

2014-02-04 Thread Damien Lespiau
On Tue, Feb 04, 2014 at 12:18:24PM +0100, Daniel Vetter wrote: On Sat, Feb 01, 2014 at 12:43:48AM +0530, sagar.a.kam...@intel.com wrote: From: Sagar Kamble sagar.a.kam...@intel.com This test will verify the 180 degree rotation of sprite and crtc planes. It will allow user to control

Re: [Intel-gfx] [PATCH 0/5] Add power feature debugfs disabling

2014-02-04 Thread Daniel Vetter
On Fri, Jan 31, 2014 at 03:42:47PM -0600, jeff.mc...@intel.com wrote: From: Jeff McGee jeff.mc...@intel.com This series has recently been accepted into the Haswell Android kernel and helps with debugging and profiling these power features. I would like it to be considered for upstream

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Add RPS debugfs manual mode

2014-02-04 Thread Daniel Vetter
On Fri, Jan 31, 2014 at 03:42:48PM -0600, jeff.mc...@intel.com wrote: From: Jeff McGee jeff.mc...@intel.com RPS manual mode disables/ignores load-based inputs and allows render performance state to be controlled externally. The enabling of manual mode and setting of desired frequency is done

Re: [Intel-gfx] [PATCH 0/5] Add power feature debugfs disabling

2014-02-04 Thread Daniel Vetter
On Sat, Feb 01, 2014 at 05:14:22PM +, Chris Wilson wrote: On Fri, Jan 31, 2014 at 03:42:47PM -0600, jeff.mc...@intel.com wrote: From: Jeff McGee jeff.mc...@intel.com This series has recently been accepted into the Haswell Android kernel and helps with debugging and profiling these

Re: [Intel-gfx] [PATCH 1/1] demos: Add intel_plane_rotate test to verify rotation of planes and crtc

2014-02-04 Thread Sagar Arun Kamble
On Tue, 2014-02-04 at 11:25 +, Damien Lespiau wrote: On Tue, Feb 04, 2014 at 12:18:24PM +0100, Daniel Vetter wrote: On Sat, Feb 01, 2014 at 12:43:48AM +0530, sagar.a.kam...@intel.com wrote: From: Sagar Kamble sagar.a.kam...@intel.com This test will verify the 180 degree rotation

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Treat using a purged buffer as a source of EFAULT

2014-02-04 Thread Chris Wilson
On Tue, Feb 04, 2014 at 12:05:13PM +0100, Daniel Vetter wrote: On Fri, Jan 31, 2014 at 11:34:58AM +, Chris Wilson wrote: Since a purged buffer is one without any associated pages, attempting to use it should generate EFAULT rather than EINVAL, as it is not strictly an invalid parameter.

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Add RPS debugfs manual mode

2014-02-04 Thread Chris Wilson
On Tue, Feb 04, 2014 at 12:31:37PM +0100, Daniel Vetter wrote: On Fri, Jan 31, 2014 at 03:42:48PM -0600, jeff.mc...@intel.com wrote: From: Jeff McGee jeff.mc...@intel.com RPS manual mode disables/ignores load-based inputs and allows render performance state to be controlled externally.

Re: [Intel-gfx] [PATCH 1/1] demos: Add intel_plane_rotate test to verify rotation of planes and crtc

2014-02-04 Thread Damien Lespiau
On Tue, Feb 04, 2014 at 05:05:47PM +0530, Sagar Arun Kamble wrote: On Tue, 2014-02-04 at 11:25 +, Damien Lespiau wrote: On Tue, Feb 04, 2014 at 12:18:24PM +0100, Daniel Vetter wrote: On Sat, Feb 01, 2014 at 12:43:48AM +0530, sagar.a.kam...@intel.com wrote: From: Sagar Kamble

Re: [Intel-gfx] [PATCH 1/1] demos: Add intel_plane_rotate test to verify rotation of planes and crtc

2014-02-04 Thread Ville Syrjälä
On Tue, Feb 04, 2014 at 11:46:46AM +, Damien Lespiau wrote: On Tue, Feb 04, 2014 at 05:05:47PM +0530, Sagar Arun Kamble wrote: On Tue, 2014-02-04 at 11:25 +, Damien Lespiau wrote: On Tue, Feb 04, 2014 at 12:18:24PM +0100, Daniel Vetter wrote: On Sat, Feb 01, 2014 at 12:43:48AM

[Intel-gfx] [PATCH] fix make distcheck

2014-02-04 Thread Daniel Vetter
Bunch of explicit include paths needed adjustments and eviction_common.c needs to be added to the dist files. This has been broken in the following three commits: commit 42bcd05eb3f1545fbf9c397c3f37c3f6a27c5da4 Author: Tvrtko Ursulin tvrtko.ursu...@intel.com Date: Mon Feb 3 10:59:41 2014 +

Re: [Intel-gfx] [PATCH 1/1] demos: Add intel_plane_rotate test to verify rotation of planes and crtc

2014-02-04 Thread Damien Lespiau
On Tue, Feb 04, 2014 at 02:08:27PM +0200, Ville Syrjälä wrote: On Tue, Feb 04, 2014 at 11:46:46AM +, Damien Lespiau wrote: On Tue, Feb 04, 2014 at 05:05:47PM +0530, Sagar Arun Kamble wrote: On Tue, 2014-02-04 at 11:25 +, Damien Lespiau wrote: On Tue, Feb 04, 2014 at 12:18:24PM

[Intel-gfx] [PATCH] [RFC] drm/i915: Generate a hang error code

2014-02-04 Thread Ben Widawsky
We get a large number of bugs which have a, hey I have that too because they see a GPU hang in dmesg. While two machines of the same model having a GPU hang is indeed a coincidence, it is far from enough evidence to suggest they are the same. In order to reduce this effect, and hopefully get

[Intel-gfx] [PATCH v7] drm/i915: Reorganize display pipe register accesses

2014-02-04 Thread Antti Koskipaa
RFCv2: Reorganize array indexing so that full offsets can be used as is. It makes grepping for registers in i915_reg.h much easier. Also move offset arrays to intel_device_info. v1: Fixed offsets for VLV, proper eDP handling v2: Fixed BCLRPAT, PIPESRC, PIPECONF and DSP* macros. v3: Added EDP

Re: [Intel-gfx] [PATCH 2/4] tests/eviction_common: Avoid submitting duplicate objects

2014-02-04 Thread Daniel Vetter
On Mon, Feb 03, 2014 at 10:59:41AM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin tvrtko.ursu...@intel.com Make sure selection loop does not generate duplicates when it picks a subset of objects for a single exec buffer. Signed-off-by: Tvrtko Ursulin tvrtko.ursu...@intel.com I've

Re: [Intel-gfx] [PATCH] [RFC] drm/i915: Generate a hang error code

2014-02-04 Thread Daniel Vetter
On Tue, Feb 4, 2014 at 1:18 PM, Ben Widawsky benjamin.widaw...@intel.com wrote: We get a large number of bugs which have a, hey I have that too because they see a GPU hang in dmesg. While two machines of the same model having a GPU hang is indeed a coincidence, it is far from enough evidence

Re: [Intel-gfx] [PATCH v3 2/6] x86: Add Intel graphics stolen memory quirk for gen2 platforms

2014-02-04 Thread Ville Syrjälä
Hi x86 folks, Ping on getting the gen2 stolen memory early quirk patches into the x86 tree. From our side Daniel and Chris both seemed happy with them, so I'd like to get them in at some point. -- Ville Syrjälä Intel OTC ___ Intel-gfx mailing list

[Intel-gfx] [RFC PATCH 7/7] drm/i915/dp: use the new drm helpers for dp i2c-over-aux

2014-02-04 Thread Jani Nikula
The main differences are: * Many of the native aux differences mentioned in the relevant commit apply. * Native aux and i2c-over-aux defer timeouts are increased to be safe for all use cases instead of depending on DP device type and properties. * i2c start/stop/reset are not done. * i2c

[Intel-gfx] [RFC PATCH 5/7] drm/i915/dp: use the new drm helpers for dp aux

2014-02-04 Thread Jani Nikula
The main differences are: * Native aux has retry limit of 7 instead of infinite retry. * Sleep in native reply defer increases from udelay(100) to usleep_range(400, 500). * Unknown native reply results in retry instead of -EIO. * Lower level -EBUSY results in retry instead of fail. * Write

[Intel-gfx] [RFC PATCH 4/7] drm/i915/dp: move edp vdd enable/disable at a lower level in i2c-over-aux

2014-02-04 Thread Jani Nikula
This is prep work for conversion to generic drm i2c-over-aux helpers where we won't have the function to do this at. Signed-off-by: Jani Nikula jani.nik...@intel.com --- drivers/gpu/drm/i915/intel_dp.c |9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[Intel-gfx] [RFC PATCH 3/7] drm/i915/dp: split edp_panel_vdd_on() for reuse

2014-02-04 Thread Jani Nikula
Introduce _edp_panel_vdd_on() that returns true if the call enabled vdd, and a matching disable is needed. Keep edp_panel_vdd_on() as a helper for when it is expected the vdd is off. Signed-off-by: Jani Nikula jani.nik...@intel.com --- drivers/gpu/drm/i915/intel_dp.c | 22

[Intel-gfx] [RFC PATCH 2/7] drm/i915/dp: fix dp aux native read return value checks

2014-02-04 Thread Jani Nikula
There's some confusion between ints and bools. Signed-off-by: Jani Nikula jani.nik...@intel.com --- drivers/gpu/drm/i915/intel_dp.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index

[Intel-gfx] [RFC PATCH 6/7] drm/i915/dp: move dp aux ch register init to aux init

2014-02-04 Thread Jani Nikula
Signed-off-by: Jani Nikula jani.nik...@intel.com --- drivers/gpu/drm/i915/intel_dp.c | 45 ++- 1 file changed, 25 insertions(+), 20 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 835f16da80af..0a8a2b189ed0

[Intel-gfx] [RFC PATCH 1/7] drm/i915/dp: clean up cargo culted intel_dp_aux_native_read_retry() usage

2014-02-04 Thread Jani Nikula
intel_dp_aux_native_read_retry() is only needed when the sink might be asleep. Use the regular read without retries otherwise. Signed-off-by: Jani Nikula jani.nik...@intel.com --- drivers/gpu/drm/i915/intel_dp.c | 33 + 1 file changed, 13 insertions(+), 20

[Intel-gfx] [RFC PATCH 0/7] drm/i915/dp: convert to new aux channel helpers

2014-02-04 Thread Jani Nikula
These are based on drm-intel-nightly plus Thierry's aux channel infrastructure patches [1], and supersede my dp aux fixes [2]. Patches 1-4 are prep work and fixes to our current code. Patches 5 and 7 do the actual conversion for native aux and i2c over aux, respectively. Patch 6 is minor cleanup

Re: [Intel-gfx] [PATCH] drm/i915: Add Baytrail PSR Support.

2014-02-04 Thread Rodrigo Vivi
In the case of a moving cursor that means indefinitely. That's true... So I think we really need a work queue delaying the enable. Or do you have any better idea? Yeah, sounds like we need a delayed work-queue to re-enable psr, also for gtt mmap writes. See Chris' latest crazy example of

[Intel-gfx] [RFC PATCH] drm/i915: Prefault the entire object on first page fault

2014-02-04 Thread Chris Wilson
Inserting additional PTEs has no side-effect for us as the pfn are fixed for the entire time the object is resident in the global GTT. The downside is that we pay the entire cost of faulting the object upon the first hit, for which we in return receive the benefit of removing the per-page faulting

Re: [Intel-gfx] [RFC PATCH] drm/i915: Prefault the entire object on first page fault

2014-02-04 Thread Daniel Vetter
On Tue, Feb 04, 2014 at 01:30:19PM +, Chris Wilson wrote: Inserting additional PTEs has no side-effect for us as the pfn are fixed for the entire time the object is resident in the global GTT. The downside is that we pay the entire cost of faulting the object upon the first hit, for which

Re: [Intel-gfx] [RFC PATCH] drm/i915: Prefault the entire object on first page fault

2014-02-04 Thread Chris Wilson
On Tue, Feb 04, 2014 at 03:15:26PM +0100, Daniel Vetter wrote: On Tue, Feb 04, 2014 at 03:12:49PM +0100, Daniel Vetter wrote: On Tue, Feb 04, 2014 at 01:30:19PM +, Chris Wilson wrote: Inserting additional PTEs has no side-effect for us as the pfn are fixed for the entire time the

Re: [Intel-gfx] [RFC PATCH 1/7] drm/i915/dp: clean up cargo culted intel_dp_aux_native_read_retry() usage

2014-02-04 Thread Ville Syrjälä
On Tue, Feb 04, 2014 at 03:40:44PM +0200, Jani Nikula wrote: intel_dp_aux_native_read_retry() is only needed when the sink might be asleep. Use the regular read without retries otherwise. I guess I should repeat here what I mentioned to Jani: The DP spec seems to indicate that AUX transactions

[Intel-gfx] [RFC] tests: add gem_fence_upload

2014-02-04 Thread Chris Wilson
This test demonstrates the performance cliff clients face when they unwittingly use too many fenced surfaces in a looped upload. Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk --- Note that pass/fail here is based on arbitrary value, and passing the test is a wishlist item. Is it worthwhile

Re: [Intel-gfx] [RFC PATCH] drm/i915: Prefault the entire object on first page fault

2014-02-04 Thread Daniel Vetter
On Tue, Feb 04, 2014 at 03:12:49PM +0100, Daniel Vetter wrote: On Tue, Feb 04, 2014 at 01:30:19PM +, Chris Wilson wrote: Inserting additional PTEs has no side-effect for us as the pfn are fixed for the entire time the object is resident in the global GTT. The downside is that we pay the

Re: [Intel-gfx] [PATCH] drm/i915: Add Baytrail PSR Support.

2014-02-04 Thread Daniel Vetter
On Tue, Feb 04, 2014 at 11:03:25AM -0200, Rodrigo Vivi wrote: In the case of a moving cursor that means indefinitely. That's true... So I think we really need a work queue delaying the enable. Or do you have any better idea? Yeah, sounds like we need a delayed work-queue to re-enable

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Treat using a purged buffer as a source of EFAULT

2014-02-04 Thread Daniel Vetter
On Tue, Feb 04, 2014 at 11:36:39AM +, Chris Wilson wrote: On Tue, Feb 04, 2014 at 12:05:13PM +0100, Daniel Vetter wrote: On Fri, Jan 31, 2014 at 11:34:58AM +, Chris Wilson wrote: Since a purged buffer is one without any associated pages, attempting to use it should generate EFAULT

Re: [Intel-gfx] [PATCH 3/4] tests/gem_userptr_blits: Expanded userptr test cases

2014-02-04 Thread Daniel Vetter
On Mon, Feb 03, 2014 at 10:59:42AM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin tvrtko.ursu...@intel.com A set of userptr test cases to support the new feature. For the eviction and swapping stress testing I have extracted some common behaviour from gem_evict_everything and made both

[Intel-gfx] [PATCH] tests: Add gem_madvise

2014-02-04 Thread Chris Wilson
Exercise that calling madvise produces expected results Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk --- tests/.gitignore | 1 + tests/Makefile.sources | 1 + tests/gem_madvise.c| 155 + 3 files changed, 157 insertions(+)

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Add RPS debugfs manual mode

2014-02-04 Thread Jeff McGee
On Tue, Feb 04, 2014 at 11:40:20AM +, Chris Wilson wrote: On Tue, Feb 04, 2014 at 12:31:37PM +0100, Daniel Vetter wrote: On Fri, Jan 31, 2014 at 03:42:48PM -0600, jeff.mc...@intel.com wrote: From: Jeff McGee jeff.mc...@intel.com RPS manual mode disables/ignores load-based inputs

Re: [Intel-gfx] [PATCH] tests: Add gem_madvise

2014-02-04 Thread Daniel Vetter
On Tue, Feb 04, 2014 at 02:14:31PM +, Chris Wilson wrote: Exercise that calling madvise produces expected results Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Thanks a lot for the testcase and patches, all pulled in. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation

Re: [Intel-gfx] [PATCH] drm/i915: fix dp/sdvo i2c cleanup

2014-02-04 Thread Daniel Vetter
On Fri, Jan 24, 2014 at 10:15 AM, Imre Deak imre.d...@intel.com wrote: Atm we try to remove the connector's i2c sysfs entry too late in the encoder's destroy callback. By that time the kobject used as the parent for all connector sysfs entries is already removed when we do an early removal of

Re: [Intel-gfx] [PATCH] drm/i915: fix dp/sdvo i2c cleanup

2014-02-04 Thread Imre Deak
On Tue, 2014-02-04 at 17:13 +0100, Daniel Vetter wrote: On Fri, Jan 24, 2014 at 10:15 AM, Imre Deak imre.d...@intel.com wrote: Atm we try to remove the connector's i2c sysfs entry too late in the encoder's destroy callback. By that time the kobject used as the parent for all connector sysfs

[Intel-gfx] [PATCH v2] drm/i915: Restore rps/rc6 on reset

2014-02-04 Thread jeff . mcgee
From: Jeff McGee jeff.mc...@intel.com A check of rps/rc6 state after i915_reset determined that the ring MAX_IDLE registers were returned to their hardware defaults and that the GEN6_PMIMR register was set to mask all interrupts. This change restores those values to their pre-reset states by

[Intel-gfx] [PATCH v3] drm/i915: Update rps interrupt limits

2014-02-04 Thread jeff . mcgee
From: Jeff McGee jeff.mc...@intel.com sysfs changes to rps min and max delay were only triggering an update of the rps interrupt limits if the active delay required an update. This change ensures that interrupt limits are always updated. v2: correct compile issue missed on rebase v3: add igt

Re: [Intel-gfx] [PATCH v7] drm/i915: Reorganize display pipe register accesses

2014-02-04 Thread Ville Syrjälä
On Tue, Feb 04, 2014 at 02:22:24PM +0200, Antti Koskipaa wrote: RFCv2: Reorganize array indexing so that full offsets can be used as is. It makes grepping for registers in i915_reg.h much easier. Also move offset arrays to intel_device_info. v1: Fixed offsets for VLV, proper eDP handling

Re: [Intel-gfx] [PATCH 02/13] drm/i915: Implement command buffer parsing logic

2014-02-04 Thread Volkin, Bradley D
On Tue, Feb 04, 2014 at 02:20:36AM -0800, Daniel Vetter wrote: On Mon, Feb 03, 2014 at 03:00:19PM -0800, Volkin, Bradley D wrote: Ping. Daniel or Chris, can one of you clarify this request? Thanks. I've been enjoying fosdem ... On Thu, Jan 30, 2014 at 10:05:27AM -0800, Volkin, Bradley D

[Intel-gfx] [PATCH] MAINTAINERS: Update drm/i915 git repo

2014-02-04 Thread Daniel Vetter
Moved to a common location so that Jani also can push to it, to avoid moving it every time I go on vacation. Please update autobuilders and everything else pointing at the drm-intel.git repo, the old one won't be updated any more. Cc: Dave Airlie airl...@gmail.com Cc: Jani Nikula

[Intel-gfx] [PATCH] lib/drmtest: don't use asprintf on signal paths

2014-02-04 Thread Imre Deak
It's not signal safe and I got kms_flip in hung state with the backtrace below, while the parent process waiting for the signal helper to exit. It was quite easy to reproduce the bug by running kms_flip --run-subtest=flip-vs-dpms-off-vs-modeset With the change I couldn't reproduce it. 0

Re: [Intel-gfx] [PATCH] MAINTAINERS: Update drm/i915 git repo

2014-02-04 Thread Daniel Vetter
On Tue, Feb 4, 2014 at 8:00 PM, Daniel Vetter daniel.vet...@ffwll.ch wrote: Moved to a common location so that Jani also can push to it, to avoid moving it every time I go on vacation. Please update autobuilders and everything else pointing at the drm-intel.git repo, the old one won't be

[Intel-gfx] [PATCH 0/7] drm/i915: vlv: handle only enabled pipestat interrupts

2014-02-04 Thread Imre Deak
Atm on VLV we handle any pending pipestat interruts, whether or not these were actually enabled explicitly with i915_enable_pipestat(). This may or may not cause any real problem, but for consistency it's worth fixing. See the last patch for more details. I also need this as a dependency for the

[Intel-gfx] [PATCH 6/7] drm/i915: vlv: fix mapping of pipestat enable to status bits

2014-02-04 Thread Imre Deak
At least on VLV we can't get at the pipestat status bits by simply right shifting the corresponding enable bits. The mapping between enable and status bits for the sprite0,1 flip done and the PSR events don't follow this rule, so we need to map them separately. The PSR enable for pipe A is

[Intel-gfx] [PATCH 2/7] drm/i915: factor out valleyview_pipestat_irq_handler

2014-02-04 Thread Imre Deak
This will be used by other platforms too, so factor it out. The only functional change is the reordeing of gmbus_irq_handler() wrt. the hotplug handling, but since it only schedules a work, it isn't an issue. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_irq.c | 76

[Intel-gfx] [PATCH 4/7] drm/i915: unify FLIP_DONE macro names

2014-02-04 Thread Imre Deak
s/FLIPDONE/FLIP_DONE/ to make all FLIP_DONE macro names consistent. No functional change. Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_irq.c | 2 +- drivers/gpu/drm/i915/i915_reg.h | 18 +- 2 files changed, 10 insertions(+), 10 deletions(-) diff

[Intel-gfx] [PATCH 5/7] drm/i915: pass status instead of enable flags to i915_enable_pipestat

2014-02-04 Thread Imre Deak
There isn't any PSR interrupt enable bit for pipe A, so we couldn't enable it through the current API. Passing the corresponding status bits solves this and also makes the mapping between enable and status bits simpler on VLV (addressed in an upcoming patch). Except of checking for invalid status

Re: [Intel-gfx] [PATCH] MAINTAINERS: Update drm/i915 git repo

2014-02-04 Thread Daniel Vetter
On Tue, Feb 4, 2014 at 8:00 PM, Daniel Vetter daniel.vet...@ffwll.ch wrote: Moved to a common location so that Jani also can push to it, to avoid moving it every time I go on vacation. Please update autobuilders and everything else pointing at the drm-intel.git repo, the old one won't be

[Intel-gfx] [PATCH 7/7] drm/i915: vlv: handle only enabled pipestat interrupt events

2014-02-04 Thread Imre Deak
Atm we call the handlers for pending pipestat interrupt events even if they aren't explicitly enabled by i915_enable_pipestat(). This isn't an issue for events other than the vblank start event, since those are always enabled anyways. Otoh, we enable the vblank start event on-demand, so we'll end

Re: [Intel-gfx] [PATCH 02/13] drm/i915: Implement command buffer parsing logic

2014-02-04 Thread Daniel Vetter
On Tue, Feb 04, 2014 at 10:45:45AM -0800, Volkin, Bradley D wrote: The current table structure is that we have tables per-ring and per-gen (plus the table for common MI commands) and all tables are treated as blacklist/greylist. The proposed flow here would indicate that we need tables

[Intel-gfx] [PATCH 3/7] drm/i915: vlv: s/spin_lock_irqsave/spin_lock/ in irq handler

2014-02-04 Thread Imre Deak
Signed-off-by: Imre Deak imre.d...@intel.com --- drivers/gpu/drm/i915/i915_irq.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c index b5524ea..e0e5190 100644 --- a/drivers/gpu/drm/i915/i915_irq.c +++

[Intel-gfx] [PATCH 1/7] drm/i915: vlv: don't unmask IIR[DISPLAY_PIPE_A/B_VBLANK] interrupt

2014-02-04 Thread Imre Deak
Bspec and the code suggests that the interrupt signaled by IIR[7,5] (DISPLAY_PIPE_A/B_VBLANK) is a first level IRQ flag for the second level PIPEA/BSTAT[2] (Start of Vertical Blank) interrupt. Measuring the relative timings of when IIR[7] and PIPEASTAT[1,2] get set and checking the effect of

[Intel-gfx] [PATCH 0/7] drm/i915: Some more w/a'ish stuff

2014-02-04 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com So I accidentally looked at gen6_init_clock_gating() and noticed a few weird things that should have gotten cleaned up years ago. So I did that. While doing that I also noticed the WIZ hashing bits, and the fact that we weren't following the

[Intel-gfx] [PATCH 5/7] drm/i915: Change IVB WIZ hashing mode to 16x4

2014-02-04 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com BSpec recommends using 8x4 hashing mode when MSAA is used. But in practice 16x4 seems to have a slight edge in performance (on IVB and HSW at least). So just use 16x4. Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com ---

[Intel-gfx] [PATCH 1/7] drm/i915: Fix SNB GT_MODE register setup

2014-02-04 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com On SNB we set up WaSetupGtModeTdRowDispatch:snb early in gen6_init_clock_gating(). That sets a bit in the GEN6_GT_MODE register. However later we go and disable all the bits in the same register. And then we go on to set some other bit. So

[Intel-gfx] [PATCH 7/7] drm/i915: Change BDW WIZ hashing mode to 16x4

2014-02-04 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com BSpec recommends using 8x4 hashing mode when MSAA is used. But in practice 16x4 seems to have a slight edge in performance (on IVB and HSW at least). So just use 16x4. Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com ---

[Intel-gfx] [PATCH 6/7] drm/i915: Change HSW WIZ hashing mode to 16x4

2014-02-04 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com BSpec recommends using 8x4 hashing mode when MSAA is used. But in practice 16x4 seems to have a slight edge in performance (on IVB and HSW at least). So just use 16x4. Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com ---

[Intel-gfx] [PATCH 3/7] drm/i915: There's no need to mask all 3D_CHICKEN bits on SNB

2014-02-04 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com The need to set all of the mask bits for 3D_CHICKEN3 was required only for pre-production hardware. Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/i915/intel_pm.c | 5 ++--- 1 file changed, 2 insertions(+), 3

[Intel-gfx] [PATCH 4/7] drm/i915: Disable SF pipelined attribute fetch for SNB

2014-02-04 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com According to Bspec we need to disable SF pipelined attribute fetch whenever SF outputs exceed 16 and normal clip mode is used. A quick glance at Mesa suggests that these conditions could happen. So let's just always set the magic bit.

[Intel-gfx] [PATCH 2/7] drm/i915: Assume we implement WaStripsFansDisableFastClipPerformanceFix:snb

2014-02-04 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com Based on the name, the workaround we implement is WaStripsFansDisableFastClipPerformanceFix. Unfortunately there's no description in the w/a database, so this is just a guess. Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com ---

[Intel-gfx] 3.13.1 - WARNING at drivers/gpu/drm/i915/i915_irq.c:1240

2014-02-04 Thread Thomas Meyer
Hi, I see a *lot* of these warning in 3.13.1. 3.12.x never showed this problem. Any ideas?! [ 9373.175179] WARNING: CPU: 0 PID: 7715 at drivers/gpu/drm/i915/i915_irq.c:1240 i965_irq_handler+0x4ee/0x670() [ 9373.175181] Received HPD interrupt although disabled [ 9373.175183] Modules linked in:

Re: [Intel-gfx] [PATCH 2/2] drm/i915: demote opregion excessive timeout WARN_ONCE to DRM_INFO_ONCE

2014-02-04 Thread Daniel Vetter
On Fri, Jan 31, 2014 at 01:48:39PM +, Chris Wilson wrote: On Fri, Jan 31, 2014 at 03:49:08PM +0200, Jani Nikula wrote: The WARN_ONCE is a bit too verbose, make it a DRM_INFO_ONCE. While at it, add a #define for MAX_DSLP and make the message a bit more informative. v2: use

Re: [Intel-gfx] 3.13.1 - WARNING at drivers/gpu/drm/i915/i915_irq.c:1240

2014-02-04 Thread Daniel Vetter
On Tue, Feb 04, 2014 at 08:37:02PM +0100, Thomas Meyer wrote: Hi, I see a *lot* of these warning in 3.13.1. 3.12.x never showed this problem. Any ideas?! Can you please try latest the drm-intel-nightly git branch from git://anongit.freedesktop.org/drm-intel ? If that one's still affected

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Fix SNB GT_MODE register setup

2014-02-04 Thread Chris Wilson
On Tue, Feb 04, 2014 at 09:59:15PM +0200, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com On SNB we set up WaSetupGtModeTdRowDispatch:snb early in gen6_init_clock_gating(). That sets a bit in the GEN6_GT_MODE register. However later we go and disable

Re: [Intel-gfx] [PATCH] lib/drmtest: don't use asprintf on signal paths

2014-02-04 Thread Chris Wilson
On Tue, Feb 04, 2014 at 09:15:14PM +0200, Imre Deak wrote: It's not signal safe and I got kms_flip in hung state with the backtrace below, while the parent process waiting for the signal helper to exit. It was quite easy to reproduce the bug by running kms_flip

Re: [Intel-gfx] [PATCH] lib/drmtest: don't use asprintf on signal paths

2014-02-04 Thread Imre Deak
On Tue, 2014-02-04 at 21:29 +, Chris Wilson wrote: On Tue, Feb 04, 2014 at 09:15:14PM +0200, Imre Deak wrote: It's not signal safe and I got kms_flip in hung state with the backtrace below, while the parent process waiting for the signal helper to exit. It was quite easy to reproduce

Re: [Intel-gfx] [PATCH] MAINTAINERS: Update drm/i915 git repo

2014-02-04 Thread Stephen Rothwell
Hi Daniel, On Tue, 4 Feb 2014 20:15:03 +0100 Daniel Vetter daniel.vet...@ffwll.ch wrote: On Tue, Feb 4, 2014 at 8:00 PM, Daniel Vetter daniel.vet...@ffwll.ch wrote: Moved to a common location so that Jani also can push to it, to avoid moving it every time I go on vacation. Please update

Re: [Intel-gfx] [PATCH] lib/drmtest: don't use asprintf on signal paths

2014-02-04 Thread Daniel Vetter
On Wed, Feb 05, 2014 at 12:04:46AM +0200, Imre Deak wrote: On Tue, 2014-02-04 at 21:29 +, Chris Wilson wrote: On Tue, Feb 04, 2014 at 09:15:14PM +0200, Imre Deak wrote: It's not signal safe and I got kms_flip in hung state with the backtrace below, while the parent process waiting for

Re: [Intel-gfx] [PATCH v5] ACPI: Fix acpi_evaluate_object() return value check

2014-02-04 Thread Bjorn Helgaas
On Wed, Jan 22, 2014 at 8:42 PM, Yijing Wang wangyij...@huawei.com wrote: Since acpi_evaluate_object() returns acpi_status and not plain int, ACPI_FAILURE() should be used for checking its return value. Also add some detailed debug info when acpi_evaluate_object() failed. Reviewed-by: Jani

Re: [Intel-gfx] [PATCH v4] ACPI: Fix acpi_evaluate_object() return value check

2014-02-04 Thread Bjorn Helgaas
On Mon, Jan 20, 2014 at 7:46 PM, Yijing Wang wangyij...@huawei.com wrote: Since acpi_evaluate_object() returns acpi_status and not plain int, ACPI_FAILURE() should be used for checking its return value. Reviewed-by: Jani Nikula jani.nik...@intel.com Signed-off-by: Yijing Wang

Re: [Intel-gfx] [PATCH v5] ACPI: Fix acpi_evaluate_object() return value check

2014-02-04 Thread Bjorn Helgaas
On Fri, Jan 24, 2014 at 8:36 AM, Rafael J. Wysocki r...@rjwysocki.net wrote: On Friday, January 24, 2014 07:54:29 AM Bjorn Helgaas wrote: On Thu, Jan 23, 2014 at 5:33 PM, Rafael J. Wysocki r...@rjwysocki.net wrote: On Thursday, January 23, 2014 11:21:01 AM Bjorn Helgaas wrote: On Wed, Jan

Re: [Intel-gfx] [PATCH v5] ACPI: Fix acpi_evaluate_object() return value check

2014-02-04 Thread Bjorn Helgaas
On Thu, Jan 23, 2014 at 5:33 PM, Rafael J. Wysocki r...@rjwysocki.net wrote: On Thursday, January 23, 2014 11:21:01 AM Bjorn Helgaas wrote: On Wed, Jan 22, 2014 at 8:42 PM, Yijing Wang wangyij...@huawei.com wrote: Since acpi_evaluate_object() returns acpi_status and not plain int,

Re: [Intel-gfx] [PATCH v4 5/5] drm/tegra: Add eDP support

2014-02-04 Thread Stephen Warren
On 01/21/2014 12:24 PM, Thierry Reding wrote: Add support for eDP functionality found on Tegra124 and later SoCs. Only fast link training is currently supported. Signed-off-by: Thierry Reding tred...@nvidia.com --- .../bindings/gpu/nvidia,tegra20-host1x.txt | 42 + This part

Re: [Intel-gfx] kernfs oops with i915+i2c_core in 3.14 merge window

2014-02-04 Thread Tejun Heo
On Thu, Jan 30, 2014 at 02:03:18PM -0500, Josh Boyer wrote: Hi All, I'm seeing the oops below on my MacBookPro 10,2 machine using i915 graphics. It's after the DRM merge for 3.14 ( v3.13-10094-g9b0cd30) , but we seem to have one report[1] of this happening well before that, in

[Intel-gfx] [PATCH 71/73] drivers/gpu: delete non-required instances of linux/init.h

2014-02-04 Thread Paul Gortmaker
None of these files are actually using any __init type directives and hence don't need to include linux/init.h. Most are just a left over from __devinit and __cpuinit removal, or simply due to code getting copied from one driver to the next. Cc: David Airlie airl...@linux.ie Cc: Daniel Vetter

[Intel-gfx] Intel HD 4400 with SLES 11 SP1

2014-02-04 Thread Tim Zoeller
Hi, I am installing a DELL Inspiron 7737 with an Intel HD 4400. PCI Device (8086:0a16). So it should be a HD 4400. The Operating System is SLES 11 SP1. I tried to install the sources, but the are so many dependencies. From the one package to the next package an so on. Do you have an

Re: [Intel-gfx] [alsa-devel] Need your advice: Add a new communication inteface between HD-Audio and Gfx drivers for hotplug notification/ELD update

2014-02-04 Thread Raymond Yau
And I have a question: how to assure the audio/gfx client find its right peer? On a x86 platform, there can be an integrated GPU and an discrete GPU. So there can be two audio controllers and two GPUs. We need to assure audio controller find the proper GPU, and vice versa. Maybe we need the

[Intel-gfx] Haswell DisplayPort Multi Stream Transport status

2014-02-04 Thread Mike Hommey
Hi, What is the current status for DP MST support on Haswell? Are there experimental patches that can be tested? If not, what can be done to help progress? Supposing the kernel parts are figured, will userland need updates accordingly, or would the existing multi-head support work as-is?

  1   2   >