diff options
author | Hugh Dickins <hugh@veritas.com> | 2007-06-08 16:46:46 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-06-08 20:23:32 -0400 |
commit | a210906c1b791af1434323f69427286039c9c8b9 (patch) | |
tree | 17f3deeda2c7c6972b9a72113a14571c2cabaae7 /mm/shmem.c | |
parent | 9739ed1c8eafd554788d8e2ba09847b5718a34e7 (diff) |
mount -t tmpfs -o mpol=: check nodes online
Randy Dunlap reports that a tmpfs, mounted with NUMA mpol= specifying an
offline node, crashes as soon as data is allocated upon it. Now restrict it
to online nodes, where before it restricted to MAX_NUMNODES.
Signed-off-by: Hugh Dickins <hugh@veritas.com>
Cc: Robin Holt <holt@sgi.com>
Cc: Christoph Lameter <clameter@sgi.com>
Cc: Andi Kleen <ak@suse.de>
Tested-and-acked-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/shmem.c')
-rw-r--r-- | mm/shmem.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/mm/shmem.c b/mm/shmem.c index e537317bec4d..b6aae2b33393 100644 --- a/mm/shmem.c +++ b/mm/shmem.c | |||
@@ -967,6 +967,8 @@ static inline int shmem_parse_mpol(char *value, int *policy, nodemask_t *policy_ | |||
967 | *nodelist++ = '\0'; | 967 | *nodelist++ = '\0'; |
968 | if (nodelist_parse(nodelist, *policy_nodes)) | 968 | if (nodelist_parse(nodelist, *policy_nodes)) |
969 | goto out; | 969 | goto out; |
970 | if (!nodes_subset(*policy_nodes, node_online_map)) | ||
971 | goto out; | ||
970 | } | 972 | } |
971 | if (!strcmp(value, "default")) { | 973 | if (!strcmp(value, "default")) { |
972 | *policy = MPOL_DEFAULT; | 974 | *policy = MPOL_DEFAULT; |