On Mon, Mar 25, 2019 at 12:44 AM Gary Guo <g...@garyguo.net> wrote: > > > > > -----Original Message----- > > From: linux-riscv <linux-riscv-boun...@lists.infradead.org> On Behalf Of > > Anup > > Patel > > Sent: Sunday, March 24, 2019 11:08 > > To: Palmer Dabbelt <pal...@sifive.com>; Albert Ou <a...@eecs.berkeley.edu> > > Cc: Anup Patel <anup.pa...@wdc.com>; linux-kernel@vger.kernel.org; Mike > > Rapoport <r...@linux.ibm.com>; Christoph Hellwig <h...@infradead.org>; Atish > > Patra <atish.pa...@wdc.com>; Paul Walmsley <paul.walms...@sifive.com>; > > linux-ri...@lists.infradead.org; Christoph Hellwig <h...@lst.de> > > Subject: [PATCH] RISC-V: Always compile mm/init.c with cmodel=medany > > > > diff --git a/arch/riscv/mm/Makefile b/arch/riscv/mm/Makefile > > index eb22ab49b3e0..7307609d405b 100644 > > --- a/arch/riscv/mm/Makefile > > +++ b/arch/riscv/mm/Makefile > > @@ -3,3 +3,5 @@ obj-y += fault.o > > obj-y += extable.o > > obj-y += ioremap.o > > obj-y += cacheflush.o > > + > > +CFLAGS_init.o := -mcmodel=medany > > I think you should also remove CFLAGS_setup.o from kernel/Makefile, as I > think it was there solely for the correctness of setup_vm. As setup_vm is now > in mm/init.o, the old CFLAGS probably isn't needed and should be removed in > the same patch. Disclaimer: I haven't tested it.
Sure, I will remove it in v2 patch. Thanks for catching. Regards, Anup
