<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 5, 2018 at 6:44 PM, Peter Krempa <span dir="ltr"><<a href="mailto:pkrempa@redhat.com" target="_blank">pkrempa@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, Feb 05, 2018 at 18:39:24 +0530, Shivaprasad bhat wrote:<br>
> Thanks for the reply Peter, Christian<br>
><br>
> On Fri, Feb 2, 2018 at 3:21 PM, Peter Krempa <<a href="mailto:pkrempa@redhat.com">pkrempa@redhat.com</a>> wrote:<br>
><br>
> > On Fri, Feb 02, 2018 at 08:14:29 +0100, Christian Ehrhardt wrote:<br>
> > > On Fri, Jan 19, 2018 at 3:55 PM, Shivaprasad G Bhat<br>
> > > <<a href="mailto:sbhat@linux.vnet.ibm.com">sbhat@linux.vnet.ibm.com</a>> wrote:<br>
> > > > The virt-aa-helper fails to parse the xmls with the memory/cpu<br>
> > > > hotplug features or user assigned aliases. Set the features in<br>
> > > > xmlopt->config for the parsing to succeed.<br>
> > > ><br>
> > > > Signed-off-by: Shivaprasad G Bhat <<a href="mailto:sbhat@linux.vnet.ibm.com">sbhat@linux.vnet.ibm.com</a>><br>
> > > > ---<br>
> > > >  src/security/virt-aa-helper.c |    8 +++++++-<br>
> > > >  1 file changed, 7 insertions(+), 1 deletion(-)<br>
> > > ><br>
> > > > diff --git a/src/security/virt-aa-helper.<wbr>c<br>
> > b/src/security/virt-aa-helper.<wbr>c<br>
> > > > index f7ccae0..29a459d 100644<br>
> > > > --- a/src/security/virt-aa-helper.<wbr>c<br>
> > > > +++ b/src/security/virt-aa-helper.<wbr>c<br>
> > > > @@ -654,6 +654,11 @@ caps_mockup(vahControl * ctl, const char *xmlStr)<br>
> > > >      return rc;<br>
> > > >  }<br>
> > > ><br>
> > > > +virDomainDefParserConfig virAAHelperDomainDefParserConf<wbr>ig = {<br>
> > > > +    .features = VIR_DOMAIN_DEF_FEATURE_MEMORY_<wbr>HOTPLUG |<br>
> > > > +                VIR_DOMAIN_DEF_FEATURE_<wbr>OFFLINE_VCPUPIN |<br>
> > > > +                VIR_DOMAIN_DEF_FEATURE_<wbr>INDIVIDUAL_VCPUS,<br>
> > > > +};<br>
> > ><br>
> > > Sure we can't link against qemu_domain.c to get "the original"<br>
> > > virQEMUDriverDomainDefParserCo<wbr>nfig.<br>
> > > But a comment here that the define is essentially taken there might<br>
> > > helpful to later on follow any updates there.<br>
> > ><br>
> > > Also mentioning why exactly you dropped<br>
> > > VIR_DOMAIN_DEF_FEATURE_USER_<wbr>ALIAS for the same reason.<br>
> ><br>
><br>
> The reason being the parsing code just ignores the user specified aliases<br>
> and doesn't fail the parsing if the VIR_DOMAIN_DEF_FEATURE_USER_<wbr>ALIAS<br>
> feature flag is not set.<br>
<br>
</div></div>Ah, right. Sorry about that, I've only remembered the rest of the flags<br>
which usually end in a parse failure.<br>
<br>
In such case, ACK even without any modification currently. We can figure<br>
out the skipping of the parser features later.</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra"><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">In that case, are you planning to push this patch to git ?</span></div><div class="gmail_extra"><br></div><div class="gmail_extra">Or Should I rework on the V3 I sent already ?<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks,</div><div class="gmail_extra">Shivaprasad</div></div>