[Skiboot] [PATCH] phys-map: fix hole in NPU2 BARs

Michael Neuling mikey at neuling.org
Fri May 12 15:55:00 AEST 2017


On Fri, 2017-05-12 at 15:39 +1000, Andrew Donnellan wrote:
> The global physical memory map added in commit 5f67c1e25378 ("Add global
> physical memory map") leaves a hole between NPU_PHY_1 and NPU_NTL_0 of size
> 0x20000.
> 
> Move the BARs in the map from NPU_NTL_0 to NPU_GENID_2 up by 0x20000 to fix
> this.
> 
> Reported-by: Alistair Popple <alistair at popple.id.au>
> Fixes: 5f67c1e25378 ("Add global physical memory map")
> Signed-off-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>

I've send the identical patch as part of a series that fixes another issue plus
adds some tests to ensure we don't create a hole like this again in the future.

  https://lists.ozlabs.org/pipermail/skiboot/2017-May/007287.html

Stewart, it's probably easier for you to just pickup my series and ignore ajd's
patch here.

Mikey

> ---
>  hw/phys-map.c | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/hw/phys-map.c b/hw/phys-map.c
> index 768f464f..95bb3035 100644
> --- a/hw/phys-map.c
> +++ b/hw/phys-map.c
> @@ -98,16 +98,16 @@ static const struct phys_map_entry phys_map_table_nimbus[]
> = {
>  	{ NPU_REGS       , 0, 0x0006030200000000ull, 0x0000000001000000ull },
>  	{ NPU_PHY        , 0, 0x0006030201200000ull, 0x0000000000200000ull },
>  	{ NPU_PHY	 , 1, 0x0006030201400000ull, 0x0000000000200000ull
> },
> -	{ NPU_NTL	 , 0, 0x0006030201620000ull, 0x0000000000020000ull
> },
> -	{ NPU_NTL	 , 1, 0x0006030201640000ull, 0x0000000000020000ull
> },
> -	{ NPU_NTL	 , 2, 0x0006030201660000ull, 0x0000000000020000ull
> },
> -	{ NPU_NTL	 , 3, 0x0006030201680000ull, 0x0000000000020000ull
> },
> -	{ NPU_NTL	 , 4, 0x00060302016a0000ull, 0x0000000000020000ull
> },
> -	{ NPU_NTL	 , 5, 0x00060302016c0000ull, 0x0000000000020000ull
> },
> -	{ NPU_GENID	 , 0, 0x00060302016e0000ull, 0x0000000000020000ull
> },
> -	{ NPU_GENID	 , 1, 0x0006030201700000ull, 0x0000000000020000ull
> },
> -	{ NPU_GENID	 , 2, 0x0006030201720000ull, 0x0000000000020000ull
> },
> -	{ RESV	         , 5, 0x0006030201740000ull,
> 0x00000000018c0000ull },
> +	{ NPU_NTL	 , 0, 0x0006030201600000ull, 0x0000000000020000ull
> },
> +	{ NPU_NTL	 , 1, 0x0006030201620000ull, 0x0000000000020000ull
> },
> +	{ NPU_NTL	 , 2, 0x0006030201640000ull, 0x0000000000020000ull
> },
> +	{ NPU_NTL	 , 3, 0x0006030201660000ull, 0x0000000000020000ull
> },
> +	{ NPU_NTL	 , 4, 0x0006030201680000ull, 0x0000000000020000ull
> },
> +	{ NPU_NTL	 , 5, 0x00060302016a0000ull, 0x0000000000020000ull
> },
> +	{ NPU_GENID	 , 0, 0x00060302016c0000ull, 0x0000000000020000ull
> },
> +	{ NPU_GENID	 , 1, 0x00060302016e0000ull, 0x0000000000020000ull
> },
> +	{ NPU_GENID	 , 2, 0x0006030201700000ull, 0x0000000000020000ull
> },
> +	{ RESV	         , 5, 0x0006030201720000ull,
> 0x00000000018e0000ull },
>  	{ PSI_LINK       , 0, 0x0006030203000000ull, 0x0000000000100000ull },
>  	{ XIVE_IC        , 0, 0x0006030203100000ull, 0x0000000000080000ull },
>  	{ XIVE_TM        , 0, 0x0006030203180000ull, 0x0000000000040000ull },


More information about the Skiboot mailing list