On 31/10/2023 1:45 pm, Roger Pau Monné wrote: > On Tue, Oct 31, 2023 at 01:29:04PM +0000, Andrew Cooper wrote: >> On 31/10/2023 12:24 pm, Roger Pau Monné wrote: >>> On Tue, Oct 31, 2023 at 12:02:15PM +0000, Andrew Cooper wrote: >>>> Make the command line docs match the actual implementation, and state that >>>> the >>>> default behaviour is selected at compile time. >>>> >>>> Fixes: 980d6acf1517 ("IOMMU: make DMA containment of quarantined devices >>>> optional") >>>> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Reviewed-by: Roger Pau Monné <roger....@citrix.com>
Thanks. > >>>> --- >>>> CC: Jan Beulich <jbeul...@suse.com> >>>> CC: Roger Pau Monné <roger....@citrix.com> >>>> CC: Wei Liu <w...@xen.org> >>>> CC: Marek Marczykowski-Górecki <marma...@invisiblethingslab.com> >>>> CC: Henry Wang <henry.w...@arm.com> >>>> --- >>>> docs/misc/xen-command-line.pandoc | 6 ++++-- >>>> 1 file changed, 4 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/docs/misc/xen-command-line.pandoc >>>> b/docs/misc/xen-command-line.pandoc >>>> index 6b07d0f3a17f..9a19a04157cb 100644 >>>> --- a/docs/misc/xen-command-line.pandoc >>>> +++ b/docs/misc/xen-command-line.pandoc >>>> @@ -1480,7 +1480,8 @@ detection of systems known to misbehave upon >>>> accesses to that port. >>>> > Default: `new` unless directed-EOI is supported >>>> >>>> ### iommu >>>> - = List of [ <bool>, verbose, debug, force, required, >>>> quarantine[=scratch-page], >>>> + = List of [ <bool>, verbose, debug, force, required, >>>> + quarantine=<bool>|scratch-page, >>> I think this should be quarantine=[<bool>|scratch-page], as just using >>> iommu=quarantine is a valid syntax and will enable basic quarantine. >>> IOW: the bool or scratch-page parameters are optional. >> =<bool> already has that meaning, and this is the form we use elsewhere. > I guess I got confused by some other options using `[ ]` to denote > optional parameters, but I see it's not used by all of them. Yeah, it's a mess, sadly. One of many things I've not had time to fix, but at least this is closer to the normal syntax than before. ~Andrew