Nick Piggin [off-list ref] wrote:

Hi Nikita,

Just having a look at your patch. I think maybe this array
should be in a seperate cacheline per node?

+/* dummy pages used to scan active lists */
+static struct page scan_pages[MAX_NUMNODES][MAX_NR_ZONES];
+

You really want it a member of `struct zone', don't you?

That'll cause include dependency hell, so maybe a page* in struct zone.


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"aart@kvack.org"> aart@kvack.org </a>