Re: [PATCH v7 17/17] iommu/vt-d: Add a command line parameter for VT-d posted-interrupts

From: Paolo Bonzini
Date: Fri Sep 11 2015 - 07:05:43 EST




On 25/08/2015 10:50, Feng Wu wrote:
> Enable VT-d Posted-Interrtups and add a command line
> parameter for it.
>
> Signed-off-by: Feng Wu <feng.wu@xxxxxxxxx>
> ---
> Documentation/kernel-parameters.txt | 1 +
> drivers/iommu/irq_remapping.c | 12 ++++++++----
> 2 files changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
> index 1d6f045..52aca36 100644
> --- a/Documentation/kernel-parameters.txt
> +++ b/Documentation/kernel-parameters.txt
> @@ -1547,6 +1547,7 @@ bytes respectively. Such letter suffixes can also be entirely omitted.
> nosid disable Source ID checking
> no_x2apic_optout
> BIOS x2APIC opt-out request will be ignored
> + nopost disable Interrupt Posting
>
> iomem= Disable strict checking of access to MMIO memory
> strict regions from userspace.
> diff --git a/drivers/iommu/irq_remapping.c b/drivers/iommu/irq_remapping.c
> index 2d99930..d8c3997 100644
> --- a/drivers/iommu/irq_remapping.c
> +++ b/drivers/iommu/irq_remapping.c
> @@ -22,7 +22,7 @@ int irq_remap_broken;
> int disable_sourceid_checking;
> int no_x2apic_optout;
>
> -int disable_irq_post = 1;
> +int disable_irq_post = 0;
>
> static int disable_irq_remap;
> static struct irq_remap_ops *remap_ops;
> @@ -58,14 +58,18 @@ static __init int setup_irqremap(char *str)
> return -EINVAL;
>
> while (*str) {
> - if (!strncmp(str, "on", 2))
> + if (!strncmp(str, "on", 2)) {
> disable_irq_remap = 0;
> - else if (!strncmp(str, "off", 3))
> + disable_irq_post = 0;
> + } else if (!strncmp(str, "off", 3)) {
> disable_irq_remap = 1;
> - else if (!strncmp(str, "nosid", 5))
> + disable_irq_post = 1;
> + } else if (!strncmp(str, "nosid", 5))
> disable_sourceid_checking = 1;
> else if (!strncmp(str, "no_x2apic_optout", 16))
> no_x2apic_optout = 1;
> + else if (!strncmp(str, "nopost", 6))
> + disable_irq_post = 1;
>
> str += strcspn(str, ",");
> while (*str == ',')
>

Reviewed-by: Paolo Bonzini <pbonzini@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/