linux-kernel - Re: Suspend/Resume bug in PCI layer wrt quirks

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: <200805150907.06930.jbarnes@virtuousgeek.org>
Date:	Thu, 15 May 2008 09:07:06 -0700
From:	Jesse Barnes <jbarnes@...tuousgeek.org>
To:	Arjan van de Ven <arjan@...radead.org>
Cc:	linux-kernel@...r.kernel.org, akpm@...ux-foundation.org,
	len.brown@...el.com, "Rafael J. Wysocki" <rjw@...k.pl>
Subject: Re: Suspend/Resume bug in PCI layer wrt quirks

On Thursday, May 15, 2008 7:05 am Arjan van de Ven wrote:
> Hi,
>
> the following is a bug in the PCI layer wrt suspend/resume and quirks:
>
> pci_device_resume_early does the following:
>
> static int pci_device_resume_early(struct device * dev)
> {
>         int error = 0;
>         struct pci_dev * pci_dev = to_pci_dev(dev);
>         struct pci_driver * drv = pci_dev->driver;
>
>         pci_fixup_device(pci_fixup_resume, pci_dev);
>
> eg it calls the resume quirks.
>
> However, one of these quirks is
> DECLARE_PCI_FIXUP_RESUME(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_ICH6_1,  
>   asus_hides_smbus_lpc_ich6);
>
> and asus_hides_smbus_lpc_ich6 does
>
>         base = ioremap_nocache(rcba & 0xFFFFC000, 0x4000); /* use bits
> 31:14, 16 kB aligned */ if (base == NULL) return;
>         val=readl(base + 0x3418); /* read the Function Disable register,
> dword mode only */ writel(val & 0xFFFFFFF7, base + 0x3418); /* enable the
> SMBus device */ iounmap(base);
>
> internally. ioremap()/iounmap() does a global tlb flush of course, which
> includes doing an IPI to other cpus. However, during early resume,
> interrupts are disabled, and it's not allowed to do a IPI/global tlbflush
> with interrupts off...

Yeah I just got a similar report for quirk_vialatency().  But why are we only 
seeing these issues now?  Is our interrupt disabled check catching more stuff 
these days?

Rafael said he's looking at the VIA issue, and iirc we had talked about 
changing this stuff around a few weeks ago anyway.  So yeah we may need 
either a new fixup phase or maybe we can just move the resume fixups to occur 
a little later, when interrupts are enabled.

Jesse
--
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/