Re: [PATCH] Free swap suspend from dependency on PageReserved

[Nigel Cunningham <ncunningham@cyclades.com>]

Hi Dave.

On Thu, 2005-10-06 at 08:03, Dave Hansen wrote:
> On Thu, 2005-10-06 at 07:04 +1000, Nigel Cunningham wrote:
> > 
> > +       for (tmp = 0; tmp < max_low_pfn; tmp++, addr += PAGE_SIZE) {
> > +               if (page_is_ram(tmp)) {
> > +                       /*
> > +                        * Only count reserved RAM pages
> > +                        */
> > +                       if (PageReserved(mem_map+tmp))
> > +                               reservedpages++;
> 
> Please don't reference mem_map[] directly outside of #ifdef
> CONFIG_FLATMEM areas.  It is not defined for all config cases.  Please
> use pfn_to_page(), instead.  It should work in all cases where the page
> is valid.
> 
> Also, instead of keeping addr defined like you do, and comparing it
> during each run of the loop, why not just use pfn_is_nosave(), which is
> already defined?  Then, you won't even need the variable.

Thanks for the comments. Will revise and resend.

Regards,

Nigel

> -- Dave
--