[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