[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 3/9] dump: Move struct definition into dump_m
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v4 3/9] dump: Move struct definition into dump_memroy.h |
Date: |
Wed, 19 Jun 2013 15:08:15 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6 |
"dump_memory.h"
Am 28.05.2013 04:50, schrieb address@hidden:
> From: Qiao Nuohan <address@hidden>
>
> Move definition of struct DumpState into include/sysemu/dump_memory.h.
>
> Signed-off-by: Qiao Nuohan <address@hidden>
> Reviewed-by: Zhang Xiaohe <address@hidden>
> ---
> dump.c | 22 +---------------------
> include/sysemu/dump_memory.h | 40 ++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 41 insertions(+), 21 deletions(-)
> create mode 100644 include/sysemu/dump_memory.h
>
> diff --git a/dump.c b/dump.c
> index c0d3da5..9ac66be 100644
> --- a/dump.c
> +++ b/dump.c
> @@ -15,10 +15,9 @@
> #include "elf.h"
> #include "cpu.h"
> #include "exec/cpu-all.h"
> -#include "exec/hwaddr.h"
> #include "monitor/monitor.h"
> #include "sysemu/kvm.h"
> -#include "sysemu/dump.h"
> +#include "sysemu/dump_memory.h"
> #include "sysemu/sysemu.h"
> #include "sysemu/memory_mapping.h"
> #include "qapi/error.h"
> @@ -57,25 +56,6 @@ static uint64_t cpu_convert_to_target64(uint64_t val, int
> endian)
> return val;
> }
>
> -typedef struct DumpState {
> - ArchDumpInfo dump_info;
> - MemoryMappingList list;
> - uint16_t phdr_num;
> - uint32_t sh_info;
> - bool have_section;
> - bool resume;
> - size_t note_size;
> - hwaddr memory_offset;
> - int fd;
> -
> - RAMBlock *block;
> - ram_addr_t start;
> - bool has_filter;
> - int64_t begin;
> - int64_t length;
> - Error **errp;
> -} DumpState;
> -
> static int dump_cleanup(DumpState *s)
> {
> int ret = 0;
> diff --git a/include/sysemu/dump_memory.h b/include/sysemu/dump_memory.h
> new file mode 100644
> index 0000000..ce22c05
> --- /dev/null
> +++ b/include/sysemu/dump_memory.h
> @@ -0,0 +1,40 @@
> +/*
> + * QEMU dump memory
> + *
> + * Copyright (C) 2013 FUJITSU LIMITED
> + *
> + * Authors:
> + * Qiao Nuohan <address@hidden>
I wondered whether you may want to keep Wen Congyang here as original
dump.c author and the corresponding dump.c copyright - but since it's
all Fujitsu that's none of my business. ;)
Regards,
Andreas
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or later.
> + * See the COPYING file in the top-level directory.
> + *
> + */
> +
> +#ifndef DUMP_MEMORY_H
> +#define DUMP_MEMORY_H
> +
> +#include "exec/cpu-all.h"
> +#include "sysemu/memory_mapping.h"
> +#include "sysemu/dump.h"
> +
> +typedef struct DumpState {
> + ArchDumpInfo dump_info;
> + MemoryMappingList list;
> + uint16_t phdr_num;
> + uint32_t sh_info;
> + bool have_section;
> + bool resume;
> + size_t note_size;
> + hwaddr memory_offset;
> + int fd;
> +
> + RAMBlock *block;
> + ram_addr_t start;
> + bool has_filter;
> + int64_t begin;
> + int64_t length;
> + Error **errp;
> +} DumpState;
> +
> +#endif
>
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v4 3/9] dump: Move struct definition into dump_memroy.h,
Andreas Färber <=