Re: [patch V4 part 1 14/36] x86/entry: Get rid of ist_begin/end_non_atomic()

2020-05-15 Thread Mathieu Desnoyers
- On May 15, 2020, at 5:34 AM, Thomas Gleixner t...@linutronix.de wrote: [...] > > Can you please trim your replies? Sorry, will do, Thanks, Mathieu -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com

Re: [patch V4 part 1 14/36] x86/entry: Get rid of ist_begin/end_non_atomic()

2020-05-15 Thread Thomas Gleixner
Mathieu Desnoyers writes: > - On May 5, 2020, at 9:16 AM, Thomas Gleixner t...@linutronix.de wrote: > >> This is completely overengineered and definitely not an interface which >> should be made available to anything else than this particular MCE case. > > This patch introduces a significant

Re: [patch V4 part 1 14/36] x86/entry: Get rid of ist_begin/end_non_atomic()

2020-05-13 Thread Steven Rostedt
On Wed, 13 May 2020 18:57:25 -0400 (EDT) Mathieu Desnoyers wrote: > - On May 5, 2020, at 9:16 AM, Thomas Gleixner t...@linutronix.de wrote: > > > This is completely overengineered and definitely not an interface which > > should be made available to anything else than this particular MCE cas

Re: [patch V4 part 1 14/36] x86/entry: Get rid of ist_begin/end_non_atomic()

2020-05-13 Thread Mathieu Desnoyers
- On May 5, 2020, at 9:16 AM, Thomas Gleixner t...@linutronix.de wrote: > This is completely overengineered and definitely not an interface which > should be made available to anything else than this particular MCE case. This patch introduces a significant change under the radar (not explaine

Re: [patch V4 part 1 14/36] x86/entry: Get rid of ist_begin/end_non_atomic()

2020-05-07 Thread Andy Lutomirski
On Tue, May 5, 2020 at 7:13 AM Thomas Gleixner wrote: > > This is completely overengineered and definitely not an interface which > should be made available to anything else than this particular MCE case. Sorry for the overengineering. :) --Andy

Re: [patch V4 part 1 14/36] x86/entry: Get rid of ist_begin/end_non_atomic()

2020-05-06 Thread Alexandre Chartre
On 5/5/20 3:16 PM, Thomas Gleixner wrote: This is completely overengineered and definitely not an interface which should be made available to anything else than this particular MCE case. Signed-off-by: Thomas Gleixner --- arch/x86/include/asm/traps.h |2 -- arch/x86/kernel/cpu/mce/co