lkml.org 
[lkml]   [2005]   [Oct]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH]: Clean up of __alloc_pages
On Sat, 1 Oct 2005, Seth, Rohit wrote:

> - goto zone_reclaim_retry;
> - }
> + if (order == 0) {
> + for (i = 0; (z = zones[i]) != NULL; i++) {
> + page = buffered_rmqueue(z, 0, gfp_mask, 0);
> + if (page)
> + goto got_pg;
> }
> -

This is checking all zones for pages on the pcp before going the more
expensive route?

Seems that this removes the logic intended to prefer local
allocations over remote pages present in the existing alloc_pages? There
is the danger that this modification will lead to the allocation of remote
pages even if local pages are available. Thus reducing performance.

I would suggest to just check the first zone's pcp instead of all zones.

-
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-03 17:36    [W:0.052 / U:0.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site