[libvirt PATCH 0/2] lxc: remove most use of libvirt allocation APIs

Daniel P. Berrangé posted 2 patches 3 years, 10 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20200513162241.3929790-1-berrange@redhat.com
src/lxc/lxc_cgroup.c     |  15 +--
src/lxc/lxc_conf.c       |  19 ++-
src/lxc/lxc_container.c  | 262 ++++++++++++++-------------------------
src/lxc/lxc_controller.c | 148 ++++++++--------------
src/lxc/lxc_domain.c     |  40 +++---
src/lxc/lxc_driver.c     |  89 ++++++-------
src/lxc/lxc_fuse.c       |  14 +--
src/lxc/lxc_fuse.h       |   1 +
src/lxc/lxc_hostdev.c    |   1 -
src/lxc/lxc_monitor.c    |   9 +-
src/lxc/lxc_native.c     |  81 ++++++------
src/lxc/lxc_process.c    | 135 ++++++++------------
12 files changed, 306 insertions(+), 508 deletions(-)
[libvirt PATCH 0/2] lxc: remove most use of libvirt allocation APIs
Posted by Daniel P. Berrangé 3 years, 10 months ago
This removes VIR_ALLOC / VIR_FREE and friends, but not
the VIR_*_ELEMENT API usage.

Daniel P. Berrangé (2):
  lxc: replace VIR_FREE with g_autofree / g_free
  lxc: replace VIR_ALLOC/REALLOC with g_new0/renew

 src/lxc/lxc_cgroup.c     |  15 +--
 src/lxc/lxc_conf.c       |  19 ++-
 src/lxc/lxc_container.c  | 262 ++++++++++++++-------------------------
 src/lxc/lxc_controller.c | 148 ++++++++--------------
 src/lxc/lxc_domain.c     |  40 +++---
 src/lxc/lxc_driver.c     |  89 ++++++-------
 src/lxc/lxc_fuse.c       |  14 +--
 src/lxc/lxc_fuse.h       |   1 +
 src/lxc/lxc_hostdev.c    |   1 -
 src/lxc/lxc_monitor.c    |   9 +-
 src/lxc/lxc_native.c     |  81 ++++++------
 src/lxc/lxc_process.c    | 135 ++++++++------------
 12 files changed, 306 insertions(+), 508 deletions(-)

-- 
2.24.1

Re: [libvirt PATCH 0/2] lxc: remove most use of libvirt allocation APIs
Posted by Laine Stump 3 years, 10 months ago
On 5/13/20 12:22 PM, Daniel P. Berrangé wrote:
> This removes VIR_ALLOC / VIR_FREE and friends, but not
> the VIR_*_ELEMENT API usage.
>
> Daniel P. Berrangé (2):
>    lxc: replace VIR_FREE with g_autofree / g_free
>    lxc: replace VIR_ALLOC/REALLOC with g_new0/renew
>
>   src/lxc/lxc_cgroup.c     |  15 +--
>   src/lxc/lxc_conf.c       |  19 ++-
>   src/lxc/lxc_container.c  | 262 ++++++++++++++-------------------------
>   src/lxc/lxc_controller.c | 148 ++++++++--------------
>   src/lxc/lxc_domain.c     |  40 +++---
>   src/lxc/lxc_driver.c     |  89 ++++++-------
>   src/lxc/lxc_fuse.c       |  14 +--
>   src/lxc/lxc_fuse.h       |   1 +
>   src/lxc/lxc_hostdev.c    |   1 -
>   src/lxc/lxc_monitor.c    |   9 +-
>   src/lxc/lxc_native.c     |  81 ++++++------
>   src/lxc/lxc_process.c    | 135 ++++++++------------
>   12 files changed, 306 insertions(+), 508 deletions(-)
>
Reviewed-by: Laine Stump <laine@redhat.com>