Re: [PATCH] riscv: Add sfence.vma after page table changed

2020-09-18 Thread Palmer Dabbelt
On Mon, 14 Sep 2020 20:58:13 PDT (-0700), greentime...@sifive.com wrote: Palmer Dabbelt 於 2020年8月5日 週三 上午10:03寫道: On Mon, 03 Aug 2020 20:29:32 PDT (-0700), a...@brainfault.org wrote: > On Tue, Aug 4, 2020 at 8:32 AM Greentime Hu wrote: >> >> This patch addes local_flush_tlb_page(addr) to use

Re: [PATCH] riscv: Add sfence.vma after page table changed

2020-09-14 Thread Greentime Hu
Palmer Dabbelt 於 2020年8月5日 週三 上午10:03寫道: > > On Mon, 03 Aug 2020 20:29:32 PDT (-0700), a...@brainfault.org wrote: > > On Tue, Aug 4, 2020 at 8:32 AM Greentime Hu wrote: > >> > >> This patch addes local_flush_tlb_page(addr) to use sfence.vma after the > > > > s/addes/adds > > > >> page table chang

Re: [PATCH] riscv: Add sfence.vma after page table changed

2020-08-05 Thread Will Deacon
On Tue, Aug 04, 2020 at 07:03:06PM -0700, Palmer Dabbelt wrote: > > On Tue, Aug 4, 2020 at 8:32 AM Greentime Hu wrote: > > > diff --git a/arch/riscv/mm/init.c b/arch/riscv/mm/init.c > > > index f4adb3684f3d..29b0f7108054 100644 > > > --- a/arch/riscv/mm/init.c > > > +++ b/arch/riscv/mm/init.c > >

Re: [PATCH] riscv: Add sfence.vma after page table changed

2020-08-04 Thread Palmer Dabbelt
On Mon, 03 Aug 2020 20:29:32 PDT (-0700), a...@brainfault.org wrote: On Tue, Aug 4, 2020 at 8:32 AM Greentime Hu wrote: This patch addes local_flush_tlb_page(addr) to use sfence.vma after the s/addes/adds page table changed. That address will be used immediately in memset(nextp, 0, PAGE_SI

Re: [PATCH] riscv: Add sfence.vma after page table changed

2020-08-03 Thread Anup Patel
On Tue, Aug 4, 2020 at 8:32 AM Greentime Hu wrote: > > This patch addes local_flush_tlb_page(addr) to use sfence.vma after the s/addes/adds > page table changed. That address will be used immediately in > memset(nextp, 0, PAGE_SIZE) to cause this issue so we should add the > sfence.vma before we

[PATCH] riscv: Add sfence.vma after page table changed

2020-08-03 Thread Greentime Hu
This patch addes local_flush_tlb_page(addr) to use sfence.vma after the page table changed. That address will be used immediately in memset(nextp, 0, PAGE_SIZE) to cause this issue so we should add the sfence.vma before we use it. Fixes: f2c17aabc917 ("RISC-V: Implement compile-time fixed mappings