From: Ville Syrjälä
Point crtc->fb the the new framebuffer only after we know that the flip
was succesfully queued.
While at it, move the intel_fb and obj assignments a bit close to where
they're used.
Cc: sta...@vger.kernel.org
Signed-off-by: Ville Syrjälä
---
It looks like Mika hit this rath
On Fri, Feb 22, 2013 at 02:17:24PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Point crtc->fb the the new framebuffer only after we know that the flip
> was succesfully queued.
>
> While at it, move the intel_fb and obj assignments a bit close to where
> they're used.
>
ville.syrj...@linux.intel.com writes:
> From: Ville Syrjälä
>
> Point crtc->fb the the new framebuffer only after we know that the flip
> was succesfully queued.
>
> While at it, move the intel_fb and obj assignments a bit close to where
> they're used.
>
> Cc: sta...@vger.kernel.org
> Signed-off
On Fri, Feb 22, 2013 at 01:31:35PM +, Chris Wilson wrote:
> On Fri, Feb 22, 2013 at 02:17:24PM +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Point crtc->fb the the new framebuffer only after we know that the flip
> > was succesfully queued.
> >
> > While at it,
On Fri, Feb 22, 2013 at 04:36:38PM +0200, Ville Syrjälä wrote:
> On Fri, Feb 22, 2013 at 01:31:35PM +, Chris Wilson wrote:
> > On Fri, Feb 22, 2013 at 02:17:24PM +0200, ville.syrj...@linux.intel.com
> > wrote:
> > > From: Ville Syrjälä
> > >
> > > Point crtc->fb the the new framebuffer only
On Fri, Feb 22, 2013 at 04:36:38PM +0200, Ville Syrjälä wrote:
> I don't think anyone should be poking at crtc->fb w/o holding the crtc
> mutex. Except that intel_update_fbc() actually does. That thing would
> appear to be just broken since it crawls around in the crtc state w/o
> proper protection