linux-kernel - Re: a7839e96 (PNP: increase max resources) breaks my ALSA intel8x0 sound card

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-next>] [day] [month] [year] [list]

Message-id: <479CCAAD.4080706@shaw.ca>
Date:	Sun, 27 Jan 2008 12:17:17 -0600
From:	Robert Hancock <hancockr@...w.ca>
To:	Avuton Olrich <avuton@...il.com>
Cc:	Zhao Yakui <yakui.zhao@...el.com>,
	Li Shaohua <shaohua.li@...el.com>,
	Bjorn Helgaas <bjorn.helgaas@...com>,
	Thomas Renninger <trenn@...e.de>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	"Linux-Kernel," <linux-kernel@...r.kernel.org>,
	Alsa-Devel <alsa-devel@...a-project.org>
Subject: Re: a7839e96 (PNP: increase max resources) breaks my ALSA intel8x0
 sound card

Avuton Olrich wrote:
> With v2.6.24 my second ALSA sound device stopped working.
> 
> After bisection it says this was the offending commit.
> 
> a7839e960675b549f06209d18283d5cee2ce9261 is first bad commit
> commit a7839e960675b549f06209d18283d5cee2ce9261
> Author: Zhao Yakui <yakui.zhao@...el.com>
> Date:   Wed Nov 28 16:21:21 2007 -0800
> 
>     PNP: increase the maximum number of resources
> 
>     On some systems the number of resources(IO,MEM) returnedy by PNP device is
>     greater than the PNP constant, for example motherboard devices.  It brings
>     that some resources can't be reserved and resource confilicts.  This will
>     cause PCI resources are assigned wrongly in some systems, and cause hang.
>     This is a regression since we deleted ACPI motherboard driver and use PNP
>     system driver.
> 
>     [akpm@...ux-foundation.org: fix text and coding-style a bit]
>     Signed-off-by: Li Shaohua <shaohua.li@...el.com>
>     Signed-off-by: Zhao Yakui <yakui.zhao@...el.com>
>     Cc: Bjorn Helgaas <bjorn.helgaas@...com>
>     Cc: Thomas Renninger <trenn@...e.de>
>     Cc: <stable@...nel.org>
>     Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
>     Signed-off-by: Linus Torvalds <torvalds@...ux-foundation.org>
> 
> The audio device is 00:1b.0 (see my lspci -vvv output), the other
> audio device works fine.
> 
> http://avuton.googlepages.com/v2.6-before (dmesg revision before)
> http://avuton.googlepages.com/v2.6-after (dmesg broken revision)
> http://avuton.googlepages.com/lspci-vvv
> http://avuton.googlepages.com/config (from the broken revision)
> http://avuton.googlepages.com/iomem
> http://avuton.googlepages.com/ioports

Here's why the driver fails to load:

[   31.133060] ACPI: PCI Interrupt 0000:00:1b.0[A] -> GSI 22 (level, 
low) -> IRQ 22
[   31.133141] PCI: Unable to reserve mem region #1:4000@...f8000 for 
device 0000:00:1b.0
[   31.133197] ACPI: PCI interrupt for device 0000:00:1b.0 disabled
[   31.133244] HDA Intel: probe of 0000:00:1b.0 failed with error -16

The iomem location of the HDA controller conflicts with this reservation 
by the BIOS:

[   22.906654] system 00:08: iomem range 0xfebfa000-0xfebfac00 has been 
reserved

There was a patch floating around to ignore PnPACPI reservations which 
conflict with PCI BARs, which appears to be what's happening in this 
case. That patch originally worked for any board, but was later made 
specific to a certain Supermicro motherboard which had the sata_nv 
controller MMIO regions marked as reserved, preventing the driver from 
loading. We may need a more general solution. See:

https://bugzilla.redhat.com/show_bug.cgi?id=313491
--
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/