linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [patch] IPMI driver for Linux
@ 2002-08-21 15:47 Corey Minyard
  2002-08-21 16:02 ` Alan Cox
  2002-08-27 14:55 ` [patch] IPMI driver for Linux Pavel Machek
  0 siblings, 2 replies; 11+ messages in thread
From: Corey Minyard @ 2002-08-21 15:47 UTC (permalink / raw)
  To: linux-kernel

I have been working on an IPMI driver for Linux for MontaVista, and I 
think it's ready to see the light of day :-).  I would like to see this 
included in the mainstream kernel eventually.   You can get it at 
http://home.attbi.com/~minyard.  It should work on any kernel version, 
although you will have to fix up the Config.in and Makefile, and the 
Configure.help stuff may not work (it's currently in the 2.4 location).

The web page has documentation on the driver, and documentation is 
included in the patch, too.  This is a fairly full-featured driver with 
a watchdog, panic event generation, full kernel and userland access to 
the driver, multi-user/multi-interface support, and emulators for other 
IPMI device drivers.

-Corey
minyard@acm.org


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [patch] IPMI driver for Linux
  2002-08-21 15:47 [patch] IPMI driver for Linux Corey Minyard
@ 2002-08-21 16:02 ` Alan Cox
  2002-08-21 16:52   ` Corey Minyard
  2002-08-21 19:28   ` Preempt kernel patch for 2.4.19? Jurgen Kramer
  2002-08-27 14:55 ` [patch] IPMI driver for Linux Pavel Machek
  1 sibling, 2 replies; 11+ messages in thread
From: Alan Cox @ 2002-08-21 16:02 UTC (permalink / raw)
  To: Corey Minyard; +Cc: linux-kernel

On Wed, 2002-08-21 at 16:47, Corey Minyard wrote:
> I have been working on an IPMI driver for Linux for MontaVista, and I 
> think it's ready to see the light of day :-).  I would like to see this 
> included in the mainstream kernel eventually.   You can get it at 
> http://home.attbi.com/~minyard.  It should work on any kernel version, 
> although you will have to fix up the Config.in and Makefile, and the 
> Configure.help stuff may not work (it's currently in the 2.4 location).
> 
> The web page has documentation on the driver, and documentation is 
> included in the patch, too.  This is a fairly full-featured driver with 
> a watchdog, panic event generation, full kernel and userland access to 
> the driver, multi-user/multi-interface support, and emulators for other 
> IPMI device drivers.

Comments in general.

It touches user space with spinlocks held -> bad idea
It doesnt check copy_*_user returns instead commenting that some other
driver didnt so it wont - bad idea too
It seems to be allocating a major - can you have > 1 ipmi per host, can
it use misc devices, can it get one registered properly with lanana

Otherwise its way way way nicer than the hideous thing a certain chip
vendor sent me.


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [patch] IPMI driver for Linux
  2002-08-21 16:02 ` Alan Cox
@ 2002-08-21 16:52   ` Corey Minyard
  2002-08-21 19:28   ` Preempt kernel patch for 2.4.19? Jurgen Kramer
  1 sibling, 0 replies; 11+ messages in thread
From: Corey Minyard @ 2002-08-21 16:52 UTC (permalink / raw)
  To: Alan Cox; +Cc: linux-kernel

Alan Cox wrote:

>On Wed, 2002-08-21 at 16:47, Corey Minyard wrote:
>  
>
>>I have been working on an IPMI driver for Linux for MontaVista, and I 
>>think it's ready to see the light of day :-).  I would like to see this 
>>included in the mainstream kernel eventually.   You can get it at 
>>http://home.attbi.com/~minyard.  It should work on any kernel version, 
>>although you will have to fix up the Config.in and Makefile, and the 
>>Configure.help stuff may not work (it's currently in the 2.4 location).
>>
>>The web page has documentation on the driver, and documentation is 
>>included in the patch, too.  This is a fairly full-featured driver with 
>>a watchdog, panic event generation, full kernel and userland access to 
>>the driver, multi-user/multi-interface support, and emulators for other 
>>IPMI device drivers.
>>    
>>
>
>Comments in general.
>
>It touches user space with spinlocks held -> bad idea
>
Oops, thanks.  I've uploaded a version that fixes this.  I only found 
one instance of this, but it's pretty bad.

>It doesnt check copy_*_user returns instead commenting that some other
>driver didnt so it wont - bad idea too
>
This was only in the emulation code.  I debated about this, but it's 
quite possible that doing the check will break the current users of this 
code.  I'm afraid if I add the checks it will cause other broken code to 
not work.  I could pull out the emulation code and supply it separately; 
I would probably choose to not put that part into the mainstream kernel, 
anyway.

>It seems to be allocating a major - can you have > 1 ipmi per host, can
>it use misc devices, can it get one registered properly with lanana
>
Yes, you can have multiple IPMI interfaces on a host (I have a board 
that has 3!).  There are serial-port interfaces planned that could also 
easily have multiple instances as well as an on-board KCS.  If there's 
an easy way to do this with a minor device, I'm all ears, but I'd prefer 
to have a separate device for each interface.  This is one of the things 
I wanted discussion about.  Once that gets settled, I'll go to lanana. 
 Right now it's just being auto-assigned.

>Otherwise its way way way nicer than the hideous thing a certain chip
>vendor sent me.
>  
>
I know what you mean.

Thank you for your response and suggestions.

-Corey
minyard@acm.org


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Preempt kernel patch for 2.4.19?
  2002-08-21 16:02 ` Alan Cox
  2002-08-21 16:52   ` Corey Minyard
@ 2002-08-21 19:28   ` Jurgen Kramer
  2002-08-21 19:32     ` J Sloan
  1 sibling, 1 reply; 11+ messages in thread
From: Jurgen Kramer @ 2002-08-21 19:28 UTC (permalink / raw)
  To: linux-kernel

I am looking for the preemptive kernel patch for 2.4.19. The latest
2.4.19 patch is for 2.4.19rc5-ac1 dated August 1st. And there is also a
patch for 2.4.20pre2 but not for plain 2.4.19.

Any change somebody made a patch for 2.4.19?
 

Thanks,

Jurgen



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Preempt kernel patch for 2.4.19?
  2002-08-21 19:28   ` Preempt kernel patch for 2.4.19? Jurgen Kramer
@ 2002-08-21 19:32     ` J Sloan
  2002-08-21 20:41       ` J Sloan
  0 siblings, 1 reply; 11+ messages in thread
From: J Sloan @ 2002-08-21 19:32 UTC (permalink / raw)
  To: Jurgen Kramer; +Cc: linux-kernel

IIRC 2.4.19 is basically identical to 2.4.19-rc5 -

(except for the version number )

I'm running 2.4.19rc5aa1 on top of 2.4.19 -

Joe

Jurgen Kramer wrote:

>I am looking for the preemptive kernel patch for 2.4.19. The latest
>2.4.19 patch is for 2.4.19rc5-ac1 dated August 1st. And there is also a
>patch for 2.4.20pre2 but not for plain 2.4.19.
>
>Any change somebody made a patch for 2.4.19?
> 
>  
>



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Preempt kernel patch for 2.4.19?
  2002-08-21 19:32     ` J Sloan
@ 2002-08-21 20:41       ` J Sloan
  0 siblings, 0 replies; 11+ messages in thread
From: J Sloan @ 2002-08-21 20:41 UTC (permalink / raw)
  To: J Sloan; +Cc: Jurgen Kramer, linux-kernel

ACK!

That should have read

    return ENOCAFEINE;

Anyway, running -ac is the best bet for 2.4-preempt...

Joe

J Sloan wrote:

> IIRC 2.4.19 is basically identical to 2.4.19-rc5 -
>
> (except for the version number )
>
> I'm running 2.4.19rc5aa1 on top of 2.4.19 -
>
> Joe
>
> Jurgen Kramer wrote:
>
>> I am looking for the preemptive kernel patch for 2.4.19. The latest
>> 2.4.19 patch is for 2.4.19rc5-ac1 dated August 1st. And there is also a
>> patch for 2.4.20pre2 but not for plain 2.4.19.
>>
>> Any change somebody made a patch for 2.4.19?
>>
>>  
>>
>
>
> -
> 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/
>




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [patch] IPMI driver for Linux
  2002-08-21 15:47 [patch] IPMI driver for Linux Corey Minyard
  2002-08-21 16:02 ` Alan Cox
@ 2002-08-27 14:55 ` Pavel Machek
  2002-08-27 22:14   ` Matthew Dharm
  1 sibling, 1 reply; 11+ messages in thread
From: Pavel Machek @ 2002-08-27 14:55 UTC (permalink / raw)
  To: Corey Minyard; +Cc: linux-kernel

Hi!

> I have been working on an IPMI driver for Linux for MontaVista, and I 
> think it's ready to see the light of day :-).  I would like to see this 

Just out of curiosity, what is IPMI?
								Pavel
-- 
Philips Velo 1: 1"x4"x8", 300gram, 60, 12MB, 40bogomips, linux, mutt,
details at http://atrey.karlin.mff.cuni.cz/~pavel/velo/index.html.


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [patch] IPMI driver for Linux
  2002-08-27 14:55 ` [patch] IPMI driver for Linux Pavel Machek
@ 2002-08-27 22:14   ` Matthew Dharm
  2002-08-28  2:01     ` Corey Minyard
  0 siblings, 1 reply; 11+ messages in thread
From: Matthew Dharm @ 2002-08-27 22:14 UTC (permalink / raw)
  To: Pavel Machek; +Cc: Corey Minyard, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1186 bytes --]

The "Intelligent Perhiperal Management Interface".  It's a managment system
for communicating to intelligent or semi-intelligent devices on a separate
communication channel from the 'typical' data paths.

I think you can download the specs from the Intel web site.

Matt

On Tue, Aug 27, 2002 at 02:55:13PM +0000, Pavel Machek wrote:
> Hi!
> 
> > I have been working on an IPMI driver for Linux for MontaVista, and I 
> > think it's ready to see the light of day :-).  I would like to see this 
> 
> Just out of curiosity, what is IPMI?
> 								Pavel
> -- 
> Philips Velo 1: 1"x4"x8", 300gram, 60, 12MB, 40bogomips, linux, mutt,
> details at http://atrey.karlin.mff.cuni.cz/~pavel/velo/index.html.
> 
> -
> 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/

-- 
Matthew Dharm                              Home: mdharm-usb@one-eyed-alien.net 
Maintainer, Linux USB Mass Storage Driver

We can customize our colonels.
					-- Tux
User Friendly, 12/1/1998

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [patch] IPMI driver for Linux
  2002-08-27 22:14   ` Matthew Dharm
@ 2002-08-28  2:01     ` Corey Minyard
  0 siblings, 0 replies; 11+ messages in thread
From: Corey Minyard @ 2002-08-28  2:01 UTC (permalink / raw)
  To: Matthew Dharm; +Cc: Pavel Machek, linux-kernel

Matthew Dharm wrote:

>The "Intelligent Perhiperal Management Interface".  It's a managment system
>for communicating to intelligent or semi-intelligent devices on a separate
>communication channel from the 'typical' data paths.
>
>I think you can download the specs from the Intel web site.
>
It's at http://www.intel.com/design/servers/ipmi/index.htm.  Note that 
many people are doing things far beyond what the spec says in multi-card 
chassis like CompactPCI and PICMG2.16.  Someone mentioned earlier that I 
must have access to documentation they don't have, I just know stuff 
lots of people are doing.

-Corey




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [patch] IPMI driver for Linux
  2002-08-21 20:41 Larry Butler
@ 2002-08-21 20:53 ` Corey Minyard
  0 siblings, 0 replies; 11+ messages in thread
From: Corey Minyard @ 2002-08-21 20:53 UTC (permalink / raw)
  To: Larry Butler; +Cc: linux-kernel

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/
>  
>




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [patch] IPMI driver for Linux
@ 2002-08-21 20:41 Larry Butler
  2002-08-21 20:53 ` Corey Minyard
  0 siblings, 1 reply; 11+ messages in thread
From: Larry Butler @ 2002-08-21 20:41 UTC (permalink / raw)
  To: linux-kernel


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


^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2002-08-28  1:56 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-08-21 15:47 [patch] IPMI driver for Linux Corey Minyard
2002-08-21 16:02 ` Alan Cox
2002-08-21 16:52   ` Corey Minyard
2002-08-21 19:28   ` Preempt kernel patch for 2.4.19? Jurgen Kramer
2002-08-21 19:32     ` J Sloan
2002-08-21 20:41       ` J Sloan
2002-08-27 14:55 ` [patch] IPMI driver for Linux Pavel Machek
2002-08-27 22:14   ` Matthew Dharm
2002-08-28  2:01     ` Corey Minyard
2002-08-21 20:41 Larry Butler
2002-08-21 20:53 ` Corey Minyard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).