qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

missing boot rom: is it really a fatal error?


From: Michael Tokarev
Subject: missing boot rom: is it really a fatal error?
Date: Sun, 7 May 2023 20:56:23 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0

Hi!

In old good world ;), there was qemu which didn't require boot roms to be 
present
for all devices for which bootrom file is defined, missing rom was just a 
warning.
But this changed in 2014, 9 years ago, with this commit:

commit 178e785fb4507ec3462dc772bbe08303416ece47
From: Marcel Apfelbaum <marcel.a@redhat.com>
Date: Mon, 27 Oct 2014 19:34:41 +0200
Subject: [PATCH] hw/pci: fixed error flow in pci_qdev_init

  Verify return code for pci_add_option_rom.

where inability to load rom file started being treated as an error.
Up until now I didn't even know about this change, until today when someone 
bugged
me about non-working qemu on debian, due to missing network boot roms (this a
packaging issue due to me being unaware of the above change).

What is the reason to require boot roms to be present and throw an error if not?

I'm about to revert that old change on debian, to make it just a warning instead
of an error (the code is different now, but the same principle applies), - 
because
I dislike dependencies which are useless 99.9% of the time and are trivial to
install when actually needed.

Thanks,

/mjt



reply via email to

[Prev in Thread] Current Thread [Next in Thread]