Re: [Xen-devel] [PATCH] add LSB header to xen-watchdog runlevel script [and 1 more messages]

[Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>]

On Mon, 2010-07-26 at 11:22 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [PATCH] add LSB header to xen-watchdog 
> runlevel script [and 1 more messages]"):
> > On Mon, 2010-07-26 at 11:08 +0100, Ian Jackson wrote:
> > > I meant: adding these lines would seem to be likely to cause some
> > > systems to automatically start the watchdog when previously they would
> > > not have done so.  I wanted to know whether that was a good idea.
> > 
> > IMHO no -- enabling a watchdog should be an explicit admin action.
> 
> So I conclude that you think the patch is wrong ?

If it causes the watchdog to be enabled then yes.

However, looking back over the whole thread I think Olaf suggested that
just adding this header won't do that and that the admin (or the
packages) has to add an explicit request (I guess to chkconfig or
something similar) to cause the header to have any actual effect, is
that right?

BTW, from the original patch:
+# Required-Start:    xend

Is that accurate? I'd have thought the watchdog ought to function
independent of xend, e.g. on a pure xl system. I don't see anything to
the contrary in a quick glance through the code.

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel