Re: 2.6.25 -- long standing bug with usb after resume with 4GB ram

Previous thread: fix aperture vs. suspend problem by Pavel Machek on Thursday, March 13, 2008 - 3:05 am. (7 messages)

Next thread: [PATCH v4] UIO: Implement a UIO interface for the SMX Cryptengine by Ben Nizette on Thursday, March 13, 2008 - 4:27 am. (4 messages)
From: Pavel Machek
Date: Thursday, March 13, 2008 - 4:01 am

Hi!

There are problems with usb after suspend/resume... it works as long
as memory is limited to 3G -- probably because iommu/agpgart/something
is not used in such case.

rmmod/insmod of ehci_hcd & ohci_hcd fixes that.

For testing with 2.6.25, you'll need aperture_64.c patch that makes
iommu as nosave (or it oopses).

Problem is also present in opensuse10.3.
								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--

From: Rafael J. Wysocki
Date: Thursday, March 13, 2008 - 1:27 pm

Any chance to push that one upstream?

Rafael
--

From: Greg KH
Date: Thursday, March 13, 2008 - 2:58 pm

Any pointer to where this patch is?

Can it go into 2.6.25?

thanks,

greg k-h
--

From: Pavel Machek
Date: Thursday, March 13, 2008 - 3:05 pm

Here it is: Yes, I guess it could go into 2.6.25...
								Pavel


diff --git a/arch/x86/kernel/aperture_64.c b/arch/x86/kernel/aperture_64.c
index 608152a..88747f3 100644
--- a/arch/x86/kernel/aperture_64.c
+++ b/arch/x86/kernel/aperture_64.c
@@ -18,6 +18,7 @@ #include <linux/pci_ids.h>
 #include <linux/pci.h>
 #include <linux/bitops.h>
 #include <linux/ioport.h>
+#include <linux/suspend.h>
 #include <asm/e820.h>
 #include <asm/io.h>
 #include <asm/gart.h>
@@ -76,6 +77,7 @@ static u32 __init allocate_aperture(void
 	printk(KERN_INFO "Mapping aperture over %d KB of RAM @ %lx\n",
 			aper_size >> 10, __pa(p));
 	insert_aperture_resource((u32)__pa(p), aper_size);
+	register_nosave_region((u32)__pa(p) >> PAGE_SHIFT, (u32)__pa(p+aper_size) >> PAGE_SHIFT);
 
 	return (u32)__pa(p);
 }


-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
pomozte zachranit klanovicky les:  http://www.ujezdskystrom.info/
--

Previous thread: fix aperture vs. suspend problem by Pavel Machek on Thursday, March 13, 2008 - 3:05 am. (7 messages)

Next thread: [PATCH v4] UIO: Implement a UIO interface for the SMX Cryptengine by Ben Nizette on Thursday, March 13, 2008 - 4:27 am. (4 messages)