Re: [Xen-devel] [v3 15/15] Add a command line parameter for VT-d posted-interrupts

["Tian, Kevin" <kevin.tian@xxxxxxxxx>]

> From: Wu, Feng
> Sent: Wednesday, June 24, 2015 1:18 PM
> 
> Enable VT-d Posted-Interrupts and add a command line
> parameter for it.
> 
> Signed-off-by: Feng Wu <feng.wu@xxxxxxxxx>
> ---
> v3:
> Remove the redundant "no intremp then no intpost" logic
> 
>  docs/misc/xen-command-line.markdown | 9 ++++++++-
>  xen/drivers/passthrough/iommu.c     | 4 +++-
>  2 files changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/misc/xen-command-line.markdown
> b/docs/misc/xen-command-line.markdown
> index aa684c0..f8ec15f 100644
> --- a/docs/misc/xen-command-line.markdown
> +++ b/docs/misc/xen-command-line.markdown
> @@ -875,6 +875,13 @@ debug hypervisor only).
>  >> Control the use of interrupt remapping (DMA remapping will always be 
> enabled
>  >> if IOMMU functionality is enabled).
> 
> +> `intpost`
> +
> +> Default: `true`
> +
> +>> Control the use of interrupt posting, interrupt posting is dependant on
> +>> interrupt remapping.

"Control the use of interrupt posting, which depends on the availability of 
interrupt remapping."

> +
>  > `qinval` (VT-d)
> 
>  > Default: `true`
> diff --git a/xen/drivers/passthrough/iommu.c b/xen/drivers/passthrough/iommu.c
> index 597f676..e13251c 100644
> --- a/xen/drivers/passthrough/iommu.c
> +++ b/xen/drivers/passthrough/iommu.c
> @@ -52,7 +52,7 @@ bool_t __read_mostly iommu_passthrough;
>  bool_t __read_mostly iommu_snoop = 1;
>  bool_t __read_mostly iommu_qinval = 1;
>  bool_t __read_mostly iommu_intremap = 1;
> -bool_t __read_mostly iommu_intpost;
> +bool_t __read_mostly iommu_intpost = 1;
>  bool_t __read_mostly iommu_hap_pt_share = 1;
>  bool_t __read_mostly iommu_debug;
>  bool_t __read_mostly amd_iommu_perdev_intremap = 1;
> @@ -97,6 +97,8 @@ static void __init parse_iommu_param(char *s)
>              iommu_qinval = val;
>          else if ( !strcmp(s, "intremap") )
>              iommu_intremap = val;
> +        else if ( !strcmp(s, "intpost") )
> +            iommu_intpost = val;
>          else if ( !strcmp(s, "debug") )
>          {
>              iommu_debug = val;
> --
> 2.1.0

Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel