diff options
author | Wen Congyang <wency@cn.fujitsu.com> | 2012-07-11 17:02:31 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-11 19:04:46 -0400 |
commit | 41b9e2d7ec3f618fd076cb3466edd0a8ebabae5a (patch) | |
tree | 5f4a6d9062f096cbbe839f5dbe14638c6e2816ff /mm | |
parent | 213ab3f9fcfc1ba471906a4f2c24d7b21370c859 (diff) |
mm/memory_hotplug.c: release memory resources if hotadd_new_pgdat() fails
We should goto error to release memory resource if hotadd_new_pgdat()
failed.
Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
Cc: Yasuaki ISIMATU <isimatu.yasuaki@jp.fujitsu.com>
Acked-by: David Rientjes <rientjes@google.com>
Cc: Len Brown <lenb@kernel.org>
Cc: "Brown, Len" <len.brown@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/memory_hotplug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c index 0d7e3ec8e0f3..427bb291dd0f 100644 --- a/mm/memory_hotplug.c +++ b/mm/memory_hotplug.c | |||
@@ -618,7 +618,7 @@ int __ref add_memory(int nid, u64 start, u64 size) | |||
618 | pgdat = hotadd_new_pgdat(nid, start); | 618 | pgdat = hotadd_new_pgdat(nid, start); |
619 | ret = -ENOMEM; | 619 | ret = -ENOMEM; |
620 | if (!pgdat) | 620 | if (!pgdat) |
621 | goto out; | 621 | goto error; |
622 | new_pgdat = 1; | 622 | new_pgdat = 1; |
623 | } | 623 | } |
624 | 624 | ||