Re: [PATCH v2 3/8] Validate remote store NVRAM

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

 



On Thu, Apr 21, 2022 at 16:52:54 +0200, Peter Krempa wrote:
> On Fri, Apr 08, 2022 at 10:48:46 -0700, Rohit Kumar wrote:
> > Remote store NVRAM feature is being enabled only
> > if it supports 'blockdev' capability.
> > 
> > Signed-off-by: Prerna Saxena <prerna.saxena@xxxxxxxxxxx>
> > Signed-off-by: Florian Schmidt <flosch@xxxxxxxxxxx>
> > Signed-off-by: Rohit Kumar <rohit.kumar3@xxxxxxxxxxx>
> > ---
> >  src/qemu/qemu_validate.c | 22 ++++++++++++++++++++++
> >  1 file changed, 22 insertions(+)
> > 
> > diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c
> > index 96f5427678..2a961b1f50 100644
> > --- a/src/qemu/qemu_validate.c
> > +++ b/src/qemu/qemu_validate.c
> > @@ -611,6 +611,25 @@ qemuValidateDomainDefBoot(const virDomainDef *def)
> >  }
> 
> As noted in 2/8, this will need to be moved earlier.
> 
> >  
> >  
> > +static int
> > +qemuValidateDomainDefNvram(const virDomainDef *def,
> > +                           virQEMUCaps *qemuCaps)
> > +{
> 
> Return early if there's nothing to validate to decrease indentation
> level.
> 
> > +    if (def->os.loader && def->os.loader->nvram) {
> > +        if (def->os.loader->nvram->type !=  VIR_STORAGE_TYPE_FILE &&
> > +            !virQEMUCapsGet(qemuCaps, QEMU_CAPS_BLOCKDEV)) {
> > +            virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> > +                           _("This Qemu does not support 'blockdev' capability "
> > +                             "for remote store NVRAM. NVRAM type other than "
> > +                             "'file' is not supported with this QEMU"));
> 
> "Remote NVRAM is not supported by this qemu".
> 
> > +            return -1;
> > +        }
> > +    }
> 
> Also you need to reject all the other unsupported configs here.

Additionally you'll also need to call the function that validates a
storage source definition too at this point.




[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