[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH qemu] spapr/rtas: Force big endian compile for r
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH qemu] spapr/rtas: Force big endian compile for rtas |
Date: |
Thu, 14 Feb 2019 14:30:40 +1100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Wed, Feb 13, 2019 at 05:00:50PM +1100, Alexey Kardashevskiy wrote:
>
>
> On 01/02/2019 11:40, Alexey Kardashevskiy wrote:
> > At the moment the rtas's Makefile uses generic QEMU rules which means
> > that when QEMU is compiled on a little endian system, the spapr-rtas.bin
> > is compiled as little endian too which is incorrect as it is always
> > executed in big endian mode.
> >
> > This enforces -mbig by defining %.o:%.S rule as spapr-rtas.bin is
> > a standalone guest binary which should not depend on QEMU flags anyway.
>
> Bag? Good? Useless? :)
I missed the original send of this, I'm afraid. Looks like a good
idea.
>
>
> >
> > Signed-off-by: Alexey Kardashevskiy <address@hidden>
> > ---
> > pc-bios/spapr-rtas/Makefile | 5 ++++-
> > 1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/pc-bios/spapr-rtas/Makefile b/pc-bios/spapr-rtas/Makefile
> > index f26dd42..4b9bb12 100644
> > --- a/pc-bios/spapr-rtas/Makefile
> > +++ b/pc-bios/spapr-rtas/Makefile
> > @@ -14,8 +14,11 @@ $(call set-vpath, $(SRC_PATH)/pc-bios/spapr-rtas)
> >
> > build-all: spapr-rtas.bin
> >
> > +%.o: %.S
> > + $(call quiet-command,$(CCAS) -mbig -c -o $@ $<,"CCAS","$(TARGET_DIR)$@")
> > +
> > %.img: %.o
> > - $(call quiet-command,$(CC) -nostdlib -o $@
> > $<,"Building","$(TARGET_DIR)$@")
> > + $(call quiet-command,$(CC) -nostdlib -mbig -o $@
> > $<,"Building","$(TARGET_DIR)$@")
> >
> > %.bin: %.img
> > $(call quiet-command,$(OBJCOPY) -O binary -j .text $<
> > $@,"Building","$(TARGET_DIR)$@")
> >
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature