Jakub Kicinski [k...@kernel.org] wrote: > On Fri, 20 Nov 2020 16:40:49 -0600 Lijun Pan wrote: > > From: Sukadev Bhattiprolu <suka...@linux.ibm.com> > > > > We sometimes run into situations where a soft/hard reset of the adapter > > takes a long time or fails to complete. Having additional messages that > > include important adapter state info will hopefully help understand what > > is happening, reduce the guess work and minimize requests to reproduce > > problems with debug patches. > > This doesn't qualify as a bug fix, please send it to net-next.
Ok. > > > + netdev_err(adapter->netdev, > > + "[S:%d FRR:%d WFR:%d] Done processing resets\n", > > + adapter->state, adapter->force_reset_recovery, > > + adapter->wait_for_reset); > > Does reset only happen as a result of an error? Should this be a > netdev_info() instead? It is an informational message, will change to netdev_info(). Thanks, Sukadev
