linux-kernel - Re: [BUG] sdhci regression in 2.6.21-rc2

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]

Message-ID: <45EC38C5.1060600@rtr.ca>
Date:	Mon, 05 Mar 2007 10:35:33 -0500
From:	Mark Lord <lkml@....ca>
To:	Pierre Ossman <drzeus-list@...eus.cx>
Cc:	sdhci-devel@...t.drzeus.cx, Adrian Bunk <bunk@...sta.de>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [BUG] sdhci regression in 2.6.21-rc2

Pierre Ossman wrote:
> Pierre Ossman wrote:
>> I'd say it's the kernel calling the interrupt handler of a
>> currently sleeping device. Since we're seeing this problem I assume the
>> kernel's interrupt code isn't aware of PM states?
>
> Hmm... I guess it can't be as the interrupt handler isn't associated with a
> device, just a random pointer.
> 
> So either release the interrupt (which seems a bit unsafe as then we might not
> get it back), or handle states at the start of the isr.

>From linux/Documentation/power/pci.txt:

> A reference implementation
> -------------------------
> .suspend()
> {
>         /* driver specific operations */
> 
>         /* Disable IRQ */
>         free_irq();
>         /* If using MSI */
>         pci_disable_msi();
> 
>         pci_save_state();
>         pci_enable_wake();
>         /* Disable IO/bus master/irq router */
>         pci_disable_device();
>         pci_set_power_state(pci_choose_state());
> }
> 
> .resume()
> {
>         pci_set_power_state(PCI_D0);
>         pci_restore_state();
>         /* device's irq possibly is changed, driver should take care */
>         pci_enable_device();
>         pci_set_master();
> 
>         /* if using MSI, device's vector possibly is changed */
>         pci_enable_msi();
> 
>         request_irq();
>         /* driver specific operations; */
> }
-
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/