ibmvnic: Skip fatal error reset after passive init
authorJuliet Kim <julietk@linux.vnet.ibm.com>
Thu, 30 Apr 2020 18:22:11 +0000 (13:22 -0500)
committerDavid S. Miller <davem@davemloft.net>
Thu, 30 Apr 2020 20:27:17 +0000 (13:27 -0700)
commitf9c6cea0b38518741c8dcf26ac056d26ee2fd61d
treec534cf24c38d4439651a27d8eb1a738aed1640f8
parent81d6bc44fa943c105d42cca8c2926bbdad80f90e
ibmvnic: Skip fatal error reset after passive init

During MTU change, the following events may happen.
Client-driven CRQ initialization fails due to partner’s CRQ closed,
causing client to enqueue a reset task for FATAL_ERROR. Then passive
(server-driven) CRQ initialization succeeds, causing client to
release CRQ and enqueue a reset task for failover. If the passive
CRQ initialization occurs before the FATAL reset task is processed,
the FATAL error reset task would try to access a CRQ message queue
that was freed, causing an oops. The problem may be most likely to
occur during DLPAR add vNIC with a non-default MTU, because the DLPAR
process will automatically issue a change MTU request.

Fix this by not processing fatal error reset if CRQ is passively
initialized after client-driven CRQ initialization fails.

Signed-off-by: Juliet Kim <julietk@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/ibm/ibmvnic.c