Am 08.01.19 um 21:03 schrieb Marek Olšák: > On Tue, Jan 8, 2019 at 12:54 PM Roland Scheidegger <srol...@vmware.com > <mailto:srol...@vmware.com>> wrote: > > Am 08.01.19 um 17:17 schrieb Marek Olšák: > > From: Marek Olšák <marek.ol...@amd.com <mailto:marek.ol...@amd.com>> > > > > We have found some pipe_surface leaks internally. > > > > This is the same code as surface_destroy in radeonsi. > > Ideally, surface_destroy would be in pipe_screen. > > No, pipe_surfaces are not context objects. > Well they are supposed to be... > But yes mesa/st doesn't play by the rules there, so I guess that's > better than a leak... > > > If it was possible, I would change the rules. create_surface could stay > in pipe_context, but I would move surface_destroy into pipe_screen.
I think st/mesa would still use (not just delete) the pipe_surface in other contexts? If so I don't like the proposal, because it's still a lie that pipe_surface is a context-based object. Roland > > Marek _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev