On Fri, May 04, 2018 at 12:55:42PM +1000, Michael Ellerman wrote:
> Sam Bobroff <sbobr...@linux.ibm.com> writes:
>
> > Add a single log line at the end of successful EEH recovery, so that
> > it's clear that event processing has finished.
> >
> > Signed-off-by: Sam Bobroff <sbobr...@linux.ibm.com>
> > ---
> > arch/powerpc/kernel/eeh_driver.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/powerpc/kernel/eeh_driver.c
> > b/arch/powerpc/kernel/eeh_driver.c
> > index 56a60b9eb397..07e0a42035ce 100644
> > --- a/arch/powerpc/kernel/eeh_driver.c
> > +++ b/arch/powerpc/kernel/eeh_driver.c
> > @@ -910,6 +910,7 @@ void eeh_handle_normal_event(struct eeh_pe *pe)
> > pr_info("EEH: Notify device driver to resume\n");
> > eeh_pe_dev_traverse(pe, eeh_report_resume, NULL);
> >
> > + pr_info("EEH: Recovery successful.\n");
>
> Is it possible for recovery for multiple devices to be interleaved?
>
> Should that message include the device?No, all the calls are serialized. They're only called from eeh_event_handler() which is in the main loop of a kernel thread (see eeh_event_init()). > cheers >
signature.asc
Description: PGP signature
