On Tue, 24 Jan 2012, Sebastian Kuzminsky wrote:

> Date: Tue, 24 Jan 2012 08:17:54 -0700
> From: Sebastian Kuzminsky <[email protected]>
> To: andy pugh <[email protected]>
> Cc: EMC developers <[email protected]>
> Subject: Re: [Emc-developers] hm2 watchdog changes
> 
> On 01/24/2012 03:15 AM, andy pugh wrote:
>> On 24 January 2012 06:27, Sebastian Kuzminsky<[email protected]>  wrote:
>>
>>> 2.  The first time you call pet_watchdog() in HAL, the watchdog wakes up
>>> and becomes active.
>> Does that mean that an unpetted watchdog never bites? Fir example, if
>> you don't link the pet_watchdog function you can run hm2 without a
>> watchdog?
>
> Yes, that's the behavior that the current hm2-watchdog branch
> implements.  But now that you mention it, that sounds kind of stupid...
>
>
>> Would it make sense for the read() and write() functions to also wake
>> the dog? In fact, why can't read() and write() also pet it
>
> That makes some sense, but I'm think there are sensible configs where
> you never read and sensible configs where you never write (stepper-only
> machines with no home switches, using hm2 as a logic analyser maybe,
> using one hm2 card for inputs and another one for outputs, etc).  If we
> get rid of the pet_watchdog() function, I think we'd have to pet in both
> read() and write() to cover all cases, and that'd be wasteful.
>
> I like read() and write() waking up the watchdog, but i think i want to
> keep pet_watchdog() to do the work.
>
> I'll update the patchset tonight and repost.  Thanks for the feedback!
>
>
> -- 
> Sebastian Kuzminsky
>



This looks like a big improvement on watchdog operation, and maybe fixes
clytles troubles.

I do like the idea of the first read or write enabling the watchdog. This 
This means its impossible to run the hardware without a watchdog
(which is almost always a good thing) It might be a little nuisance
while testing things with halrun but I would feel better if the watchdog was 
always enabled (and has a short default timeout).

Is there any way to make the pet_watchdog dependent on a cookie (dog_biscuit)?

That would make it possible for the watchdog to be conditionally enabled in 
HAL

(so for example additional sanity checks could be logically ANDxed if desired)



Peter Wallace
Mesa Electronics

(\__/)
(='.'=) This is Bunny. Copy and paste bunny into your
(")_(") signature to help him gain world domination.


------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
_______________________________________________
Emc-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/emc-developers

Reply via email to