Messages in this thread | | | From | "Rafael J. Wysocki" <> | Subject | Re: [discuss] Re: [PATCH][Fix] swsusp: avoid possible page tables corruption during resume on x86-64 | Date | Thu, 6 Oct 2005 10:07:45 +0200 |
| |
Hi,
On Thursday, 6 of October 2005 00:49, Pavel Machek wrote: > Hi! > > > Summary ========= > > The following patch makes swsusp avoid the possible temporary corruption of > > page translation tables during resume on x86-64. This is achieved by creating > > a copy of the relevant page tables that will not be modified by swsusp and can > > be safely used by it on resume. > > Andi, this means swsusp fails 50% of time on x86-64. I believe we even > have one report in suse bugzilla by now... Could we get this somehow > merged? > > > > Index: linux-2.6.14-rc3-git5/kernel/power/swsusp.c > > =================================================================== > > --- linux-2.6.14-rc3-git5.orig/kernel/power/swsusp.c 2005-10-05 21:12:41.000000000 +0200 > > +++ linux-2.6.14-rc3-git5/kernel/power/swsusp.c 2005-10-05 21:24:50.000000000 +0200 > > @@ -672,7 +672,6 @@ > > return 0; > > > > page = pfn_to_page(pfn); > > - BUG_ON(PageReserved(page) && PageNosave(page)); > > if (PageNosave(page)) > > return 0; > > if (PageReserved(page) && pfn_is_nosave(pfn)) { > > Rafael, are you sure?
Yes, I am. The pages allocated in init_memory_mapping() are marked with PG_reserved by the init code.
> This will clash with snapshot.c split and probably belongs to some other patch.
I am aware of that. This will conflict with the Nigel's patch, so we probably can arrange to apply that patch before this one, if you prefer.
As far as the split is concerned, if you recall my doubts wrt it, the "bugfixes pending" is the first point on the list. :-)
Greetings, Rafael
- 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/
| |