[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/7] ppc/pnv: Discard internal BMC initialization when BMC is
From: |
Joel Stanley |
Subject: |
Re: [PATCH 5/7] ppc/pnv: Discard internal BMC initialization when BMC is external |
Date: |
Thu, 28 Jan 2021 00:48:42 +0000 |
On Tue, 26 Jan 2021 at 17:11, Cédric Le Goater <clg@kaod.org> wrote:
>
> The PowerNV machine can be run with an external IPMI BMC device
> connected to a remote QEMU machine acting as BMC, using these options :
>
> -chardev socket,id=ipmi0,host=localhost,port=9002,reconnect=10 \
> -device ipmi-bmc-extern,id=bmc0,chardev=ipmi0 \
> -device isa-ipmi-bt,bmc=bmc0,irq=10 \
> -nodefaults
Should this information also go in docs/system/ppc similar to the
descriptions we have in docs/system/arm?
>
> In that case, some aspects of the BMC initialization should be
> skipped, since they rely on the simulator interface.
>
> Signed-off-by: Cédric Le Goater <clg@kaod.org>
Reviewed-by: Joel Stanley <joel@jms.id.au>
> ---
> hw/ppc/pnv_bmc.c | 15 ++++++++++++++-
> 1 file changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/hw/ppc/pnv_bmc.c b/hw/ppc/pnv_bmc.c
> index 86d16b493539..b9bf5735ea0f 100644
> --- a/hw/ppc/pnv_bmc.c
> +++ b/hw/ppc/pnv_bmc.c
> @@ -51,6 +51,11 @@ typedef struct OemSel {
> #define SOFT_OFF 0x00
> #define SOFT_REBOOT 0x01
>
> +static bool pnv_bmc_is_simulator(IPMIBmc *bmc)
> +{
> + return object_dynamic_cast(OBJECT(bmc), TYPE_IPMI_BMC_SIMULATOR);
> +}
> +
> static void pnv_gen_oem_sel(IPMIBmc *bmc, uint8_t reboot)
> {
> /* IPMI SEL Event are 16 bytes long */
> @@ -79,6 +84,10 @@ void pnv_dt_bmc_sensors(IPMIBmc *bmc, void *fdt)
> const struct ipmi_sdr_compact *sdr;
> uint16_t nextrec;
>
> + if (!pnv_bmc_is_simulator(bmc)) {
> + return;
> + }
> +
> offset = fdt_add_subnode(fdt, 0, "bmc");
> _FDT(offset);
>
> @@ -243,6 +252,10 @@ static const IPMINetfn hiomap_netfn = {
>
> void pnv_bmc_set_pnor(IPMIBmc *bmc, PnvPnor *pnor)
> {
> + if (!pnv_bmc_is_simulator(bmc)) {
> + return;
> + }
> +
> object_ref(OBJECT(pnor));
> object_property_add_const_link(OBJECT(bmc), "pnor", OBJECT(pnor));
>
> @@ -286,7 +299,7 @@ static int bmc_find(Object *child, void *opaque)
>
> IPMIBmc *pnv_bmc_find(Error **errp)
> {
> - ForeachArgs args = { TYPE_IPMI_BMC_SIMULATOR, NULL };
> + ForeachArgs args = { TYPE_IPMI_BMC, NULL };
> int ret;
>
> ret = object_child_foreach_recursive(object_get_root(), bmc_find, &args);
> --
> 2.26.2
>
>