[08/21] ipmi: introduce an ipmi_bmc_sdr_find() API

This patch exposes a new IPMI routine to query a sdr entry from the
sdr table maintained by the IPMI BMC simulator. The API is very
similar to the internal sdr_find_entry() routine and should be used
the same way to query one or all sdrs.

A typical use would be to loop on the sdrs to build nodes of a device
tree.

Signed-off-by: Cédric Le Goater <clg@kaod.org>
Acked-by: Corey Minyard <cminyard@mvista.com>
---
 hw/ipmi/ipmi_bmc_sim.c | 16 ++++++++++++++++
 include/hw/ipmi/ipmi.h |  2 ++
 2 files changed, 18 insertions(+)

On Wed, Apr 05, 2017 at 02:41:33PM +0200, Cédric Le Goater wrote:
> This patch exposes a new IPMI routine to query a sdr entry from the
> sdr table maintained by the IPMI BMC simulator. The API is very
> similar to the internal sdr_find_entry() routine and should be used
> the same way to query one or all sdrs.
> 
> A typical use would be to loop on the sdrs to build nodes of a device
> tree.
> 
> Signed-off-by: Cédric Le Goater <clg@kaod.org>
> Acked-by: Corey Minyard <cminyard@mvista.com>

I don't know much about IPMI, but since it has Corey's ack, I've
staged 5..8/21 in my for-2.10 tree.  9 I've left for now, since Corey
has comments.

> ---
>  hw/ipmi/ipmi_bmc_sim.c | 16 ++++++++++++++++
>  include/hw/ipmi/ipmi.h |  2 ++
>  2 files changed, 18 insertions(+)
> 
> diff --git a/hw/ipmi/ipmi_bmc_sim.c b/hw/ipmi/ipmi_bmc_sim.c
> index eae7b2d8c92b..8185a84b76b9 100644
> --- a/hw/ipmi/ipmi_bmc_sim.c
> +++ b/hw/ipmi/ipmi_bmc_sim.c
> @@ -416,6 +416,22 @@ static int sdr_find_entry(IPMISdr *sdr, uint16_t recid,
>      return 1;
>  }
>  
> +int ipmi_bmc_sdr_find(IPMIBmc *b, uint16_t recid,
> +                      const struct ipmi_sdr_compact **sdr, uint16_t *nextrec)
> +
> +{
> +    IPMIBmcSim *ibs = IPMI_BMC_SIMULATOR(b);
> +    unsigned int pos;
> +
> +    pos = 0;
> +    if (sdr_find_entry(&ibs->sdr, recid, &pos, nextrec)) {
> +        return -1;
> +    }
> +
> +    *sdr = (const struct ipmi_sdr_compact *) &ibs->sdr.sdr[pos];
> +    return 0;
> +}
> +
>  static void sel_inc_reservation(IPMISel *sel)
>  {
>      sel->reservation++;
> diff --git a/include/hw/ipmi/ipmi.h b/include/hw/ipmi/ipmi.h
> index 91b83b5bb0b6..0d36cfc6b7f3 100644
> --- a/include/hw/ipmi/ipmi.h
> +++ b/include/hw/ipmi/ipmi.h
> @@ -259,4 +259,6 @@ struct ipmi_sdr_compact {
>  
>  typedef uint8_t ipmi_sdr_compact_buffer[sizeof(struct ipmi_sdr_compact)];
>  
> +int ipmi_bmc_sdr_find(IPMIBmc *b, uint16_t recid,
> +                      const struct ipmi_sdr_compact **sdr, uint16_t *nextrec);
>  #endif

On 04/06/2017 07:36 AM, David Gibson wrote:
> On Wed, Apr 05, 2017 at 02:41:33PM +0200, Cédric Le Goater wrote:
>> This patch exposes a new IPMI routine to query a sdr entry from the
>> sdr table maintained by the IPMI BMC simulator. The API is very
>> similar to the internal sdr_find_entry() routine and should be used
>> the same way to query one or all sdrs.
>>
>> A typical use would be to loop on the sdrs to build nodes of a device
>> tree.
>>
>> Signed-off-by: Cédric Le Goater <clg@kaod.org>
>> Acked-by: Corey Minyard <cminyard@mvista.com>
> 
> I don't know much about IPMI, but since it has Corey's ack, I've
> staged 5..8/21 in my for-2.10 tree. 

thanks for doing so. The patchset needs the interface introduced in    
patch 9 and 10 to build the sensor nodes in the device tree and to 
generate shutdown events which is important for the machine.

> 9 I've left for now, since Corey has comments.

It's patch 10 in fact which is about the SET_SENSOR_READING command. 

I think this is the most complex command in the IPMI specs ... It's 
not a strong requirement for the firmware but as I have started it 
I am trying to reach a minimal and correct implementation ... It really
is giving me headaches tough. I am glad that Corey is giving some 
support.

C. 
 
>> ---
>>  hw/ipmi/ipmi_bmc_sim.c | 16 ++++++++++++++++
>>  include/hw/ipmi/ipmi.h |  2 ++
>>  2 files changed, 18 insertions(+)
>>
>> diff --git a/hw/ipmi/ipmi_bmc_sim.c b/hw/ipmi/ipmi_bmc_sim.c
>> index eae7b2d8c92b..8185a84b76b9 100644
>> --- a/hw/ipmi/ipmi_bmc_sim.c
>> +++ b/hw/ipmi/ipmi_bmc_sim.c
>> @@ -416,6 +416,22 @@ static int sdr_find_entry(IPMISdr *sdr, uint16_t recid,
>>      return 1;
>>  }
>>  
>> +int ipmi_bmc_sdr_find(IPMIBmc *b, uint16_t recid,
>> +                      const struct ipmi_sdr_compact **sdr, uint16_t *nextrec)
>> +
>> +{
>> +    IPMIBmcSim *ibs = IPMI_BMC_SIMULATOR(b);
>> +    unsigned int pos;
>> +
>> +    pos = 0;
>> +    if (sdr_find_entry(&ibs->sdr, recid, &pos, nextrec)) {
>> +        return -1;
>> +    }
>> +
>> +    *sdr = (const struct ipmi_sdr_compact *) &ibs->sdr.sdr[pos];
>> +    return 0;
>> +}
>> +
>>  static void sel_inc_reservation(IPMISel *sel)
>>  {
>>      sel->reservation++;
>> diff --git a/include/hw/ipmi/ipmi.h b/include/hw/ipmi/ipmi.h
>> index 91b83b5bb0b6..0d36cfc6b7f3 100644
>> --- a/include/hw/ipmi/ipmi.h
>> +++ b/include/hw/ipmi/ipmi.h
>> @@ -259,4 +259,6 @@ struct ipmi_sdr_compact {
>>  
>>  typedef uint8_t ipmi_sdr_compact_buffer[sizeof(struct ipmi_sdr_compact)];
>>  
>> +int ipmi_bmc_sdr_find(IPMIBmc *b, uint16_t recid,
>> +                      const struct ipmi_sdr_compact **sdr, uint16_t *nextrec);
>>  #endif
>

On 04/06/2017 09:36 AM, Cédric Le Goater wrote:
> On 04/06/2017 07:36 AM, David Gibson wrote:
>> On Wed, Apr 05, 2017 at 02:41:33PM +0200, Cédric Le Goater wrote:
>>> This patch exposes a new IPMI routine to query a sdr entry from the
>>> sdr table maintained by the IPMI BMC simulator. The API is very
>>> similar to the internal sdr_find_entry() routine and should be used
>>> the same way to query one or all sdrs.
>>>
>>> A typical use would be to loop on the sdrs to build nodes of a device
>>> tree.
>>>
>>> Signed-off-by: Cédric Le Goater <clg@kaod.org>
>>> Acked-by: Corey Minyard <cminyard@mvista.com>
>>
>> I don't know much about IPMI, but since it has Corey's ack, I've
>> staged 5..8/21 in my for-2.10 tree. 
> 
> thanks for doing so. The patchset needs the interface introduced in    
> patch 9 and 10 to build the sensor nodes in the device tree and to 

oups, I meant patch 8 and 9 :

[PATCH 08/21] ipmi: introduce an ipmi_bmc_sdr_find() API
[PATCH 09/21] ipmi: introduce an ipmi_bmc_gen_event() API

sorry for the noise.

C. 

> generate shutdown events which is important for the machine.
> 
>> 9 I've left for now, since Corey has comments.
> 
> It's patch 10 in fact which is about the SET_SENSOR_READING command. 
> 
> I think this is the most complex command in the IPMI specs ... It's 
> not a strong requirement for the firmware but as I have started it 
> I am trying to reach a minimal and correct implementation ... It really
> is giving me headaches tough. I am glad that Corey is giving some 
> support.
> 
> C. 
>  
>>> ---
>>>  hw/ipmi/ipmi_bmc_sim.c | 16 ++++++++++++++++
>>>  include/hw/ipmi/ipmi.h |  2 ++
>>>  2 files changed, 18 insertions(+)
>>>
>>> diff --git a/hw/ipmi/ipmi_bmc_sim.c b/hw/ipmi/ipmi_bmc_sim.c
>>> index eae7b2d8c92b..8185a84b76b9 100644
>>> --- a/hw/ipmi/ipmi_bmc_sim.c
>>> +++ b/hw/ipmi/ipmi_bmc_sim.c
>>> @@ -416,6 +416,22 @@ static int sdr_find_entry(IPMISdr *sdr, uint16_t recid,
>>>      return 1;
>>>  }
>>>  
>>> +int ipmi_bmc_sdr_find(IPMIBmc *b, uint16_t recid,
>>> +                      const struct ipmi_sdr_compact **sdr, uint16_t *nextrec)
>>> +
>>> +{
>>> +    IPMIBmcSim *ibs = IPMI_BMC_SIMULATOR(b);
>>> +    unsigned int pos;
>>> +
>>> +    pos = 0;
>>> +    if (sdr_find_entry(&ibs->sdr, recid, &pos, nextrec)) {
>>> +        return -1;
>>> +    }
>>> +
>>> +    *sdr = (const struct ipmi_sdr_compact *) &ibs->sdr.sdr[pos];
>>> +    return 0;
>>> +}
>>> +
>>>  static void sel_inc_reservation(IPMISel *sel)
>>>  {
>>>      sel->reservation++;
>>> diff --git a/include/hw/ipmi/ipmi.h b/include/hw/ipmi/ipmi.h
>>> index 91b83b5bb0b6..0d36cfc6b7f3 100644
>>> --- a/include/hw/ipmi/ipmi.h
>>> +++ b/include/hw/ipmi/ipmi.h
>>> @@ -259,4 +259,6 @@ struct ipmi_sdr_compact {
>>>  
>>>  typedef uint8_t ipmi_sdr_compact_buffer[sizeof(struct ipmi_sdr_compact)];
>>>  
>>> +int ipmi_bmc_sdr_find(IPMIBmc *b, uint16_t recid,
>>> +                      const struct ipmi_sdr_compact **sdr, uint16_t *nextrec);
>>>  #endif
>>
> 
>