Re: [PATCH RFC] powerpc/powernv: sysfs entry to force full IPL reboot

2016-12-22 Thread Stewart Smith
Andrew Donnellan writes: > skiboot now supports "fast reboot", a reboot procedure where skiboot > reinitialises hardware and loads a new kernel without re-IPLing the > machine. At present, fast reboot support is still experimental and is not > enabled by default, however it is intended that it wil

Re: [PATCH RFC] powerpc/powernv: sysfs entry to force full IPL reboot

2016-12-22 Thread Stewart Smith
Michael Ellerman writes: > Andrew Donnellan writes: > >> On 23/11/16 12:37, Andrew Donnellan wrote: There's existing logic in kernel/reboot.c to handle a reboot= command line parameter, which can set the reboot_mode, so my preference would be that we use that. Currently w

Re: [PATCH RFC] powerpc/powernv: sysfs entry to force full IPL reboot

2016-11-22 Thread Michael Ellerman
Andrew Donnellan writes: > On 23/11/16 12:37, Andrew Donnellan wrote: >>> There's existing logic in kernel/reboot.c to handle a reboot= command >>> line parameter, which can set the reboot_mode, so my preference would be >>> that we use that. >>> >>> Currently we completely ignore the reboot_mode

Re: [PATCH RFC] powerpc/powernv: sysfs entry to force full IPL reboot

2016-11-22 Thread Andrew Donnellan
On 23/11/16 12:37, Andrew Donnellan wrote: There's existing logic in kernel/reboot.c to handle a reboot= command line parameter, which can set the reboot_mode, so my preference would be that we use that. Currently we completely ignore the reboot_mode, so there's no backward compatibility issue.

Re: [PATCH RFC] powerpc/powernv: sysfs entry to force full IPL reboot

2016-11-22 Thread Andrew Donnellan
On 22/11/16 21:27, Michael Ellerman wrote: Can I suggest: - Fix OPAL - Fix OPAL - Fix OPAL ... Oh OK, it's firmware :D Yep :D There's existing logic in kernel/reboot.c to handle a reboot= command line parameter, which can set the reboot_mode, so my preference would be that we use that.

Re: [PATCH RFC] powerpc/powernv: sysfs entry to force full IPL reboot

2016-11-22 Thread Michael Ellerman
Andrew Donnellan writes: > skiboot now supports "fast reboot", a reboot procedure where skiboot > reinitialises hardware and loads a new kernel without re-IPLing the > machine. At present, fast reboot support is still experimental and is not > enabled by default, however it is intended that it wi