diff options
author | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2010-08-04 00:17:17 -0400 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2010-08-04 00:17:17 -0400 |
commit | f1c2c19c498e27de48bf0dc4221e6e31b1823169 (patch) | |
tree | 61c1b8b9ab32cd76128e33186148597a8b8b39b1 /mm/memblock.c | |
parent | 3a09b1be53d23df780a0cd0e4087a05e2ca4a00c (diff) |
memblock: Fix memblock_is_region_reserved() to return a boolean
All callers expect a boolean result which is true if the region
overlaps a reserved region. However, the implementation actually
returns -1 if there is no overlap, and a region index (0 based)
if there is.
Make it behave as callers (and common sense) expect.
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Diffstat (limited to 'mm/memblock.c')
-rw-r--r-- | mm/memblock.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/memblock.c b/mm/memblock.c index 3024eb30fc2..43840b305ec 100644 --- a/mm/memblock.c +++ b/mm/memblock.c | |||
@@ -504,7 +504,7 @@ int __init memblock_is_reserved(u64 addr) | |||
504 | 504 | ||
505 | int memblock_is_region_reserved(u64 base, u64 size) | 505 | int memblock_is_region_reserved(u64 base, u64 size) |
506 | { | 506 | { |
507 | return memblock_overlaps_region(&memblock.reserved, base, size); | 507 | return memblock_overlaps_region(&memblock.reserved, base, size) >= 0; |
508 | } | 508 | } |
509 | 509 | ||
510 | /* | 510 | /* |