Hi On Sun, Aug 7, 2016 at 11:15 PM Peter Maydell <peter.mayd...@linaro.org> wrote:
> On 5 August 2016 at 19:01, Marc-André Lureau <marcandre.lur...@gmail.com> > wrote: > > Hi Peter, > > > > All patches have a reviewed-by tag now, Given the mix of domains, I can > send > > you a pull request. What do you think? > > 36 patches is a lot to be landing on the eve of rc2. How many > of these are actual leaks rather than leaks in test cases > or things that are in theory leaks but of objects that > in practice last the life of the QEMU process anyway ? > They should be fairly safe, but I don't mind moving it to after the release. Yes, many of the patches are one-time leaks and for tests. This would be a possible shortlist for 2.7 of things that could be more than one-timer though: numa: do not leak NumaOptions char: free the tcp connection data when closing char: free MuxDriver when closing ahci: free irqs array qjson: free str virtio-input: free config list usb: free USBDevice.strings usb: free leaking path ahci: fix sglist leak on retry thanks -- Marc-André Lureau