commit e72629e5149aba6f44122ea6d2a803ef136a0c6b
author Janosch Frank <frankja@linux.ibm.com> Thu Nov 09 12:04:42 2023 +0000
committer Thomas Huth <thuth@redhat.com> Tue Nov 14 10:42:32 2023 +0100
tree 23a560b59a445063d2f849e956a29681e620fede
parent 816644b1219900875f47d7adf9bfb283f1b29aa0

dump: Add arch cleanup function

Some architectures (s390x) need to cleanup after a failed dump to be
able to continue to run the vm. Add a cleanup function pointer and
call it if it's set.

Signed-off-by: Janosch Frank <frankja@linux.ibm.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-ID: <20231109120443.185979-3-frankja@linux.ibm.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>

dump/dump.c

diff --git a/dump/dump.c b/dump/dump.c
index ad5294e..4819050 100644
--- a/dump/dump.c
+++ b/dump/dump.c
@@ -96,6 +96,10 @@
 
 static int dump_cleanup(DumpState *s)
 {
+    if (s->dump_info.arch_cleanup_fn) {
+        s->dump_info.arch_cleanup_fn(s);
+    }
+
     guest_phys_blocks_free(&s->guest_phys_blocks);
     memory_mapping_list_free(&s->list);
     close(s->fd);

include/sysemu/dump-arch.h

diff --git a/include/sysemu/dump-arch.h b/include/sysemu/dump-arch.h
index 59bbc9b..743916e 100644
--- a/include/sysemu/dump-arch.h
+++ b/include/sysemu/dump-arch.h
@@ -24,6 +24,7 @@
     void (*arch_sections_add_fn)(DumpState *s);
     uint64_t (*arch_sections_write_hdr_fn)(DumpState *s, uint8_t *buff);
     int (*arch_sections_write_fn)(DumpState *s, uint8_t *buff);
+    void (*arch_cleanup_fn)(DumpState *s);
 } ArchDumpInfo;
 
 struct GuestPhysBlockList; /* memory_mapping.h */