Hi!
> From: Lijun Pan <ljp@xxxxxxxxxxxxx>
>
> [ Upstream commit 98025bce3a6200a0c4637272a33b5913928ba5b8 ]
>
> Commit 61d3e1d9bc2a ("ibmvnic: Remove netdev notify for failover resets")
> excluded the failover case for notify call because it said
> netdev_notify_peers() can cause network traffic to stall or halt.
> Current testing does not show network traffic stall
> or halt because of the notify call for failover event.
> netdev_notify_peers may be used when a device wants to inform the
> rest of the network about some sort of a reconfiguration
> such as failover or migration.
>
> It is unnecessary to call that in other events like
> FATAL, NON_FATAL, CHANGE_PARAM, and TIMEOUT resets
> since in those scenarios the hardware does not change.
> If the driver must do a hard reset, it is necessary to notify peers.
Something went wrong here.
> @@ -1877,8 +1877,9 @@ static int do_reset(struct ibmvnic_adapt
> for (i = 0; i < adapter->req_rx_queues; i++)
> napi_schedule(&adapter->napi[i]);
>
> - if (adapter->reset_reason != VNIC_RESET_FAILOVER &&
> - adapter->reset_reason != VNIC_RESET_CHANGE_PARAM) {
> + if ((adapter->reset_reason != VNIC_RESET_FAILOVER &&
> + adapter->reset_reason != VNIC_RESET_CHANGE_PARAM) ||
> + adapter->reset_reason == VNIC_RESET_MOBILITY) {
This condition does not make sense... part after || is redundant.
Mainline changed != in FAILOVER test to ==, so it does not have same
problem.
Best regards,
Pavel
--
http://www.livejournal.com/~pavelmachek
Attachment:
signature.asc
Description: PGP signature
