[lkml]   [2011]   [Nov]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH RFC] watchdog: add a new driver for VIA chipsets
    Wim Van Sebroeck <> wrote:

    > > Regarding the timer, the driver can not read the timeout value set in
    > > the BIOS, so there is still a risk of reboot if the BIOS value is lower
    > > than driver one. I assume that other drivers may be in the same case,
    > > and that is required for user compatibility anyway.
    > The driver assumes 1 second as heartbeat of the watchdog device.
    > It thus takes half of this time do actually 'tickle' the watchdog. (or 500ms).
    > We should set this 1 second value to the smallest time that is possible (either
    > the smallest value that can be set in the BIOS or the smallest value that is
    > listed in the datasheets).
    The smallest possible value is 1 second, both in BIOS and datasheet. For
    info, the maximum value is 1023 seconds, approx. 17 min.

    I think it is dangerous to set the timer to 1s, both in BIOS, obviously
    as the boot is still in progress when it expires, and even in the driver,
    where you loose a chance to avoid mandatory reboot after only 1 second
    of latency, even if the hardware timer is higher.

    The value you have set, 15 seconds, is reasonable to me, and should not
    be lowered. If the BIOS was well designed, it should also not allow less
    than, say 1 minute.

    Kind regards,

     \ /
      Last update: 2011-11-24 15:45    [W:0.025 / U:0.044 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site