Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-18 Thread Ville Syrjälä
On Fri, Nov 18, 2016 at 11:26:18AM +0100, Maarten Lankhorst wrote: > Op 17-11-16 om 16:06 schreef Ville Syrjälä: > > On Tue, Nov 15, 2016 at 02:53:00PM +0100, Maarten Lankhorst wrote: > >> Op 15-11-16 om 14:41 schreef Ville Syrjälä: > >>> On Tue, Nov 15, 2016 at 11:14:29AM +0100, Maarten Lankhorst

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-18 Thread Maarten Lankhorst
Op 17-11-16 om 16:06 schreef Ville Syrjälä: > On Tue, Nov 15, 2016 at 02:53:00PM +0100, Maarten Lankhorst wrote: >> Op 15-11-16 om 14:41 schreef Ville Syrjälä: >>> On Tue, Nov 15, 2016 at 11:14:29AM +0100, Maarten Lankhorst wrote: Op 14-11-16 om 17:35 schreef ville.syrj...@linux.intel.com:

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-17 Thread Ville Syrjälä
On Tue, Nov 15, 2016 at 02:53:00PM +0100, Maarten Lankhorst wrote: > Op 15-11-16 om 14:41 schreef Ville Syrjälä: > > On Tue, Nov 15, 2016 at 11:14:29AM +0100, Maarten Lankhorst wrote: > >> Op 14-11-16 om 17:35 schreef ville.syrj...@linux.intel.com: > >>> From: Ville Syrjälä

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-15 Thread Maarten Lankhorst
Op 15-11-16 om 14:41 schreef Ville Syrjälä: > On Tue, Nov 15, 2016 at 11:14:29AM +0100, Maarten Lankhorst wrote: >> Op 14-11-16 om 17:35 schreef ville.syrj...@linux.intel.com: >>> From: Ville Syrjälä >>> >>> A modeset on one pipe can update

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-15 Thread Ville Syrjälä
On Tue, Nov 15, 2016 at 11:14:29AM +0100, Maarten Lankhorst wrote: > Op 14-11-16 om 17:35 schreef ville.syrj...@linux.intel.com: > > From: Ville Syrjälä > > > > A modeset on one pipe can update dev_priv->atomic_cdclk_freq without > > actually touching the hardware,

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-15 Thread Maarten Lankhorst
Op 14-11-16 om 17:35 schreef ville.syrj...@linux.intel.com: > From: Ville Syrjälä > > A modeset on one pipe can update dev_priv->atomic_cdclk_freq without > actually touching the hardware, in which case we won't force a modeset > on all the pipes, and thus won't

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-15 Thread Ville Syrjälä
On Tue, Nov 15, 2016 at 10:08:10AM +0100, Daniel Vetter wrote: > On Mon, Nov 14, 2016 at 06:35:10PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > A modeset on one pipe can update dev_priv->atomic_cdclk_freq without > > actually

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-15 Thread Daniel Vetter
On Tue, Nov 15, 2016 at 10:08:10AM +0100, Daniel Vetter wrote: > On Mon, Nov 14, 2016 at 06:35:10PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > A modeset on one pipe can update dev_priv->atomic_cdclk_freq without > > actually

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-15 Thread Daniel Vetter
On Mon, Nov 14, 2016 at 06:35:10PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > A modeset on one pipe can update dev_priv->atomic_cdclk_freq without > actually touching the hardware, in which case we won't force a modeset > on all the

[Intel-gfx] [PATCH 2/3] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks

2016-11-14 Thread ville . syrjala
From: Ville Syrjälä A modeset on one pipe can update dev_priv->atomic_cdclk_freq without actually touching the hardware, in which case we won't force a modeset on all the pipes, and thus won't lock any of the other pipes either. That means a parallel plane update