[v2] dump: Arch info function pointer addition and cleanup

[PATCH v2 0/3] dump: Arch info function pointer addition and cleanup

Janosch Frank posted 3 patches 1 year ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231109120443.185979-1-frankja@linux.ibm.com

Maintainers: "Marc-André Lureau" <marcandre.lureau@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, David Hildenbrand <david@redhat.com>, Ilya Leoshkevich <iii@linux.ibm.com>, Thomas Huth <thuth@redhat.com>

dump/dump.c                |  4 ++++
include/sysemu/dump-arch.h |  1 +
target/s390x/arch_dump.c   | 21 +++++++++++++++++----
3 files changed, 22 insertions(+), 4 deletions(-)

Expand all Fold all

[PATCH v2 0/3] dump: Arch info function pointer addition and cleanup

Posted by Janosch Frank 1 year ago

Small cleanups/fixes to the dump info function pointer assignments as
well as a new function pointer for cleanup of residual state.

This has come up because test managed to dump a s390 PV vm onto a disk
that was too small for the dump. After the dump failed, the vm wasn't
able to resume running since KVM was still in dump mode which blocks
vcpu entry.

The new function pointer allows cleanup of such a situation.

v2:
	- Usage of g_autofree
	- Dropped explicit NULLing of function pointers

Janosch Frank (3):
  target/s390x/dump: Remove unneeded dump info function pointer init
  dump: Add arch cleanup function
  target/s390x/arch_dump: Add arch cleanup function for PV dumps

 dump/dump.c                |  4 ++++
 include/sysemu/dump-arch.h |  1 +
 target/s390x/arch_dump.c   | 21 +++++++++++++++++----
 3 files changed, 22 insertions(+), 4 deletions(-)

-- 
2.34.1

Re: [PATCH v2 0/3] dump: Arch info function pointer addition and cleanup

Posted by Marc-André Lureau 1 year ago

Hi

On Thu, Nov 9, 2023 at 4:05 PM Janosch Frank <frankja@linux.ibm.com> wrote:
>
> Small cleanups/fixes to the dump info function pointer assignments as
> well as a new function pointer for cleanup of residual state.
>
> This has come up because test managed to dump a s390 PV vm onto a disk
> that was too small for the dump. After the dump failed, the vm wasn't
> able to resume running since KVM was still in dump mode which blocks
> vcpu entry.
>
> The new function pointer allows cleanup of such a situation.
>
> v2:
>         - Usage of g_autofree
>         - Dropped explicit NULLing of function pointers
>
> Janosch Frank (3):
>   target/s390x/dump: Remove unneeded dump info function pointer init
>   dump: Add arch cleanup function
>   target/s390x/arch_dump: Add arch cleanup function for PV dumps
>
>  dump/dump.c                |  4 ++++
>  include/sysemu/dump-arch.h |  1 +
>  target/s390x/arch_dump.c   | 21 +++++++++++++++++----
>  3 files changed, 22 insertions(+), 4 deletions(-)
>

Series:
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>



-- 
Marc-André Lureau