[Skiboot] [PATCH] phb4: Reassign link_retries counter in IODA purge

Stewart Smith stewart at linux.vnet.ibm.com
Thu Oct 5 18:10:26 AEDT 2017


"Guilherme G. Piccoli" <gpiccoli at linux.vnet.ibm.com> writes:
> Recently, a link_retries counter was added in pci/phb4 in order
> Skiboot can retry to train a link some times - default number of
> attempts to retrain a link is 3.
>
> Happens that, if during a regular boot process we exhaust the
> link retries and fail to train a PHB, the variable link_retries
> is stuck in 0. If a kdump happens later, a PHB reset procedure is
> triggered by Linux and, since we have a decrement-and-test in this
> variable, we end up setting it to -1; it's unsigned, hence we get
> an overflow.
>
> This patch fixes the issue by reassigning the default value to
> link_retries in every IODA purge.
>
> Signed-off-by: Guilherme G. Piccoli <gpiccoli at linux.vnet.ibm.com>
> ---
>  hw/phb4.c | 2 ++
>  1 file changed, 2 insertions(+)

Merged as of 784da79b85537ded99e8d1890b46e3a804d93295

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list