bnxt_en: Remove unused field "ref_count" in struct bnxt_ulp
authorKalesh AP <kalesh-anakkur.purayil@broadcom.com>
Thu, 17 Apr 2025 17:24:47 +0000 (10:24 -0700)
committerJakub Kicinski <kuba@kernel.org>
Tue, 22 Apr 2025 01:50:34 +0000 (18:50 -0700)
The "ref_count" field in struct bnxt_ulp is unused after
commit a43c26fa2e6c ("RDMA/bnxt_re: Remove the sriov config callback").
So we can just remove it now.

Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
Signed-off-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Link: https://patch.msgid.link/20250417172448.1206107-4-michael.chan@broadcom.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c
drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.h

index a8e930d5dbb09f33e4b2afbea96d4f8bb299ba98..238db9a1aebf86fd59fdfe496d9cb02ad33b189f 100644 (file)
@@ -148,7 +148,6 @@ void bnxt_unregister_dev(struct bnxt_en_dev *edev)
        struct net_device *dev = edev->net;
        struct bnxt *bp = netdev_priv(dev);
        struct bnxt_ulp *ulp;
-       int i = 0;
 
        ulp = edev->ulp_tbl;
        netdev_lock(dev);
@@ -164,10 +163,6 @@ void bnxt_unregister_dev(struct bnxt_en_dev *edev)
        synchronize_rcu();
        ulp->max_async_event_id = 0;
        ulp->async_events_bmap = NULL;
-       while (atomic_read(&ulp->ref_count) != 0 && i < 10) {
-               msleep(100);
-               i++;
-       }
        mutex_unlock(&edev->en_dev_lock);
        netdev_unlock(dev);
        return;
index 7fa3b8d1ebd288c543efb7c429295dab7dffdeeb..f6b5efb5e7753518b17dee0181a466f1c227e376 100644 (file)
@@ -50,7 +50,6 @@ struct bnxt_ulp {
        unsigned long   *async_events_bmap;
        u16             max_async_event_id;
        u16             msix_requested;
-       atomic_t        ref_count;
 };
 
 struct bnxt_en_dev {