Hi
On Tue, Nov 7, 2023 at 6:22 PM Janosch Frank <fran...@linux.ibm.com> wrote:
>
> dump_state_prepare() now sets the fucntion pointers to NULL so we only
function
> need to touch them if we're going to use them.
>
> Signed-off-by: Janosch Frank <fran...@linux.ibm.com>
regardless if the previous patch is applied, this patch lgtm
Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com>
> ---
> target/s390x/arch_dump.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/target/s390x/arch_dump.c b/target/s390x/arch_dump.c
> index 51a2116515..bdb0bfa0e7 100644
> --- a/target/s390x/arch_dump.c
> +++ b/target/s390x/arch_dump.c
> @@ -448,10 +448,6 @@ int cpu_get_dump_info(ArchDumpInfo *info,
> info->arch_sections_add_fn = *arch_sections_add;
> info->arch_sections_write_hdr_fn = *arch_sections_write_hdr;
> info->arch_sections_write_fn = *arch_sections_write;
> - } else {
> - info->arch_sections_add_fn = NULL;
> - info->arch_sections_write_hdr_fn = NULL;
> - info->arch_sections_write_fn = NULL;
> }
> return 0;
> }
> --
> 2.34.1
>
>
--
Marc-André Lureau
