On Mon, Mar 25, 2019 at 12:44 AM Gary Guo <[email protected]> wrote: > > > > > -----Original Message----- > > From: linux-riscv <[email protected]> On Behalf Of > > Anup > > Patel > > Sent: Sunday, March 24, 2019 11:08 > > To: Palmer Dabbelt <[email protected]>; Albert Ou <[email protected]> > > Cc: Anup Patel <[email protected]>; [email protected]; Mike > > Rapoport <[email protected]>; Christoph Hellwig <[email protected]>; Atish > > Patra <[email protected]>; Paul Walmsley <[email protected]>; > > [email protected]; Christoph Hellwig <[email protected]> > > 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
