[libvirt PATCH] qemu_firmware: don't error out for unknown firmware features

Pavel Hrdina phrdina at redhat.com
Mon Jan 31 14:55:03 UTC 2022

On Mon, Jan 31, 2022 at 03:45:18PM +0100, Michal Prívozník wrote:
> On 1/31/22 15:38, Daniel P. Berrangé wrote:
> > 
> > Oh, I hadn't checked how number prioritization affects it. So you're
> > saying that we stop parsing the firmware file as soon as we find a
> > valid match. That does indeed limit the impact of the change.
> Ah, my memory is failing me. We don't do that. Sorry for the noise. But
> changing the code to do that should be easy enough.

I was just checking that but we do parse all files.

Anyway I was wondering that we could modify the code to make the
`nvram-template` mandatory base on the firmware mode or if there is no
firmware mode specified.

That way we would keep the current behavior for old firmware descriptors
where no firmware mode exists and with the new mode we could do the
right thing. Unless I'm missing someting.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20220131/c93b1950/attachment-0001.sig>

More information about the libvir-list mailing list