Re: [PATCH 2/3] x86/asm: Move 'status' from thread_struct to thread_info

2018-01-28 Thread Linus Torvalds
On Sun, Jan 28, 2018 at 10:38 AM, Andy Lutomirski wrote: > > Linus suggested further changing: > > ti->status &= ~(TS_COMPAT|TS_I386_REGS_POKED); > > to: > > if (unlikely(ti->status & (TS_COMPAT|TS_I386_REGS_POKED))) > ti->status &= ~(TS_COMPAT|TS_I386_REGS_POKED); > > on the theory

Re: [PATCH 2/3] x86/asm: Move 'status' from thread_struct to thread_info

2018-01-28 Thread Ingo Molnar
* Andy Lutomirski wrote: > The TS_COMPAT bit is very hot and is accessed from code paths that > mostly also touch thread_info::flags. Move it into struct > thread_info to improve cache locality. > > The only reason it was in thread_struct is that there was a brief > period during which we didn