[Skiboot] [PATCH] hw/occ: Fix fast-reboot crash in P8 platforms.

[Stewart Smith]

Pridhiviraj Paidipeddi <ppaidipe at linux.vnet.ibm.com> writes:
> commit 85a1de35cbe4 ("fast-boot: occ: Re-parse the pstate table during fast-boot" )
> breaks the fast-reboot on P8 platforms while reiniting the OCC pstates. On P8
> platforms OPAL adds additional two properties #address-cells and #size-cells
> under ibm,opal/power-mgmt/ DT node. While in fast-reboot same properties adding
> back to the same node results in Duplicate properties and hence fast-reboot fails
> with below traces.
>
> [  541.410373292,5] OCC: All Chip Rdy after 0 ms
> [  541.410488745,3] Duplicate property "#address-cells" in node /ibm,opal/power-mgt
> [  541.410694290,0] Aborting!
> CPU 0058 Backtrace:
>  S: 0000000031d639d0 R: 000000003001367c   .backtrace+0x48
>  S: 0000000031d63a60 R: 000000003001a03c   ._abort+0x4c
>  S: 0000000031d63ae0 R: 00000000300267d8   .new_property+0xd8
>  S: 0000000031d63b70 R: 0000000030026a28   .__dt_add_property_cells+0x30
>  S: 0000000031d63c10 R: 000000003003ea3c   .occ_pstates_init+0x984
>  S: 0000000031d63d90 R: 00000000300142d8   .load_and_boot_kernel+0x86c
>  S: 0000000031d63e70 R: 000000003002586c   .fast_reboot_entry+0x358
>  S: 0000000031d63f00 R: 00000000300029f4   fast_reset_entry+0x2c
>
> This patch fixes this issue by removing these two properties on P8 while doing
> OCC pstates re-init in fast-reboot code path.
>
> Signed-off-by: Pridhiviraj Paidipeddi <ppaidipe at linux.vnet.ibm.com>
> ---
>  hw/occ.c | 2 ++
>  1 file changed, 2 insertions(+)

Whoops. Thanks for finding and fixing that.

Merged to master as of 92a303dfeeb61c73ea4d9226af6c4d874db6bc89

-- 
Stewart Smith
OPAL Architect, IBM.