[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH V3 0/3] Cleanup: flexarray taking gc.



This two patches do a bit of cleanup in the memomy managment in libxl,
regarding the use of flexarray.

The first one modify flexarray_make to take gc as argument and update every
user in libxl to pass gc and to not call flexarray_free anymore.

The second one does some cleanup only in libxl_json to make it use the gc.

Change since v2:
  - New patch to expose gc_is_real to libxl.

Change since v1:
  - Change order of the two patch to not end up with leaks.
  - Add a comment on the first patch to tell why it OK to store the gc in the
    flexarray struct.
  - Keep the _free functions (libxl__json_object_free and flexarray_free, as
    they can be used if gc in NOGC.


Anthony PERARD (3):
  libxl: Move gc_is_real to libxl_internal.h.
  libxl: Have flexarray using the GC
  libxl_json: Use libxl alloc function.

 tools/libxl/flexarray.c      | 43 +++++++++++--------
 tools/libxl/flexarray.h      |  7 +++-
 tools/libxl/libxl.c          | 99 +++++++++-----------------------------------
 tools/libxl/libxl_dm.c       | 15 ++-----
 tools/libxl/libxl_internal.c | 11 ++---
 tools/libxl/libxl_internal.h |  5 +++
 tools/libxl/libxl_json.c     | 93 +++++++++--------------------------------
 tools/libxl/libxl_pci.c      | 18 ++------
 tools/libxl/libxl_qmp.c      | 29 ++-----------
 9 files changed, 86 insertions(+), 234 deletions(-)

-- 
Anthony PERARD


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.