dump: Replace opaque DumpState pointer with a typed one

It's always better to convey the type of a pointer if at all
possible. So let's add the DumpState typedef to typedefs.h and move
the dump note functions from the opaque pointers to DumpState
pointers.

Signed-off-by: Janosch Frank <frankja@linux.ibm.com>
CC: Peter Maydell <peter.maydell@linaro.org>
CC: Cédric Le Goater <clg@kaod.org>
CC: Daniel Henrique Barboza <danielhb413@gmail.com>
CC: David Gibson <david@gibson.dropbear.id.au>
CC: Greg Kurz <groug@kaod.org>
CC: Palmer Dabbelt <palmer@dabbelt.com>
CC: Alistair Francis <alistair.francis@wdc.com>
CC: Bin Meng <bin.meng@windriver.com>
CC: Cornelia Huck <cohuck@redhat.com>
CC: Thomas Huth <thuth@redhat.com>
CC: Richard Henderson <richard.henderson@linaro.org>
CC: David Hildenbrand <david@redhat.com>
Acked-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-Id: <20220811121111.9878-2-frankja@linux.ibm.com>
This commit is contained in:
Janosch Frank 2022-08-11 12:10:54 +00:00 committed by Marc-André Lureau
parent f1d33f55c4
commit 1af0006ab9
12 changed files with 49 additions and 52 deletions

View File

@ -53,25 +53,25 @@ typedef struct SysemuCPUOps {
* 32-bit VM coredump. * 32-bit VM coredump.
*/ */
int (*write_elf32_note)(WriteCoreDumpFunction f, CPUState *cpu, int (*write_elf32_note)(WriteCoreDumpFunction f, CPUState *cpu,
int cpuid, void *opaque); int cpuid, DumpState *s);
/** /**
* @write_elf64_note: Callback for writing a CPU-specific ELF note to a * @write_elf64_note: Callback for writing a CPU-specific ELF note to a
* 64-bit VM coredump. * 64-bit VM coredump.
*/ */
int (*write_elf64_note)(WriteCoreDumpFunction f, CPUState *cpu, int (*write_elf64_note)(WriteCoreDumpFunction f, CPUState *cpu,
int cpuid, void *opaque); int cpuid, DumpState *s);
/** /**
* @write_elf32_qemunote: Callback for writing a CPU- and QEMU-specific ELF * @write_elf32_qemunote: Callback for writing a CPU- and QEMU-specific ELF
* note to a 32-bit VM coredump. * note to a 32-bit VM coredump.
*/ */
int (*write_elf32_qemunote)(WriteCoreDumpFunction f, CPUState *cpu, int (*write_elf32_qemunote)(WriteCoreDumpFunction f, CPUState *cpu,
void *opaque); DumpState *s);
/** /**
* @write_elf64_qemunote: Callback for writing a CPU- and QEMU-specific ELF * @write_elf64_qemunote: Callback for writing a CPU- and QEMU-specific ELF
* note to a 64-bit VM coredump. * note to a 64-bit VM coredump.
*/ */
int (*write_elf64_qemunote)(WriteCoreDumpFunction f, CPUState *cpu, int (*write_elf64_qemunote)(WriteCoreDumpFunction f, CPUState *cpu,
void *opaque); DumpState *s);
/** /**
* @virtio_is_big_endian: Callback to return %true if a CPU which supports * @virtio_is_big_endian: Callback to return %true if a CPU which supports
* runtime configurable endianness is currently big-endian. * runtime configurable endianness is currently big-endian.

View File

@ -131,6 +131,7 @@ typedef struct VirtIODevice VirtIODevice;
typedef struct Visitor Visitor; typedef struct Visitor Visitor;
typedef struct VMChangeStateEntry VMChangeStateEntry; typedef struct VMChangeStateEntry VMChangeStateEntry;
typedef struct VMStateDescription VMStateDescription; typedef struct VMStateDescription VMStateDescription;
typedef struct DumpState DumpState;
/* /*
* Pointer types * Pointer types

View File

@ -232,12 +232,11 @@ static int aarch64_write_elf64_sve(WriteCoreDumpFunction f,
#endif #endif
int arm_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int arm_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
struct aarch64_note note; struct aarch64_note note;
ARMCPU *cpu = ARM_CPU(cs); ARMCPU *cpu = ARM_CPU(cs);
CPUARMState *env = &cpu->env; CPUARMState *env = &cpu->env;
DumpState *s = opaque;
uint64_t pstate, sp; uint64_t pstate, sp;
int ret, i; int ret, i;
@ -360,12 +359,11 @@ static int arm_write_elf32_vfp(WriteCoreDumpFunction f, CPUARMState *env,
} }
int arm_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs, int arm_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
struct arm_note note; struct arm_note note;
ARMCPU *cpu = ARM_CPU(cs); ARMCPU *cpu = ARM_CPU(cs);
CPUARMState *env = &cpu->env; CPUARMState *env = &cpu->env;
DumpState *s = opaque;
int ret, i; int ret, i;
bool fpvalid = cpu_isar_feature(aa32_vfp_simd, cpu); bool fpvalid = cpu_isar_feature(aa32_vfp_simd, cpu);

View File

@ -1102,9 +1102,9 @@ int arm_gen_dynamic_svereg_xml(CPUState *cpu, int base_reg);
const char *arm_gdb_get_dynamic_xml(CPUState *cpu, const char *xmlname); const char *arm_gdb_get_dynamic_xml(CPUState *cpu, const char *xmlname);
int arm_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int arm_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque); int cpuid, DumpState *s);
int arm_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs, int arm_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque); int cpuid, DumpState *s);
#ifdef TARGET_AARCH64 #ifdef TARGET_AARCH64
int aarch64_cpu_gdb_read_register(CPUState *cpu, GByteArray *buf, int reg); int aarch64_cpu_gdb_read_register(CPUState *cpu, GByteArray *buf, int reg);

View File

@ -42,7 +42,7 @@ typedef struct {
static int x86_64_write_elf64_note(WriteCoreDumpFunction f, static int x86_64_write_elf64_note(WriteCoreDumpFunction f,
CPUX86State *env, int id, CPUX86State *env, int id,
void *opaque) DumpState *s)
{ {
x86_64_user_regs_struct regs; x86_64_user_regs_struct regs;
Elf64_Nhdr *note; Elf64_Nhdr *note;
@ -94,7 +94,7 @@ static int x86_64_write_elf64_note(WriteCoreDumpFunction f,
buf += descsz - sizeof(x86_64_user_regs_struct)-sizeof(target_ulong); buf += descsz - sizeof(x86_64_user_regs_struct)-sizeof(target_ulong);
memcpy(buf, &regs, sizeof(x86_64_user_regs_struct)); memcpy(buf, &regs, sizeof(x86_64_user_regs_struct));
ret = f(note, note_size, opaque); ret = f(note, note_size, s);
g_free(note); g_free(note);
if (ret < 0) { if (ret < 0) {
return -1; return -1;
@ -148,7 +148,7 @@ static void x86_fill_elf_prstatus(x86_elf_prstatus *prstatus, CPUX86State *env,
} }
static int x86_write_elf64_note(WriteCoreDumpFunction f, CPUX86State *env, static int x86_write_elf64_note(WriteCoreDumpFunction f, CPUX86State *env,
int id, void *opaque) int id, DumpState *s)
{ {
x86_elf_prstatus prstatus; x86_elf_prstatus prstatus;
Elf64_Nhdr *note; Elf64_Nhdr *note;
@ -170,7 +170,7 @@ static int x86_write_elf64_note(WriteCoreDumpFunction f, CPUX86State *env,
buf += ROUND_UP(name_size, 4); buf += ROUND_UP(name_size, 4);
memcpy(buf, &prstatus, sizeof(prstatus)); memcpy(buf, &prstatus, sizeof(prstatus));
ret = f(note, note_size, opaque); ret = f(note, note_size, s);
g_free(note); g_free(note);
if (ret < 0) { if (ret < 0) {
return -1; return -1;
@ -180,7 +180,7 @@ static int x86_write_elf64_note(WriteCoreDumpFunction f, CPUX86State *env,
} }
int x86_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int x86_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
X86CPU *cpu = X86_CPU(cs); X86CPU *cpu = X86_CPU(cs);
int ret; int ret;
@ -189,10 +189,10 @@ int x86_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
bool lma = !!(first_x86_cpu->env.hflags & HF_LMA_MASK); bool lma = !!(first_x86_cpu->env.hflags & HF_LMA_MASK);
if (lma) { if (lma) {
ret = x86_64_write_elf64_note(f, &cpu->env, cpuid, opaque); ret = x86_64_write_elf64_note(f, &cpu->env, cpuid, s);
} else { } else {
#endif #endif
ret = x86_write_elf64_note(f, &cpu->env, cpuid, opaque); ret = x86_write_elf64_note(f, &cpu->env, cpuid, s);
#ifdef TARGET_X86_64 #ifdef TARGET_X86_64
} }
#endif #endif
@ -201,7 +201,7 @@ int x86_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
} }
int x86_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs, int x86_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
X86CPU *cpu = X86_CPU(cs); X86CPU *cpu = X86_CPU(cs);
x86_elf_prstatus prstatus; x86_elf_prstatus prstatus;
@ -224,7 +224,7 @@ int x86_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
buf += ROUND_UP(name_size, 4); buf += ROUND_UP(name_size, 4);
memcpy(buf, &prstatus, sizeof(prstatus)); memcpy(buf, &prstatus, sizeof(prstatus));
ret = f(note, note_size, opaque); ret = f(note, note_size, s);
g_free(note); g_free(note);
if (ret < 0) { if (ret < 0) {
return -1; return -1;
@ -329,7 +329,7 @@ static void qemu_get_cpustate(QEMUCPUState *s, CPUX86State *env)
static inline int cpu_write_qemu_note(WriteCoreDumpFunction f, static inline int cpu_write_qemu_note(WriteCoreDumpFunction f,
CPUX86State *env, CPUX86State *env,
void *opaque, DumpState *s,
int type) int type)
{ {
QEMUCPUState state; QEMUCPUState state;
@ -369,7 +369,7 @@ static inline int cpu_write_qemu_note(WriteCoreDumpFunction f,
buf += ROUND_UP(name_size, 4); buf += ROUND_UP(name_size, 4);
memcpy(buf, &state, sizeof(state)); memcpy(buf, &state, sizeof(state));
ret = f(note, note_size, opaque); ret = f(note, note_size, s);
g_free(note); g_free(note);
if (ret < 0) { if (ret < 0) {
return -1; return -1;
@ -379,19 +379,19 @@ static inline int cpu_write_qemu_note(WriteCoreDumpFunction f,
} }
int x86_cpu_write_elf64_qemunote(WriteCoreDumpFunction f, CPUState *cs, int x86_cpu_write_elf64_qemunote(WriteCoreDumpFunction f, CPUState *cs,
void *opaque) DumpState *s)
{ {
X86CPU *cpu = X86_CPU(cs); X86CPU *cpu = X86_CPU(cs);
return cpu_write_qemu_note(f, &cpu->env, opaque, 1); return cpu_write_qemu_note(f, &cpu->env, s, 1);
} }
int x86_cpu_write_elf32_qemunote(WriteCoreDumpFunction f, CPUState *cs, int x86_cpu_write_elf32_qemunote(WriteCoreDumpFunction f, CPUState *cs,
void *opaque) DumpState *s)
{ {
X86CPU *cpu = X86_CPU(cs); X86CPU *cpu = X86_CPU(cs);
return cpu_write_qemu_note(f, &cpu->env, opaque, 0); return cpu_write_qemu_note(f, &cpu->env, s, 0);
} }
int cpu_get_dump_info(ArchDumpInfo *info, int cpu_get_dump_info(ArchDumpInfo *info,

View File

@ -1938,13 +1938,13 @@ extern const VMStateDescription vmstate_x86_cpu;
int x86_cpu_pending_interrupt(CPUState *cs, int interrupt_request); int x86_cpu_pending_interrupt(CPUState *cs, int interrupt_request);
int x86_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cpu, int x86_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cpu,
int cpuid, void *opaque); int cpuid, DumpState *s);
int x86_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cpu, int x86_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cpu,
int cpuid, void *opaque); int cpuid, DumpState *s);
int x86_cpu_write_elf64_qemunote(WriteCoreDumpFunction f, CPUState *cpu, int x86_cpu_write_elf64_qemunote(WriteCoreDumpFunction f, CPUState *cpu,
void *opaque); DumpState *s);
int x86_cpu_write_elf32_qemunote(WriteCoreDumpFunction f, CPUState *cpu, int x86_cpu_write_elf32_qemunote(WriteCoreDumpFunction f, CPUState *cpu,
void *opaque); DumpState *s);
void x86_cpu_get_memory_mapping(CPUState *cpu, MemoryMappingList *list, void x86_cpu_get_memory_mapping(CPUState *cpu, MemoryMappingList *list,
Error **errp); Error **errp);

View File

@ -270,23 +270,23 @@ ssize_t cpu_get_note_size(int class, int machine, int nr_cpus)
static int ppc_write_all_elf_notes(const char *note_name, static int ppc_write_all_elf_notes(const char *note_name,
WriteCoreDumpFunction f, WriteCoreDumpFunction f,
PowerPCCPU *cpu, int id, PowerPCCPU *cpu, int id,
void *opaque) DumpState *s)
{ {
NoteFuncArg arg = { .state = opaque }; NoteFuncArg arg = { .state = s };
int ret = -1; int ret = -1;
int note_size; int note_size;
const NoteFuncDesc *nf; const NoteFuncDesc *nf;
for (nf = note_func; nf->note_contents_func; nf++) { for (nf = note_func; nf->note_contents_func; nf++) {
arg.note.hdr.n_namesz = cpu_to_dump32(opaque, sizeof(arg.note.name)); arg.note.hdr.n_namesz = cpu_to_dump32(s, sizeof(arg.note.name));
arg.note.hdr.n_descsz = cpu_to_dump32(opaque, nf->contents_size); arg.note.hdr.n_descsz = cpu_to_dump32(s, nf->contents_size);
strncpy(arg.note.name, note_name, sizeof(arg.note.name)); strncpy(arg.note.name, note_name, sizeof(arg.note.name));
(*nf->note_contents_func)(&arg, cpu); (*nf->note_contents_func)(&arg, cpu);
note_size = note_size =
sizeof(arg.note) - sizeof(arg.note.contents) + nf->contents_size; sizeof(arg.note) - sizeof(arg.note.contents) + nf->contents_size;
ret = f(&arg.note, note_size, opaque); ret = f(&arg.note, note_size, s);
if (ret < 0) { if (ret < 0) {
return -1; return -1;
} }
@ -295,15 +295,15 @@ static int ppc_write_all_elf_notes(const char *note_name,
} }
int ppc64_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int ppc64_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
PowerPCCPU *cpu = POWERPC_CPU(cs); PowerPCCPU *cpu = POWERPC_CPU(cs);
return ppc_write_all_elf_notes("CORE", f, cpu, cpuid, opaque); return ppc_write_all_elf_notes("CORE", f, cpu, cpuid, s);
} }
int ppc32_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs, int ppc32_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
PowerPCCPU *cpu = POWERPC_CPU(cs); PowerPCCPU *cpu = POWERPC_CPU(cs);
return ppc_write_all_elf_notes("CORE", f, cpu, cpuid, opaque); return ppc_write_all_elf_notes("CORE", f, cpu, cpuid, s);
} }

View File

@ -1354,9 +1354,9 @@ void ppc_gdb_gen_spr_xml(PowerPCCPU *cpu);
const char *ppc_gdb_get_dynamic_xml(CPUState *cs, const char *xml_name); const char *ppc_gdb_get_dynamic_xml(CPUState *cs, const char *xml_name);
#endif #endif
int ppc64_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int ppc64_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque); int cpuid, DumpState *s);
int ppc32_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs, int ppc32_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque); int cpuid, DumpState *s);
#ifndef CONFIG_USER_ONLY #ifndef CONFIG_USER_ONLY
void ppc_cpu_do_interrupt(CPUState *cpu); void ppc_cpu_do_interrupt(CPUState *cpu);
bool ppc_cpu_exec_interrupt(CPUState *cpu, int int_req); bool ppc_cpu_exec_interrupt(CPUState *cpu, int int_req);

View File

@ -64,12 +64,11 @@ static void riscv64_note_init(struct riscv64_note *note, DumpState *s,
} }
int riscv_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int riscv_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
struct riscv64_note note; struct riscv64_note note;
RISCVCPU *cpu = RISCV_CPU(cs); RISCVCPU *cpu = RISCV_CPU(cs);
CPURISCVState *env = &cpu->env; CPURISCVState *env = &cpu->env;
DumpState *s = opaque;
int ret, i = 0; int ret, i = 0;
const char name[] = "CORE"; const char name[] = "CORE";
@ -134,12 +133,11 @@ static void riscv32_note_init(struct riscv32_note *note, DumpState *s,
} }
int riscv_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs, int riscv_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
struct riscv32_note note; struct riscv32_note note;
RISCVCPU *cpu = RISCV_CPU(cs); RISCVCPU *cpu = RISCV_CPU(cs);
CPURISCVState *env = &cpu->env; CPURISCVState *env = &cpu->env;
DumpState *s = opaque;
int ret, i; int ret, i;
const char name[] = "CORE"; const char name[] = "CORE";

View File

@ -534,9 +534,9 @@ extern const char * const riscv_fpr_regnames[];
const char *riscv_cpu_get_trap_name(target_ulong cause, bool async); const char *riscv_cpu_get_trap_name(target_ulong cause, bool async);
void riscv_cpu_do_interrupt(CPUState *cpu); void riscv_cpu_do_interrupt(CPUState *cpu);
int riscv_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int riscv_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque); int cpuid, DumpState *s);
int riscv_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs, int riscv_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque); int cpuid, DumpState *s);
int riscv_cpu_gdb_read_register(CPUState *cpu, GByteArray *buf, int reg); int riscv_cpu_gdb_read_register(CPUState *cpu, GByteArray *buf, int reg);
int riscv_cpu_gdb_write_register(CPUState *cpu, uint8_t *buf, int reg); int riscv_cpu_gdb_write_register(CPUState *cpu, uint8_t *buf, int reg);
int riscv_cpu_hviprio_index2irq(int index, int *out_irq, int *out_rdzero); int riscv_cpu_hviprio_index2irq(int index, int *out_irq, int *out_rdzero);

View File

@ -204,7 +204,7 @@ static const NoteFuncDesc note_linux[] = {
static int s390x_write_elf64_notes(const char *note_name, static int s390x_write_elf64_notes(const char *note_name,
WriteCoreDumpFunction f, WriteCoreDumpFunction f,
S390CPU *cpu, int id, S390CPU *cpu, int id,
void *opaque, DumpState *s,
const NoteFuncDesc *funcs) const NoteFuncDesc *funcs)
{ {
Note note; Note note;
@ -222,7 +222,7 @@ static int s390x_write_elf64_notes(const char *note_name,
(*nf->note_contents_func)(&note, cpu, id); (*nf->note_contents_func)(&note, cpu, id);
note_size = sizeof(note) - sizeof(note.contents) + nf->contents_size; note_size = sizeof(note) - sizeof(note.contents) + nf->contents_size;
ret = f(&note, note_size, opaque); ret = f(&note, note_size, s);
if (ret < 0) { if (ret < 0) {
return -1; return -1;
@ -235,16 +235,16 @@ static int s390x_write_elf64_notes(const char *note_name,
int s390_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int s390_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque) int cpuid, DumpState *s)
{ {
S390CPU *cpu = S390_CPU(cs); S390CPU *cpu = S390_CPU(cs);
int r; int r;
r = s390x_write_elf64_notes("CORE", f, cpu, cpuid, opaque, note_core); r = s390x_write_elf64_notes("CORE", f, cpu, cpuid, s, note_core);
if (r) { if (r) {
return r; return r;
} }
return s390x_write_elf64_notes("LINUX", f, cpu, cpuid, opaque, note_linux); return s390x_write_elf64_notes("LINUX", f, cpu, cpuid, s, note_linux);
} }
int cpu_get_dump_info(ArchDumpInfo *info, int cpu_get_dump_info(ArchDumpInfo *info,

View File

@ -227,7 +227,7 @@ static inline hwaddr decode_basedisp_s(CPUS390XState *env, uint32_t ipb,
/* arch_dump.c */ /* arch_dump.c */
int s390_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs, int s390_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
int cpuid, void *opaque); int cpuid, DumpState *s);
/* cc_helper.c */ /* cc_helper.c */