linux-kernel - Re: >3G => iommu => suspend problems -- was Re: SB600 AHCI: Hard Disk Corruption

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: <20080527112236.GA13957@elf.ucw.cz>
Date:	Tue, 27 May 2008 13:22:37 +0200
From:	Pavel Machek <pavel@...e.cz>
To:	Patrick <ragamuffin@...acomm.ch>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: >3G => iommu => suspend problems -- was Re: SB600 AHCI: Hard
	Disk Corruption

Hi!

> > Can you try this one? It should prevent suspend in the broken cases,
> > but allow it in mem=4G config.
> 
> Sure!
> 
> root@...ar:/usr/src/linux-2.6.25# patch -p1 < /home/pat/patch-2.6.26-rc3.gart-suspend
> patching file arch/x86/kernel/pci-gart_64.c
> Hunk #4 succeeded at 629 with fuzz 2 (offset 11 lines).
> 
> .....make; cp bzImage /boot; reboot....

Thanks!

This goes on top of the second patch... it makes it work.

								Pavel

For iommu suspend/resume code to work, functions it calls may not be
__init.

Signed-off-by: Pavel Machek <pavel@...e.cz>

---
commit 0ea376de01be797f9563c2c2464149f8f0af6329
tree 4b5179fe97fe045cc770091bce94f898f26e4499
parent 017834f8541b8ded8ef831e5fe2b5f9cead4f6b0
author Pavel <pavel@....ucw.cz> Tue, 27 May 2008 13:21:05 +0200
committer Pavel <pavel@....ucw.cz> Tue, 27 May 2008 13:21:05 +0200

 arch/x86/kernel/k8.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/x86/kernel/k8.c b/arch/x86/kernel/k8.c
index 7377ccb..acf4770 100644
--- a/arch/x86/kernel/k8.c
+++ b/arch/x86/kernel/k8.c
@@ -76,7 +76,7 @@ EXPORT_SYMBOL_GPL(cache_k8_northbridges)
 
 /* Ignores subdevice/subvendor but as far as I can figure out
    they're useless anyways */
-int __init early_is_k8_nb(u32 device)
+int early_is_k8_nb(u32 device)
 {
 	struct pci_device_id *id;
 	u32 vendor = device & 0xffff;


-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
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/