On Thu, 2025-07-31 at 11:05 +0530, Shivendra Pratap wrote: > > > On 7/30/2025 8:53 PM, André Draszik wrote: > > More importantly, if e.g. an OOPS / panic happens after the reboot > > notifier has run (and set vendor_reset.valid because a reboot mode > > was requested), a panic handler changing reboot_mode to warm to > > retain RAM contents will have no effect, because the the original > > code above making those distinctions can not be reached anymore. > > > > Above scenario with OOPS / panic after reboot notifier could e.g. > > happen as part of device_shutdown() - see kernel_shutdown_prepare() > > We can handle the panic path by adding a panic_notifier in psci > and make vendor_reset.valid = false. Do you think adding this can clear > the panic scenario above or there can still be some leak? I think that would work. You then can't convey the reboot command, but at that stage (panic/oops) it probably doesn't matter anymore, it only cares about the crash handling which probably is enough. Cheers, Andre'