Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread Dave Young
On 01/15/14 at 11:55am, Vivek Goyal wrote: > On Wed, Jan 15, 2014 at 12:15:56PM +, One Thousand Gnomes wrote: > > > watchdog and crash dump really conflicts to some degree, from the watchdog > > > point of view it can reboot system whhen kdump kernel hangs. But from > > > kdump > > > point of

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread Dave Young
On 01/15/14 at 11:46am, Vivek Goyal wrote: > On Wed, Jan 15, 2014 at 09:11:42AM +0800, Dave Young wrote: > > [..] > > > I thought this problem was resolved (atleast conceptually) last time > > > when Don Zickus brought it up. > > > > > > He mentioned that it was concluded that keep watchdog

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread Vivek Goyal
On Wed, Jan 15, 2014 at 12:15:56PM +, One Thousand Gnomes wrote: > > watchdog and crash dump really conflicts to some degree, from the watchdog > > point of view it can reboot system whhen kdump kernel hangs. But from kdump > > point of view it want ensure saving the vmcore for later

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread Vivek Goyal
On Wed, Jan 15, 2014 at 09:11:42AM +0800, Dave Young wrote: [..] > > I thought this problem was resolved (atleast conceptually) last time > > when Don Zickus brought it up. > > > > He mentioned that it was concluded that keep watchdog interval long > > enough, say 60 seconds and keep on kicking

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread One Thousand Gnomes
> watchdog and crash dump really conflicts to some degree, from the watchdog > point of view it can reboot system whhen kdump kernel hangs. But from kdump > point of view it want ensure saving the vmcore for later debugging. > > Maybe we can only select only one in this case. You want to be able

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread One Thousand Gnomes
watchdog and crash dump really conflicts to some degree, from the watchdog point of view it can reboot system whhen kdump kernel hangs. But from kdump point of view it want ensure saving the vmcore for later debugging. Maybe we can only select only one in this case. You want to be able to

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread Vivek Goyal
On Wed, Jan 15, 2014 at 09:11:42AM +0800, Dave Young wrote: [..] I thought this problem was resolved (atleast conceptually) last time when Don Zickus brought it up. He mentioned that it was concluded that keep watchdog interval long enough, say 60 seconds and keep on kicking it fast

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread Vivek Goyal
On Wed, Jan 15, 2014 at 12:15:56PM +, One Thousand Gnomes wrote: watchdog and crash dump really conflicts to some degree, from the watchdog point of view it can reboot system whhen kdump kernel hangs. But from kdump point of view it want ensure saving the vmcore for later debugging.

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread Dave Young
On 01/15/14 at 11:46am, Vivek Goyal wrote: On Wed, Jan 15, 2014 at 09:11:42AM +0800, Dave Young wrote: [..] I thought this problem was resolved (atleast conceptually) last time when Don Zickus brought it up. He mentioned that it was concluded that keep watchdog interval long

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-15 Thread Dave Young
On 01/15/14 at 11:55am, Vivek Goyal wrote: On Wed, Jan 15, 2014 at 12:15:56PM +, One Thousand Gnomes wrote: watchdog and crash dump really conflicts to some degree, from the watchdog point of view it can reboot system whhen kdump kernel hangs. But from kdump point of view it want

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Dave Young
On 01/14/14 at 11:24am, Vivek Goyal wrote: > On Tue, Jan 14, 2014 at 12:16:39PM +, One Thousand Gnomes wrote: > > On Tue, 14 Jan 2014 16:23:23 +0800 > > Dave Young wrote: > > > > > In kdump kernel watchdog could interrupt vmcore capturing because we > > > have no way to disable/stop it while

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Dave Young
On 01/14/14 at 12:16pm, One Thousand Gnomes wrote: > On Tue, 14 Jan 2014 16:23:23 +0800 > Dave Young wrote: > > > In kdump kernel watchdog could interrupt vmcore capturing because we > > have no way to disable/stop it while crashing happens. > > Lots of watchdogs cannot be stopped. > > > Add a

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Vivek Goyal
On Tue, Jan 14, 2014 at 12:16:39PM +, One Thousand Gnomes wrote: > On Tue, 14 Jan 2014 16:23:23 +0800 > Dave Young wrote: > > > In kdump kernel watchdog could interrupt vmcore capturing because we > > have no way to disable/stop it while crashing happens. > > Lots of watchdogs cannot be

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread One Thousand Gnomes
On Tue, 14 Jan 2014 16:23:23 +0800 Dave Young wrote: > In kdump kernel watchdog could interrupt vmcore capturing because we > have no way to disable/stop it while crashing happens. Lots of watchdogs cannot be stopped. > Add a module parameter stop_before_register so watchdog can be stopped >

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Dave Young
On 01/14/14 at 04:41pm, Dave Young wrote: > On 01/14/14 at 09:26am, Wim Van Sebroeck wrote: > > Hi Dave, > > > > > In kdump kernel watchdog could interrupt vmcore capturing because we > > > have no way to disable/stop it while crashing happens. > > > > > > Add a module parameter

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Dave Young
On 01/14/14 at 09:26am, Wim Van Sebroeck wrote: > Hi Dave, > > > In kdump kernel watchdog could interrupt vmcore capturing because we > > have no way to disable/stop it while crashing happens. > > > > Add a module parameter stop_before_register so watchdog can be stopped > > before register in

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Wim Van Sebroeck
Hi Dave, > In kdump kernel watchdog could interrupt vmcore capturing because we > have no way to disable/stop it while crashing happens. > > Add a module parameter stop_before_register so watchdog can be stopped > before register in driver loading path. Thus we can try to load the > watchdog

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Wim Van Sebroeck
Hi Dave, In kdump kernel watchdog could interrupt vmcore capturing because we have no way to disable/stop it while crashing happens. Add a module parameter stop_before_register so watchdog can be stopped before register in driver loading path. Thus we can try to load the watchdog driver as

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Dave Young
On 01/14/14 at 09:26am, Wim Van Sebroeck wrote: Hi Dave, In kdump kernel watchdog could interrupt vmcore capturing because we have no way to disable/stop it while crashing happens. Add a module parameter stop_before_register so watchdog can be stopped before register in driver

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Dave Young
On 01/14/14 at 04:41pm, Dave Young wrote: On 01/14/14 at 09:26am, Wim Van Sebroeck wrote: Hi Dave, In kdump kernel watchdog could interrupt vmcore capturing because we have no way to disable/stop it while crashing happens. Add a module parameter stop_before_register so watchdog

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread One Thousand Gnomes
On Tue, 14 Jan 2014 16:23:23 +0800 Dave Young dyo...@redhat.com wrote: In kdump kernel watchdog could interrupt vmcore capturing because we have no way to disable/stop it while crashing happens. Lots of watchdogs cannot be stopped. Add a module parameter stop_before_register so watchdog can

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Vivek Goyal
On Tue, Jan 14, 2014 at 12:16:39PM +, One Thousand Gnomes wrote: On Tue, 14 Jan 2014 16:23:23 +0800 Dave Young dyo...@redhat.com wrote: In kdump kernel watchdog could interrupt vmcore capturing because we have no way to disable/stop it while crashing happens. Lots of watchdogs

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Dave Young
On 01/14/14 at 12:16pm, One Thousand Gnomes wrote: On Tue, 14 Jan 2014 16:23:23 +0800 Dave Young dyo...@redhat.com wrote: In kdump kernel watchdog could interrupt vmcore capturing because we have no way to disable/stop it while crashing happens. Lots of watchdogs cannot be stopped.

Re: [PATCH] watchdog: add a parameter for stop wdt before register

2014-01-14 Thread Dave Young
On 01/14/14 at 11:24am, Vivek Goyal wrote: On Tue, Jan 14, 2014 at 12:16:39PM +, One Thousand Gnomes wrote: On Tue, 14 Jan 2014 16:23:23 +0800 Dave Young dyo...@redhat.com wrote: In kdump kernel watchdog could interrupt vmcore capturing because we have no way to disable/stop it