The code to update link status is not safe in secondary process.
If called from secondary it will crash, example from dumpcap:
    eth_em_link_update

Signed-off-by: Jun Wang <junwan...@cestc.cn>
---
 drivers/net/e1000/em_ethdev.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/net/e1000/em_ethdev.c b/drivers/net/e1000/em_ethdev.c
index c5a4dec..f6875b0 100644
--- a/drivers/net/e1000/em_ethdev.c
+++ b/drivers/net/e1000/em_ethdev.c
@@ -1136,6 +1136,9 @@ static int eth_em_pci_remove(struct rte_pci_device 
*pci_dev)
        struct rte_eth_link link;
        int link_up, count;
 
+       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
+               return -1;
+
        link_up = 0;
        hw->mac.get_link_status = 1;
 
-- 
1.8.3.1



Reply via email to