On 05.10.2020 11:49, Paul Durrant wrote:
Just two nits, in case the op is really needed:
> --- a/xen/drivers/passthrough/iommu.c
> +++ b/xen/drivers/passthrough/iommu.c
> @@ -515,6 +515,14 @@ static int iommu_ctl(
>
> switch ( ctl->op )
> {
> + case XEN_DOMCTL_IOMMU_SET_ALLOCATION:
> + {
> + struct xen_domctl_iommu_set_allocation *set_allocation =
> + &ctl->u.set_allocation;
const please, or drop the local variable.
> + rc = iommu_set_allocation(d, set_allocation->nr_pages);
> + break;
> + }
> default:
Blank line above here please.
Jan
