On Thu, 11 Dec 2008, Daniel P. Berrange wrote: > > * a virNodeInfo is a structure filled by virNodeGetInfo() and providing > > @@ -504,6 +567,10 @@ int virDomainSetMaxMemory (virDomainPtr domain, > > int virDomainSetMemory (virDomainPtr domain, > > unsigned long memory); > > int virDomainGetMaxVcpus (virDomainPtr domain); > > +int virDomainGetSecLabel (virDomainPtr domain, > > + virDomainSecLabelPtr seclabel); > > +int virDomainGetSecModel (virDomainPtr domain, > > + virDomainSecModelPtr secmodel); > > I'm leaning two ways on this. On the one hand I could see the > virDomainGetSecModel being done against the node to match the > fact that we record it in the node capabilities XML, so perhaps > virNodeGetSecurityModel(virConnectPtr). Actually, this is a call to get the node information, so I think the name should be changed. > On the other hand, we already have this info against the node, Which came from the above call. > and conceivably you could have a domain configured with a model > that doesn't match the node's model, so an explicit per-domain > call is right. In that scenario, could we just put the security > model data into the security label struct and have a single API The domain doesn't have a security label until it's running, and then it must match the node's model, so I'm not sure we need to change anything except the name of virDomainGetSecModel (to virNodeGetSecurityMode). - James -- James Morris <jmorris@xxxxxxxxx> -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list