Re: [Intel-gfx] Bad flicker on skylake HQD due to code in the 4.7 merge window

2016-07-08 Thread Ville Syrjälä
On Fri, Jul 08, 2016 at 07:48:29AM -0700, James Bottomley wrote: > On Fri, 2016-07-08 at 13:19 +0300, Ville Syrjälä wrote: > > On Thu, Jul 07, 2016 at 12:19:36PM -0700, James Bottomley wrote: > > > On Thu, 2016-07-07 at 09:55 -0700, James Bottomley wrote: > > > > On Thu, 2016-07-07 at 19:14 +0300,

Re: [Intel-gfx] Bad flicker on skylake HQD due to code in the 4.7 merge window

2016-07-08 Thread James Bottomley
On Fri, 2016-07-08 at 13:19 +0300, Ville Syrjälä wrote: > On Thu, Jul 07, 2016 at 12:19:36PM -0700, James Bottomley wrote: > > On Thu, 2016-07-07 at 09:55 -0700, James Bottomley wrote: > > > On Thu, 2016-07-07 at 19:14 +0300, Ville Syrjälä wrote: > > > > On Tue, Jun 21, 2016 at 06:44:34PM +0300, Vi

Re: [Intel-gfx] [PATCH] drm/i915: Fill unused GGTT with scratch pages for VT-d

2016-07-08 Thread Chris Wilson
On Fri, Jul 08, 2016 at 03:29:30PM +0300, David Weinehall wrote: > On Fri, Jun 24, 2016 at 02:07:14PM +0100, Chris Wilson wrote: > > One of the numerous VT-d workarounds we require is that the display > > hardware reads past the end of the buffer triggering VT-d faults. This > > is acknowledged in

Re: [Intel-gfx] [PATCH] drm/i915: Fill unused GGTT with scratch pages for VT-d

2016-07-08 Thread David Weinehall
On Fri, Jun 24, 2016 at 02:07:14PM +0100, Chris Wilson wrote: > One of the numerous VT-d workarounds we require is that the display > hardware reads past the end of the buffer triggering VT-d faults. This > is acknowledged in the code as being safe "since we fill the unused > portions of the GGTT w

Re: [Intel-gfx] [PATCH] drm/i915: Fill unused GGTT with scratch pages for VT-d

2016-07-08 Thread Chris Wilson
On Fri, Jun 24, 2016 at 02:07:14PM +0100, Chris Wilson wrote: > One of the numerous VT-d workarounds we require is that the display > hardware reads past the end of the buffer triggering VT-d faults. This > is acknowledged in the code as being safe "since we fill the unused > portions of the GGTT w

Re: [Intel-gfx] [PATCH 04/15] drm/i915: Walk over encoders in crtc enable/disable using atomic state.

2016-07-08 Thread Ville Syrjälä
On Thu, Jul 07, 2016 at 01:55:46PM +0200, Maarten Lankhorst wrote: > This cleans up another possible use of the connector list, > encoder->crtc is legacy state and should not be used. > > With the atomic state as argument it's easy to find the encoder from > the connector it belongs to. > > intel

Re: [Intel-gfx] [PATCH 03/15] drm/i915: Remove unused mode_set hook from encoder

2016-07-08 Thread Ville Syrjälä
On Thu, Jul 07, 2016 at 01:55:45PM +0200, Maarten Lankhorst wrote: > Signed-off-by: Maarten Lankhorst Reviewed-by: Ville Syrjälä > --- > drivers/gpu/drm/i915/intel_drv.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_drv.h > b/drivers/gpu/drm/i915/intel_dr

Re: [Intel-gfx] Bad flicker on skylake HQD due to code in the 4.7 merge window

2016-07-08 Thread Ville Syrjälä
On Thu, Jul 07, 2016 at 12:19:36PM -0700, James Bottomley wrote: > On Thu, 2016-07-07 at 09:55 -0700, James Bottomley wrote: > > On Thu, 2016-07-07 at 19:14 +0300, Ville Syrjälä wrote: > > > On Tue, Jun 21, 2016 at 06:44:34PM +0300, Ville Syrjälä wrote: > > > > On Tue, Jun 21, 2016 at 09:53:15AM -0

Re: [Intel-gfx] [PATCH 01/15] drm/i915: handle DP_MST correctly in bxt_get_dpll

2016-07-08 Thread Ville Syrjälä
On Thu, Jul 07, 2016 at 01:55:43PM +0200, Maarten Lankhorst wrote: > No idea if it supports it, but this is the minimum required from get_dpll. > > Signed-off-by: Maarten Lankhorst > --- > drivers/gpu/drm/i915/intel_dpll_mgr.c | 10 -- > 1 file changed, 8 insertions(+), 2 deletions(-) >

Re: [Intel-gfx] [PATCH 08/64] drm/i915: Retire oldest completed request before allocating next

2016-07-08 Thread Chris Wilson
On Fri, Jul 08, 2016 at 12:21:18PM +0300, Mika Kuoppala wrote: > Chris Wilson writes: > > > In order to keep the memory allocated for requests reasonably tight, try > > to reuse the oldest request (so long as it is completed and has no > > external references) for the next allocation. > > > > Sig

Re: [Intel-gfx] [PATCH 08/64] drm/i915: Retire oldest completed request before allocating next

2016-07-08 Thread Mika Kuoppala
Chris Wilson writes: > In order to keep the memory allocated for requests reasonably tight, try > to reuse the oldest request (so long as it is completed and has no > external references) for the next allocation. > > Signed-off-by: Chris Wilson > --- > drivers/gpu/drm/i915/i915_gem_request.c |

Re: [Intel-gfx] [RFC v2] drm/i915/chv: Clip cursor for CHV pipe C HW Cursor pos < 0

2016-07-08 Thread Shobhit Kumar
On Wed, Jun 29, 2016 at 6:24 PM, Shobhit Kumar wrote: > > From: Shobhit Kumar > > CHV pipe C hits underrun when we get negative crtc_x values of cursor. > To avoid this we clip and shift the cursor image by negative crtc_x > value. > > v2: Make a copy of cursor plane state and allocate new gem ob

Re: [Intel-gfx] ✗ Ro.CI.BAT: warning for series starting with [01/64] drm/i915/breadcrumbs: Queue hangcheck before sleeping

2016-07-08 Thread Chris Wilson
On Thu, Jul 07, 2016 at 10:28:42AM +0100, Chris Wilson wrote: > On Thu, Jul 07, 2016 at 09:12:01AM -, Patchwork wrote: > > == Series Details == > > > > Series: series starting with [01/64] drm/i915/breadcrumbs: Queue hangcheck > > before sleeping > > URL : https://patchwork.freedesktop.org/