[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 12/15] s390: remove bios_name
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH 12/15] s390: remove bios_name |
Date: |
Tue, 27 Oct 2020 14:26:01 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1 |
On 27/10/20 09:38, Christian Borntraeger wrote:
>>
>> diff --git a/hw/s390x/s390-virtio-ccw.c b/hw/s390x/s390-virtio-ccw.c
>> index e52182f946..a521eba673 100644
>> --- a/hw/s390x/s390-virtio-ccw.c
>> +++ b/hw/s390x/s390-virtio-ccw.c
>> @@ -258,7 +258,8 @@ static void ccw_init(MachineState *machine)
>> /* get a BUS */
>> css_bus = virtual_css_bus_init();
>> s390_init_ipl_dev(machine->kernel_filename, machine->kernel_cmdline,
>> - machine->initrd_filename, "s390-ccw.img",
>> + machine->initrd_filename,
>> + machine->firmware ?: "s390-ccw.img",
> Adding the elvis operator is actually a fix, no?
>
I think it was already doing the equivalent here in s390_ipl_realize
if (bios_name == NULL) {
bios_name = ipl->firmware;
}
bios_filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
so it was just an encapsulation violation on part of the IPL device.
Paolo
- Re: [PATCH 02/15] digic: stash firmware into DigicState, (continued)
- [PATCH 04/15] hppa: remove bios_name, Paolo Bonzini, 2020/10/26
- [PATCH 12/15] s390: remove bios_name, Paolo Bonzini, 2020/10/26
- [PATCH 09/15] moxie: remove bios_name, Paolo Bonzini, 2020/10/26
- [PATCH 03/15] arm: remove bios_name, Paolo Bonzini, 2020/10/26
- [PATCH 07/15] m68k: remove bios_name, Paolo Bonzini, 2020/10/26