diff options
| author | Benjamin Herrenschmidt | 2010-08-04 06:17:17 +0200 |
|---|---|---|
| committer | Benjamin Herrenschmidt | 2010-08-09 03:21:38 +0200 |
| commit | 2ed9aae0facd603f9f9b8347d1d7e9de23bde286 (patch) | |
| tree | ddecb718dcbe08e1491dee0df45ef0ec3f488309 /mm | |
| parent | powerpc: Trim defconfigs (diff) | |
| download | kernel-qcow2-linux-2ed9aae0facd603f9f9b8347d1d7e9de23bde286.tar.gz kernel-qcow2-linux-2ed9aae0facd603f9f9b8347d1d7e9de23bde286.tar.xz kernel-qcow2-linux-2ed9aae0facd603f9f9b8347d1d7e9de23bde286.zip | |
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')
| -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 3024eb30fc27..43840b305ecb 100644 --- a/mm/memblock.c +++ b/mm/memblock.c @@ -504,7 +504,7 @@ int __init memblock_is_reserved(u64 addr) int memblock_is_region_reserved(u64 base, u64 size) { - return memblock_overlaps_region(&memblock.reserved, base, size); + return memblock_overlaps_region(&memblock.reserved, base, size) >= 0; } /* |