Quoting Chris Wilson (2017-12-08 11:31:01) > Quoting Ville Syrjälä (2017-12-08 11:14:17) > > On Thu, Dec 07, 2017 at 10:00:25PM +0000, Chris Wilson wrote: > > > When intel_modeset_setup_plane_state() fails drop the local framebuffer > > > reference before jumping to the error, otherwise we leak the framebuffer. > > > > Or we could just > > https://patchwork.freedesktop.org/patch/165486/ > > Why not both! :) > > > This is > > Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > nonetheless. > > Ta, less controversial patch for more, grand improvement for you :) And pushed, thanks for the review. Let's have a gander at the more intelligent load-detect patch. :) -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
