![]() | |||||||||||||
Messages in this thread |
On Thu, 2006-05-04 at 11:46 -0400, Bob Picco wrote: > Dave Hansen wrote: [Thu May 04 2006, 11:21:06AM EDT] > > I haven't thought through it completely, but these two lines worry me: > > > > > + start = pgdat->node_start_pfn & ~((1 << (MAX_ORDER - 1)) - 1); > > > + end = start + pgdat->node_spanned_pages; > > > > Should the "end" be based off of the original "start", or the aligned > > "start"? > Yes. I failed to quilt refresh before sending. You mean end should be > end = pgdat->node_start_pfn + pgdat->node_spanned_pages before rounding > up. Yep. Looks good. -- 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: 2006-05-04 18:10 [from the cache] ©2003-2008 | |||||||||||||