On Thu, Aug 06, 2020 at 09:04:49PM +0200, Luc Van Oostenryck wrote: > On Thu, Aug 06, 2020 at 06:07:24AM +0900, Stafford Horne wrote: > > --- > > arch/openrisc/kernel/signal.c | 14 +++++++------- > > 1 file changed, 7 insertions(+), 7 deletions(-) > > > > diff --git a/arch/openrisc/kernel/signal.c b/arch/openrisc/kernel/signal.c > > index 4f0754874d78..7ce0728412f6 100644 > > --- a/arch/openrisc/kernel/signal.c > > +++ b/arch/openrisc/kernel/signal.c > > @@ -76,7 +76,7 @@ asmlinkage long _sys_rt_sigreturn(struct pt_regs *regs) > > * then frame should be dword aligned here. If it's > > * not, then the user is trying to mess with us. > > */ > > - if (((long)frame) & 3) > > + if (((__force unsigned long)frame) & 3) > > goto badframe; > > Same as patch 6, the __force is not needed.
Thanks, I thought this was complaining before, I tested now and there is no problem so I must have been mixed up with something else. -Stafford