These little things came up when looking at behaviour of IPMI with the bmc-sim implementation running the ppc powernv machine, and trying to clean up error messages and missing features.
Since v1 (thanks to Corey for review and suggestions): - Added fwinfo to PCI devices - Report interrupt number in Get Channel Info for ISA, PCI, and unknown/unassigned. - Fix error reporting for Get Channel Info unsupported channels. Verify it is the correct error code that ipmitool looks for https://github.com/ipmitool/ipmitool/blob/master/lib/ipmi_channel.c#L256C16-L256C45 - Change _CH_ to _CHANNEL_ in some defines names. - Also avoid adding event logs with watchdog don't log flag. Thanks, Nick Nicholas Piggin (5): ipmi/pci-ipmi-bt: Rename copy-paste variables ipmi: add fwinfo to pci ipmi devices ipmi/bmc-sim: Add 'Get Channel Info' command ipmi/bmc-sim: implement watchdog dont log flag ipmi/bmc-sim: add error handling for 'Set BMC Global Enables' command include/hw/ipmi/ipmi.h | 19 +++++++ hw/acpi/ipmi.c | 2 +- hw/ipmi/ipmi_bmc_sim.c | 107 ++++++++++++++++++++++++++++++++----- hw/ipmi/ipmi_bt.c | 2 + hw/ipmi/ipmi_kcs.c | 1 + hw/ipmi/isa_ipmi_bt.c | 1 + hw/ipmi/isa_ipmi_kcs.c | 1 + hw/ipmi/pci_ipmi_bt.c | 50 ++++++++++------- hw/ipmi/pci_ipmi_kcs.c | 11 ++++ hw/smbios/smbios_type_38.c | 6 ++- 10 files changed, 166 insertions(+), 34 deletions(-) -- 2.47.1