diff options
| author | KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> | 2006-08-05 15:15:01 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-08-06 11:57:48 -0400 |
| commit | 58c1b5b079071d82b2f924000b7e8fb5585ce7d8 (patch) | |
| tree | 74178ceb6462b68cba9f1b144d450dafcaf01c22 /kernel | |
| parent | 0f04ab5efbca73ab366a156d96b073d2da35b158 (diff) | |
[PATCH] memory hotadd fixes: find_next_system_ram catch range fix
find_next_system_ram() is used to find available memory resource at onlining
newly added memory. This patch fixes following problem.
find_next_system_ram() cannot catch this case.
Resource: (start)-------------(end)
Section : (start)-------------(end)
Signed-off-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Keith Mannthey <kmannth@gmail.com>
Cc: Yasunori Goto <y-goto@jp.fujitsu.com>
Cc: Dave Hansen <haveblue@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/resource.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/resource.c b/kernel/resource.c index 63e879379dbd..46286434af80 100644 --- a/kernel/resource.c +++ b/kernel/resource.c | |||
| @@ -244,6 +244,7 @@ int find_next_system_ram(struct resource *res) | |||
| 244 | 244 | ||
| 245 | start = res->start; | 245 | start = res->start; |
| 246 | end = res->end; | 246 | end = res->end; |
| 247 | BUG_ON(start >= end); | ||
| 247 | 248 | ||
| 248 | read_lock(&resource_lock); | 249 | read_lock(&resource_lock); |
| 249 | for (p = iomem_resource.child; p ; p = p->sibling) { | 250 | for (p = iomem_resource.child; p ; p = p->sibling) { |
| @@ -254,7 +255,7 @@ int find_next_system_ram(struct resource *res) | |||
| 254 | p = NULL; | 255 | p = NULL; |
| 255 | break; | 256 | break; |
| 256 | } | 257 | } |
| 257 | if (p->start >= start) | 258 | if ((p->end >= start) && (p->start < end)) |
| 258 | break; | 259 | break; |
| 259 | } | 260 | } |
| 260 | read_unlock(&resource_lock); | 261 | read_unlock(&resource_lock); |
