RE: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-12 Thread Liu, Chuansheng
.@google.com; > linux-kernel@vger.kernel.org > Subject: Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic > > On Wed 2018-12-12 01:16:11, Liu, Chuansheng wrote: > > > > > > > -Original Message- > > > From: Tetsuo Handa [mailto:pengu

Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-12 Thread Petr Mladek
; > Cc: a...@linux-foundation.org; pmla...@suse.com; > > sergey.senozhat...@gmail.com; rost...@goodmis.org; dvyu...@google.com; > > linux-kernel@vger.kernel.org > > Subject: Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic > > &g

Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-11 Thread Petr Mladek
On Tue 2018-12-11 01:16:10, Liu, Chuansheng wrote: > > From: Tetsuo Handa [mailto:penguin-ker...@i-love.sakura.ne.jp] > > On 2018/12/10 15:11, Sergey Senozhatsky wrote: > > > On (12/10/18 05:58), Liu, Chuansheng wrote: > > >>> On (12/10/18 05:40), Liu, Chuansheng wrote: > > @@ -130,6 +130,13

Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-11 Thread Tetsuo Handa
On 2018/12/11 10:16, Liu, Chuansheng wrote: > We may enhance it by: > - if (sysctl_hung_task_warnings) { > + if (sysctl_hung_task_panic || sysctl_hung_task_warnings) { > if (sysctl_hung_task_warnings > 0) > sysctl_hung_task_warnings--; Why

RE: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-10 Thread Liu, Chuansheng
...@goodmis.org; dvyu...@google.com; > linux-kernel@vger.kernel.org > Subject: Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic > > On 2018/12/10 15:11, Sergey Senozhatsky wrote: > > On (12/10/18 05:58), Liu, Chuansheng wrote: > >>> On (12/10/18 05:4

Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-10 Thread Sergey Senozhatsky
On (12/10/18 18:58), Tetsuo Handa wrote: > +*/ > + if (sysctl_hung_task_panic && !ignore_loglevel) > + ignore_loglevel = true; > >>> > >>> console_verbose()? > >> > >> Thanks Sergey, it is really my need. I will prepare

Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-10 Thread Tetsuo Handa
On 2018/12/10 15:11, Sergey Senozhatsky wrote: > On (12/10/18 05:58), Liu, Chuansheng wrote: >>> On (12/10/18 05:40), Liu, Chuansheng wrote: @@ -130,6 +130,13 @@ static void check_hung_task(struct task_struct *t, >>> unsigned long timeout)

Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-09 Thread Sergey Senozhatsky
On (12/10/18 05:58), Liu, Chuansheng wrote: > > On (12/10/18 05:40), Liu, Chuansheng wrote: > > > @@ -130,6 +130,13 @@ static void check_hung_task(struct task_struct *t, > > unsigned long timeout) > > > init_utsname()->version); > > > pr_err("\"echo 0 > > >

RE: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-09 Thread Liu, Chuansheng
dvyu...@google.com; > penguin-ker...@i-love.sakura.ne.jp; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic > > On (12/10/18 05:40), Liu, Chuansheng wrote: > > @@ -130,6 +130,13 @@ static void check_hung_task(struct tas

Re: [PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-09 Thread Sergey Senozhatsky
On (12/10/18 05:40), Liu, Chuansheng wrote: > @@ -130,6 +130,13 @@ static void check_hung_task(struct task_struct *t, > unsigned long timeout) > init_utsname()->version); > pr_err("\"echo 0 > /proc/sys/kernel/hung_task_timeout_secs\"" >

[PATCH] kernel/hung_task.c: force ignore_loglevel before panic

2018-12-09 Thread Liu, Chuansheng
Based on patch commit 401c636a0eeb ("kernel/hung_task.c: show all hung tasks before panic"), we could get the call stack of hung task. However, if the console loglevel is not high, we still can not get the useful information in practice, and in most cases users don't set console loglevel to