[libvirt] [PATCH 0/3] Adjust some formatting for numa distance code and resolve a memory leak

John Ferlan posted 3 patches 6 years, 4 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20171124171842.28448-1-jferlan@redhat.com
src/conf/numa_conf.c | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
[libvirt] [PATCH 0/3] Adjust some formatting for numa distance code and resolve a memory leak
Posted by John Ferlan 6 years, 4 months ago
While reviewing the recent distance adjustments for Michal, I noted a
couple of minor formatting things as well as a memory leak when removing
the distances.

John Ferlan (3):
  conf: Clean up virDomainNumaDefNodeDistanceParseXML
  conf: Clean up virDomainNumaDefCPUFormatXML
  conf: Fix memory leak for distances in virDomainNumaFree

 src/conf/numa_conf.c | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

-- 
2.13.6

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/3] Adjust some formatting for numa distance code and resolve a memory leak
Posted by John Ferlan 6 years, 3 months ago
ping?  Patch 3 is a memory leak bugfix for code added in this release...

Tks,

John

On 11/24/2017 12:18 PM, John Ferlan wrote:
> While reviewing the recent distance adjustments for Michal, I noted a
> couple of minor formatting things as well as a memory leak when removing
> the distances.
> 
> John Ferlan (3):
>   conf: Clean up virDomainNumaDefNodeDistanceParseXML
>   conf: Clean up virDomainNumaDefCPUFormatXML
>   conf: Fix memory leak for distances in virDomainNumaFree
> 
>  src/conf/numa_conf.c | 14 +++++++++-----
>  1 file changed, 9 insertions(+), 5 deletions(-)
> 

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/3] Adjust some formatting for numa distance code and resolve a memory leak
Posted by Michal Privoznik 6 years, 3 months ago
On 11/24/2017 06:18 PM, John Ferlan wrote:
> While reviewing the recent distance adjustments for Michal, I noted a
> couple of minor formatting things as well as a memory leak when removing
> the distances.
> 
> John Ferlan (3):
>   conf: Clean up virDomainNumaDefNodeDistanceParseXML
>   conf: Clean up virDomainNumaDefCPUFormatXML
>   conf: Fix memory leak for distances in virDomainNumaFree
> 
>  src/conf/numa_conf.c | 14 +++++++++-----
>  1 file changed, 9 insertions(+), 5 deletions(-)
> 

ACK series and safe for freeze. BTW: while reviewing this I've found
couple of other memleaks. Will post patches shortly.

Michal

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list