Re: Reserved page flaging of 2.4 kernel memory changed recently?

[Michael Frank]

On Wednesday 11 February 2004 02:51, Andrea Arcangeli wrote:
> On Tue, Feb 10, 2004 at 11:24:01PM +0800, Michael Frank wrote:
> > 
> > diff -uN -r -X /home/mhf/sys/dont/dontdiff linux-2.4.24-Vanilla/arch/i386/mm/init.c linux-2.4.24-mhf179/arch/i386/mm/init.c
> > --- linux-2.4.24-Vanilla/arch/i386/mm/init.c	2004-01-21 15:53:01.000000000 +0800
> > +++ linux-2.4.24-mhf179/arch/i386/mm/init.c	2004-02-10 06:15:31.000000000 +0800
> > @@ -451,15 +451,18 @@
> >  {
> >  	if (!page_is_ram(pfn)) {
> >  		SetPageReserved(page);
> > +		SetPageNosave(page);
> >  		return;
> >  	}
> >  	
> >  	if (bad_ppro && page_kills_ppro(pfn)) {
> >  		SetPageReserved(page);
> > +		SetPageNosave(page);
> >  		return;
> >  	}
> >  	
> >  	ClearPageReserved(page);
> > +	ClearPageNosave(page);
> 
> why this clearpagenosave? looks superflous, you're not doing it in the
> normal zone anyways.

I'll sleep on it and get back to you with my arguments.

> 
> > +#if defined(__nosave_begin)
> 
> this won't work right, __nosave_begin isn't a preprocessor thing so it
> will be ignored when you uncomment it. You probably can use #if 0
> instead and a comment near __nosave_begin to turn it to 1 when enabling
> the suspend code.

Oh sh*t, this is what one gets for fixing things up for a demo after
a long night... Will bite my lower rear portion after the nap.

> 
> > What is your opinion of this approach?
> 
> except for the above two nitpicks, the patch is correct and needed for
> safe suspend IMHO. 2.6 seems to miss this thing too, why not add it to
> 2.6 first?

Swsusp won't be in 2.4 anyway, if Nigel accepts the patch, it will become part 
of his next releases for 2.4 and 2.6.

Anyway, I'll fix the patch up for 2.6, test it and post the patch in a few days.

Regards
Michael

-
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/