| Message ID | 20190520063326.26083-1-hch@lst.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | riscv: remove free_initrd_mem | expand |
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
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>
Paul, are you going to pick this one up?
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
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)
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(-)