linux-kernel - Re: [PATCH] ACPI: Fix the irq0 assigning issue for AMILO PRO V2030 platform

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 01 Jun 2012 12:53:00 -0400
From:	Len Brown <lenb@...nel.org>
To:	Feng Tang <feng.tang@...el.com>
CC:	linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
	rui.zhang@...el.com, fazerxlo@...pl, x86@...nel.org
Subject: Re: [PATCH] ACPI: Fix the irq0 assigning issue for AMILO PRO V2030
 platform

Hello Tang-Feng,

This looks good, but especially w/ interrupt patches,
it is important to take logically independent baby steps.
So please break this patch into two, and add another small bit
in a 3 patch series:

1. allow the workaround to apply to non global_irq 2
2. delete the WARN from dmi_ignore_irq0_timer_override
	as it causes alarm, but gives no benefit
3. add the DMI to invoke the workaround for this platform

thanks,
-Len Brown, Intel Open Source Technology Center

On 05/28/2012 10:43 PM, Feng Tang wrote:

> This is to fix the kernel bugzilla 40002: "IRQ 0 assigned to VGA"
>  https://bugzilla.kernel.org/show_bug.cgi?id=40002
> 
> The root cause is the buggy FW, whose ACPI tables assign the GSI 16
> to 2 irqs 0 and 16(VGA), and the VGA is the right owner of GSI 16.
> So add a quirk to ignore the irq0 overriding GSI 16 for the
> FUJITSU SIEMENS AMILO PRO V2030 platform will solve this issue.
> 
> This also slightly modify the current irq0 override handling, as it
> only covered the irq0 overriding GSI 2 case.
> 
> Reported-and-tested-by: Szymon Kowalczyk <fazerxlo@...pl>
> Signed-off-by: Feng Tang <feng.tang@...el.com>
> Cc: x86@...nel.org
> ---
>  arch/x86/kernel/acpi/boot.c |   17 ++++++++++++++---
>  1 files changed, 14 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c
> index 4558f0d..06c78c3 100644
> --- a/arch/x86/kernel/acpi/boot.c
> +++ b/arch/x86/kernel/acpi/boot.c
> @@ -416,12 +416,14 @@ acpi_parse_int_src_ovr(struct acpi_subtable_header * header,
>  		return 0;
>  	}
>  
> -	if (intsrc->source_irq == 0 && intsrc->global_irq == 2) {
> +	if (intsrc->source_irq == 0) {
>  		if (acpi_skip_timer_override) {
> -			printk(PREFIX "BIOS IRQ0 pin2 override ignored.\n");
> +			printk(PREFIX "BIOS IRQ0 override ignored.\n");
>  			return 0;
>  		}
> -		if (acpi_fix_pin2_polarity && (intsrc->inti_flags & ACPI_MADT_POLARITY_MASK)) {
> +
> +		if ((intsrc->global_irq == 2) && acpi_fix_pin2_polarity
> +			&& (intsrc->inti_flags & ACPI_MADT_POLARITY_MASK)) {
>  			intsrc->inti_flags &= ~ACPI_MADT_POLARITY_MASK;
>  			printk(PREFIX "BIOS IRQ0 pin2 override: forcing polarity to high active.\n");
>  		}
> @@ -1466,6 +1468,15 @@ static struct dmi_system_id __initdata acpi_dmi_table_late[] = {
>  		     DMI_MATCH(DMI_PRODUCT_NAME, "HP Compaq 6715b"),
>  		     },
>  	 },
> +	{
> +	 .callback = dmi_ignore_irq0_timer_override,
> +	 .ident = "FUJITSU SIEMENS",
> +	 .matches = {
> +		     DMI_MATCH(DMI_SYS_VENDOR, "FUJITSU SIEMENS"),
> +		     DMI_MATCH(DMI_PRODUCT_NAME, "AMILO PRO V2030"),
> +		     },
> +	 },
> +
>  	{}
>  };
>  


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/