On 5/25/20 1:02 PM, Gerd Hoffmann wrote:
> On Wed, May 20, 2020 at 03:25:53PM +0200, Philippe Mathieu-Daudé wrote:
>> Hi Gerd,
>>
>> On 5/20/20 3:19 PM, Gerd Hoffmann wrote:
>>> qboot isn't a bios and shouldnt be named that way.
>>
>> Still it resides in a directory named pc-bios =)
>
> Yep, that should be named "firmware" but changing that is beyond the
> scope of this patch ;)
>
>> It is used later in x86_bios_rom_init() as:
>>
>> bios = g_malloc(sizeof(*bios));
>> memory_region_init_ram(bios, NULL, "pc.bios", bios_size, &error_fatal);
>> if (!isapc_ram_fw) {
>> memory_region_set_readonly(bios, true);
>> }
>> ret = rom_add_file_fixed(bios_name, (uint32_t)(-bios_size), -1);
>> if (ret != 0) {
>> bios_error:
>> fprintf(stderr, "qemu: could not load PC BIOS '%s'\n", bios_name);
>> exit(1);
>> }
>> g_free(filename);
>>
>> Many 'bios' occurrences.
>>
>> Should it be called 'qboot.rom' instead?
>
> I don't care much, I only want bios*.bin for seabios which actually is a
> complete bios implementation ...
OK.
Preferably named 'qboot.rom':
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
