lkml.org 
[lkml]   [2005]   [Oct]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] Free swap suspend from dependency on PageReserved
From
Date
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
--


-
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 00:29    [W:0.054 / U:0.176 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site