[Qemu-devel] some ROMs questions

[Qemu-devel] some ROMs questions

[Michael Tokarev]

In the recent 2.12-rc1 tarball, there are the following things present:

roms/u-boot-sam460ex/tools/updater/stubs.c -
 it is some strange symlink pointing to a strange place, probably should be removed?

When making u-boot-am460ex image, in roms/Makefile, we do a plain copy, while 5
lines above, u-boot image is being stripped on copy. Should we perform strip for
u-boot-am460ex image too, if not only to be consistent?

BTW, can't u-boot-am460ex image be made from regular u-boot sources, maybe with an
additional board added? :)

Also, skiboot image can't be built from roms/skiboot in the source tarball, it fails
because there's no .version file present in roms/skiboot/ directory, but this file
is present in the git repository (submodule). Perhaps this .version file is wrongly
removed when making release tarball?

Maybe these trivial issues can be fixed for 2.12 release?

Thanks,

/mjt

Re: [Qemu-devel] some ROMs questions

[BALATON Zoltan]

On Mon, 2 Apr 2018, Michael Tokarev wrote:
> roms/u-boot-sam460ex/tools/updater/stubs.c -
> it is some strange symlink pointing to a strange place, probably should be removed?

This does not seem to matter for building the rom image but this can be 
fixed by converting it to a relative path. I've done that and will send a 
patch to update the submodule as well to use the QEMU repo now that we 
have a mirror there. Thanks for finding this.

> When making u-boot-am460ex image, in roms/Makefile, we do a plain copy, while 5
> lines above, u-boot image is being stripped on copy. Should we perform strip for
> u-boot-am460ex image too, if not only to be consistent?

No, this is correct. These use different images. For e500 the u-boot ELF 
image is used after stripping it while for sam460ex the u-boot.bin is 
needed which is another build target and does not contain symbols to be 
stripped so a plain copy is enough.

> BTW, can't u-boot-am460ex image be made from regular u-boot sources, maybe with an
> additional board added? :)

No. This was discussed before several times. Look up those threads for 
more info or if you don't beleive it just compare the two u-boot versions. 
I don't intend to take up the task of supporting this board in latest 
u-boot versions which even removed support for the CPU type it needs. Also 
we aim to be as compatible to real hardware as possible because of 
assumptions of closed source guests.

Regards,
BALATON Zoltan

Re: [Qemu-devel] some ROMs questions

[Michael Tokarev]

02.04.2018 17:30, BALATON Zoltan wrote:
> On Mon, 2 Apr 2018, Michael Tokarev wrote:
>> roms/u-boot-sam460ex/tools/updater/stubs.c -
>> it is some strange symlink pointing to a strange place, probably should be removed?
> 
> This does not seem to matter for building the rom image but this can be fixed by converting it to a relative path. I've done that and will send a patch to update the submodule as well to use the QEMU repo now that we have a mirror there. Thanks for finding this.

Hmm, I've no idea how to pull a submodule change.. :)

But I've one more question about this rom.  When I'm trying to build it,
the build fails:

roms/u-boot-sam460ex$ make Sam460ex_config CROSS_COMPILE=powerpc64-linux-gnu-
Generating include/autoconf.mk
cc1: error: -mcall not supported in this configuration
cc1: error: -mrelocatable not supported in this configuration
cc1: error: -meabi not supported in this configuration
cc1: error: -m64 requires a PowerPC64 cpu
Generating include/autoconf.mk.dep
cc1: error: -mcall not supported in this configuration
cc1: error: -mrelocatable not supported in this configuration
cc1: error: -meabi not supported in this configuration
cc1: error: -m64 requires a PowerPC64 cpu
Configuring for Sam460ex board...

roms/u-boot-sam460ex$ make CROSS_COMPILE=powerpc64-linux-gnu-
...
make -C arch/powerpc/cpu/ppc4xx start.o
make[1]: Entering directory '/build/qemu/debian-qemu/roms/u-boot-sam460ex/arch/powerpc/cpu/ppc4xx'
powerpc64-linux-gnu-gcc   -D__ASSEMBLY__ -g  -Os   -mrelocatable -fPIC -meabi -ffunction-sections -fdata-sections -D__KERNEL__ -DTEXT_BASE=0xFFF80000 -I/build/qemu/debian-qemu/roms/u-boot-sam460ex/include -fno-builtin -ffreestanding -nostdinc -isystem /usr/lib/gcc-cross/powerpc64-linux-gnu/6/include -pipe  -DCONFIG_PPC -D__powerpc__ -DCONFIG_4xx -ffixed-r2 -mstring -msoft-float -Wa,-m440 -mcpu=440 -DCONFIG_440=1 -I../bios_emulator/scitech/include -I../bios_emulator/scitech/src/x86emu -Dprintk=printf   \
        -o start.o start.S -c
start.S:1:0: error: -mcall not supported in this configuration
 /*

start.S:1:0: error: -mrelocatable not supported in this configuration
start.S:1:0: error: -meabi not supported in this configuration
start.S:1:0: error: -m64 requires a PowerPC64 cpu
/build/qemu/debian-qemu/roms/u-boot-sam460ex/config.mk:249: recipe for target 'start.o' failed
make[1]: *** [start.o] Error 1
make[1]: Leaving directory '/build/qemu/debian-qemu/roms/u-boot-sam460ex/arch/powerpc/cpu/ppc4xx'
Makefile:353: recipe for target 'arch/powerpc/cpu/ppc4xx/start.o' failed
make: *** [arch/powerpc/cpu/ppc4xx/start.o] Error 2

What I'm doing wrong?

$ powerpc64-linux-gnu-gcc --version
powerpc64-linux-gnu-gcc (Debian 6.3.0-18) 6.3.0 20170516

Thanks!

/mjt

Re: [Qemu-devel] some ROMs questions

[Michael Tokarev]

02.04.2018 17:30, BALATON Zoltan wrote:
> On Mon, 2 Apr 2018, Michael Tokarev wrote:
>> roms/u-boot-sam460ex/tools/updater/stubs.c -
>> it is some strange symlink pointing to a strange place, probably should be removed?
> 
> This does not seem to matter for building the rom image but this can be fixed by converting it to a relative path. I've done that and will send a patch to update the submodule as well to use the QEMU repo now that we have a mirror there. Thanks for finding this.

There are 2 more files which we might want to remove:
 tools/multiplier/multiplier.image
 tools/multiplier/multiplier

FWIW.

Thanks,

/mjt

Re: [Qemu-devel] some ROMs questions

[BALATON Zoltan]

On Fri, 6 Apr 2018, Michael Tokarev wrote:
> 02.04.2018 17:30, BALATON Zoltan wrote:
>> On Mon, 2 Apr 2018, Michael Tokarev wrote:
>>> roms/u-boot-sam460ex/tools/updater/stubs.c -
>>> it is some strange symlink pointing to a strange place, probably should be removed?
>>
>> This does not seem to matter for building the rom image but this can be fixed by converting it to a relative path. I've done that and will send a patch to update the submodule as well to use the QEMU repo now that we have a mirror there. Thanks for finding this.
>
> There are 2 more files which we might want to remove:
> tools/multiplier/multiplier.image
> tools/multiplier/multiplier

These all come from the upstream source from the hardware vendor which 
I've taken as is to follow GPL. Thanks for noticing, I've removed these 
and will send another patch to update the submodule. Are there any more of 
these before I do that?

Regards,
BALATON Zoltan

Re: [Qemu-devel] some ROMs questions

[BALATON Zoltan]

On Fri, 6 Apr 2018, Michael Tokarev wrote:
> 02.04.2018 17:30, BALATON Zoltan wrote:
>> On Mon, 2 Apr 2018, Michael Tokarev wrote:
>>> roms/u-boot-sam460ex/tools/updater/stubs.c -
>>> it is some strange symlink pointing to a strange place, probably should be removed?
>>
>> This does not seem to matter for building the rom image but this can be fixed by converting it to a relative path. I've done that and will send a patch to update the submodule as well to use the QEMU repo now that we have a mirror there. Thanks for finding this.
>
> Hmm, I've no idea how to pull a submodule change.. :)

git submodule update? But it's not in master yet only in 
David's ppc-for-2.12 branch I think.

> But I've one more question about this rom.  When I'm trying to build it,
> the build fails:
>
> roms/u-boot-sam460ex$ make Sam460ex_config CROSS_COMPILE=powerpc64-linux-gnu-
> Generating include/autoconf.mk
> cc1: error: -mcall not supported in this configuration
> cc1: error: -mrelocatable not supported in this configuration
> cc1: error: -meabi not supported in this configuration
> cc1: error: -m64 requires a PowerPC64 cpu
> Generating include/autoconf.mk.dep
> cc1: error: -mcall not supported in this configuration
> cc1: error: -mrelocatable not supported in this configuration
> cc1: error: -meabi not supported in this configuration
> cc1: error: -m64 requires a PowerPC64 cpu
> Configuring for Sam460ex board...
>
> roms/u-boot-sam460ex$ make CROSS_COMPILE=powerpc64-linux-gnu-
> ...
> make -C arch/powerpc/cpu/ppc4xx start.o
> make[1]: Entering directory '/build/qemu/debian-qemu/roms/u-boot-sam460ex/arch/powerpc/cpu/ppc4xx'
> powerpc64-linux-gnu-gcc   -D__ASSEMBLY__ -g  -Os   -mrelocatable -fPIC -meabi -ffunction-sections -fdata-sections -D__KERNEL__ -DTEXT_BASE=0xFFF80000 -I/build/qemu/debian-qemu/roms/u-boot-sam460ex/include -fno-builtin -ffreestanding -nostdinc -isystem /usr/lib/gcc-cross/powerpc64-linux-gnu/6/include -pipe  -DCONFIG_PPC -D__powerpc__ -DCONFIG_4xx -ffixed-r2 -mstring -msoft-float -Wa,-m440 -mcpu=440 -DCONFIG_440=1 -I../bios_emulator/scitech/include -I../bios_emulator/scitech/src/x86emu -Dprintk=printf   \
>        -o start.o start.S -c
> start.S:1:0: error: -mcall not supported in this configuration
> /*
>
> start.S:1:0: error: -mrelocatable not supported in this configuration
> start.S:1:0: error: -meabi not supported in this configuration
> start.S:1:0: error: -m64 requires a PowerPC64 cpu
> /build/qemu/debian-qemu/roms/u-boot-sam460ex/config.mk:249: recipe for target 'start.o' failed
> make[1]: *** [start.o] Error 1
> make[1]: Leaving directory '/build/qemu/debian-qemu/roms/u-boot-sam460ex/arch/powerpc/cpu/ppc4xx'
> Makefile:353: recipe for target 'arch/powerpc/cpu/ppc4xx/start.o' failed
> make: *** [arch/powerpc/cpu/ppc4xx/start.o] Error 2
>
> What I'm doing wrong?
>
> $ powerpc64-linux-gnu-gcc --version
> powerpc64-linux-gnu-gcc (Debian 6.3.0-18) 6.3.0 20170516

Note sure, maybe building it with ppc64 compiler or a too new one? This 
board is 32bit and the source is from 2011 so I'm not sure what it does 
with recent compilers. I have powerpc-elf-gcc 4.9.3 which works.

Regards,
BALATON Zoltan

Re: [Qemu-devel] some ROMs questions

[David Gibson]

[-- Attachment #1: Type: text/plain, Size: 3699 bytes --]

On Fri, Apr 06, 2018 at 01:59:03PM +0200, BALATON Zoltan wrote:
> On Fri, 6 Apr 2018, Michael Tokarev wrote:
> > 02.04.2018 17:30, BALATON Zoltan wrote:
> > > On Mon, 2 Apr 2018, Michael Tokarev wrote:
> > > > roms/u-boot-sam460ex/tools/updater/stubs.c -
> > > > it is some strange symlink pointing to a strange place, probably should be removed?
> > > 
> > > This does not seem to matter for building the rom image but this can be fixed by converting it to a relative path. I've done that and will send a patch to update the submodule as well to use the QEMU repo now that we have a mirror there. Thanks for finding this.
> > 
> > Hmm, I've no idea how to pull a submodule change.. :)
> 
> git submodule update? But it's not in master yet only in David's
> ppc-for-2.12 branch I think.
> 
> > But I've one more question about this rom.  When I'm trying to build it,
> > the build fails:
> > 
> > roms/u-boot-sam460ex$ make Sam460ex_config CROSS_COMPILE=powerpc64-linux-gnu-
> > Generating include/autoconf.mk
> > cc1: error: -mcall not supported in this configuration
> > cc1: error: -mrelocatable not supported in this configuration
> > cc1: error: -meabi not supported in this configuration
> > cc1: error: -m64 requires a PowerPC64 cpu
> > Generating include/autoconf.mk.dep
> > cc1: error: -mcall not supported in this configuration
> > cc1: error: -mrelocatable not supported in this configuration
> > cc1: error: -meabi not supported in this configuration
> > cc1: error: -m64 requires a PowerPC64 cpu
> > Configuring for Sam460ex board...
> > 
> > roms/u-boot-sam460ex$ make CROSS_COMPILE=powerpc64-linux-gnu-
> > ...
> > make -C arch/powerpc/cpu/ppc4xx start.o
> > make[1]: Entering directory '/build/qemu/debian-qemu/roms/u-boot-sam460ex/arch/powerpc/cpu/ppc4xx'
> > powerpc64-linux-gnu-gcc   -D__ASSEMBLY__ -g  -Os   -mrelocatable -fPIC -meabi -ffunction-sections -fdata-sections -D__KERNEL__ -DTEXT_BASE=0xFFF80000 -I/build/qemu/debian-qemu/roms/u-boot-sam460ex/include -fno-builtin -ffreestanding -nostdinc -isystem /usr/lib/gcc-cross/powerpc64-linux-gnu/6/include -pipe  -DCONFIG_PPC -D__powerpc__ -DCONFIG_4xx -ffixed-r2 -mstring -msoft-float -Wa,-m440 -mcpu=440 -DCONFIG_440=1 -I../bios_emulator/scitech/include -I../bios_emulator/scitech/src/x86emu -Dprintk=printf   \
> >        -o start.o start.S -c
> > start.S:1:0: error: -mcall not supported in this configuration
> > /*
> > 
> > start.S:1:0: error: -mrelocatable not supported in this configuration
> > start.S:1:0: error: -meabi not supported in this configuration
> > start.S:1:0: error: -m64 requires a PowerPC64 cpu
> > /build/qemu/debian-qemu/roms/u-boot-sam460ex/config.mk:249: recipe for target 'start.o' failed
> > make[1]: *** [start.o] Error 1
> > make[1]: Leaving directory '/build/qemu/debian-qemu/roms/u-boot-sam460ex/arch/powerpc/cpu/ppc4xx'
> > Makefile:353: recipe for target 'arch/powerpc/cpu/ppc4xx/start.o' failed
> > make: *** [arch/powerpc/cpu/ppc4xx/start.o] Error 2
> > 
> > What I'm doing wrong?
> > 
> > $ powerpc64-linux-gnu-gcc --version
> > powerpc64-linux-gnu-gcc (Debian 6.3.0-18) 6.3.0 20170516
> 
> Note sure, maybe building it with ppc64 compiler or a too new one? This
> board is 32bit and the source is from 2011 so I'm not sure what it does with
> recent compilers. I have powerpc-elf-gcc 4.9.3 which works.

Yeah, if it's a 32-bit board, I'm pretty sure you'll need a 32-bit
compiler.  Or at least add -m32 if it's a biarch compiler.

-- 
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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]