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

Dave Hansen
2005-10-05T22:03:38Z

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.

-- Dave