From: "Mike Rapoport (Microsoft)" <[email protected]>

free_reserved_area() expects end parameter to point to the first address
after the area, but reserve_mem_release_by_name() passes it the last
address inside the area.

Remove subtraction of one in calculation of the area end.

Fixes: 74e2498ccf7b ("mm/memblock: Add reserved memory release function")
Signed-off-by: Mike Rapoport (Microsoft) <[email protected]>
---
 mm/memblock.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mm/memblock.c b/mm/memblock.c
index b3ddfdec7a80..d4a02f1750e9 100644
--- a/mm/memblock.c
+++ b/mm/memblock.c
@@ -2434,7 +2434,7 @@ int reserve_mem_release_by_name(const char *name)
                return 0;
 
        start = phys_to_virt(map->start);
-       end = start + map->size - 1;
+       end = start + map->size;
        snprintf(buf, sizeof(buf), "reserve_mem:%s", name);
        free_reserved_area(start, end, 0, buf);
        map->size = 0;
-- 
2.53.0


Reply via email to