I have reworked the QEMU IPMI patches, and I believe this design
improves greatly on the previous one.  The suggestions made all
resulted in better design.  To start an IPMI interface
with this, you would use:

  -device ipmi-bmc-sim,id=bmc0 -device isa-ipmi-bt,bmc=bmc0

Note that I did not make the BMCs objects.  I started this way, but
adding properties was ending up duplicating a lot of code already
in the device core, especially for the chardev link.

I'm not sure of the importance of having a BMC be an object instead
of a device, but it seems to me that if we do that, it would be
best to add helper functions to the object core for the chardev
and integer properties.

Thanks,

-corey


Reply via email to