* Re: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
@ 2011-07-19 18:47 ` Vivek Goyal
0 siblings, 0 replies; 11+ messages in thread
From: Vivek Goyal @ 2011-07-19 18:47 UTC (permalink / raw)
To: Seiji Aguchi
Cc: kexec@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-mtd@lists.infradead.org, Eric W. Biederman, KOSAKI Motohiro,
Americo Wang, Matthew Garrett, tony.luck@intel.com, Andrew Morton,
Jarod Wilson, hpa@zytor.com, dzickus@redhat.com,
dle-develop@lists.sourceforge.net, Satoru Moriya
On Tue, Jul 19, 2011 at 02:23:26PM -0400, Seiji Aguchi wrote:
> Hi,
>
> [Upstream status]
> Discussion about kmsg_dump() in kdump path:
> - Eric and Vivek are worried about reliability of existing kmsg_dump().
> - Especially, Vivek would like to remove a RCU function call chain in kdump path
> which kernel modules can register their function calls freely.
>
> Discussion about pstore in nmi_hander.
> - Don Zickus found an issue of pstore in nmi_handler due to its mutex_lock.
You did not answer my questions in the last posting mail thread and gone
ahead with the new posting. How are we supposed to discuss something. This
has been a problem on this mail thread since the beginneing. There is
little open discussion.
So if you want to make any progress in this direction, what will help
is open discussion.
Locking is going to be a problem. So atleast we can remove kmsg_dump()
infrastructure from reboot path. Why can't we log something in user
space when user initiates a reboot, let it get logged in /var/log/messages
and then umount the file root and go ahead with reboot. Why does kernel
need to capture that info in NVRAM.
If we can get rid of all the logging thing on reboot path, then at least
it does not need to be lock protected with crash path.
Thanks
Vivek
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
@ 2011-07-19 18:47 ` Vivek Goyal
0 siblings, 0 replies; 11+ messages in thread
From: Vivek Goyal @ 2011-07-19 18:47 UTC (permalink / raw)
To: Seiji Aguchi
Cc: dzickus@redhat.com, Satoru Moriya,
dle-develop@lists.sourceforge.net, tony.luck@intel.com,
hpa@zytor.com, kexec@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org,
Eric W. Biederman, KOSAKI Motohiro, Jarod Wilson, Americo Wang,
Andrew Morton, Matthew Garrett
On Tue, Jul 19, 2011 at 02:23:26PM -0400, Seiji Aguchi wrote:
> Hi,
>
> [Upstream status]
> Discussion about kmsg_dump() in kdump path:
> - Eric and Vivek are worried about reliability of existing kmsg_dump().
> - Especially, Vivek would like to remove a RCU function call chain in kdump path
> which kernel modules can register their function calls freely.
>
> Discussion about pstore in nmi_hander.
> - Don Zickus found an issue of pstore in nmi_handler due to its mutex_lock.
You did not answer my questions in the last posting mail thread and gone
ahead with the new posting. How are we supposed to discuss something. This
has been a problem on this mail thread since the beginneing. There is
little open discussion.
So if you want to make any progress in this direction, what will help
is open discussion.
Locking is going to be a problem. So atleast we can remove kmsg_dump()
infrastructure from reboot path. Why can't we log something in user
space when user initiates a reboot, let it get logged in /var/log/messages
and then umount the file root and go ahead with reboot. Why does kernel
need to capture that info in NVRAM.
If we can get rid of all the logging thing on reboot path, then at least
it does not need to be lock protected with crash path.
Thanks
Vivek
^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
2011-07-19 18:47 ` Vivek Goyal
(?)
@ 2011-07-19 19:18 ` Seiji Aguchi
-1 siblings, 0 replies; 11+ messages in thread
From: Seiji Aguchi @ 2011-07-19 19:18 UTC (permalink / raw)
To: Vivek Goyal
Cc: dzickus@redhat.com, Satoru Moriya,
dle-develop@lists.sourceforge.net, tony.luck@intel.com,
hpa@zytor.com, kexec@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org,
Eric W. Biederman, KOSAKI Motohiro, Jarod Wilson, Americo Wang,
Andrew Morton, Matthew Garrett
Hi,
>Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
Thank you for coordinating discussion.
I will reply to your question.
Regards,
Seiji
>-----Original Message-----
>From: Vivek Goyal [mailto:vgoyal@redhat.com]
>Sent: Tuesday, July 19, 2011 2:48 PM
>To: Seiji Aguchi
>Cc: kexec@lists.infradead.org; linux-kernel@vger.kernel.org; linux-mtd@lists.infradead.org; Eric W. Biederman; KOSAKI
>Motohiro; Americo Wang; Matthew Garrett; tony.luck@intel.com; Andrew Morton; Jarod Wilson; hpa@zytor.com;
>dzickus@redhat.com; dle-develop@lists.sourceforge.net; Satoru Moriya
>Subject: Re: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
>
>On Tue, Jul 19, 2011 at 02:23:26PM -0400, Seiji Aguchi wrote:
>> Hi,
>>
>> [Upstream status]
>> Discussion about kmsg_dump() in kdump path:
>> - Eric and Vivek are worried about reliability of existing kmsg_dump().
>> - Especially, Vivek would like to remove a RCU function call chain in kdump path
>> which kernel modules can register their function calls freely.
>>
>> Discussion about pstore in nmi_hander.
>> - Don Zickus found an issue of pstore in nmi_handler due to its mutex_lock.
>
>You did not answer my questions in the last posting mail thread and gone
>ahead with the new posting. How are we supposed to discuss something. This
>has been a problem on this mail thread since the beginneing. There is
>little open discussion.
>
>So if you want to make any progress in this direction, what will help
>is open discussion.
>
>Locking is going to be a problem. So atleast we can remove kmsg_dump()
>infrastructure from reboot path. Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
>
>If we can get rid of all the logging thing on reboot path, then at least
>it does not need to be lock protected with crash path.
>
>Thanks
>Vivek
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
@ 2011-07-19 19:18 ` Seiji Aguchi
0 siblings, 0 replies; 11+ messages in thread
From: Seiji Aguchi @ 2011-07-19 19:18 UTC (permalink / raw)
To: Vivek Goyal
Cc: kexec@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-mtd@lists.infradead.org, Eric W. Biederman, KOSAKI Motohiro,
Americo Wang, Matthew Garrett, tony.luck@intel.com, Andrew Morton,
Jarod Wilson, hpa@zytor.com, dzickus@redhat.com,
dle-develop@lists.sourceforge.net, Satoru Moriya
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 2333 bytes --]
Hi,
>Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
Thank you for coordinating discussion.
I will reply to your question.
Regards,
Seiji
>-----Original Message-----
>From: Vivek Goyal [mailto:vgoyal@redhat.com]
>Sent: Tuesday, July 19, 2011 2:48 PM
>To: Seiji Aguchi
>Cc: kexec@lists.infradead.org; linux-kernel@vger.kernel.org; linux-mtd@lists.infradead.org; Eric W. Biederman; KOSAKI
>Motohiro; Americo Wang; Matthew Garrett; tony.luck@intel.com; Andrew Morton; Jarod Wilson; hpa@zytor.com;
>dzickus@redhat.com; dle-develop@lists.sourceforge.net; Satoru Moriya
>Subject: Re: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
>
>On Tue, Jul 19, 2011 at 02:23:26PM -0400, Seiji Aguchi wrote:
>> Hi,
>>
>> [Upstream status]
>> Discussion about kmsg_dump() in kdump path:
>> - Eric and Vivek are worried about reliability of existing kmsg_dump().
>> - Especially, Vivek would like to remove a RCU function call chain in kdump path
>> ããã which kernel modules can register their function calls freely.
>>
>> Discussion about pstore in nmi_hander.
>> - Don Zickus found an issue of pstore in nmi_handler due to its mutex_lock.
>
>You did not answer my questions in the last posting mail thread and gone
>ahead with the new posting. How are we supposed to discuss something. This
>has been a problem on this mail thread since the beginneing. There is
>little open discussion.
>
>So if you want to make any progress in this direction, what will help
>is open discussion.
>
>Locking is going to be a problem. So atleast we can remove kmsg_dump()
>infrastructure from reboot path. Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
>
>If we can get rid of all the logging thing on reboot path, then at least
>it does not need to be lock protected with crash path.
>
>Thanks
>Vivek
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
^ permalink raw reply [flat|nested] 11+ messages in thread* RE: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
@ 2011-07-19 19:18 ` Seiji Aguchi
0 siblings, 0 replies; 11+ messages in thread
From: Seiji Aguchi @ 2011-07-19 19:18 UTC (permalink / raw)
To: Vivek Goyal
Cc: dzickus@redhat.com, Satoru Moriya,
dle-develop@lists.sourceforge.net, tony.luck@intel.com,
hpa@zytor.com, kexec@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org,
Eric W. Biederman, KOSAKI Motohiro, Jarod Wilson, Americo Wang,
Andrew Morton, Matthew Garrett
Hi,
>Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
Thank you for coordinating discussion.
I will reply to your question.
Regards,
Seiji
>-----Original Message-----
>From: Vivek Goyal [mailto:vgoyal@redhat.com]
>Sent: Tuesday, July 19, 2011 2:48 PM
>To: Seiji Aguchi
>Cc: kexec@lists.infradead.org; linux-kernel@vger.kernel.org; linux-mtd@lists.infradead.org; Eric W. Biederman; KOSAKI
>Motohiro; Americo Wang; Matthew Garrett; tony.luck@intel.com; Andrew Morton; Jarod Wilson; hpa@zytor.com;
>dzickus@redhat.com; dle-develop@lists.sourceforge.net; Satoru Moriya
>Subject: Re: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
>
>On Tue, Jul 19, 2011 at 02:23:26PM -0400, Seiji Aguchi wrote:
>> Hi,
>>
>> [Upstream status]
>> Discussion about kmsg_dump() in kdump path:
>> - Eric and Vivek are worried about reliability of existing kmsg_dump().
>> - Especially, Vivek would like to remove a RCU function call chain in kdump path
>> which kernel modules can register their function calls freely.
>>
>> Discussion about pstore in nmi_hander.
>> - Don Zickus found an issue of pstore in nmi_handler due to its mutex_lock.
>
>You did not answer my questions in the last posting mail thread and gone
>ahead with the new posting. How are we supposed to discuss something. This
>has been a problem on this mail thread since the beginneing. There is
>little open discussion.
>
>So if you want to make any progress in this direction, what will help
>is open discussion.
>
>Locking is going to be a problem. So atleast we can remove kmsg_dump()
>infrastructure from reboot path. Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
>
>If we can get rid of all the logging thing on reboot path, then at least
>it does not need to be lock protected with crash path.
>
>Thanks
>Vivek
^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
2011-07-19 18:47 ` Vivek Goyal
(?)
@ 2011-07-20 22:34 ` Seiji Aguchi
-1 siblings, 0 replies; 11+ messages in thread
From: Seiji Aguchi @ 2011-07-20 22:34 UTC (permalink / raw)
To: Vivek Goyal
Cc: dzickus@redhat.com, Satoru Moriya,
dle-develop@lists.sourceforge.net, tony.luck@intel.com,
hpa@zytor.com, kexec@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org,
Eric W. Biederman, KOSAKI Motohiro, Jarod Wilson, Americo Wang,
Andrew Morton, Matthew Garrett
Hi Vivek,
> Why can't we log something in user space when user initiates a reboot,
> Why does kernel need to capture that info in NVRAM.
As I explained in following thread,
customers sometimes ask us to investigate the reason why system reboots.
https://lkml.org/lkml/2010/11/17/249
We can solve following scenario by logging messages in kernel space.
- Syslogd is killed by mistake.
(user kills syslogd with command or other applications send signal).
- And then, system reboots.
If we log messages in user space, we can't see messages.
Seiji
>-----Original Message-----
>From: Vivek Goyal [mailto:vgoyal@redhat.com]
>Sent: Tuesday, July 19, 2011 2:48 PM
>To: Seiji Aguchi
>Cc: kexec@lists.infradead.org; linux-kernel@vger.kernel.org; linux-mtd@lists.infradead.org; Eric W. Biederman; KOSAKI
>Motohiro; Americo Wang; Matthew Garrett; tony.luck@intel.com; Andrew Morton; Jarod Wilson; hpa@zytor.com;
>dzickus@redhat.com; dle-develop@lists.sourceforge.net; Satoru Moriya
>Subject: Re: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
>
>On Tue, Jul 19, 2011 at 02:23:26PM -0400, Seiji Aguchi wrote:
>> Hi,
>>
>> [Upstream status]
>> Discussion about kmsg_dump() in kdump path:
>> - Eric and Vivek are worried about reliability of existing kmsg_dump().
>> - Especially, Vivek would like to remove a RCU function call chain in kdump path
>> which kernel modules can register their function calls freely.
>>
>> Discussion about pstore in nmi_hander.
>> - Don Zickus found an issue of pstore in nmi_handler due to its mutex_lock.
>
>You did not answer my questions in the last posting mail thread and gone
>ahead with the new posting. How are we supposed to discuss something. This
>has been a problem on this mail thread since the beginneing. There is
>little open discussion.
>
>So if you want to make any progress in this direction, what will help
>is open discussion.
>
>Locking is going to be a problem. So atleast we can remove kmsg_dump()
>infrastructure from reboot path. Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
>
>If we can get rid of all the logging thing on reboot path, then at least
>it does not need to be lock protected with crash path.
>
>Thanks
>Vivek
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 11+ messages in thread* RE: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
@ 2011-07-20 22:34 ` Seiji Aguchi
0 siblings, 0 replies; 11+ messages in thread
From: Seiji Aguchi @ 2011-07-20 22:34 UTC (permalink / raw)
To: Vivek Goyal
Cc: kexec@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-mtd@lists.infradead.org, Eric W. Biederman, KOSAKI Motohiro,
Americo Wang, Matthew Garrett, tony.luck@intel.com, Andrew Morton,
Jarod Wilson, hpa@zytor.com, dzickus@redhat.com,
dle-develop@lists.sourceforge.net, Satoru Moriya
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 2585 bytes --]
Hi Vivek,
> Why can't we log something in user space when user initiates a reboot,
> Why does kernel need to capture that info in NVRAM.
As I explained in following thread,
customers sometimes ask us to investigate the reason why system reboots.
https://lkml.org/lkml/2010/11/17/249
We can solve following scenario by logging messages in kernel space.
- Syslogd is killed by mistake.
(user kills syslogd with command or other applications send signal).
- And then, system reboots.
If we log messages in user space, we can't see messages.
Seiji
>-----Original Message-----
>From: Vivek Goyal [mailto:vgoyal@redhat.com]
>Sent: Tuesday, July 19, 2011 2:48 PM
>To: Seiji Aguchi
>Cc: kexec@lists.infradead.org; linux-kernel@vger.kernel.org; linux-mtd@lists.infradead.org; Eric W. Biederman; KOSAKI
>Motohiro; Americo Wang; Matthew Garrett; tony.luck@intel.com; Andrew Morton; Jarod Wilson; hpa@zytor.com;
>dzickus@redhat.com; dle-develop@lists.sourceforge.net; Satoru Moriya
>Subject: Re: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
>
>On Tue, Jul 19, 2011 at 02:23:26PM -0400, Seiji Aguchi wrote:
>> Hi,
>>
>> [Upstream status]
>> Discussion about kmsg_dump() in kdump path:
>> - Eric and Vivek are worried about reliability of existing kmsg_dump().
>> - Especially, Vivek would like to remove a RCU function call chain in kdump path
>> ããã which kernel modules can register their function calls freely.
>>
>> Discussion about pstore in nmi_hander.
>> - Don Zickus found an issue of pstore in nmi_handler due to its mutex_lock.
>
>You did not answer my questions in the last posting mail thread and gone
>ahead with the new posting. How are we supposed to discuss something. This
>has been a problem on this mail thread since the beginneing. There is
>little open discussion.
>
>So if you want to make any progress in this direction, what will help
>is open discussion.
>
>Locking is going to be a problem. So atleast we can remove kmsg_dump()
>infrastructure from reboot path. Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
>
>If we can get rid of all the logging thing on reboot path, then at least
>it does not need to be lock protected with crash path.
>
>Thanks
>Vivek
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
^ permalink raw reply [flat|nested] 11+ messages in thread* RE: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
@ 2011-07-20 22:34 ` Seiji Aguchi
0 siblings, 0 replies; 11+ messages in thread
From: Seiji Aguchi @ 2011-07-20 22:34 UTC (permalink / raw)
To: Vivek Goyal
Cc: dzickus@redhat.com, Satoru Moriya,
dle-develop@lists.sourceforge.net, tony.luck@intel.com,
hpa@zytor.com, kexec@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org,
Eric W. Biederman, KOSAKI Motohiro, Jarod Wilson, Americo Wang,
Andrew Morton, Matthew Garrett
Hi Vivek,
> Why can't we log something in user space when user initiates a reboot,
> Why does kernel need to capture that info in NVRAM.
As I explained in following thread,
customers sometimes ask us to investigate the reason why system reboots.
https://lkml.org/lkml/2010/11/17/249
We can solve following scenario by logging messages in kernel space.
- Syslogd is killed by mistake.
(user kills syslogd with command or other applications send signal).
- And then, system reboots.
If we log messages in user space, we can't see messages.
Seiji
>-----Original Message-----
>From: Vivek Goyal [mailto:vgoyal@redhat.com]
>Sent: Tuesday, July 19, 2011 2:48 PM
>To: Seiji Aguchi
>Cc: kexec@lists.infradead.org; linux-kernel@vger.kernel.org; linux-mtd@lists.infradead.org; Eric W. Biederman; KOSAKI
>Motohiro; Americo Wang; Matthew Garrett; tony.luck@intel.com; Andrew Morton; Jarod Wilson; hpa@zytor.com;
>dzickus@redhat.com; dle-develop@lists.sourceforge.net; Satoru Moriya
>Subject: Re: [RFC][PATCH -mmotm 0/4] Improvement of pstore/kmsg_dump in kexec/panic path
>
>On Tue, Jul 19, 2011 at 02:23:26PM -0400, Seiji Aguchi wrote:
>> Hi,
>>
>> [Upstream status]
>> Discussion about kmsg_dump() in kdump path:
>> - Eric and Vivek are worried about reliability of existing kmsg_dump().
>> - Especially, Vivek would like to remove a RCU function call chain in kdump path
>> which kernel modules can register their function calls freely.
>>
>> Discussion about pstore in nmi_hander.
>> - Don Zickus found an issue of pstore in nmi_handler due to its mutex_lock.
>
>You did not answer my questions in the last posting mail thread and gone
>ahead with the new posting. How are we supposed to discuss something. This
>has been a problem on this mail thread since the beginneing. There is
>little open discussion.
>
>So if you want to make any progress in this direction, what will help
>is open discussion.
>
>Locking is going to be a problem. So atleast we can remove kmsg_dump()
>infrastructure from reboot path. Why can't we log something in user
>space when user initiates a reboot, let it get logged in /var/log/messages
>and then umount the file root and go ahead with reboot. Why does kernel
>need to capture that info in NVRAM.
>
>If we can get rid of all the logging thing on reboot path, then at least
>it does not need to be lock protected with crash path.
>
>Thanks
>Vivek
^ permalink raw reply [flat|nested] 11+ messages in thread