On Thu, Feb 15, 2018 at 12:49:51PM -0600, Bryant G. Ly wrote: > From: "Juan J. Alvarez" <jjalv...@linux.vnet.ibm.com> > > EEH structure is not populated with function > notify resume when running on systems that do not support > it, i.e: BMC. Hence adding a conditional check for NULL for > systems that don't add function notify_resume. > > Signed-off-by: Juan J. Alvarez <jjalv...@linux.vnet.ibm.com> > Reviewed-by: Bryant G. Ly <bryan...@linux.vnet.ibm.com> > Tested-by: Carol L. Soto <cls...@us.ibm.com>
Tested-by: Mauro S. M. Rodrigues <maur...@linux.vnet.ibm.com> Thanks! I was debugging it and I'm glad you proposed a fix already. > --- > arch/powerpc/kernel/eeh_driver.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/powerpc/kernel/eeh_driver.c > b/arch/powerpc/kernel/eeh_driver.c > index beea218..0c0b66f 100644 > --- a/arch/powerpc/kernel/eeh_driver.c > +++ b/arch/powerpc/kernel/eeh_driver.c > @@ -384,7 +384,8 @@ static void *eeh_report_resume(void *data, void *userdata) > eeh_pcid_put(dev); > pci_uevent_ers(dev, PCI_ERS_RESULT_RECOVERED); > #ifdef CONFIG_PCI_IOV > - eeh_ops->notify_resume(eeh_dev_to_pdn(edev)); > + if (eeh_ops->notify_resume && eeh_dev_to_pdn(edev)) > + eeh_ops->notify_resume(eeh_dev_to_pdn(edev)); > #endif > return NULL; > } > -- > 2.7.2 >
