login
Header Space

 
 

Re: w83697hf_wdt.c stops watchdog on load

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Wim Van Sebroeck <wim@...>
Cc: Pádraig <P@...>, <junker@...>, LKML <linux-kernel@...>
Date: Tuesday, March 11, 2008 - 4:28 pm

On 11/03, Wim Van Sebroeck wrote:

| > > Isn't it a two-edged sword?
| > 
| > True, but one should be able to set things up so that there is no race.
| > I.E. if you add the autodisable, it should only be done as an option.
| > Best have it =no by default, but as least one could turn this off.
| 
| In my opinion it should be the other way around: the default behaviour is to
| stop the watchdog and to let userspace (the watchdog-daemon) control the
| watchdog. So if we add a module parameter to take over the watchdog's
| bios-setting, then the default behaviour should be to stop the watchdog and
| add an option that takes the value from the bios.

Hi Wim.

That was my intent: disable the watchdog by default, and add an option
not to disable it (let it untouched, so getting the BIOS setting) at
module initialization, as Pádraig request is totally reasonable.

  Sam

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: w83697hf_wdt.c stops watchdog on load, Wim Van Sebroeck, (Tue Mar 11, 4:22 pm)
Re: w83697hf_wdt.c stops watchdog on load, Samuel Tardieu, (Tue Mar 11, 4:28 pm)
Re: w83697hf_wdt.c stops watchdog on load, Pádraig Brady, (Wed Mar 12, 7:54 am)
Re: w83697hf_wdt.c stops watchdog on load, Samuel Tardieu, (Wed Mar 12, 8:00 am)
Re: w83697hf_wdt.c stops watchdog on load, Wim Van Sebroeck, (Tue Mar 11, 5:09 pm)
speck-geostationary