[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.7 v4 00/36] Various memory leak fixes
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH for-2.7 v4 00/36] Various memory leak fixes |
Date: |
Sun, 7 Aug 2016 20:38:30 +0100 |
On 7 August 2016 at 20:31, Marc-André Lureau <address@hidden> wrote:
> Hi
>
> On Sun, Aug 7, 2016 at 11:15 PM Peter Maydell <address@hidden>
> wrote:
>>
>> On 5 August 2016 at 19:01, Marc-André Lureau <address@hidden>
>> 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
That looks like a much shorter list which I'd definitely
be happier with applying.
thanks
-- PMM
- [Qemu-devel] [PATCH for-2.7 v4 32/36] tests: add qtest_add_data_func_full, (continued)
- [Qemu-devel] [PATCH for-2.7 v4 32/36] tests: add qtest_add_data_func_full, marcandre . lureau, 2016/08/05
- [Qemu-devel] [PATCH for-2.7 v4 33/36] tests: pc-cpu-test leaks fixes, marcandre . lureau, 2016/08/05
- [Qemu-devel] [PATCH for-2.7 v4 34/36] tests: fix rsp leak in postcopy-test, marcandre . lureau, 2016/08/05
- [Qemu-devel] [PATCH for-2.7 v4 35/36] ahci: fix sglist leak on retry, marcandre . lureau, 2016/08/05
- [Qemu-devel] [PATCH for-2.7 v4 36/36] tests: fix postcopy-test leaks, marcandre . lureau, 2016/08/05
- Re: [Qemu-devel] [PATCH for-2.7 v4 00/36] Various memory leak fixes, Daniel P. Berrange, 2016/08/05
- Re: [Qemu-devel] [PATCH for-2.7 v4 00/36] Various memory leak fixes, Marc-André Lureau, 2016/08/05