svn commit: r325378 - head/sys/dev/ipmi

[Xin Li]

On 11/3/17 22:53, Warner Losh wrote:
> 
> 
> On Fri, Nov 3, 2017 at 11:29 PM, Xin Li <delphij at delphij.net
> <mailto:delphij at delphij.net>> wrote:
> 
> 
> 
>     On 11/3/17 20:01, Warner Losh wrote:
>     > Author: imp
>     > Date: Sat Nov  4 03:01:58 2017
>     > New Revision: 325378
>     > URL: https://svnweb.freebsd.org/changeset/base/325378
>     <https://svnweb.freebsd.org/changeset/base/325378>
>     >
>     > Log:
>     >   Make the startup timeout 0 seconds by default rathern than 420s.  This
>     >   makes the default fail safe when watchdogd is disabled (which is also
>     >   the default).
> 
>     I'm not sure if this is good: what if watchdogd is enabled, but the
>     system get stuck before watchdogd starts?
> 
> 
> Correct. This feature has to be opt-in on top of watchdogd. If you don't
> add-in this feature, you won't have this new protection. You have to set
> hw.ipmi.wd_startup_countdown to activate it.
> 
> More complicated logic may be possible, but the ipmi driver has no clue
> if watchdogd is enabled, so I'm not sure I see a path forward to
> automatically doing the right thing with sensible defaults. Have I
> overlooked something?

No you didn't.  I missed the fact that hw.ipmi.wd_startup_countdown can
be set in loader.  Sorry for the noise.

Cheers,

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/svn-src-head/attachments/20171103/dd46bfca/attachment.sig>