Re: [PATCH] powerpc/64: Initialise thread_info for emergency stacks

2017-06-21 Thread Abdul Haleem
On Tue, 2017-06-20 at 23:58 +1000, Nicholas Piggin wrote: > Emergency stacks have their thread_info mostly uninitialised, which in > particular means garbage preempt_count values. > > Emergency stack code runs with interrupts disabled entirely, and is > used very rarely, so this has been unnoticed

Re: [PATCH] powerpc/64: Initialise thread_info for emergency stacks

2017-06-21 Thread Nicholas Piggin
On Wed, 21 Jun 2017 12:01:37 +1000 Michael Ellerman wrote: > Nicholas Piggin writes: > > > diff --git a/arch/powerpc/include/asm/thread_info.h > > b/arch/powerpc/include/asm/thread_info.h > > index a941cc6fc3e9..5995e4b2996d 100644 > > --- a/arch/powerpc/include/asm/thread_info.h > > +++ b/arc

Re: [PATCH] powerpc/64: Initialise thread_info for emergency stacks

2017-06-21 Thread Michael Ellerman
Nicholas Piggin writes: > diff --git a/arch/powerpc/include/asm/thread_info.h > b/arch/powerpc/include/asm/thread_info.h > index a941cc6fc3e9..5995e4b2996d 100644 > --- a/arch/powerpc/include/asm/thread_info.h > +++ b/arch/powerpc/include/asm/thread_info.h > @@ -62,6 +63,24 @@ struct thread_info

[PATCH] powerpc/64: Initialise thread_info for emergency stacks

2017-06-20 Thread Nicholas Piggin
Emergency stacks have their thread_info mostly uninitialised, which in particular means garbage preempt_count values. Emergency stack code runs with interrupts disabled entirely, and is used very rarely, so this has been unnoticed so far. It was found by a proposed new powerpc watchdog that takes