On Tue, Jun 30, 2015 at 03:22:51PM +0100, Chris Wilson wrote: > On Tue, Jun 30, 2015 at 10:53:05AM -0300, Paulo Zanoni wrote: > > From: Paulo Zanoni <[email protected]> > > > > We first set the threshold value when we're allocating the CFB, and > > then later at {ilk,gen7}_fbc_enable() we increment it in case we're > > using 16bpp. While that is correct, it is dangerous: if we rework the > > code a little bit in a way that allows us to call intel_fbc_enable() > > without necessarily calling i915_gem_stolen_setup_compression() first, > > we might end up incrementing threshold more than once. To prevent > > that, increment a temporary variable instead. > > > > v2: Rebase. > > > > Signed-off-by: Paulo Zanoni <[email protected]> > > Reviewed-by: Chris Wilson <[email protected]>
Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
