Re: [PATCH v2 1/5] mm: add numa node symlink for memory section insysfs

From: David Rientjes
Date: Tue Oct 27 2009 - 17:28:12 EST


On Tue, 27 Oct 2009, Alex Chiang wrote:

> Thank you for ACKing, David.
>
> S390 guys, I cc'ed you on this patch because I heard a rumour
> that your memory sections may belong to more than one NUMA node?
> Is that true? If so, how would you like me to handle that
> situation?
>

You're referring to how unregister_mem_sect_under_nodes() should be
handled, right? register_mem_sect_under_node() already looks supported by
your patch.

Since the unregister function includes a plural "nodes," I assume that
it's possible for hotplug to register a memory section to more than one
node. That's probably lacking on x86 currently, however, because we lack
node hotplug.

I'd suggest a similiar iteration through pfn's that the register function
does checking for multiple nodes and then removing the link from all
applicable node_devices kobj when unregistering.

Maybe one of the s390 maintainers will test that?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/