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

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



On Friday 23 December 2011 12:36:32 Ian Campbell wrote:
> 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.
Good point, Will use xenstore_read in next try.
Thanks,
Wei


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.