Re: [libvirt PATCH 00/10] Cleanup and test more firmware handling scenarios

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

 



On Mon, Apr 04, 2022 at 02:45:41PM -0400, Cole Robinson wrote:
> On 2/16/22 8:17 AM, Michal Prívozník wrote:
> > On 2/15/22 19:54, Daniel P. Berrangé wrote:
> >> There are a mind bending number of possible ways to configure the
> >> firmware with/without NVRAM. Only a small portion are tested and
> >> many error scenarios are silently ignored.
> >>
> >> This series attempts to get coverage of every possible XML config
> >> scenario and report explicit errors in all invalid configs.
> >>
> >> There is an open question on patch 4.  Essentially the use of NVRAM
> >> combined with writable executable feels like an accidental feature
> >> in libvirt that hasn't really been thought through. I'd like to
> >> better define expectations here but there are several possible
> >> strategies and I'm undecided which is best.
> >>
> >> Daniel P. Berrangé (10):
> >>   qemu: fix bad indentation for qemuDomainNVRAMPathFormat
> >>   tests: add explicit test case for pflash loader lacking path
> >>   tests: add test case for NVRAM with template
> >>   conf: validate NVRAM template usage with R/W loader binary
> >>   tests: don't permit NVRAM path when using firmware auto-select
> >>   qemu: inline code for filling in per-VM NVRAM path
> >>   conf: rename struct field for NVRAM template
> >>   conf: switch nvram parsing to use XML node / property helpers
> >>   conf: move nvram parsing into virDomainLoaderDefParseXML
> >>   conf: stop ignoring <loader>/<nvram> with firmware auto-select
> >>
> 
> >>
> > 
> > Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> > 
> > Michal
> > 
> 
> I don't see the last 3 patches in git. Daniel was that intentional?

I didn't merge the ones with comments from Michal on, and that also
invalidated the last few. I need to revisit the remaining ones again.


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux