[PATCH] net/ixgbe: do not update link status in secondary process

[Stephen Hemminger]

The code to update link status is not safe in secondary process.
If called from secondary it will crash, example from dumpcap:
        ixgbe_dev_link_update_share()
        ixgbe_dev_link_update()
        rte_eth_link_get()

Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
Reported-by: Jun Wang <junwan...@cestc.cn>
---
Simpler version of earlier patch, and add explanation.

 drivers/net/ixgbe/ixgbe_ethdev.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c
index c61c52b2966b..86ccbdd78292 100644
--- a/drivers/net/ixgbe/ixgbe_ethdev.c
+++ b/drivers/net/ixgbe/ixgbe_ethdev.c
@@ -4293,6 +4293,9 @@ ixgbe_dev_link_update_share(struct rte_eth_dev *dev,
        int wait = 1;
        u32 esdp_reg;
 
+       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
+               return -1;
+
        memset(&link, 0, sizeof(link));
        link.link_status = RTE_ETH_LINK_DOWN;
        link.link_speed = RTE_ETH_SPEED_NUM_NONE;
-- 
2.43.0