[v2] Fix memory leak of some device state in migration

[PATCH v2 0/3] Fix memory leak of some device state in migration

g00517791 posted 3 patches 4 years, 10 months ago

Diff against v1 v3
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201228090053.346-1-gaojinhao@huawei.com

Maintainers: Greg Kurz <groug@kaod.org>, David Gibson <david@gibson.dropbear.id.au>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Juan Quintela <quintela@redhat.com>

There is a newer version of this series

hw/ppc/spapr_pci.c  | 11 +++++++++++
migration/savevm.c  | 31 +++++++++++++++++++++++++++----
migration/vmstate.c |  1 +
3 files changed, 39 insertions(+), 4 deletions(-)

Expand all Fold all

[PATCH v2 0/3] Fix memory leak of some device state in migration

Posted by g00517791 4 years, 10 months ago

From: Jinhao Gao <gaojinhao@huawei.com>

For some device state having some fields of VMS_ALLOC flag, they
don't free memory allocated for the fields in vmstate_save_state
and vmstate_load_state. We add funcs or sentences of free memory
before and after VM saves or loads device state to avoid memory leak.

v2
 - Drop patch1-3,6-8 of v1
 - Address Michael's comment (free memory before load vmsd centrally)
 - Add David's Acked-by and Michael's Signed-off-by

Jinhao Gao (3):
  spapr_pci: Fix memory leak of vmstate_spapr_pci
  savevm: Fix memory leak of vmstate_configuration
  vmstate: Fix memory leak in vmstate_handle_alloc()

 hw/ppc/spapr_pci.c  | 11 +++++++++++
 migration/savevm.c  | 31 +++++++++++++++++++++++++++----
 migration/vmstate.c |  1 +
 3 files changed, 39 insertions(+), 4 deletions(-)

-- 
2.23.0