All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <linux@roeck-us.net>
To: minyard@acm.org, Don Zickus <dzickus@redhat.com>
Cc: openipmi-developer@lists.sourceforge.net, linux-watchdog@vger.kernel.org
Subject: Re: ipmi watchdog questions
Date: Thu, 01 May 2014 18:11:12 -0700	[thread overview]
Message-ID: <5362F0B0.4030405@roeck-us.net> (raw)
In-Reply-To: <5362E8FA.9050700@acm.org>

On 05/01/2014 05:38 PM, Corey Minyard wrote:
> On 05/01/2014 08:58 AM, Don Zickus wrote:
>> Hi Corey,
>>
>> I stumbled upon an issue with a partner of ours, where they booted their
>> machine and tried to load the ipmi_watchdog module by hand and it failed.
>>
>> The reason it failed was that the iTCO watchdog driver was already loaded
>> and it registered the misc device /dev/watchdog first.
>>
>> I looked at the ipmi watchdog driver and realized it was never converted
>> to the new watchdog framework where the watchdog_core module manages the
>> '/dev/watchdog' misc device.
>>
>> So being naive and not knowing much about IPMI, I decided to follow the
>> helpful document Documentation/watchdog/convert_drivers_to_kernel_api.txt
>> and convert the ipmi_watchdog to use the new watchdog framework.
>>
>> I ran into a few issues and then realized the driver itself never really
>> binds to any hardware, so it makes the conversion process a little more
>> challenging.
>>
>> So a few questions to you before I waste my time in this area:
>>
>> - Is there any prior history about why the ipmi_watchdog was never
>>    converted to the new watchdog framework?  Lack of interest? Technical
>> hurdles?
>
> Mostly lack of interest, but there are some technical hurdles.
>
> It would be hard to implement some things.  The watchdog framework has
> no concept of pretimeouts.  And IPMI is message based, you send a

Are you saying that WDIOC_SETPRETIMEOUT and WDIOC_GETPRETIMEOUT don't work
for ipmi ? If so, can you explain ?

> message to a controller to do anything, and you have to wait for the
> response.  That doesn't work very well with the watchdog interface,
> which assumes you can do everything immediately.
>
Does it ? How so ? Please elaborate; I don't immediately see how the watchdog
subsystem would prevent you from using, say, kernel threads or delayed workers
to implement asynchronous access to or from any underlying hardware.

Guenter


  reply	other threads:[~2014-05-02  1:11 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-01 13:58 ipmi watchdog questions Don Zickus
2014-05-02  0:38 ` Corey Minyard
2014-05-02  1:11   ` Guenter Roeck [this message]
2014-05-02  4:38     ` Corey Minyard
2014-05-02 13:17       ` Guenter Roeck
2014-05-02 16:44         ` Don Zickus
2014-05-02 17:18           ` Guenter Roeck
2014-05-02 17:46             ` Don Zickus
2014-05-02 21:52               ` Corey Minyard
2014-05-02 23:20                 ` Guenter Roeck
2014-05-03  2:10                 ` Don Zickus
2014-05-03  2:51                   ` Corey Minyard
2014-05-03  4:23                   ` Guenter Roeck
2014-05-02 15:10   ` Don Zickus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5362F0B0.4030405@roeck-us.net \
    --to=linux@roeck-us.net \
    --cc=dzickus@redhat.com \
    --cc=linux-watchdog@vger.kernel.org \
    --cc=minyard@acm.org \
    --cc=openipmi-developer@lists.sourceforge.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.