diff mbox series

MIPS: Loongson64: Fix up reserving kernel memory range

Message ID 1606299062-7427-1-git-send-email-hejinyang@loongson.cn (mailing list archive)
State Accepted
Commit 1dac4585f585d3aa1a5af4821128ea2642700e48
Headers show
Series MIPS: Loongson64: Fix up reserving kernel memory range | expand

Commit Message

Jinyang He Nov. 25, 2020, 10:11 a.m. UTC
Reserve memory from &_text to &_end. Otherwise if kernel address
was modified, the memory range of start_pfn to kernel_start_pfn
would be reserved. Then we could not use this range.

Signed-off-by: Jinyang He <hejinyang@loongson.cn>
---
 arch/mips/loongson64/numa.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

Comments

Thomas Bogendoerfer Nov. 27, 2020, 10:34 a.m. UTC | #1
On Wed, Nov 25, 2020 at 06:11:02PM +0800, Jinyang He wrote:
> Reserve memory from &_text to &_end. Otherwise if kernel address
> was modified, the memory range of start_pfn to kernel_start_pfn
> would be reserved. Then we could not use this range.
> 
> Signed-off-by: Jinyang He <hejinyang@loongson.cn>
> ---
>  arch/mips/loongson64/numa.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)

applied to mips-next.

Thomas.
Jiaxun Yang Dec. 3, 2020, 9:30 a.m. UTC | #2
于 2020年11月25日 GMT+08:00 下午6:11:02, Jinyang He <hejinyang@loongson.cn> 写到:
>Reserve memory from &_text to &_end. Otherwise if kernel address
>was modified, the memory range of start_pfn to kernel_start_pfn
>would be reserved. Then we could not use this range.

I suspect you meant to make use lower memory after relocation?

Reviewed-by: Jiaxun Yang <jiaxun.yang@flygoat.com>

Thanks.

- Jiaxun

>
>Signed-off-by: Jinyang He <hejinyang@loongson.cn>
>---
> arch/mips/loongson64/numa.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
>diff --git a/arch/mips/loongson64/numa.c b/arch/mips/loongson64/numa.c
>index 509b360..c6f0c48 100644
>--- a/arch/mips/loongson64/numa.c
>+++ b/arch/mips/loongson64/numa.c
>@@ -151,6 +151,9 @@ static void __init node_mem_init(unsigned int node)
> 	NODE_DATA(node)->node_spanned_pages = end_pfn - start_pfn;
> 
> 	if (node == 0) {
>+		/* kernel start address */
>+		unsigned long kernel_start_pfn = PFN_DOWN(__pa_symbol(&_text));
>+
> 		/* kernel end address */
> 		unsigned long kernel_end_pfn = PFN_UP(__pa_symbol(&_end));
> 
>@@ -158,8 +161,8 @@ static void __init node_mem_init(unsigned int node)
> 		max_low_pfn = end_pfn;
> 
> 		/* Reserve the kernel text/data/bss */
>-		memblock_reserve(start_pfn << PAGE_SHIFT,
>-				 ((kernel_end_pfn - start_pfn) << PAGE_SHIFT));
>+		memblock_reserve(kernel_start_pfn << PAGE_SHIFT,
>+				 ((kernel_end_pfn - kernel_start_pfn) << PAGE_SHIFT));
> 
> 		/* Reserve 0xfe000000~0xffffffff for RS780E integrated GPU */
> 		if (node_end_pfn(0) >= (0xffffffff >> PAGE_SHIFT))
diff mbox series

Patch

diff --git a/arch/mips/loongson64/numa.c b/arch/mips/loongson64/numa.c
index 509b360..c6f0c48 100644
--- a/arch/mips/loongson64/numa.c
+++ b/arch/mips/loongson64/numa.c
@@ -151,6 +151,9 @@  static void __init node_mem_init(unsigned int node)
 	NODE_DATA(node)->node_spanned_pages = end_pfn - start_pfn;
 
 	if (node == 0) {
+		/* kernel start address */
+		unsigned long kernel_start_pfn = PFN_DOWN(__pa_symbol(&_text));
+
 		/* kernel end address */
 		unsigned long kernel_end_pfn = PFN_UP(__pa_symbol(&_end));
 
@@ -158,8 +161,8 @@  static void __init node_mem_init(unsigned int node)
 		max_low_pfn = end_pfn;
 
 		/* Reserve the kernel text/data/bss */
-		memblock_reserve(start_pfn << PAGE_SHIFT,
-				 ((kernel_end_pfn - start_pfn) << PAGE_SHIFT));
+		memblock_reserve(kernel_start_pfn << PAGE_SHIFT,
+				 ((kernel_end_pfn - kernel_start_pfn) << PAGE_SHIFT));
 
 		/* Reserve 0xfe000000~0xffffffff for RS780E integrated GPU */
 		if (node_end_pfn(0) >= (0xffffffff >> PAGE_SHIFT))