* [PATCH] riscv: remove free_initrd_mem
@ 2019-05-20 6:33 Christoph Hellwig
2019-05-20 8:04 ` Anup Patel
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Christoph Hellwig @ 2019-05-20 6:33 UTC (permalink / raw)
To: linux-riscv
The RISC-V free_initrd_mem is identical to the default one, except
that it doesn't poison the freed memory. Remove it so that the
default implementations gets used instead.
Signed-off-by: Christoph Hellwig <hch@lst.de>
---
arch/riscv/mm/init.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/arch/riscv/mm/init.c b/arch/riscv/mm/init.c
index 8bf6f9c2d48c..b2341979d3eb 100644
--- a/arch/riscv/mm/init.c
+++ b/arch/riscv/mm/init.c
@@ -92,11 +92,6 @@ static void __init setup_initrd(void)
initrd_start = 0;
initrd_end = 0;
}
-
-void __init free_initrd_mem(unsigned long start, unsigned long end)
-{
- free_reserved_area((void *)start, (void *)end, -1, "initrd");
-}
#endif /* CONFIG_BLK_DEV_INITRD */
void __init setup_bootmem(void)
--
2.20.1
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] riscv: remove free_initrd_mem
2019-05-20 6:33 [PATCH] riscv: remove free_initrd_mem Christoph Hellwig
@ 2019-05-20 8:04 ` Anup Patel
2019-05-29 21:13 ` Palmer Dabbelt
2019-07-04 9:51 ` Paul Walmsley
2 siblings, 0 replies; 5+ messages in thread
From: Anup Patel @ 2019-05-20 8:04 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: linux-riscv
On Mon, May 20, 2019 at 12:04 PM Christoph Hellwig <hch@lst.de> wrote:
>
> The RISC-V free_initrd_mem is identical to the default one, except
> that it doesn't poison the freed memory. Remove it so that the
> default implementations gets used instead.
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
> arch/riscv/mm/init.c | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/arch/riscv/mm/init.c b/arch/riscv/mm/init.c
> index 8bf6f9c2d48c..b2341979d3eb 100644
> --- a/arch/riscv/mm/init.c
> +++ b/arch/riscv/mm/init.c
> @@ -92,11 +92,6 @@ static void __init setup_initrd(void)
> initrd_start = 0;
> initrd_end = 0;
> }
> -
> -void __init free_initrd_mem(unsigned long start, unsigned long end)
> -{
> - free_reserved_area((void *)start, (void *)end, -1, "initrd");
> -}
> #endif /* CONFIG_BLK_DEV_INITRD */
>
> void __init setup_bootmem(void)
> --
> 2.20.1
>
>
> _______________________________________________
> linux-riscv mailing list
> linux-riscv@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv
LGTM.
Reviewed-by: Anup Patel <anup@brainfault.org>
Regards,
Anup
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] riscv: remove free_initrd_mem
2019-05-20 6:33 [PATCH] riscv: remove free_initrd_mem Christoph Hellwig
2019-05-20 8:04 ` Anup Patel
@ 2019-05-29 21:13 ` Palmer Dabbelt
2019-07-03 13:01 ` Christoph Hellwig
2019-07-04 9:51 ` Paul Walmsley
2 siblings, 1 reply; 5+ messages in thread
From: Palmer Dabbelt @ 2019-05-29 21:13 UTC (permalink / raw)
To: Christoph Hellwig, Paul Walmsley; +Cc: linux-riscv
On Sun, 19 May 2019 23:33:26 PDT (-0700), Christoph Hellwig wrote:
> The RISC-V free_initrd_mem is identical to the default one, except
> that it doesn't poison the freed memory. Remove it so that the
> default implementations gets used instead.
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
> arch/riscv/mm/init.c | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/arch/riscv/mm/init.c b/arch/riscv/mm/init.c
> index 8bf6f9c2d48c..b2341979d3eb 100644
> --- a/arch/riscv/mm/init.c
> +++ b/arch/riscv/mm/init.c
> @@ -92,11 +92,6 @@ static void __init setup_initrd(void)
> initrd_start = 0;
> initrd_end = 0;
> }
> -
> -void __init free_initrd_mem(unsigned long start, unsigned long end)
> -{
> - free_reserved_area((void *)start, (void *)end, -1, "initrd");
> -}
> #endif /* CONFIG_BLK_DEV_INITRD */
>
> void __init setup_bootmem(void)
Reviewed-by: Palmer Dabbelt <palmer@sifive.com>
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] riscv: remove free_initrd_mem
2019-05-29 21:13 ` Palmer Dabbelt
@ 2019-07-03 13:01 ` Christoph Hellwig
0 siblings, 0 replies; 5+ messages in thread
From: Christoph Hellwig @ 2019-07-03 13:01 UTC (permalink / raw)
To: Palmer Dabbelt; +Cc: linux-riscv, Christoph Hellwig, Paul Walmsley
Paul, are you going to pick this one up?
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] riscv: remove free_initrd_mem
2019-05-20 6:33 [PATCH] riscv: remove free_initrd_mem Christoph Hellwig
2019-05-20 8:04 ` Anup Patel
2019-05-29 21:13 ` Palmer Dabbelt
@ 2019-07-04 9:51 ` Paul Walmsley
2 siblings, 0 replies; 5+ messages in thread
From: Paul Walmsley @ 2019-07-04 9:51 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: linux-riscv
On Mon, 20 May 2019, Christoph Hellwig wrote:
> The RISC-V free_initrd_mem is identical to the default one, except
> that it doesn't poison the freed memory. Remove it so that the
> default implementations gets used instead.
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
Thanks Christoph, queued for v5.3 with Anup and Palmer's Reviewed-by:s.
- Paul
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-07-04 9:51 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-20 6:33 [PATCH] riscv: remove free_initrd_mem Christoph Hellwig
2019-05-20 8:04 ` Anup Patel
2019-05-29 21:13 ` Palmer Dabbelt
2019-07-03 13:01 ` Christoph Hellwig
2019-07-04 9:51 ` Paul Walmsley
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.