Re: [Intel-gfx] [PATCH] drm/i915: Remove the preliminary_hw_support shackles from CHV

2015-03-11 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 5937 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -4 281/281

Re: [Intel-gfx] [PATCH] drm/i915: add i915 specific connector debugfs file for DPCD

2015-03-11 Thread Todd Previte
On 2/25/2015 5:34 AM, Jani Nikula wrote: Occasionally it would be interesting to read some of the DPCD registers for debug purposes, without having to resort to logging. Add an i915 specific i915_dpcd debugfs file for DP and eDP connectors to dump parts of the DPCD. Currently the DPCD addresses t

Re: [Intel-gfx] [PATCH] drm/i915: Read CHV_PLL_DW8 from the correct offset

2015-03-11 Thread Todd Previte
On 3/11/2015 1:52 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä We accidentally pass 'pipe' instead of 'port' to CHV_PLL_DW8() and with PIPE_C we end up at register offset 0x8320 which isn't the 0x8020 we wanted. Fix it. The problem was fortunately caught by the sanity check in

Re: [Intel-gfx] [PATCH] drm/i915: Read CHV_PLL_DW8 from the correct offset

2015-03-11 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 5936 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -15 281/281

Re: [Intel-gfx] [Beignet] [PATCH 2/2 v2] Query the driver directly for compute units and subslice

2015-03-11 Thread Zhigang Gong
LGTM, Reviewed-by: Zhigang Gong Thanks. > -Original Message- > From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of > jeff.mc...@intel.com > Sent: Tuesday, March 10, 2015 7:36 AM > To: beig...@lists.freedesktop.org > Cc: intel-gfx@lists.freedesktop.org; dri-de...@li

Re: [Intel-gfx] [PATCH v2] drm/i915: Optimistically spin for the request completion

2015-03-11 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 5934 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -9 281/281

Re: [Intel-gfx] [PATCH v2] drm/i915: Optimistically spin for the request completion

2015-03-11 Thread Chris Wilson
On Wed, Mar 11, 2015 at 03:29:19PM +, Chris Wilson wrote: > + while (!need_resched()) { > + if (i915_gem_request_completed(req, true)) { > + ret = 0; > + goto out; > + } > + > + if (timeout && time_after_eq(jiffies,

Re: [Intel-gfx] [PATCH 11/11] drm/i915/skl: Enable the RPS interrupts programming

2015-03-11 Thread Jesse Barnes
On 03/05/2015 09:37 PM, akash.g...@intel.com wrote: > From: Akash Goel > > Enable the RPS interrupts programming(enable/disable/reset) for GEN9, > as missing changes to enable the RPS support on GEN9 have been added. > > Signed-off-by: Akash Goel > --- > drivers/gpu/drm/i915/intel_pm.c | 17 ++

[Intel-gfx] [PATCH] drm/i915: Read CHV_PLL_DW8 from the correct offset

2015-03-11 Thread ville . syrjala
From: Ville Syrjälä We accidentally pass 'pipe' instead of 'port' to CHV_PLL_DW8() and with PIPE_C we end up at register offset 0x8320 which isn't the 0x8020 we wanted. Fix it. The problem was fortunately caught by the sanity check in vlv_dpio_read(): WARNING: CPU: 1 PID: 238 at ../drivers/gpu/d

Re: [Intel-gfx] [PATCH 08/11] drm/i915/skl: Updated the i915_frequency_info debugfs function

2015-03-11 Thread Ville Syrjälä
On Fri, Mar 06, 2015 at 11:07:21AM +0530, akash.g...@intel.com wrote: > From: Akash Goel > > Added support for SKL in the i915_frequency_info debugfs function > > v2: > - corrected the handling of reqf (Damien) > - Reorderd the platform check for cagf (Ville) > > Signed-off-by: Akash Goel Had

Re: [Intel-gfx] [PATCH 55/56] drm/i915: Remove 'faked' request from LRC submission

2015-03-11 Thread John Harrison
On 11/03/2015 16:44, Jesse Barnes wrote: On 03/11/2015 09:14 AM, Daniel Vetter wrote: On Wed, Mar 11, 2015 at 02:53:39PM +, John Harrison wrote: On 05/03/2015 14:49, Daniel Vetter wrote: On Thu, Mar 05, 2015 at 01:57:31PM +, john.c.harri...@intel.com wrote: From: John Harrison The L

Re: [Intel-gfx] [PATCH] drm/i915: Simplify the way BC bifurcation state consistency is kept

2015-03-11 Thread Daniel Vetter
On Wed, Mar 11, 2015 at 06:58:12PM +0200, Ville Syrjälä wrote: > On Wed, Mar 11, 2015 at 11:37:54AM +, Conselvan De Oliveira, Ander wrote: > > On Wed, 2015-03-11 at 13:35 +0200, Ander Conselvan de Oliveira wrote: > > > Remove the global modeset resource function that would disable the > > > bif

Re: [Intel-gfx] [PATCH 07/11] drm/i915/skl: Updated the gen9_enable_rps function

2015-03-11 Thread Daniel Vetter
On Wed, Mar 11, 2015 at 12:27:59PM -0700, Jesse Barnes wrote: > On 03/05/2015 09:37 PM, akash.g...@intel.com wrote: > > From: Akash Goel > > > > On SKL, GT frequency is programmed in units of 16.66 MHZ units compared > > to 50 MHZ for older platforms. Also the time value specified for Up/Down EI

Re: [Intel-gfx] [PATCH] drm/i915: Simplify the way BC bifurcation state consistency is kept

2015-03-11 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 5933 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -3 281/281

Re: [Intel-gfx] [PATCH] drm/i915: Update intel_dp_hpd_pulse() to check link status for non-MST operation

2015-03-11 Thread Daniel Vetter
On Wed, Mar 11, 2015 at 09:10:29PM +0200, Ville Syrjälä wrote: > On Wed, Mar 11, 2015 at 11:37:23AM -0700, Jesse Barnes wrote: > > On 03/09/2015 02:04 PM, Ville Syrjälä wrote: > > > On Mon, Mar 09, 2015 at 12:07:31PM -0700, Jesse Barnes wrote: > > >> On 03/09/2015 10:29 AM, Daniel Vetter wrote: > >

Re: [Intel-gfx] [PATCH 07/11] drm/i915/skl: Updated the gen9_enable_rps function

2015-03-11 Thread Jesse Barnes
On 03/05/2015 09:37 PM, akash.g...@intel.com wrote: > From: Akash Goel > > On SKL, GT frequency is programmed in units of 16.66 MHZ units compared > to 50 MHZ for older platforms. Also the time value specified for Up/Down EI & > Up/Down thresholds are expressed in units of 1.33 us, compared to 1.

Re: [Intel-gfx] [PATCH 10/11] drm/i915/skl: Enabling processing of Turbo interrupts

2015-03-11 Thread Jesse Barnes
On 03/05/2015 09:37 PM, akash.g...@intel.com wrote: > From: Akash Goel > > Earlier Turbo interrupts were not being processed for SKL, > as something was amiss in turbo programming for SKL. > Now missing changes have been added, so enabling the Turbo > interrupt processing for SKL. > > Signed-off

Re: [Intel-gfx] [PATCH 01/11] drm/i915/skl: Added new macros

2015-03-11 Thread Ville Syrjälä
On Fri, Mar 06, 2015 at 11:07:14AM +0530, akash.g...@intel.com wrote: > From: Akash Goel > > For SKL, register definition for RPNSWREQ (A008), RPSTAT1(A01C) > have changed slightly. Also on SKL, frequency is specified in > units of 16.66 MHZ, compared to 50 MHZ for most of the earlier > platforms

Re: [Intel-gfx] [PATCH 03/11] drm/i915/skl: Updated the gen6_init_rps_frequencies function

2015-03-11 Thread Ville Syrjälä
On Fri, Mar 06, 2015 at 11:07:16AM +0530, akash.g...@intel.com wrote: > From: Akash Goel > > On SKL the frequency is specified in units of 16.66 MHZ, barring the > RP_STATE_CAP(0x5998) register, which still reports frequency in units > of 50 MHZ. So an extra conversion is required in gen6_init_rp

Re: [Intel-gfx] [PATCH] drm/i915: Update intel_dp_hpd_pulse() to check link status for non-MST operation

2015-03-11 Thread Ville Syrjälä
On Wed, Mar 11, 2015 at 11:37:23AM -0700, Jesse Barnes wrote: > On 03/09/2015 02:04 PM, Ville Syrjälä wrote: > > On Mon, Mar 09, 2015 at 12:07:31PM -0700, Jesse Barnes wrote: > >> On 03/09/2015 10:29 AM, Daniel Vetter wrote: > >>> On Mon, Mar 09, 2015 at 08:34:49AM -0700, Jesse Barnes wrote: >

Re: [Intel-gfx] [PATCH] drm/i915: Update intel_dp_hpd_pulse() to check link status for non-MST operation

2015-03-11 Thread Jesse Barnes
On 03/09/2015 02:04 PM, Ville Syrjälä wrote: > On Mon, Mar 09, 2015 at 12:07:31PM -0700, Jesse Barnes wrote: >> On 03/09/2015 10:29 AM, Daniel Vetter wrote: >>> On Mon, Mar 09, 2015 at 08:34:49AM -0700, Jesse Barnes wrote: On 03/06/2015 08:34 AM, Daniel Vetter wrote: > On Thu, Mar 05, 2015

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Add ULL postfix to VGT_MAGIC constant

2015-03-11 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 5931 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -7 281/281

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Add debugfs functions for Displayport compliance testing

2015-03-11 Thread Todd Previte
On 3/9/2015 10:57 AM, Jani Nikula wrote: On Thu, 19 Feb 2015, Todd Previte wrote: This patch is the amalgamation of 7 patches from the V2 series. These patches all involve the implementation of the debugfs mechanism for handling Displayport compliance testing. The following are the commit mes

Re: [Intel-gfx] [PATCH] drm/i915: Simplify the way BC bifurcation state consistency is kept

2015-03-11 Thread Ville Syrjälä
On Wed, Mar 11, 2015 at 11:37:54AM +, Conselvan De Oliveira, Ander wrote: > On Wed, 2015-03-11 at 13:35 +0200, Ander Conselvan de Oliveira wrote: > > Remove the global modeset resource function that would disable the > > bifurcation bit, and instead enable/disable it when enabling the pch > > t

[Intel-gfx] [PATCH 1/2] drm/i915: Rewrite some some of the FDI lane checks

2015-03-11 Thread ville . syrjala
From: Ville Syrjälä The logic in the FDI lane checks is very hard for my poor brain to grasp. Rewrite it in a more straightforward way. Cc: Ander Conselvan de Oliveira Cc: Daniel Vetter Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 15 +++ 1 file changed

[Intel-gfx] [PATCH 2/2] drm/i915: Rewrite IVB FDI bifurcation conflict checks

2015-03-11 Thread ville . syrjala
From: Ville Syrjälä Ignore the current state of the pipe and just check crtc_state->enable and the number of FDI lanes required. This means we don't accidentally mistake the FDI lanes as being available of one of the pipes just happens to be disabled at the time of the check. Also we no longer co

[Intel-gfx] [PATCH 0/2] drm/i915: Try to sort out ironlake_check_fdi_lanes()

2015-03-11 Thread ville . syrjala
From: Ville Syrjälä So these on top of Ander's latest FDI bifurcation patch [1] should hopefully result in something sane. Didn't test them though, but assuming Ander has that test somewhere maybe some of the problematic cases I identified [2] could be checked with these. [1] http://lists.freede

Re: [Intel-gfx] [PATCH 55/56] drm/i915: Remove 'faked' request from LRC submission

2015-03-11 Thread Jesse Barnes
On 03/11/2015 09:14 AM, Daniel Vetter wrote: > On Wed, Mar 11, 2015 at 02:53:39PM +, John Harrison wrote: >> On 05/03/2015 14:49, Daniel Vetter wrote: >>> On Thu, Mar 05, 2015 at 01:57:31PM +, john.c.harri...@intel.com wrote: From: John Harrison The LRC submission code requi

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Make derived plane state correct after crtc_enable

2015-03-11 Thread Daniel Vetter
On Wed, Mar 11, 2015 at 02:19:38PM +0200, Ville Syrjälä wrote: > On Wed, Mar 11, 2015 at 11:24:34AM +0100, Daniel Vetter wrote: > > On Wed, Mar 11, 2015 at 12:05:39PM +0200, Ville Syrjälä wrote: > > > On Wed, Mar 11, 2015 at 10:52:29AM +0100, Daniel Vetter wrote: > > > > On Tue, Mar 10, 2015 at 07:

Re: [Intel-gfx] [PATCH 55/56] drm/i915: Remove 'faked' request from LRC submission

2015-03-11 Thread Daniel Vetter
On Wed, Mar 11, 2015 at 02:53:39PM +, John Harrison wrote: > On 05/03/2015 14:49, Daniel Vetter wrote: > >On Thu, Mar 05, 2015 at 01:57:31PM +, john.c.harri...@intel.com wrote: > >>From: John Harrison > >> > >>The LRC submission code requires a request for tracking purposes. It does > >>n

Re: [Intel-gfx] [PATCH] drm/i915: remove indirection in the PCI ID macros

2015-03-11 Thread Daniel Vetter
On Tue, Feb 03, 2015 at 01:36:00PM +, Damien Lespiau wrote: > On Tue, Feb 03, 2015 at 02:34:05PM +0200, Jani Nikula wrote: > > Spell all the PCI IDs out to be able to quickly grep for the IDs. No > > functional changes. > > > > Signed-off-by: Jani Nikula > > > > --- > > > > I tested this by

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Move drm_framebuffer_unreference out of struct_mutex for takeover

2015-03-11 Thread Ville Syrjälä
On Tue, Mar 10, 2015 at 12:32:43PM +, Tvrtko Ursulin wrote: > > On 03/10/2015 12:19 PM, Chris Wilson wrote: > > On Tue, Mar 10, 2015 at 12:02:28PM +, Tvrtko Ursulin wrote: > >>> @@ -13707,13 +13709,18 @@ void intel_modeset_gem_init(struct drm_device > >>> *dev) > >>>

[Intel-gfx] [PATCH v2] drm/i915: Optimistically spin for the request completion

2015-03-11 Thread Chris Wilson
This provides a nice boost to mesa in swap bound scenarios (as mesa throttles itself to the previous frame and given the scenario that will complete shortly). It will also provide a good boost to systems running with semaphores disabled and so frequently waiting on the GPU as it switches rings. In

Re: [Intel-gfx] [PATCH] drm/i915: Optimistically spin for the request completion

2015-03-11 Thread Chris Wilson
On Wed, Mar 11, 2015 at 10:30:13AM +, Chris Wilson wrote: > On Wed, Mar 11, 2015 at 11:13:59AM +0100, Daniel Vetter wrote: > > Also do you have microbenchmark numbers for something midly ridiculous > > like a loop of very short batches (enough ofc to cause a bit of delay) and > > immediately st

Re: [Intel-gfx] [PATCH] drm/i915: Add polish to VLV WM shift+mask operations

2015-03-11 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 5926 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -3 281/281

Re: [Intel-gfx] [PATCH 1/5] drm/i915/chv: Remove Wait for a previous gfx force-off

2015-03-11 Thread Deepak S
On Thursday 26 February 2015 09:12 PM, Deepak S wrote: On Thursday 26 February 2015 09:13 PM, Ville Syrjälä wrote: On Thu, Feb 26, 2015 at 08:46:54PM +0530, deepa...@linux.intel.com wrote: From: Deepak S On CHV, PUNIT team confirmed that 'VLV_GFX_CLK_STATUS_BIT' is not a sticky bit and it

Re: [Intel-gfx] [PATCH 55/56] drm/i915: Remove 'faked' request from LRC submission

2015-03-11 Thread John Harrison
On 05/03/2015 14:49, Daniel Vetter wrote: On Thu, Mar 05, 2015 at 01:57:31PM +, john.c.harri...@intel.com wrote: From: John Harrison The LRC submission code requires a request for tracking purposes. It does not actually require that request to 'complete' it simply uses it for keeping hold

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

2015-03-11 Thread Deepak S
On Wednesday 11 March 2015 07:36 PM, Chris Wilson wrote: On Wed, Mar 11, 2015 at 07:23:48PM +0530, Deepak S wrote: On Thursday 26 February 2015 09:42 PM, Chris Wilson wrote: On Thu, Feb 26, 2015 at 08:46:56PM +0530, deepa...@linux.intel.com wrote: From: Deepak S After feedback from the ha

Re: [Intel-gfx] [PATCH igt 1/2] lib/kms: Add a way to override an output's mode

2015-03-11 Thread Damien Lespiau
On Wed, Mar 11, 2015 at 03:48:00PM +0200, Ander Conselvan De Oliveira wrote: > On Wed, 2015-03-11 at 13:26 +, Damien Lespiau wrote: > > On Wed, Mar 11, 2015 at 01:33:00PM +0200, Ander Conselvan de Oliveira wrote: > > > So that it is possible to use a custom mode with the simplified mode set >

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

2015-03-11 Thread Chris Wilson
On Wed, Mar 11, 2015 at 07:23:48PM +0530, Deepak S wrote: > > > On Thursday 26 February 2015 09:42 PM, Chris Wilson wrote: > >On Thu, Feb 26, 2015 at 08:46:56PM +0530, deepa...@linux.intel.com wrote: > >>From: Deepak S > >> > >>After feedback from the hardware team, now we set the GPU min freq t

Re: [Intel-gfx] [PATCH v2] drm/i915: Modifying RC6 Promotion timer for Media workloads.

2015-03-11 Thread Deepak S
On Wednesday 11 March 2015 07:26 PM, Chris Wilson wrote: On Wed, Mar 11, 2015 at 07:07:12PM +0530, Deepak S wrote: On Friday 06 March 2015 10:10 PM, Daniel Vetter wrote: On Thu, Mar 05, 2015 at 09:27:59PM +0530, deepa...@linux.intel.com wrote: From: Deepak S In normal cases, RC6 promotion

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

2015-03-11 Thread Deepak S
On Thursday 26 February 2015 09:42 PM, Chris Wilson wrote: On Thu, Feb 26, 2015 at 08:46:56PM +0530, deepa...@linux.intel.com wrote: From: Deepak S After feedback from the hardware team, now we set the GPU min freq to RPe. If we drop the freq to RPn, we found that the punit was not setting t

Re: [Intel-gfx] [PATCH v2] drm/i915: Modifying RC6 Promotion timer for Media workloads.

2015-03-11 Thread Chris Wilson
On Wed, Mar 11, 2015 at 07:07:12PM +0530, Deepak S wrote: > > > On Friday 06 March 2015 10:10 PM, Daniel Vetter wrote: > >On Thu, Mar 05, 2015 at 09:27:59PM +0530, deepa...@linux.intel.com wrote: > >>From: Deepak S > >> > >>In normal cases, RC6 promotion timer is 1700us/500us. This will > >>resu

Re: [Intel-gfx] [PATCH igt 1/2] lib/kms: Add a way to override an output's mode

2015-03-11 Thread Ander Conselvan De Oliveira
On Wed, 2015-03-11 at 13:26 +, Damien Lespiau wrote: > On Wed, Mar 11, 2015 at 01:33:00PM +0200, Ander Conselvan de Oliveira wrote: > > So that it is possible to use a custom mode with the simplified mode set > > API. > > Maybe just igt_output_set_mode()? That works too. I used override sinc

Re: [Intel-gfx] [PATCH igt 1/2] lib/kms: Add a way to override an output's mode

2015-03-11 Thread Damien Lespiau
On Wed, Mar 11, 2015 at 01:33:00PM +0200, Ander Conselvan de Oliveira wrote: > So that it is possible to use a custom mode with the simplified mode set API. Maybe just igt_output_set_mode()? -- Damien > --- > lib/igt_kms.c | 9 + > lib/igt_kms.h | 4 > 2 files changed, 13 insertio

Re: [Intel-gfx] [PATCH] drm/i915: Simplify the way BC bifurcation state consistency is kept

2015-03-11 Thread Conselvan De Oliveira, Ander
On Wed, 2015-03-11 at 15:10 +0200, Ville Syrjälä wrote: > On Wed, Mar 11, 2015 at 01:35:43PM +0200, Ander Conselvan de Oliveira wrote: > > Remove the global modeset resource function that would disable the > > bifurcation bit, and instead enable/disable it when enabling the pch > > transcoder. The

Re: [Intel-gfx] [PATCH] drm/i915: Simplify the way BC bifurcation state consistency is kept

2015-03-11 Thread Ville Syrjälä
On Wed, Mar 11, 2015 at 01:35:43PM +0200, Ander Conselvan de Oliveira wrote: > Remove the global modeset resource function that would disable the > bifurcation bit, and instead enable/disable it when enabling the pch > transcoder. The mode set consistency check should prevent us from > disabling th

Re: [Intel-gfx] [PATCH] drm/i915: Simplify the way BC bifurcation state consistency is kept

2015-03-11 Thread Ville Syrjälä
On Wed, Mar 11, 2015 at 01:35:43PM +0200, Ander Conselvan de Oliveira wrote: > Remove the global modeset resource function that would disable the > bifurcation bit, and instead enable/disable it when enabling the pch > transcoder. The mode set consistency check should prevent us from > disabling th

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Make derived plane state correct after crtc_enable

2015-03-11 Thread Ville Syrjälä
On Wed, Mar 11, 2015 at 11:24:34AM +0100, Daniel Vetter wrote: > On Wed, Mar 11, 2015 at 12:05:39PM +0200, Ville Syrjälä wrote: > > On Wed, Mar 11, 2015 at 10:52:29AM +0100, Daniel Vetter wrote: > > > On Tue, Mar 10, 2015 at 07:57:13PM +0200, Ville Syrjälä wrote: > > > > On Tue, Mar 10, 2015 at 10:

Re: [Intel-gfx] [PATCH] drm/i915: Simplify the way BC bifurcation state consistency is kept

2015-03-11 Thread Conselvan De Oliveira, Ander
On Wed, 2015-03-11 at 13:35 +0200, Ander Conselvan de Oliveira wrote: > Remove the global modeset resource function that would disable the > bifurcation bit, and instead enable/disable it when enabling the pch > transcoder. The mode set consistency check should prevent us from > disabling the bit i

[Intel-gfx] [PATCH] drm/i915: Simplify the way BC bifurcation state consistency is kept

2015-03-11 Thread Ander Conselvan de Oliveira
Remove the global modeset resource function that would disable the bifurcation bit, and instead enable/disable it when enabling the pch transcoder. The mode set consistency check should prevent us from disabling the bit if pipe C is enabled so the change should be safe. Note that this doens't affe

[Intel-gfx] [PATCH igt 1/2] lib/kms: Add a way to override an output's mode

2015-03-11 Thread Ander Conselvan de Oliveira
So that it is possible to use a custom mode with the simplified mode set API. --- lib/igt_kms.c | 9 + lib/igt_kms.h | 4 2 files changed, 13 insertions(+) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 26e4913..0dccd2d 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -895,6 +895,

[Intel-gfx] [PATCH igt 2/2] tests: Add test for pipe B and C interactions in IVB

2015-03-11 Thread Ander Conselvan de Oliveira
The tests exercise different combinations of enabling pipe B with modes that require more than 2 lanes and then enabling pipe C. v2: Added a couple more tests for different pipe transitions. (Ander) Use custom modes to make the test reliable. (Daniel) --- tests/Makefile.sources| 1 + te

Re: [Intel-gfx] [PATCH] drm/i915: Optimistically spin for the request completion

2015-03-11 Thread Chris Wilson
On Wed, Mar 11, 2015 at 11:13:59AM +0100, Daniel Vetter wrote: > On Tue, Mar 10, 2015 at 04:14:14PM +, Chris Wilson wrote: > > On Tue, Mar 10, 2015 at 04:06:19PM +, Chris Wilson wrote: > > > @@ -1235,12 +1257,20 @@ int __i915_wait_request(struct > > > drm_i915_gem_request *req, > > > if

Re: [Intel-gfx] [PATCH 9/9] drm/i915: Don't re-enable an explicitly disabled primary plane due to sprite coverage changes

2015-03-11 Thread Daniel Vetter
On Wed, Mar 11, 2015 at 12:09:24PM +0200, Ville Syrjälä wrote: > On Wed, Mar 11, 2015 at 11:00:32AM +0100, Daniel Vetter wrote: > > On Tue, Mar 10, 2015 at 01:15:29PM +0200, ville.syrj...@linux.intel.com > > wrote: > > > From: Ville Syrjälä > > > > > > When the sprite is covering the entire pipe

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Make derived plane state correct after crtc_enable

2015-03-11 Thread Daniel Vetter
On Wed, Mar 11, 2015 at 12:05:39PM +0200, Ville Syrjälä wrote: > On Wed, Mar 11, 2015 at 10:52:29AM +0100, Daniel Vetter wrote: > > On Tue, Mar 10, 2015 at 07:57:13PM +0200, Ville Syrjälä wrote: > > > On Tue, Mar 10, 2015 at 10:01:51AM -0700, Matt Roper wrote: > > > > On Tue, Mar 10, 2015 at 01:15:

Re: [Intel-gfx] [PATCH] drm/i915: Optimistically spin for the request completion

2015-03-11 Thread Daniel Vetter
On Tue, Mar 10, 2015 at 04:14:14PM +, Chris Wilson wrote: > On Tue, Mar 10, 2015 at 04:06:19PM +, Chris Wilson wrote: > > @@ -1235,12 +1257,20 @@ int __i915_wait_request(struct drm_i915_gem_request > > *req, > > if (ring->id == RCS && INTEL_INFO(dev)->gen >= 6) > > gen6_rps

Re: [Intel-gfx] [PATCH 9/9] drm/i915: Don't re-enable an explicitly disabled primary plane due to sprite coverage changes

2015-03-11 Thread Ville Syrjälä
On Wed, Mar 11, 2015 at 11:00:32AM +0100, Daniel Vetter wrote: > On Tue, Mar 10, 2015 at 01:15:29PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > When the sprite is covering the entire pipe (and color keying is not > > enabled) we currently try to automagically disab

Re: [Intel-gfx] [PATCH] drm/i915: Use FW_WM() macro for older gmch platforms too

2015-03-11 Thread Daniel Vetter
On Tue, Mar 10, 2015 at 02:50:16PM -0700, Jesse Barnes wrote: > On 03/10/2015 08:02 AM, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > Use the FW_WM() macro from the VLV wm code to polish up the wm > > code for older gmch platforms. > > > > Cc: Daniel Vetter > > Signed-off

Re: [Intel-gfx] [PATCH 5/9] drm/i915: Pass primary plane size to .update_primary_plane()

2015-03-11 Thread Daniel Vetter
On Tue, Mar 10, 2015 at 07:59:15PM +0200, Ville Syrjälä wrote: > On Tue, Mar 10, 2015 at 10:10:40AM -0700, Matt Roper wrote: > > On Tue, Mar 10, 2015 at 01:15:25PM +0200, ville.syrj...@linux.intel.com > > wrote: > > > From: Ville Syrjälä > > > > > > In preparation to movable/resizable primary pl

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Make derived plane state correct after crtc_enable

2015-03-11 Thread Daniel Vetter
On Tue, Mar 10, 2015 at 07:57:13PM +0200, Ville Syrjälä wrote: > On Tue, Mar 10, 2015 at 10:01:51AM -0700, Matt Roper wrote: > > On Tue, Mar 10, 2015 at 01:15:24PM +0200, ville.syrj...@linux.intel.com > > wrote: > > > From: Ville Syrjälä > > > -static void disable_plane_internal(struct drm_plane

Re: [Intel-gfx] [PATCH 5/9] drm/i915: Pass primary plane size to .update_primary_plane()

2015-03-11 Thread Ville Syrjälä
On Tue, Mar 10, 2015 at 01:57:09PM -0700, Matt Roper wrote: > On Tue, Mar 10, 2015 at 07:59:15PM +0200, Ville Syrjälä wrote: > > On Tue, Mar 10, 2015 at 10:10:40AM -0700, Matt Roper wrote: > > > On Tue, Mar 10, 2015 at 01:15:25PM +0200, ville.syrj...@linux.intel.com > > > wrote: > > > > From: Vill

Re: [Intel-gfx] [PATCH 3/9] drm/i915: Use plane->state->fb instead of plane->fb in intel_plane_restore()

2015-03-11 Thread Daniel Vetter
On Tue, Mar 10, 2015 at 07:48:39PM +0200, Ville Syrjälä wrote: > On Tue, Mar 10, 2015 at 10:01:47AM -0700, Matt Roper wrote: > > On Tue, Mar 10, 2015 at 01:15:23PM +0200, ville.syrj...@linux.intel.com > > wrote: > > > From: Ville Syrjälä > > > > > > plane->fb is not as reliable as plane->state->

Re: [Intel-gfx] [PATCH 5/9] drm/i915: Pass primary plane size to .update_primary_plane()

2015-03-11 Thread sonika
On Wednesday 11 March 2015 02:57 PM, Ville Syrjälä wrote: On Wed, Mar 11, 2015 at 10:39:31AM +0530, sonika wrote: On Tuesday 10 March 2015 04:45 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä In preparation to movable/resizable primary planes pass the clipped plane size to .upd

Re: [Intel-gfx] [PATCH 5/9] drm/i915: Pass primary plane size to .update_primary_plane()

2015-03-11 Thread Ville Syrjälä
On Wed, Mar 11, 2015 at 10:39:31AM +0530, sonika wrote: > > On Tuesday 10 March 2015 04:45 PM, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > In preparation to movable/resizable primary planes pass the clipped > > plane size to .update_primary_plane(). > > > > Cc: Sonika Jin

Re: [Intel-gfx] commit 440fd52 drm/mm: Support 4GiB and larger ranges oops on 32bit kernel

2015-03-11 Thread Chris Wilson
[cc'ing lists] On Tue, Mar 10, 2015 at 07:17:22PM +0100, Krzysztof Kolasa wrote: > System ( 32bit, Intel 945GM ) hangs after some short time, oops: > > [ cut here ] > kernel BUG at drivers/gpu/drm/drm_mm.c:305! > invalid opcode: [#1] SMP > Modules linked in: arc4 md4

[Intel-gfx] [PATCH 1/2] drm/fourcc: 64 #defines need ULL postfix

2015-03-11 Thread Daniel Vetter
I have no idea about the exact rules, but this angered Dave's 32bit rhel gcc. Reported-by: Dave Airlie Signed-off-by: Daniel Vetter --- include/uapi/drm/drm_fourcc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h

[Intel-gfx] [PATCH 2/2] drm/i915: Add ULL postfix to VGT_MAGIC constant

2015-03-11 Thread Daniel Vetter
Without this Dave's 32bit rhel compiler is annoyed. Don't ask me about the exact rules for this stuff though, but this should be safe. Reported-by: Dave Airlie Signed-off-by: Daniel Vetter --- drivers/gpu/drm/i915/i915_vgpu.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

Re: [Intel-gfx] [Beignet] [PATCH i-g-t 2/2] configure: Bump required libdrm version to 2.4.60

2015-03-11 Thread Daniel Vetter
On Tue, Mar 10, 2015 at 01:06:44PM -0700, Jeff McGee wrote: > On Tue, Mar 10, 2015 at 07:47:03PM +0100, Daniel Vetter wrote: > > On Tue, Mar 10, 2015 at 01:58:52PM -0400, Rob Clark wrote: > > > On Tue, Mar 10, 2015 at 12:59 PM, Jeff McGee wrote: > > > > On Tue, Mar 10, 2015 at 08:37:30AM +0100, Da