Re: [Xen-devel] [PATCH 12 of 16] hvmloader: Build IVRS table

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2011-12-23 at 11:29 +0000, Wei Wang wrote:
> diff -r b70cf1dcb110 -r bb76ba2457e6
> xen/include/public/hvm/hvm_info_table.h
> --- a/xen/include/public/hvm/hvm_info_table.h   Thu Dec 22 16:56:45
> 2011 +0100
> +++ b/xen/include/public/hvm/hvm_info_table.h   Thu Dec 22 16:56:48
> 2011 +0100
> @@ -67,6 +67,9 @@ struct hvm_info_table {
>  
>      /* Bitmap of which CPUs are online at boot time. */
>      uint8_t     vcpu_online[(HVM_MAX_VCPUS + 7)/8];
> +
> +    /* guest iommu enabled */
> +    uint8_t     iommu_enabled;
>  };
>  
>  #endif /* __XEN_PUBLIC_HVM_HVM_INFO_TABLE_H__ */

Please can we avoid adding new things to this struct and use xenstore to
pass new configuration items instead. See Paul Durrant's recent patches
to make s3 optional via the platform/acpi_s3 node.

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel