[Skiboot] [PATCH 2/2] hw/imc: don't access homer memory if it was not initialised

Madhavan Srinivasan maddy at linux.vnet.ibm.com
Mon Mar 26 13:56:46 AEDT 2018



On Sunday 25 March 2018 06:18 AM, Nicholas Piggin wrote:
> This can happen under mambo, at least.

looks good. In mambo, we could access homer via disable_unavailable_units()
which is out of mambo_quirk check.

Reviewed-by: Madhavan Srinivasan <maddy at linux.vnet.ibm.com>

> Signed-off-by: Nicholas Piggin <npiggin at gmail.com>
> ---
>   hw/imc.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/hw/imc.c b/hw/imc.c
> index 5bc59b59..c7999e1e 100644
> --- a/hw/imc.c
> +++ b/hw/imc.c
> @@ -154,6 +154,9 @@ static struct imc_chip_cb *get_imc_cb(uint32_t chip_id)
>   	struct proc_chip *chip = get_chip(chip_id);
>   	struct imc_chip_cb *cb;
>   
> +	if (!chip->homer_base)
> +		return NULL; /* The No Homers Club */
> +
>   	cb = (struct imc_chip_cb *)(chip->homer_base + P9_CB_STRUCT_OFFSET);
>   	if (!is_nest_mem_initialized(cb))
>   		return NULL;



More information about the Skiboot mailing list