[Skiboot] [PATCH] hw/imc: alway enable "imc_nest_chip" exports property
Stewart Smith
stewart at linux.vnet.ibm.com
Fri Dec 1 17:43:59 AEDT 2017
Madhavan Srinivasan <maddy at linux.vnet.ibm.com> writes:
> imc_dt_update_nest_node() adds a "imc_nest_chip" property
> to the "exports" node (under opal_node) to view nest counter
> region. This comes handy when debugging ucode runtime
> errors (like counter data update or control block update
> so on...). And current code enables the property only if
> the microcode is in running state at system boot. To aid
> the debug of ucode not running/starting issues at boot,
> enable the addition of "imc_nest_chip" property always.
>
> Fixes: 167e65d570a7c ('skiboot/hw/imc: Add nest_memory region to "exports" node')
> Signed-off-by: Madhavan Srinivasan <maddy at linux.vnet.ibm.com>
> ---
> hw/imc.c | 61 +++++++++++++++++++++++++++++++++++++++----------------------
> 1 file changed, 39 insertions(+), 22 deletions(-)
Thanks, merged to master as of 938d1d9ed964686fc6f5f4fb7ae1bfd281593ed8.
One comment on it:
> diff --git a/hw/imc.c b/hw/imc.c
> index 1db94ec6ac1b..ea8c59d62b11 100644
> --- a/hw/imc.c
> +++ b/hw/imc.c
> @@ -307,6 +307,42 @@ static void check_imc_device_type(struct dt_node *dev)
> return;
> }
>
> +static void imc_dt_exports_prop_add(struct dt_node *dev)
> +{
> + struct dt_node *node;
> + struct proc_chip *chip;
> + const struct dt_property *type;
> + uint32_t offset = 0, size = 0;
> + uint64_t baddr;
> + char namebuf[32];
> +
> +
> + dt_for_each_compatible(dev, node, "ibm,imc-counters") {
> + type = dt_find_property(node, "type");
> + if (type && is_nest_node(node)) {
> + offset = dt_prop_get_u32(node, "offset");
> + size = dt_prop_get_u32(node, "size");
> + }
> + }
This logic looks a bit suspicious, are you sure we end up with the right
thing?
--
Stewart Smith
OPAL Architect, IBM.
More information about the Skiboot
mailing list