On 03/27/2015 07:29 AM, Geoff Levand wrote: > On Thu, 2015-03-26 at 17:28 +0900, AKASHI Takahiro wrote: >> @@ -64,7 +65,11 @@ void soft_restart(unsigned long addr) >> setup_mm_for_reboot(); >> >> cpu_soft_restart(virt_to_phys(cpu_reset), >> - is_hyp_mode_available(), addr); >> +#ifdef CONFIG_KEXEC >> + !in_crash_kexec && >> +#endif > > Why not define in_crash_kexec without condition on CONFIG_KEXEC, say > here in process.c and then avoid these preprocessor conditionals. Well, I thought of that, but as its name suggested, the variable should be basically part of kdump code. Moreover, I suspect that some one might reject a whole idea of patch #3. -Takahiro AKASHI >> + is_hyp_mode_available(), >> + addr); >> >> /* Should never get here */ >> BUG(); > > >
