Carol Hebert wrote:
> Hi Corey,
>
> This patch does not appear to be in the 2.6.25-rc8 kernel although you
> sent it in to lkml over 6 weeks back.  We were sort of expecting (and
> counting on) it to be included in the mainline kernel by now.  Is there
> anything we can do to encourage the kernel folks to pull this patch in
> soon?
>
> Thank you very much for all your help, :-)
>   
Well, we can try, but it might be a little late.

Andrew, could we get this patch pushed to the mainstream kernel?

-corey

> Carol Hebert
>
> On Wed, 2008-02-13 at 10:22 -0600, Corey Minyard wrote:
>   
>> From: Carol Hebert <[EMAIL PROTECTED]>
>>
>> Currently, on systems with multiple BMC interfaces, the ipmi device
>> names are being created in reverse order relative to how they are
>> discovered on the system (e.g. on an IBM x3950 multinode server with N
>> nodes, the device name for the BMC in the first node is /dev/ipmiN-1 and
>> the device name for the BMC in the last node is /dev/ipmi0, etc.). The
>> problem is caused by the list handling routines chosen in dmi_scan.c.
>> Using list_add() causes the multiple ipmi devices to be added to the
>> device list using a stack-paradigm and so the ipmi driver subsequently
>> pulls them off during initialization in LIFO order.  This patch changes
>> the dmi_save_ipmi_device() list handling paradigm to a queue, thereby
>> allowing the ipmi driver to build the ipmi device names in the order in
>> which they are found on the system.
>>
>> Signed-off-by: Carol Hebert <[EMAIL PROTECTED]>
>> Signed-off-by: Corey Minyard <[EMAIL PROTECTED]>
>> ---
>>
>> --- linux-2.6.24.orig/drivers/firmware/dmi_scan.c    2008-01-30 
>> 11:21:55.000000000 -0800
>> +++ linux-2.6.24/drivers/firmware/dmi_scan.c 2008-01-30 11:18:05.000000000 
>> -0800
>> @@ -219,7 +219,7 @@ static void __init dmi_save_ipmi_device(
>>      dev->name = "IPMI controller";
>>      dev->device_data = data;
>>
>> -    list_add(&dev->list, &dmi_devices);
>> +    list_add_tail(&dev->list, &dmi_devices);
>>  }
>>
>>  /*
>>
>>     


-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
Openipmi-developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openipmi-developer

Reply via email to