lkml.org 
[lkml]   [2010]   [Jul]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ipmi: Fix ACPI detecting with regspacing
On 07/28/2010 12:22 PM, Andrew Morton wrote:
> On Tue, 27 Jul 2010 11:41:10 -0600
> Bjorn Helgaas <bjorn.helgaas@hp.com> wrote:
>
>> On Tuesday, July 27, 2010 11:28:28 am Yinghai Lu wrote:
>>> On 07/27/2010 10:23 AM, Bjorn Helgaas wrote:
>>>> On Tuesday, July 27, 2010 11:07:55 am Yinghai Lu wrote:
>>>>> On 07/27/2010 08:34 AM, Bjorn Helgaas wrote:
>>>>>> On Monday, July 26, 2010 03:48:02 pm Yinghai Lu wrote:
>>>>>>>
>>>>>>> after the commint that change ipmi_si detecting sequence from SMBIOS/ACPI to ACPI/SMBIOS,
>>>>>>>
>>>>>>> | commit 754d453185275951d39792865927ec494fa1ebd8
>>>>>>> | Author: Matthew Garrett <mjg@redhat.com>
>>>>>>> | Date: Wed May 26 14:43:47 2010 -0700
>>>>>>> |
>>>>>>> | ipmi: change device discovery order
>>>>>>> |
>>>>>>> | The ipmi spec provides an ordering for si discovery. Change the driver to
>>>>>>> | match, with the exception of preferring smbios to SPMI as HPs (at least)
>>>>>>> | contain accurate information in the former but not the latter.
>>>>>>>
>>>>>>> ipmi_si can not be initialized.
>>>>>>
>>>>>> I think this patch makes sense, as long as this is not a bringup
>>>>>> issue that only affects pre-release firmware. If this only affects
>>>>>> a prototype, it would be better to change the firmware so it conforms
>>>>>> to the conventional _CRS usage of systems in the field.
>>>>>>
>>>>>> To that end, I'd like to know what system this is, and whether IPMI
>>>>>> works under Windows on this system. We know that Windows doesn't
>>>>>> look at SMBIOS or SPMI, so if Windows works, it must be doing something
>>>>>> similar to your patch.
>>>>>
>>>>> Sun Fire X4800.
>>>>>
>>>>> Yes. other OSes work and pass certification.
>>>>
>>>> Please put this information in the changelog (if that wasn't obvious).
>>>
>>> I don't want to mention other os.
>>
>> Sigh. Why is this so difficult?
>
> It wasn't for me ;)
>
> Do we have acks for 2.6.36 and a -stable backport on this patch?

Matthew Garrett <mjg@redhat.com>
gave ack to it.

looks like Matthew's patch only get into 2.6.35.

So we need it for 2.6.35 and later.

Thanks

Yinghai



\
 
 \ /
  Last update: 2010-07-28 21:35    [from the cache]
©2003-2011 Jasper Spaans