Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-09 Thread Jan Beulich
>>> On 09.08.18 at 12:51, wrote: > On Thu, Aug 09, 2018 at 04:29:51AM -0600, Jan Beulich wrote: >> >>> On 09.08.18 at 12:01, wrote: >> > In fact I think this would be clearer if something like: >> > >> > enum { >> > NONE, >> > RELAXED, >> > STRICT, >> > } iommu_hwdom = RELAXED; >> >

Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-09 Thread Roger Pau Monné
On Thu, Aug 09, 2018 at 04:29:51AM -0600, Jan Beulich wrote: > >>> On 09.08.18 at 12:01, wrote: > > On Thu, Aug 09, 2018 at 01:00:59AM -0600, Jan Beulich wrote: > >> >>> On 08.08.18 at 17:50, wrote: > >> > On Wed, Aug 08, 2018 at 06:10:39AM -0600, Jan Beulich wrote: > >> >> >>> On 08.08.18 at 12:

Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-09 Thread Jan Beulich
>>> On 09.08.18 at 12:01, wrote: > On Thu, Aug 09, 2018 at 01:00:59AM -0600, Jan Beulich wrote: >> >>> On 08.08.18 at 17:50, wrote: >> > On Wed, Aug 08, 2018 at 06:10:39AM -0600, Jan Beulich wrote: >> >> >>> On 08.08.18 at 12:07, wrote: >> >> > +Note that all the above options are mutually exclu

Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-09 Thread Paul Durrant
an Jackson ; Tim > (Xen.org) ; Julien Grall ; Suravee > Suthikulpanit ; xen-devel de...@lists.xenproject.org>; Brian Woods > Subject: Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu > option > > >>> On 08.08.18 at 17:50, wrote: > > On Wed, Aug 08, 2018 at

Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-09 Thread Roger Pau Monné
On Thu, Aug 09, 2018 at 01:00:59AM -0600, Jan Beulich wrote: > >>> On 08.08.18 at 17:50, wrote: > > On Wed, Aug 08, 2018 at 06:10:39AM -0600, Jan Beulich wrote: > >> >>> On 08.08.18 at 12:07, wrote: > >> > +Note that all the above options are mutually exclusive. Specifying more > >> > than > >>

Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-09 Thread Jan Beulich
>>> On 08.08.18 at 17:50, wrote: > On Wed, Aug 08, 2018 at 06:10:39AM -0600, Jan Beulich wrote: >> >>> On 08.08.18 at 12:07, wrote: >> > +Note that all the above options are mutually exclusive. Specifying more >> > than >> > +one on the `dom0-iommu` command line will result in undefined behavior

Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-08 Thread Roger Pau Monné
On Wed, Aug 08, 2018 at 06:10:39AM -0600, Jan Beulich wrote: > >>> On 08.08.18 at 12:07, wrote: > > +Note that all the above options are mutually exclusive. Specifying more > > than > > +one on the `dom0-iommu` command line will result in undefined behavior. > > Isn't this more strict than it ne

Re: [Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-08 Thread Jan Beulich
>>> On 08.08.18 at 12:07, wrote: > @@ -1198,6 +1204,23 @@ detection of systems known to misbehave upon accesses > to that port. > > >> Enable IOMMU debugging code (implies `verbose`). > > +### dom0-iommu This is now misplaced, as the file is (meant to be) alphabetically sorted. > +> `= Lis

[Xen-devel] [PATCH v4 1/4] iommu: introduce dom0-iommu option

2018-08-08 Thread Roger Pau Monne
To select the iommu configuration used by Dom0. This option supersedes iommu=dom0-strict|dom0-passthrough. No functional change. Signed-off-by: Roger Pau Monné Reviewed-by: Paul Durrant --- Changes since v2: - Change the style and text used in the xen command line document. - Don't allow none