[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] dump: Make DumpState and endian conversion
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] dump: Make DumpState and endian conversion routines available for arch-specific dump code |
Date: |
Tue, 29 Apr 2014 11:29:20 +0200 |
On Tue, 29 Apr 2014 11:16:51 +0200
Alexander Graf <address@hidden> wrote:
>
> On 28.04.14 13:29, Greg Kurz wrote:
> > From: Bharata B Rao <address@hidden>
> >
> > Make DumpState and endian conversion routines available for arch-specific
> > dump
> > code by moving into dump.h. DumpState will be needed by arch-specific dump
> > code to access target endian information from DumpState->ArchDumpInfo. Also
> > break the dependency of dump.h from stubs/dump.c by creating a separate
> > dump-arch.h.
> >
> > This patch doesn't change any functionality.
> >
> > Signed-off-by: Bharata B Rao <address@hidden>
> > [ rebased on top of current master branch,
> > Greg Kurz <address@hidden> ]
> > Signed-off-by: Greg Kurz <address@hidden>
> > ---
> > dump.c | 39 +++---------------------------------
> > include/sysemu/dump-arch.h | 28 ++++++++++++++++++++++++++
> > include/sysemu/dump.h | 48
> > +++++++++++++++++++++++++++++++++++---------
> > stubs/dump.c | 2 +-
> > 4 files changed, 70 insertions(+), 47 deletions(-)
> > create mode 100644 include/sysemu/dump-arch.h
> >
> > diff --git a/dump.c b/dump.c
> > index 14b3d1d..13c9bf2 100644
> > --- a/dump.c
> > +++ b/dump.c
> > @@ -36,7 +36,7 @@
> > #define ELF_MACHINE_UNAME "Unknown"
> > #endif
> >
> > -static uint16_t cpu_convert_to_target16(uint16_t val, int endian)
> > +uint16_t cpu_convert_to_target16(uint16_t val, int endian)
>
> This is quite some heavy pollution of the global name space.
>
>
> Alex
>
What about moving the declarations from dump.h to a PPC only header file then ?
--
Gregory Kurz address@hidden
address@hidden
Software Engineer @ IBM/Meiosys http://www.ibm.com
Tel +33 (0)562 165 496
"Anarchy is about taking complete responsibility for yourself."
Alan Moore.
[Qemu-devel] [PATCH 2/4] ppc64-dump: Support dump for little endian ppc64, Greg Kurz, 2014/04/28
[Qemu-devel] [PATCH 3/4] target-ppc: ppc can be either endian, Greg Kurz, 2014/04/28