lkml.org 
[lkml]   [2005]   [Oct]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [discuss] Re: [PATCH][Fix] swsusp: avoid possible page tables corruption during resume on x86-64
    Date
    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/

    \
     
     \ /
      Last update: 2005-10-06 10:10    [W:0.044 / U:1.032 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site