Re: [libvirt PATCH] qemu_firmware: don't error out for unknown firmware features
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.
Pavel
Attachments:
- signature.asc (application/pgp-signature — 833 bytes)