Re: [PATCH] python/xc: add missing Py_DECREF() to fix a memory leak

[Andrew Cooper <andrew.cooper3@citrix.com>]

On 31/08/2015 13:04, Wei Liu wrote:
> On Fri, Aug 28, 2015 at 05:35:18PM -0400, Zhigang Wang wrote:
>> Python PyList_Append() will increase reference count of the item. We have to
>> decrease its reference count to let it garbage collected.
>>
>> We missed the Py_DECREF() for 'cpuinfo_obj' here, thus we have a memory leak.
>>
>> The memory leak could be easily confirmed by:
>>
>>   # python
>>   >>> import xen.lowlevel.xc
>>   >>> xc = xen.lowlevel.xc.xc()
>>   >>> for i in range(1000): xc.getcpuinfo(1)
>>
>> And check the python process memory usage before and after:
>>
>>   # ps f -o vsize,rss,%mem,size,cmd -p <pid>
>>
>> Signed-off-by: Zhigang Wang <zhigang.x.wang@oracle.com>
> Acked-by: Wei Liu <wei.liu2@citrix.com>
>
> There is no in-tree user of python binding, I was planning to remove it
> in 4.7 window.

I am sure I have mentioned this in the past, but XenServer still uses
them in places, and they are very useful to use from interactive sessions.

~Andrew

(Oracle are still using them as they are still using xend)