Re: [PATCH v2 4/4] virt-aa-helper: add DomainGuest to mockup caps

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

 



On Tue, Jul 07, 2015 at 11:38:53AM +0200, Cédric Bosdonnat wrote:
> With commit 3f9868a virt-aa-helper stopped working due to missing
> DomainGuest in the caps.
> 
> The test with -c without arch also needs to be
> removed since the new capabilities code uses the host arch when none is
> provided.
> ---
>  src/security/virt-aa-helper.c | 25 ++++++++++++++++++++++++-
>  tests/virt-aa-helper-test     |  3 ---
>  2 files changed, 24 insertions(+), 4 deletions(-)
> 
> diff --git a/src/security/virt-aa-helper.c b/src/security/virt-aa-helper.c
> index f94f337..cf729e9 100644
> --- a/src/security/virt-aa-helper.c
> +++ b/src/security/virt-aa-helper.c
> @@ -73,6 +73,7 @@ typedef struct {
>      virDomainDefPtr def;        /* VM definition */
>      virCapsPtr caps;            /* VM capabilities */
>      virDomainXMLOptionPtr xmlopt; /* XML parser data */
> +    char *hvm;                  /* type of hypervisor (eg qemu, xen, lxc) */

HVM should stand for Hardware Virtual Machine, but not all the
hypervisors provide full virtualization.

I think hypervisor would be a better name. Or virtType, which is what
domain_conf.c uses.

>      char *os;                   /* type of os (eg hvm, xen, exe) */
>      virArch arch;               /* machine architecture */
>      char *newfile;              /* newly added file */
  

> @@ -730,6 +738,21 @@ get_definition(vahControl * ctl, const char *xmlStr)
>          goto exit;
>      }
>  
> +    if ((hvmtype = virDomainVirtTypeFromString(ctl->hvm)) < 0) {
> +        vah_error(ctl, 0, _("unknown HVM type"));

unknown domain/virt type

ACK with the name changed.

Jan

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[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]