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

On Mon, 26 Apr 2004 22:11:45 +1000, Herbert Xu
<herbert@gondor.apana.org.au> wrote:

> On Mon, Apr 26, 2004 at 09:27:13PM +1000, Nigel Cunningham wrote:
>>
>> There used to be such a check. Centrinos, however, if I recall
>> correctly,
>> don't have PSE but can suspend with our current method. Perhaps we can
>
> Then it's just pure luck. Whenever you have a page whose page table
> lies in a page beyond that page itself the non-PSE case will fail.

I'm no expert on the hardware side of things, but from what I know, it's
really only these hardware devices that are accessing memory while we're
doing the copyback that are the problem. All processes are stopped and
we've called device_suspend(). Nothing but us should be using/modifying
the page tables.

>> come up with a more nuanced test? Better still, though, we should just
>> get
>> proper AGP support for suspending and resuming in.
>
> It's got nothing to do with AGP. This is a flaw in the swsusp code.
> It can be triggered by anything that plays with page attributes.

Not so much a flaw in the suspend code as something that needs to be dealt
with: it's not a bug for pages to have protection, and its not a bug for
us to need it temporarily removed in order to do the copyback. We just
need the support in the drivers to achieve that. When we have it (as we do
in some cases in 2.4), all is well.

Regards,

Nigel
--
Nigel Cunningham
C/- Westminster Presbyterian Church Belconnen
61 Templeton Street, Cook, ACT 2614, Australia.
+61 (2) 6251 7727 (wk)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:02    [from the cache]
©2003-2011 Jasper Spaans