On Tue, Aug 06, 2013 at 09:45:11PM +0100, Chris Wilson wrote: > On Tue, Aug 06, 2013 at 10:24:03PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote: > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > We don't need to store the FBC WM enabled status in each watermark > > level. We anyway have to reduce it down to a single boolean, so just > > delay checking the FBC WM limit until we're computing the final > > value. > > > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > The pay-off simply being the reduction of one bool in a temporary > struct [x3]? I don't remember anymore if I had a better reason originally. > > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
