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 ... take care, Gerd