Hi Sergey, On Wed, Dec 05, 2018 at 11:26:54AM +0900, Sergey Senozhatsky wrote: > On (12/04/18 23:49), Feng Tang wrote: > > This should be able to achieve the same goal. > > > > One thing I can think of is what mentioned by Sergey that some sysrq > > handler may want to print out something, but it should mostly be > > covered by 2 other panic debug print patches, which will print out > > task/mem/timer/lock/ftrace info runtime on demand. > > Well, not all sysrq handlers just printk stuff; some do sane things, > like emergency sync, umount, etc. Yes, I understand. I said that by assuming we will keep the irq enabled and use the panic_blinking to control the flag, as suggested by Petr and you. With irq enabled, these sync, umount will be completed as usual. Btw, just FYI, I just tried the sysrq (using minicom CTL + A + F + 'magic key'), it works with system is running, but failed after I trigger a panic, I will check more though I'm not very familiar with sysrq yet. Thanks, Feng
