Re: [PATCH 2/4] target/s390x/dump: Remove unneeded dump info function pointer init

2023-11-08 Thread Marc-André Lureau
Hi On Tue, Nov 7, 2023 at 6:22 PM Janosch Frank 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 regardless if the previous patch is applied, this patch lgtm Reviewed-b

Re: [PATCH 2/4] target/s390x/dump: Remove unneeded dump info function pointer init

2023-11-07 Thread Claudio Imbrenda
On Tue, 7 Nov 2023 14:20:46 + Janosch Frank wrote: > dump_state_prepare() now sets the fucntion pointers to NULL so we only > need to touch them if we're going to use them. > > Signed-off-by: Janosch Frank I would merge this and the previous patch > --- > target/s390x/arch_dump.c | 4 --

[PATCH 2/4] target/s390x/dump: Remove unneeded dump info function pointer init

2023-11-07 Thread Janosch Frank
dump_state_prepare() now sets the fucntion pointers to NULL so we only need to touch them if we're going to use them. Signed-off-by: Janosch Frank --- 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 51a2116