OSDN Git Service

octeontx2-pf: Fix interface down flag on error
authorGeetha sowjanya <gakula@marvell.com>
Sun, 25 Jul 2021 07:59:03 +0000 (13:29 +0530)
committerDavid S. Miller <davem@davemloft.net>
Sun, 25 Jul 2021 08:25:03 +0000 (09:25 +0100)
commit69f0aeb13bb548e2d5710a350116e03f0273302e
tree2d048b35e22b65d68fd05c7fc18b15c2c0bfdd90
parentac059d16442f30e6a9a95d41655153e01247e710
octeontx2-pf: Fix interface down flag on error

In the existing code while changing the number of TX/RX
queues using ethtool the PF/VF interface resources are
freed and reallocated (otx2_stop and otx2_open is called)
if the device is in running state. If any resource allocation
fails in otx2_open, driver free already allocated resources
and return. But again, when the number of queues changes
as the device state still running oxt2_stop is called.
In which we try to free already freed resources leading
to driver crash.
This patch fixes the issue by setting the INTF_DOWN flag on
error and free the resources in otx2_stop only if the flag is
not set.

Fixes: 50fe6c02e5ad ("octeontx2-pf: Register and handle link notifications")
Signed-off-by: Geetha sowjanya <gakula@marvell.com>
Signed-off-by: Sunil Kovvuri Goutham <Sunil.Goutham@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/marvell/octeontx2/nic/otx2_ethtool.c
drivers/net/ethernet/marvell/octeontx2/nic/otx2_pf.c