[libvirt] [PATCH 00/12] OOM reporting cleanups

Ján Tomko jtomko at redhat.com
Wed Jul 2 10:10:53 UTC 2014


Patches 1-3 don't touch any code
4-5 no functional change
6 introduces a new function
7-8 have no functional change unless virBuffer API is misused
  (that case resulted in an OOM error before, now it's an internal
   error)
9-10 add OOM error reporting to a few functions
11-12 remove double OOM error reporting

Ján Tomko (12):
  Fix indentation in bridge driver
  More indentation fixes
  usb: Remove redundant comment
  Remove useless condition in networkRadvdConfContents
  Use virStringReplace instead of openvz_replace
  Introduce virBufferCheckError
  Report errors in virCapabilitiesFormatXML
  Use virBufferCheckError everywhere we report OOM error
  Set errno on OOM in lxcProcReadMeminfo
  Add OOM error reporting to a few fucntions
  Remove double OOM error reporting from JSON monitor
  Remove double OOM error reporting

 HACKING                                   |   7 +-
 docs/hacking.html.in                      |   7 +-
 po/POTFILES.in                            |   1 +
 src/bhyve/bhyve_driver.c                  |  12 +--
 src/conf/capabilities.c                   |   2 +-
 src/conf/cpu_conf.c                       |   6 +-
 src/conf/domain_conf.c                    |  17 +---
 src/conf/interface_conf.c                 |   8 +-
 src/conf/network_conf.c                   |  12 +--
 src/conf/node_device_conf.c               |   9 +-
 src/conf/nwfilter_conf.c                  |  16 +---
 src/conf/secret_conf.c                    |   6 +-
 src/conf/snapshot_conf.c                  |   5 +-
 src/conf/storage_conf.c                   |  18 ++--
 src/cpu/cpu_map.c                         |  14 ++-
 src/cpu/cpu_x86.c                         |   5 +-
 src/esx/esx_driver.c                      |  20 +---
 src/esx/esx_util.c                        |  12 +--
 src/esx/esx_vi.c                          |  20 +---
 src/esx/esx_vi_methods.c                  |   4 +-
 src/hyperv/hyperv_wmi.c                   |   4 +-
 src/libvirt_private.syms                  |   1 +
 src/libxl/libxl_driver.c                  |  12 +--
 src/locking/lock_driver_sanlock.c         |   5 +-
 src/lxc/lxc_container.c                   |   2 +-
 src/lxc/lxc_controller.c                  |   8 +-
 src/lxc/lxc_driver.c                      |  47 +++-------
 src/lxc/lxc_fuse.c                        |   4 +-
 src/network/bridge_driver.c               | 105 ++++++++++-----------
 src/node_device/node_device_udev.c        |   8 +-
 src/nwfilter/nwfilter_ebiptables_driver.c |  10 +-
 src/nwfilter/nwfilter_gentech_driver.c    |  63 ++++++-------
 src/openvz/openvz_conf.c                  |  40 +-------
 src/parallels/parallels_driver.c          |   3 +-
 src/phyp/phyp_driver.c                    |  16 +---
 src/qemu/qemu_capabilities.c              |   2 +-
 src/qemu/qemu_command.c                   | 150 ++++++++----------------------
 src/qemu/qemu_driver.c                    |  47 +++-------
 src/qemu/qemu_hotplug.c                   |   4 +-
 src/qemu/qemu_migration.c                 |   5 +-
 src/qemu/qemu_monitor.c                   |   5 +-
 src/qemu/qemu_monitor_json.c              |  21 ++---
 src/qemu/qemu_monitor_text.c              |   9 +-
 src/rpc/virnetsocket.c                    |   4 +-
 src/rpc/virnetsshsession.c                |   8 +-
 src/storage/storage_backend.c             |   6 +-
 src/storage/storage_backend_rbd.c         |  12 +--
 src/test/test_driver.c                    |   3 +-
 src/uml/uml_conf.c                        |   4 +-
 src/uml/uml_driver.c                      |   3 +-
 src/util/virbuffer.c                      |  31 ++++++
 src/util/virbuffer.h                      |  17 ++++
 src/util/vircommand.c                     |   5 +-
 src/util/virconf.c                        |  10 +-
 src/util/virnetdevopenvswitch.c           |   4 +-
 src/util/virstoragefile.c                 |   4 +-
 src/util/virstring.c                      |   8 +-
 src/util/virsysinfo.c                     |   4 +-
 src/util/virsystemd.c                     |   8 +-
 src/util/viruri.c                         |   7 +-
 src/util/virusb.c                         |   1 -
 src/vmx/vmx.c                             |   4 +-
 src/xen/xen_driver.c                      |  12 +--
 src/xen/xen_hypervisor.c                  |   8 +-
 src/xen/xend_internal.c                   |   9 +-
 src/xenapi/xenapi_driver.c                |   8 +-
 src/xenapi/xenapi_utils.c                 |   5 +-
 src/xenxs/xen_sxpr.c                      |  12 +--
 src/xenxs/xen_xm.c                        |  17 +---
 tests/domainsnapshotxml2xmltest.c         |   4 +-
 tests/vircaps2xmltest.c                   |   4 +-
 71 files changed, 328 insertions(+), 666 deletions(-)

-- 
1.8.5.5




More information about the libvir-list mailing list