<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Jan 8, 2019 at 12:54 PM Roland Scheidegger <<a href="mailto:sroland@vmware.com">sroland@vmware.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Am 08.01.19 um 17:17 schrieb Marek Olšák:<br>
> From: Marek Olšák <<a href="mailto:marek.olsak@amd.com" target="_blank">marek.olsak@amd.com</a>><br>
> <br>
> We have found some pipe_surface leaks internally.<br>
> <br>
> This is the same code as surface_destroy in radeonsi.<br>
> Ideally, surface_destroy would be in pipe_screen.<br>
> No, pipe_surfaces are not context objects.<br>
Well they are supposed to be...<br>
But yes mesa/st doesn't play by the rules there, so I guess that's<br>
better than a leak...<br></blockquote><div><br></div><div>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.<br></div><div><br></div>Marek<br></div></div>