[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/10] S390: check if BIOS is available and crea
From: |
Dominik Dingel |
Subject: |
Re: [Qemu-devel] [PATCH 04/10] S390: check if BIOS is available and create links |
Date: |
Fri, 26 Apr 2013 20:03:34 +0200 |
On Fri, 26 Apr 2013 11:38:00 -0500
Anthony Liguori <address@hidden> wrote:
> Alexander Graf <address@hidden> writes:
>
> > On 26.04.2013, at 17:48, Dominik Dingel wrote:
> >
> >> On Fri, 26 Apr 2013 17:23:18 +0200
> >> Alexander Graf <address@hidden> wrote:
> >>
> >>>
> >>> On 26.04.2013, at 14:12, Dominik Dingel wrote:
> >>>
> >>>> Signed-off-by: Dominik Dingel <address@hidden>
> >>>>
> >>>> Check if the BIOS is available before loading it into the memory.
> >>>> Create the needed Links for build.
> >>>>
> >>>> Add a if the BIOS is available, also add the needed links for the build
> >>>> process.
> >>>>
> >>>> diff --git a/configure b/configure
> >>>> index 19777de..2bbbd54 100755
> >>>> --- a/configure
> >>>> +++ b/configure
> >>>> @@ -4541,6 +4541,7 @@ for bios_file in \
> >>>> $source_path/pc-bios/*.aml \
> >>>> $source_path/pc-bios/*.rom \
> >>>> $source_path/pc-bios/*.dtb \
> >>>> + $source_path/pc-bios/*.img \
> >>>> $source_path/pc-bios/openbios-* \
> >>>> $source_path/pc-bios/palcode-*
> >>>> do
> >>>> diff --git a/hw/s390x/ipl.c b/hw/s390x/ipl.c
> >>>> index 9758529..36daa67 100644
> >>>> --- a/hw/s390x/ipl.c
> >>>> +++ b/hw/s390x/ipl.c
> >>>> @@ -95,6 +95,10 @@ static int s390_ipl_init(SysBusDevice *dev)
> >>>> }
> >>>>
> >>>> bios_filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
> >>>> + if (bios_filename == NULL) {
> >>>> + hw_error("could not find stage1 bootloader\n");
> >>>> + }
> >>>> +
> >>>> bios_size = load_elf(bios_filename, NULL, NULL, &ipl->start_addr,
> >>>> NULL,
> >>>> NULL, 1, ELF_MACHINE, 0);
> >>>> if (bios_size == -1UL) {
> >>>> diff --git a/pc-bios/s390-ccw/Makefile b/pc-bios/s390-ccw/Makefile
> >>>> index ad55a14..1e6ab1e 100644
> >>>> --- a/pc-bios/s390-ccw/Makefile
> >>>> +++ b/pc-bios/s390-ccw/Makefile
> >>>> @@ -21,6 +21,7 @@ s390-ccw.elf: $(OBJECTS)
> >>>>
> >>>> s390-ccw.img: s390-ccw.elf
> >>>> $(call quiet-command,strip $< -o $@," Stripping $(TARGET_DIR)$@")
> >>>> + ln -s -f `pwd`/$@ ../$@
> >>>
> >>> I don't think we do this for any other blobs, so why should we here?
> >>>
> >>> Alex
> >>
> >> In case we are not running on the s390 platform we need the link to the
> >> shipped binary.
> >> But if we are running on the s390 platform we like to build the rom in the
> >> s390-ccw build folder. After that we either copy or link it to the pc-bios
> >> location.
> >> I personally preferred the link version.
> >
> > It's not what users expect when they compile code inside of pc-bios. Please
> > stick to whatever behavior other blobs that we compile on demand have.
>
> Ack.
>
> Regards,
>
> Anthony Liguori
Would be a link for all blobs okay? Otherwise it is a magic box full with bugs
and frustration for developers, as they need to copy explicitly the freshly
build blob over to the source folder.
Dominik
> >
> >
> > Alex
>
- Re: [Qemu-devel] [PATCH 09/10] S390: Pass per-device loadparm values for CCW blk and net devs., (continued)
[Qemu-devel] [PATCH 08/10] s390-ccw.img: Get queue config from host., Dominik Dingel, 2013/04/26
[Qemu-devel] [PATCH 04/10] S390: check if BIOS is available and create links, Dominik Dingel, 2013/04/26
[Qemu-devel] [PATCH 07/10] s390-ccw.img: Rudimentary error checking., Dominik Dingel, 2013/04/26
[Qemu-devel] [PATCH 05/10] s390-ccw.img: Detect devices with stsch., Dominik Dingel, 2013/04/26
[Qemu-devel] [PATCH 10/10] S390: Enabling device and program selection, Dominik Dingel, 2013/04/26
[Qemu-devel] [PATCH 01/10] Common: Add a default bootindex for all applicable devices, Dominik Dingel, 2013/04/26