RE: [char-misc-next v2 7/7] watchdog: mei_wdt: re-register device on event

From: Winkler, Tomas
Date: Sun Dec 20 2015 - 07:53:17 EST


>
> > > That breaks the existing behaviour of hot pluggable watchdog interfaces
> > > and is different to just about any other device in the kernel. Today with
> > > any desktop or server distribution you can already trivially arrange for
> > > watchdog daemons to start at the point a watchdog is detected dynamically.
> > >
> >
> > Ok, you have a point. Wonder if any distributions are doing that, though.
> > Any idea ?
>
> I don't know for any of the out of the box mainstream ones.
>
> There is a second problem if you register the watchdog but don't actually
> have it enabled. Consider the case where a user has the mei watchdog
> disabled and a more advanced watchdog card plugged in - in that case the
> naÃvely implemented existing watchdog app may bind to the non-functional
> watchdog, not the correct one.

That's correct, though WDIOF_ALARMONLY or device identity has to be checked.
And if mei is disabled one would probably hit ENODEV, still there will be no uevent to the user space
when the device is provisioned again, so still I prefer add/removing the watchdog device.

Thanks
Tomas