public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Corey Minyard <minyard@acm.org>
To: Larry Butler <larry_butler@hp.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [patch] IPMI driver for Linux
Date: Wed, 21 Aug 2002 15:53:01 -0500	[thread overview]
Message-ID: <3D63FDAD.6000009@acm.org> (raw)
In-Reply-To: 200208211441.40195.larry_butler@hp.com

I tie into the highres timer code for short sleeps.  It does require 
that you have highres timers installed in your kernel and enabled. 
 Otherwise you are right, it is very slow.

Since I had access to highres timers, that was a lot easier than hooking 
into and configuring the timer interrupt, and a lot more portable, too.

If you want to post your code or modify mine to add the timer interrupt 
support, that would be great.

-Corey

Larry Butler wrote:

>Corey,
>
>I've been working on a driver too because the busy waits in the drivers that 
>are out there can hold a CPU for too long.  I've measured as much as 120ms.
>
>First I tried sleeping in the driver until the very next jiffy.  I found that 
>my driver became unreliable under high CPU load because the scheduling delays 
>were too long.  I even managed wedge the BMC on one of my test systems in a 
>way I can't seem to fix. :)
>
>What I finally settled on was using the timer interrupt.  This seems to work 
>well both in terms of being nice to the rest of the system (I register a 
>shared irq handler only while I need it) and being reliable even under high 
>load.   So, just consider it a suggestion.  I'd like to see your driver 
>included too.  It's certainly more complete than mine.  You must have access 
>to more documentation than I do.
>
>Larry
>
>-
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at  http://www.tux.org/lkml/
>  
>




  reply	other threads:[~2002-08-21 20:48 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-21 20:41 [patch] IPMI driver for Linux Larry Butler
2002-08-21 20:53 ` Corey Minyard [this message]
  -- strict thread matches above, loose matches on Subject: below --
2002-08-21 15:47 Corey Minyard
2002-08-21 16:02 ` Alan Cox
2002-08-21 16:52   ` Corey Minyard
2002-08-27 14:55 ` Pavel Machek
2002-08-27 22:14   ` Matthew Dharm
2002-08-28  2:01     ` Corey Minyard

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=3D63FDAD.6000009@acm.org \
    --to=minyard@acm.org \
    --cc=larry_butler@hp.com \
    --cc=linux-kernel@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox