Message ID | 20160817034323.23053-4-cyrilbur@gmail.com (mailing list archive) |
---|---|
State | Changes Requested |
Headers | show |
On Wed, 2016-08-17 at 13:43 +1000, Cyril Bur wrote: > giveup_all() causes FPU/VMX/VSX facilitities to be disabled in a facilities. > threads MSR. If this thread was transactional this should be recorded > as reclaiming/recheckpointing code will need to know. Can you expand on this? It's not clear to me how this relates to the code. Mikey > Fixes: c208505 ("powerpc: create giveup_all()") > Signed-off-by: Cyril Bur <cyrilbur@gmail.com> > --- > arch/powerpc/kernel/process.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/powerpc/kernel/process.c > b/arch/powerpc/kernel/process.c > index cdf2d20..82308fd 100644 > --- a/arch/powerpc/kernel/process.c > +++ b/arch/powerpc/kernel/process.c > @@ -445,6 +445,7 @@ void giveup_all(struct task_struct *tsk) > return; > > msr_check_and_set(msr_all_available); > + check_if_tm_restore_required(tsk); > > #ifdef CONFIG_PPC_FPU > if (usermsr & MSR_FP)
diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c index cdf2d20..82308fd 100644 --- a/arch/powerpc/kernel/process.c +++ b/arch/powerpc/kernel/process.c @@ -445,6 +445,7 @@ void giveup_all(struct task_struct *tsk) return; msr_check_and_set(msr_all_available); + check_if_tm_restore_required(tsk); #ifdef CONFIG_PPC_FPU if (usermsr & MSR_FP)
giveup_all() causes FPU/VMX/VSX facilitities to be disabled in a threads MSR. If this thread was transactional this should be recorded as reclaiming/recheckpointing code will need to know. Fixes: c208505 ("powerpc: create giveup_all()") Signed-off-by: Cyril Bur <cyrilbur@gmail.com> --- arch/powerpc/kernel/process.c | 1 + 1 file changed, 1 insertion(+)