Re: [PATCH][Fix] Fix Bug #4959 (take 2)

From: Pavel Machek
Date: Tue Sep 27 2005 - 18:45:04 EST


Hi!

> > > I can reserve the static buffer (10 pages) in suspend.c and mark it as nosave.
> > > The code that creates the mappings can stay in suspend.c either except it
> > > won't need to call get_usable_page() and free_eaten_memory() any more
> > > (__next_page() can be changed to get pages from the static buffer instead
> > > of allocating them). The code can also be simplified a bit, as we assume that
> > > there will be only one PGD entry in the direct mapping.
> > >
> > > If that sounds good to you, please confirm.
> >
> > 8GB limit seems good to me -- as long as it makes code significantly
> > simpler. It would be nice if it was <20 lines.
>
> It is more than that, but it seems to be quite simple anyway.
>
> The new patch follows.

Thanks, seems okay to me.

Pavel

--
if you have sharp zaurus hardware you don't need... you know my address
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/