[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 08/29] dump: change cpu_get_note_size to retu
From: |
Wen Congyang |
Subject: |
Re: [Qemu-devel] [PATCH v3 08/29] dump: change cpu_get_note_size to return ssize_t |
Date: |
Mon, 11 Jun 2012 09:52:41 +0800 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100413 Fedora/3.0.4-2.fc13 Thunderbird/3.0.4 |
Hi, Paolo
Thank you for doing this.
At 06/07/2012 03:39 PM, Paolo Bonzini Wrote:
> So that it can use the same prototype in both cases. Check for failure.
>
> Cc: Wen Congyang <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> cpu-all.h | 4 ++--
> dump.c | 9 +++++++--
> target-i386/arch_dump.c | 2 +-
> 3 files changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/cpu-all.h b/cpu-all.h
> index e8749de..624030d 100644
> --- a/cpu-all.h
> +++ b/cpu-all.h
> @@ -552,7 +552,7 @@ int cpu_write_elf64_qemunote(write_core_dump_function f,
> CPUArchState *env,
> int cpu_write_elf32_qemunote(write_core_dump_function f, CPUArchState *env,
> void *opaque);
> int cpu_get_dump_info(ArchDumpInfo *info);
> -size_t cpu_get_note_size(int class, int machine, int nr_cpus);
> +ssize_t cpu_get_note_size(int class, int machine, int nr_cpus);
> #else
> static inline int cpu_write_elf64_note(write_core_dump_function f,
> CPUArchState *env, int cpuid,
> @@ -587,7 +587,7 @@ static inline int cpu_get_dump_info(ArchDumpInfo *info)
> return -1;
> }
>
> -static inline int cpu_get_note_size(int class, int machine, int nr_cpus)
> +static inline ssize_t cpu_get_note_size(int class, int machine, int nr_cpus)
> {
> return -1;
> }
> diff --git a/dump.c b/dump.c
> index 0ca14f8..b24d4be 100644
> --- a/dump.c
> +++ b/dump.c
> @@ -750,6 +750,13 @@ static int dump_init(DumpState *s, int fd, bool paging,
> bool has_filter,
> goto cleanup;
> }
>
> + s->note_size = cpu_get_note_size(s->dump_info.d_class,
> + s->dump_info.d_machine, nr_cpus);
> + if (ret < 0) {
Hmm, I guess you want to check cpu_get_note_size()'s return value here.
Thanks
Wen Congyang
> + error_set(errp, QERR_UNSUPPORTED);
> + goto cleanup;
> + }
> +
> /* get memory mapping */
> memory_mapping_list_init(&s->list);
> if (paging) {
> @@ -784,8 +791,6 @@ static int dump_init(DumpState *s, int fd, bool paging,
> bool has_filter,
> }
> }
>
> - s->note_size = cpu_get_note_size(s->dump_info.d_class,
> - s->dump_info.d_machine, nr_cpus);
> if (s->dump_info.d_class == ELFCLASS64) {
> if (s->have_section) {
> s->memory_offset = sizeof(Elf64_Ehdr) +
> diff --git a/target-i386/arch_dump.c b/target-i386/arch_dump.c
> index 135d855..7c2b514 100644
> --- a/target-i386/arch_dump.c
> +++ b/target-i386/arch_dump.c
> @@ -415,7 +415,7 @@ int cpu_get_dump_info(ArchDumpInfo *info)
> return 0;
> }
>
> -size_t cpu_get_note_size(int class, int machine, int nr_cpus)
> +ssize_t cpu_get_note_size(int class, int machine, int nr_cpus)
> {
> int name_size = 5; /* "CORE" or "QEMU" */
> size_t elf_note_size = 0;
- [Qemu-devel] [PATCH v3 26/29] build: limit usage of vpath, (continued)
- [Qemu-devel] [PATCH v3 26/29] build: limit usage of vpath, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 28/29] configure: ensure directory exists when creating symlink, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 15/29] build: move fsdev/ objects to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 07/29] dump: do not compile dump.o for user-mode emulation, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 29/29] build: do not create directories at configure time, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 09/29] dump: remove dumping stuff from cpu-all.h, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 23/29] build: move per-target hw/ objects to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 20/29] build: move qga/ objects to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 27/29] build: compile oslib-obj-y once, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 08/29] dump: change cpu_get_note_size to return ssize_t, Paolo Bonzini, 2012/06/07
- Re: [Qemu-devel] [PATCH v3 08/29] dump: change cpu_get_note_size to return ssize_t,
Wen Congyang <=
- [Qemu-devel] [PATCH v3 16/29] build: move ui/ objects to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 24/29] build: move device tree to per-target Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 19/29] build: move qapi/ objects to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 18/29] build: move slirp/ objects to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 22/29] build: convert libhw to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 17/29] build: move audio/ objects to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- [Qemu-devel] [PATCH v3 05/29] build: move obj-TARGET-y variables to nested Makefile.objs, Paolo Bonzini, 2012/06/07
- Re: [Qemu-devel] [PULL v3 00/29] per-directory Makefile.objs snippets, limit vpath (ab)use, Andreas Färber, 2012/06/07