lkml.org 
[lkml]   [2004]   [Apr]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
SubjectRe: Bug#234976: kernel-source-2.6.4: Software Suspend doesn't work
From
tags 234976 pending
quit

On Fri, Mar 26, 2004 at 07:17:52PM +0100, Roland Stigge wrote:
>
> On Fri, 2004-03-26 at 17:17, Pavel Machek wrote:
> > > Please first read the full conversation I had with Herbert at
> > > http://bugs.debian.org/234976 . It contains documented debug sessions of
> > > this kind. Please tell me what to try.
> >
> > Intel AGP, by chance?
>
> Yes, but as I wrote, only on one of the machines in question.

OK, I've finally found out why agpgart locks up the machine upon
resuming from swsusp/pmdisk.

The reason is that the gatt table is remapped with ioremap_nocache,
which on i386 machines capable of PSE will result in 4M pages being
split.

When swsusp/pmdisk copies the pages back, the top page table is
written before the entries that it points to are filled in.
Depending on whether the second-level table lies before or after
the 4M-page in question, this will result in a page fault.

A simple solution is to copy the pages in reverse. This way the
top page table is filled in last which should resolve this particular
issue. The following patch does exactly that and fixes the problem
for me.

Cheers,
--
Debian GNU/Linux 3.0 is out! ( http://www.debian.org/ )
Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txtIndex: arch/i386/power/swsusp.S
===================================================================
RCS file: /home/gondolin/herbert/src/CVS/debian/kernel-source-2.5/arch/i386/power/swsusp.S,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 swsusp.S
--- a/arch/i386/power/swsusp.S 28 Sep 2003 04:44:10 -0000 1.1.1.1
+++ b/arch/i386/power/swsusp.S 26 Apr 2004 09:31:46 -0000
@@ -33,8 +33,9 @@
movl %ecx,%cr3

call do_magic_resume_1
- movl $0,loop
- cmpl $0,nr_copy_pages
+ movl nr_copy_pages,%eax
+ movl %eax,loop
+ cmpl $0,%eax
je .L1453
.p2align 4,,7
.L1455:
@@ -45,8 +46,8 @@
movl loop,%eax
movl loop2,%edx
sall $4,%eax
- movl 4(%ecx,%eax),%ebx
- movl (%ecx,%eax),%eax
+ movl -12(%ecx,%eax),%ebx
+ movl -16(%ecx,%eax),%eax
movb (%edx,%eax),%al
movb %al,(%edx,%ebx)
movl %cr3, %eax;
@@ -59,11 +60,11 @@
cmpl $4095,%eax
jbe .L1459
movl loop,%eax
- leal 1(%eax),%edx
+ leal -1(%eax),%edx
movl %edx,loop
movl %edx,%eax
- cmpl nr_copy_pages,%eax
- jb .L1455
+ cmpl $0,%eax
+ jne .L1455
.p2align 4,,7
.L1453:
movl $__USER_DS,%eax
\
 
 \ /
  Last update: 2005-03-22 14:02    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site