Re: [PATCH] powerpc/fadump: account for memory_limit while reserving memory

[Michael Ellerman]

On Wed, 27 May 2020 15:14:35 +0530, Hari Bathini wrote:
> If the memory chunk found for reserving memory overshoots the memory
> limit imposed, do not proceed with reserving memory. Default behavior
> was this until commit 140777a3d8df ("powerpc/fadump: consider reserved
> ranges while reserving memory") changed it unwittingly.

Applied to powerpc/next.

[1/1] powerpc/fadump: Account for memory_limit while reserving memory
  https://git.kernel.org/powerpc/c/9a2921e5baca1d25eb8d21f21d1e90581a6d0f68

cheers

[PATCH] powerpc/fadump: account for memory_limit while reserving memory

[Hari Bathini]

If the memory chunk found for reserving memory overshoots the memory
limit imposed, do not proceed with reserving memory. Default behavior
was this until commit 140777a3d8df ("powerpc/fadump: consider reserved
ranges while reserving memory") changed it unwittingly.

Reported-by: kbuild test robot 
Fixes: 140777a3d8df ("powerpc/fadump: consider reserved ranges while reserving 
memory")
Cc: sta...@vger.kernel.org
Signed-off-by: Hari Bathini 
---

For reference:
- https://lists.ozlabs.org/pipermail/linuxppc-dev/2020-May/211136.html


 arch/powerpc/kernel/fadump.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/kernel/fadump.c b/arch/powerpc/kernel/fadump.c
index 63aac8b..78ab9a6 100644
--- a/arch/powerpc/kernel/fadump.c
+++ b/arch/powerpc/kernel/fadump.c
@@ -603,7 +603,7 @@ int __init fadump_reserve_mem(void)
 */
base = fadump_locate_reserve_mem(base, size);
 
-   if (!base) {
+   if (!base || (base + size > mem_boundary)) {
pr_err("Failed to find memory chunk for 
reservation!\n");
goto error_out;
}