* Re: [PATCH] reboot: add reboot_panic parameter
[not found] ` <EEBA739CCF11FE49B73E1FB4690F5EE649E1B93B@shsmsx102.ccr.corp.intel.com>
@ 2018-07-02 21:30 ` Andrew Morton
2018-07-05 6:47 ` Tian, Baofeng
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Morton @ 2018-07-02 21:30 UTC (permalink / raw)
To: Tian, Baofeng; +Cc: andrew.smirnov, linux-kernel, Zhuang, Qihua
On Mon, 2 Jul 2018 02:20:59 +0000 "Tian, Baofeng" <baofeng.tian@intel.com> wrote:
> This "reboot_panic" kernel cmdline parameter allow to change the
> reboot mode in case of panic only. It use the same format as the
> "reboot" parameter.
> Example, for cf9 warm reset in case of panic: reboot_panic=p,w
> currently it only support cold/warm reset in case of panic.
Why should we do this? What are the benefits, what are the usecases,
etc?
Some user-facing documentation should clear that up. `grep -r reboot=
Documentation' will direct you to the appropriate places.
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [PATCH] reboot: add reboot_panic parameter
2018-07-02 21:30 ` [PATCH] reboot: add reboot_panic parameter Andrew Morton
@ 2018-07-05 6:47 ` Tian, Baofeng
2018-07-05 21:54 ` Andrew Morton
0 siblings, 1 reply; 3+ messages in thread
From: Tian, Baofeng @ 2018-07-05 6:47 UTC (permalink / raw)
To: Andrew Morton; +Cc: andrew.smirnov, linux-kernel, Zhuang, Qihua
Hi, Morton
On android platform, if use adb reboot, it will automatically reboot with cold reset.
cold reset will reset RAM and cause RAM lose data information, if we want to enable some debug features
like pstore, and ramdump etc, warm reset is needed to keep RAM across reset
So, we add this interface to allow people to set reboot as warm to get log data across reset.
Thanks
Tim
-----Original Message-----
From: Andrew Morton [mailto:akpm@linux-foundation.org]
Sent: Tuesday, July 3, 2018 5:31 AM
To: Tian, Baofeng <baofeng.tian@intel.com>
Cc: andrew.smirnov@gmail.com; linux-kernel@vger.kernel.org; Zhuang, Qihua <qihua.zhuang@intel.com>
Subject: Re: [PATCH] reboot: add reboot_panic parameter
On Mon, 2 Jul 2018 02:20:59 +0000 "Tian, Baofeng" <baofeng.tian@intel.com> wrote:
> This "reboot_panic" kernel cmdline parameter allow to change the
> reboot mode in case of panic only. It use the same format as the
> "reboot" parameter.
> Example, for cf9 warm reset in case of panic: reboot_panic=p,w
> currently it only support cold/warm reset in case of panic.
Why should we do this? What are the benefits, what are the usecases, etc?
Some user-facing documentation should clear that up. `grep -r reboot= Documentation' will direct you to the appropriate places.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] reboot: add reboot_panic parameter
2018-07-05 6:47 ` Tian, Baofeng
@ 2018-07-05 21:54 ` Andrew Morton
0 siblings, 0 replies; 3+ messages in thread
From: Andrew Morton @ 2018-07-05 21:54 UTC (permalink / raw)
To: Tian, Baofeng; +Cc: andrew.smirnov, linux-kernel, Zhuang, Qihua
On Thu, 5 Jul 2018 06:47:37 +0000 "Tian, Baofeng" <baofeng.tian@intel.com> wrote:
> -----Original Message-----
> From: Andrew Morton [mailto:akpm@linux-foundation.org]
> Sent: Tuesday, July 3, 2018 5:31 AM
> To: Tian, Baofeng <baofeng.tian@intel.com>
> Cc: andrew.smirnov@gmail.com; linux-kernel@vger.kernel.org; Zhuang, Qihua <qihua.zhuang@intel.com>
> Subject: Re: [PATCH] reboot: add reboot_panic parameter
>
> On Mon, 2 Jul 2018 02:20:59 +0000 "Tian, Baofeng" <baofeng.tian@intel.com> wrote:
>
> > This "reboot_panic" kernel cmdline parameter allow to change the
> > reboot mode in case of panic only. It use the same format as the
> > "reboot" parameter.
> > Example, for cf9 warm reset in case of panic: reboot_panic=p,w
> > currently it only support cold/warm reset in case of panic.
>
> Why should we do this? What are the benefits, what are the usecases, etc?
>
> Some user-facing documentation should clear that up. `grep -r reboot= Documentation' will direct you to the appropriate places.
>
(top-posting repaired. Please don't top-post)
> Hi, Morton
"Andrew" :)
> On android platform, if use adb reboot, it will automatically reboot with cold reset.
> cold reset will reset RAM and cause RAM lose data information, if we want to enable some debug features
> like pstore, and ramdump etc, warm reset is needed to keep RAM across reset
>
> So, we add this interface to allow people to set reboot as warm to get log data across reset.
>
OK, sounds good. Please add this info to the changelog, add the user
documentation and resend.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-07-05 21:54 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <0038B620BAC72941BCBF48AEB892976C02CADD8C@shsmsx102.ccr.corp.intel.com>
[not found] ` <EEBA739CCF11FE49B73E1FB4690F5EE649E1B93B@shsmsx102.ccr.corp.intel.com>
2018-07-02 21:30 ` [PATCH] reboot: add reboot_panic parameter Andrew Morton
2018-07-05 6:47 ` Tian, Baofeng
2018-07-05 21:54 ` Andrew Morton
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.