Re: [PATCH 2/2] riscv: Make __fstate_clean() can work correctly.

2019-08-13 Thread Vincent Chen
On Mon, Aug 12, 2019 at 10:59 PM Christoph Hellwig wrote: > > Maybe s/can //g in the subject? > > > + regs->sstatus = (regs->sstatus & ~(SR_FS)) | SR_FS_CLEAN; > > No need for the inner braces here either. OK, I will remove them. > > Otherwise: > > Reviewed-by: Christoph Hellwig Thanks for

Re: [PATCH 2/2] riscv: Make __fstate_clean() can work correctly.

2019-08-13 Thread Vincent Chen
On Thu, Aug 8, 2019 at 6:17 PM Anup Patel wrote: > > On Thu, Aug 8, 2019 at 1:30 PM Vincent Chen wrote: > > > > Make the __fstate_clean() function can correctly set the > > state of sstatus.FS in pt_regs to SR_FS_CLEAN. > > > > Tested on both QEMU and HiFive Unleashed using BBL + Linux. > > > > S

Re: [PATCH 2/2] riscv: Make __fstate_clean() can work correctly.

2019-08-12 Thread Christoph Hellwig
Maybe s/can //g in the subject? > + regs->sstatus = (regs->sstatus & ~(SR_FS)) | SR_FS_CLEAN; No need for the inner braces here either. Otherwise: Reviewed-by: Christoph Hellwig

Re: [PATCH 2/2] riscv: Make __fstate_clean() can work correctly.

2019-08-08 Thread Anup Patel
On Thu, Aug 8, 2019 at 1:30 PM Vincent Chen wrote: > > Make the __fstate_clean() function can correctly set the > state of sstatus.FS in pt_regs to SR_FS_CLEAN. > > Tested on both QEMU and HiFive Unleashed using BBL + Linux. > > Signed-off-by: Vincent Chen > --- > arch/riscv/include/asm/switch_t

[PATCH 2/2] riscv: Make __fstate_clean() can work correctly.

2019-08-08 Thread Vincent Chen
Make the __fstate_clean() function can correctly set the state of sstatus.FS in pt_regs to SR_FS_CLEAN. Tested on both QEMU and HiFive Unleashed using BBL + Linux. Signed-off-by: Vincent Chen --- arch/riscv/include/asm/switch_to.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --gi