Re: [Qemu-devel] [PATCH 0/8] ipmi: a couple of enhancements to the BMC simulator (round 2)

[Corey Minyard <cminyard@mvista.com>]

On 02/09/2016 06:13 AM, Cédric Le Goater wrote:
> The first patches are cleanups and prepare ground for an extension of
> the BMC simulator providing a SDR loader using a file. A simple FRU
> support comes next.
>
> The last patches introduce APIs to populate a guest device tree with
> the available sensors and to generate events, which is needed by
> platforms in some occasions.
>

I have reviewed all of these, and they look good.  I only have one
comment: The naming of the properties probably needs to be
fixed.

"sdr" should be "sdrfile" to be consistent with everything else.

Technically, a "FRU" is a piece of hardware that can be replaced
in the field, "FRU data" is the data describing that FRU, and a "FRU
area" is the memory used to store FRU data.  I know this is mixed
up a lot (and I have done so) but some people are picky about this.

With the renaming of sdr (fru is your option):

Acked-by: Corey Minyard <cminyard@mvista.com>

for all patches.

Oh, and I assume you need to add documentation for the
properties to qemu-options.hx.

-corey

> Based on e4a096b1cd43 and also available here  :
>
>    https://github.com/legoater/qemu/commits/ipmi
>
> Thanks,
>
> C.
>
> Cédric Le Goater (8):
>    ipmi: add a realize function to the device class
>    ipmi: use a function to initialize the SDR table
>    ipmi: remove the need of an ending record in the SDR table
>    ipmi: add some local variables in ipmi_sdr_init
>    ipmi: use a file to load SDRs
>    ipmi: provide support for FRUs
>    ipmi: introduce an ipmi_bmc_sdr_find() API
>    ipmi: introduce an ipmi_bmc_gen_event() API
>
>   hw/ipmi/ipmi_bmc_sim.c | 256 +++++++++++++++++++++++++++++++++++++++++++------
>   include/hw/ipmi/ipmi.h |   4 +
>   2 files changed, 233 insertions(+), 27 deletions(-)
>