On Tue, Aug 21, 2018 at 05:52:39AM -0600, Jan Beulich wrote: > >>> On 17.08.18 at 17:12, <wei.l...@citrix.com> wrote: > > --- a/xen/arch/x86/sysctl.c > > +++ b/xen/arch/x86/sysctl.c > > @@ -92,8 +92,10 @@ void arch_do_physinfo(struct xen_sysctl_physinfo *pi) > > min(sizeof(pi->hw_cap), sizeof(boot_cpu_data.x86_capability))); > > if ( hvm_enabled ) > > pi->capabilities |= XEN_SYSCTL_PHYSCAP_hvm; > > - if ( iommu_enabled ) > > + if ( hvm_enabled && iommu_enabled ) > > pi->capabilities |= XEN_SYSCTL_PHYSCAP_hvm_directio; > > + else if ( iommu_enabled ) > > + pi->capabilities |= XEN_SYSCTL_PHYSCAP_directio; > > } > > At the sysctl layer I think you can, as suggested iirc by Roger, > simply replace hvm_directio with directio (or iommu). For the > "xl info" output, otoh, I'm afraid this doesn't hold, as people > may parse for the string. Depending on how this would best > be addressed in the tool stack, replacing the sysctl names may > then no longer be the most suitable solution.
In that case what do you think about the two flags this patch provides on the toolstack level? Essentially we change slightly hvm_directio's meaning to mean "hvm && iommu_enabled" while it previously mean "iommu_enabled", and then in the absence of hvm_directio, add "directio" as an indication for "iommu_enabled". Wei. > > Jan > > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
