From: "Marc-André Lureau" <marcandre.lureau@redhat.com>
To: Janosch Frank <frankja@linux.ibm.com>
Cc: qemu-devel@nongnu.org, pbonzini@redhat.com,
mhartmay@linux.ibm.com, borntraeger@linux.ibm.com,
imbrenda@linux.ibm.com, pasic@linux.ibm.com, cohuck@redhat.com,
thuth@redhat.com, qemu-s390x@nongnu.org, seiden@linux.ibm.com,
scgl@linux.ibm.com, "Peter Maydell" <peter.maydell@linaro.org>,
"Cédric Le Goater" <clg@kaod.org>,
"Daniel Henrique Barboza" <danielhb413@gmail.com>,
"David Gibson" <david@gibson.dropbear.id.au>,
"Greg Kurz" <groug@kaod.org>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Alistair Francis" <alistair.francis@wdc.com>,
"Bin Meng" <bin.meng@windriver.com>,
"Richard Henderson" <richard.henderson@linaro.org>,
"David Hildenbrand" <david@redhat.com>
Subject: Re: [PATCH v5 01/18] dump: Replace opaque DumpState pointer with a typed one
Date: Tue, 16 Aug 2022 11:58:48 +0400 [thread overview]
Message-ID: <CAMxuvaxgjW-mmHVTJ=4atQpQc2s5KyOUjnTVCyUSb+FYsciqAg@mail.gmail.com> (raw)
In-Reply-To: <20220811121111.9878-2-frankja@linux.ibm.com>
On Thu, Aug 11, 2022 at 4:13 PM Janosch Frank <frankja@linux.ibm.com> wrote:
>
> 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>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.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>
> ---
> include/hw/core/sysemu-cpu-ops.h | 8 ++++----
> include/qemu/typedefs.h | 1 +
> target/arm/arch_dump.c | 6 ++----
> target/arm/cpu.h | 4 ++--
> target/i386/arch_dump.c | 30 +++++++++++++++---------------
> target/i386/cpu.h | 8 ++++----
> target/ppc/arch_dump.c | 18 +++++++++---------
> target/ppc/cpu.h | 4 ++--
> target/riscv/arch_dump.c | 6 ++----
> target/riscv/cpu.h | 4 ++--
> target/s390x/arch_dump.c | 10 +++++-----
> target/s390x/s390x-internal.h | 2 +-
> 12 files changed, 49 insertions(+), 52 deletions(-)
>
> diff --git a/include/hw/core/sysemu-cpu-ops.h b/include/hw/core/sysemu-cpu-ops.h
> index a9ba39e5f2..ee169b872c 100644
> --- a/include/hw/core/sysemu-cpu-ops.h
> +++ b/include/hw/core/sysemu-cpu-ops.h
> @@ -53,25 +53,25 @@ typedef struct SysemuCPUOps {
> * 32-bit VM coredump.
> */
> 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
> * 64-bit VM coredump.
> */
> 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
> * note to a 32-bit VM coredump.
> */
> int (*write_elf32_qemunote)(WriteCoreDumpFunction f, CPUState *cpu,
> - void *opaque);
> + DumpState *s);
> /**
> * @write_elf64_qemunote: Callback for writing a CPU- and QEMU-specific ELF
> * note to a 64-bit VM coredump.
> */
> 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
> * runtime configurable endianness is currently big-endian.
> diff --git a/include/qemu/typedefs.h b/include/qemu/typedefs.h
> index 42f4ceb701..054fd46fa6 100644
> --- a/include/qemu/typedefs.h
> +++ b/include/qemu/typedefs.h
> @@ -129,6 +129,7 @@ typedef struct VirtIODevice VirtIODevice;
> typedef struct Visitor Visitor;
> typedef struct VMChangeStateEntry VMChangeStateEntry;
> typedef struct VMStateDescription VMStateDescription;
> +typedef struct DumpState DumpState;
>
> /*
> * Pointer types
> diff --git a/target/arm/arch_dump.c b/target/arm/arch_dump.c
> index b1f040e69f..2d8e41ab8a 100644
> --- a/target/arm/arch_dump.c
> +++ b/target/arm/arch_dump.c
> @@ -232,12 +232,11 @@ static int aarch64_write_elf64_sve(WriteCoreDumpFunction f,
> #endif
>
> int arm_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
> - int cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> struct aarch64_note note;
> ARMCPU *cpu = ARM_CPU(cs);
> CPUARMState *env = &cpu->env;
> - DumpState *s = opaque;
> uint64_t pstate, sp;
> 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 cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> struct arm_note note;
> ARMCPU *cpu = ARM_CPU(cs);
> CPUARMState *env = &cpu->env;
> - DumpState *s = opaque;
> int ret, i;
> bool fpvalid = cpu_isar_feature(aa32_vfp_simd, cpu);
>
> diff --git a/target/arm/cpu.h b/target/arm/cpu.h
> index 5168e3d837..fc8b358779 100644
> --- a/target/arm/cpu.h
> +++ b/target/arm/cpu.h
> @@ -1100,9 +1100,9 @@ int arm_gen_dynamic_svereg_xml(CPUState *cpu, int base_reg);
> const char *arm_gdb_get_dynamic_xml(CPUState *cpu, const char *xmlname);
>
> 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 cpuid, void *opaque);
> + int cpuid, DumpState *s);
>
> #ifdef TARGET_AARCH64
> int aarch64_cpu_gdb_read_register(CPUState *cpu, GByteArray *buf, int reg);
> diff --git a/target/i386/arch_dump.c b/target/i386/arch_dump.c
> index 004141fc04..c290910a04 100644
> --- a/target/i386/arch_dump.c
> +++ b/target/i386/arch_dump.c
> @@ -42,7 +42,7 @@ typedef struct {
>
> static int x86_64_write_elf64_note(WriteCoreDumpFunction f,
> CPUX86State *env, int id,
> - void *opaque)
> + DumpState *s)
> {
> x86_64_user_regs_struct regs;
> 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);
> memcpy(buf, ®s, sizeof(x86_64_user_regs_struct));
>
> - ret = f(note, note_size, opaque);
> + ret = f(note, note_size, s);
> g_free(note);
> if (ret < 0) {
> 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,
> - int id, void *opaque)
> + int id, DumpState *s)
> {
> x86_elf_prstatus prstatus;
> Elf64_Nhdr *note;
> @@ -170,7 +170,7 @@ static int x86_write_elf64_note(WriteCoreDumpFunction f, CPUX86State *env,
> buf += ROUND_UP(name_size, 4);
> memcpy(buf, &prstatus, sizeof(prstatus));
>
> - ret = f(note, note_size, opaque);
> + ret = f(note, note_size, s);
> g_free(note);
> if (ret < 0) {
> 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 cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> X86CPU *cpu = X86_CPU(cs);
> 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);
>
> 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 {
> #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
> }
> #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 cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> X86CPU *cpu = X86_CPU(cs);
> x86_elf_prstatus prstatus;
> @@ -224,7 +224,7 @@ int x86_cpu_write_elf32_note(WriteCoreDumpFunction f, CPUState *cs,
> buf += ROUND_UP(name_size, 4);
> memcpy(buf, &prstatus, sizeof(prstatus));
>
> - ret = f(note, note_size, opaque);
> + ret = f(note, note_size, s);
> g_free(note);
> if (ret < 0) {
> return -1;
> @@ -329,7 +329,7 @@ static void qemu_get_cpustate(QEMUCPUState *s, CPUX86State *env)
>
> static inline int cpu_write_qemu_note(WriteCoreDumpFunction f,
> CPUX86State *env,
> - void *opaque,
> + DumpState *s,
> int type)
> {
> QEMUCPUState state;
> @@ -369,7 +369,7 @@ static inline int cpu_write_qemu_note(WriteCoreDumpFunction f,
> buf += ROUND_UP(name_size, 4);
> memcpy(buf, &state, sizeof(state));
>
> - ret = f(note, note_size, opaque);
> + ret = f(note, note_size, s);
> g_free(note);
> if (ret < 0) {
> 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,
> - void *opaque)
> + DumpState *s)
> {
> 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,
> - void *opaque)
> + DumpState *s)
> {
> 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,
> diff --git a/target/i386/cpu.h b/target/i386/cpu.h
> index 82004b65b9..b75108d6a3 100644
> --- a/target/i386/cpu.h
> +++ b/target/i386/cpu.h
> @@ -1938,13 +1938,13 @@ extern const VMStateDescription vmstate_x86_cpu;
> int x86_cpu_pending_interrupt(CPUState *cs, int interrupt_request);
>
> 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 cpuid, void *opaque);
> + int cpuid, DumpState *s);
> int x86_cpu_write_elf64_qemunote(WriteCoreDumpFunction f, CPUState *cpu,
> - void *opaque);
> + DumpState *s);
> int x86_cpu_write_elf32_qemunote(WriteCoreDumpFunction f, CPUState *cpu,
> - void *opaque);
> + DumpState *s);
>
> void x86_cpu_get_memory_mapping(CPUState *cpu, MemoryMappingList *list,
> Error **errp);
> diff --git a/target/ppc/arch_dump.c b/target/ppc/arch_dump.c
> index 1139cead9f..f58e6359d5 100644
> --- a/target/ppc/arch_dump.c
> +++ b/target/ppc/arch_dump.c
> @@ -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,
> WriteCoreDumpFunction f,
> PowerPCCPU *cpu, int id,
> - void *opaque)
> + DumpState *s)
> {
> - NoteFuncArg arg = { .state = opaque };
> + NoteFuncArg arg = { .state = s };
> int ret = -1;
> int note_size;
> const NoteFuncDesc *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_descsz = cpu_to_dump32(opaque, nf->contents_size);
> + arg.note.hdr.n_namesz = cpu_to_dump32(s, sizeof(arg.note.name));
> + arg.note.hdr.n_descsz = cpu_to_dump32(s, nf->contents_size);
> strncpy(arg.note.name, note_name, sizeof(arg.note.name));
>
> (*nf->note_contents_func)(&arg, cpu);
>
> note_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) {
> 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 cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> 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 cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> 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);
> }
> diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h
> index a4c893cfad..f7d28fe093 100644
> --- a/target/ppc/cpu.h
> +++ b/target/ppc/cpu.h
> @@ -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);
> #endif
> 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 cpuid, void *opaque);
> + int cpuid, DumpState *s);
> #ifndef CONFIG_USER_ONLY
> void ppc_cpu_do_interrupt(CPUState *cpu);
> bool ppc_cpu_exec_interrupt(CPUState *cpu, int int_req);
> diff --git a/target/riscv/arch_dump.c b/target/riscv/arch_dump.c
> index 709f621d82..736a232956 100644
> --- a/target/riscv/arch_dump.c
> +++ b/target/riscv/arch_dump.c
> @@ -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 cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> struct riscv64_note note;
> RISCVCPU *cpu = RISCV_CPU(cs);
> CPURISCVState *env = &cpu->env;
> - DumpState *s = opaque;
> int ret, i = 0;
> 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 cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> struct riscv32_note note;
> RISCVCPU *cpu = RISCV_CPU(cs);
> CPURISCVState *env = &cpu->env;
> - DumpState *s = opaque;
> int ret, i;
> const char name[] = "CORE";
>
> diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
> index 5c7acc055a..9699504721 100644
> --- a/target/riscv/cpu.h
> +++ b/target/riscv/cpu.h
> @@ -505,9 +505,9 @@ extern const char * const riscv_fpr_regnames[];
> const char *riscv_cpu_get_trap_name(target_ulong cause, bool async);
> void riscv_cpu_do_interrupt(CPUState *cpu);
> 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 cpuid, void *opaque);
> + int cpuid, DumpState *s);
> 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_hviprio_index2irq(int index, int *out_irq, int *out_rdzero);
> diff --git a/target/s390x/arch_dump.c b/target/s390x/arch_dump.c
> index 08daf93ae1..f60a14920d 100644
> --- a/target/s390x/arch_dump.c
> +++ b/target/s390x/arch_dump.c
> @@ -204,7 +204,7 @@ static const NoteFuncDesc note_linux[] = {
> static int s390x_write_elf64_notes(const char *note_name,
> WriteCoreDumpFunction f,
> S390CPU *cpu, int id,
> - void *opaque,
> + DumpState *s,
> const NoteFuncDesc *funcs)
> {
> Note note;
> @@ -222,7 +222,7 @@ static int s390x_write_elf64_notes(const char *note_name,
> (*nf->note_contents_func)(¬e, cpu, id);
>
> note_size = sizeof(note) - sizeof(note.contents) + nf->contents_size;
> - ret = f(¬e, note_size, opaque);
> + ret = f(¬e, note_size, s);
>
> if (ret < 0) {
> 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 cpuid, void *opaque)
> + int cpuid, DumpState *s)
> {
> S390CPU *cpu = S390_CPU(cs);
> 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) {
> 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,
> diff --git a/target/s390x/s390x-internal.h b/target/s390x/s390x-internal.h
> index 6aba7fd0ca..b5ae0ae364 100644
> --- a/target/s390x/s390x-internal.h
> +++ b/target/s390x/s390x-internal.h
> @@ -227,7 +227,7 @@ static inline hwaddr decode_basedisp_s(CPUS390XState *env, uint32_t ipb,
>
> /* arch_dump.c */
> int s390_cpu_write_elf64_note(WriteCoreDumpFunction f, CPUState *cs,
> - int cpuid, void *opaque);
> + int cpuid, DumpState *s);
>
>
> /* cc_helper.c */
> --
> 2.34.1
>
next prev parent reply other threads:[~2022-08-16 8:03 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-11 12:10 [PATCH v5 00/18] dump: Add arch section and s390x PV dump Janosch Frank
2022-08-11 12:10 ` [PATCH v5 01/18] dump: Replace opaque DumpState pointer with a typed one Janosch Frank
2022-08-11 16:51 ` Daniel Henrique Barboza
2022-08-16 7:58 ` Marc-André Lureau [this message]
2022-08-11 12:10 ` [PATCH v5 02/18] dump: Rename write_elf_loads to write_elf_phdr_loads Janosch Frank
2022-08-11 12:10 ` [PATCH v5 03/18] dump: Refactor dump_iterate and introduce dump_filter_memblock_*() Janosch Frank
2022-08-16 8:12 ` Marc-André Lureau
2022-08-11 12:10 ` [PATCH v5 04/18] dump: Rework get_start_block Janosch Frank
2022-08-29 20:17 ` Janis Schoetterl-Glausch
2022-09-26 14:48 ` Janosch Frank
2022-08-11 12:10 ` [PATCH v5 05/18] dump: Rework filter area variables Janosch Frank
2022-08-16 8:19 ` Marc-André Lureau
2022-08-11 12:10 ` [PATCH v5 06/18] dump: Rework dump_calculate_size function Janosch Frank
2022-09-01 9:24 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 07/18] dump: Split elf header functions into prepare and write Janosch Frank
2022-08-16 8:26 ` Marc-André Lureau
2022-09-01 9:24 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 08/18] dump: Rename write_elf*_phdr_note to prepare_elf*_phdr_note Janosch Frank
2022-08-16 8:28 ` Marc-André Lureau
2022-09-01 9:24 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 09/18] dump: Use a buffer for ELF section data and headers Janosch Frank
2022-08-16 8:43 ` Marc-André Lureau
2022-08-29 20:43 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 10/18] dump: Reorder struct DumpState Janosch Frank
2022-08-11 12:11 ` [PATCH v5 11/18] dump: Swap segment and section header locations Janosch Frank
2022-08-11 12:11 ` [PATCH v5 12/18] dump/dump: Add section string table support Janosch Frank
2022-08-30 11:35 ` Steffen Eiden
2022-08-30 14:02 ` Janosch Frank
2022-09-01 9:25 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 13/18] dump/dump: Add arch section support Janosch Frank
2022-09-01 9:26 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 14/18] DRAFT: linux header sync Janosch Frank
2022-08-11 12:11 ` [PATCH v5 15/18] s390x: Add protected dump cap Janosch Frank
2022-08-29 11:29 ` Thomas Huth
2022-09-01 9:26 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 16/18] s390x: Introduce PV query interface Janosch Frank
2022-08-29 11:30 ` Thomas Huth
2022-09-01 9:26 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 17/18] s390x: Add KVM PV dump interface Janosch Frank
2022-08-23 15:25 ` Steffen Eiden
2022-09-01 9:26 ` Janis Schoetterl-Glausch
2022-08-11 12:11 ` [PATCH v5 18/18] s390x: pv: Add dump support Janosch Frank
2022-08-11 13:03 ` Janosch Frank
2022-08-23 15:26 ` Steffen Eiden
2022-08-29 11:57 ` Thomas Huth
2022-09-01 9:31 ` Janis Schoetterl-Glausch
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAMxuvaxgjW-mmHVTJ=4atQpQc2s5KyOUjnTVCyUSb+FYsciqAg@mail.gmail.com' \
--to=marcandre.lureau@redhat.com \
--cc=alistair.francis@wdc.com \
--cc=bin.meng@windriver.com \
--cc=borntraeger@linux.ibm.com \
--cc=clg@kaod.org \
--cc=cohuck@redhat.com \
--cc=danielhb413@gmail.com \
--cc=david@gibson.dropbear.id.au \
--cc=david@redhat.com \
--cc=frankja@linux.ibm.com \
--cc=groug@kaod.org \
--cc=imbrenda@linux.ibm.com \
--cc=mhartmay@linux.ibm.com \
--cc=palmer@dabbelt.com \
--cc=pasic@linux.ibm.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=scgl@linux.ibm.com \
--cc=seiden@linux.ibm.com \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.