Re: [Xen-devel] [PATCH v2 1/2] VT-d: re-phrase logic in vtd_set_hwdom_mapping() for clarity

[Roger Pau Monné <roger.pau@xxxxxxxxxx>]

On Tue, Jun 12, 2018 at 02:47:33PM +0100, Paul Durrant wrote:
> It is hard to reconcile the comment at the top of the loop in
> vtd_set_hwdom_mapping() with the if statement following it. This patch
> re-phrases the logic, preserving the semantics, but making it easier
> to read.
> 
> The patch also modifies the Xen command line documentation to make it
> clear that iommu_inclusive_mapping only applies to pages up to the 4GB
> boundary.
> 
> NOTE: This patch also corrects the indentation of the printk() towards
>       the end of vtd_set_hwdom_mapping().
> 
> Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

Reviewed-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

Thanks, Roger.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel