On Wed, Sep 9, 2015 at 1:51 PM, Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > On Wed, Sep 09, 2015 at 01:46:21PM +0200, Maarten Lankhorst wrote: >> This may cause issues because encoders are already destroyed so removing >> active primaries may use freed memory. Instead free the fb directly, >> ignoring refcount. > > So what about fixing the cause, not the symptom? That is remove > framebuffers before nuking crtc/encoders/etc. Also by that point we shouldn't have any framebuffers left (the WARN_ON is for that), so not sure what's the point of this patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
