Re: [Qemu-devel] [PATCH] target/riscv: Do not allow sfence.vma from user mode

2019-04-12 Thread Palmer Dabbelt
I actually missed this. I just added it to for-next on github.com/palmer-dabbelt. Thanks for the ping! On Fri, 12 Apr 2019 14:23:42 PDT (-0700), alistai...@gmail.com wrote: On Fri, Apr 12, 2019 at 2:15 PM Jonathan Behrens wrote: Just to double check, nothing further on this is need from

Re: [Qemu-devel] [PATCH] target/riscv: Do not allow sfence.vma from user mode

2019-04-12 Thread Alistair Francis
On Fri, Apr 12, 2019 at 2:15 PM Jonathan Behrens wrote: > > Just to double check, nothing further on this is need from me, right? It is > set to be merged onto the master branch once the 4.0 release is out? Thanks for checking! Yep you don't need to do anything, Palmer will merge it in the

Re: [Qemu-devel] [PATCH] target/riscv: Do not allow sfence.vma from user mode

2019-04-12 Thread Jonathan Behrens
Just to double check, nothing further on this is need from me, right? It is set to be merged onto the master branch once the 4.0 release is out? Jonathan On Wed, Apr 3, 2019 at 7:11 PM Alistair Francis wrote: > On Mon, Apr 1, 2019 at 1:39 PM Jonathan Behrens > wrote: > > > > The 'sfence.vma'

Re: [Qemu-devel] [PATCH] target/riscv: Do not allow sfence.vma from user mode

2019-04-03 Thread Alistair Francis
On Mon, Apr 1, 2019 at 1:39 PM Jonathan Behrens wrote: > > The 'sfence.vma' instruction is privileged, and should only ever be allowed > when executing in supervisor mode or higher. > > Jonathan > > Signed-off-by: Jonathan Behrens Reviewed-by: Alistair Francis Alistair > --- >

Re: [Qemu-devel] [PATCH] target/riscv: Do not allow sfence.vma from user mode

2019-04-01 Thread Richard Henderson
On 4/2/19 2:12 AM, Jonathan Behrens wrote: > The 'sfence.vma' instruction is privileged, and should only ever be allowed > when executing in supervisor mode or higher. > > Jonathan > > Signed-off-by: Jonathan Behrens > --- > target/riscv/op_helper.c | 7 --- > 1 file changed, 4

[Qemu-devel] [PATCH] target/riscv: Do not allow sfence.vma from user mode

2019-04-01 Thread Jonathan Behrens
The 'sfence.vma' instruction is privileged, and should only ever be allowed when executing in supervisor mode or higher. Jonathan Signed-off-by: Jonathan Behrens --- target/riscv/op_helper.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/target/riscv/op_helper.c