Lijun Pan [lijunp213@gmail.com] wrote:

quoted

quoted

diff --git a/drivers/net/ethernet/ibm/ibmvnic.c

b/drivers/net/ethernet/ibm/ibmvnic.c
index aed985e08e8a..11f28fd03057 100644

--- a/drivers/net/ethernet/ibm/ibmvnic.c
+++ b/drivers/net/ethernet/ibm/ibmvnic.c

@@ -2235,8 +2235,7 @@ static void __ibmvnic_reset(struct work_struct

*work)
      while (rwi) {
              spin_lock_irqsave(&adapter->state_lock, flags);

-             if (adapter->state == VNIC_REMOVING ||
-                 adapter->state == VNIC_REMOVED) {
+             if (adapter->state == VNIC_REMOVED) {

If the adapter is in REMOVING state, there is no point going
through the reset process. We could just bail out here. We
should also drain any other resets in the queue (something
my other patch set was addressing).

Sukadev

quoted

If we do get here, we would crash because ibmvnic_remove() happened. It
frees the adapter struct already.

Not exactly. viodev is gone; netdev is gone; ibmvnic_adapter is still there.

Lijun