[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7e2c75bf-3ec5-4202-8b69-04fce763e948@molgen.mpg.de>
Date: Sat, 21 Sep 2024 14:52:55 +0200
From: Paul Menzel <pmenzel@...gen.mpg.de>
To: Wander Lairson Costa <wander@...hat.com>
Cc: Tony Nguyen <anthony.l.nguyen@...el.com>,
Przemek Kitszel <przemyslaw.kitszel@...el.com>,
"David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
intel-wired-lan@...ts.osuosl.org, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [Intel-wired-lan] [PATCH 2/2] igbvf: remove unused spinlock
Dear Wander,
Thank you for your patch.
Am 20.09.24 um 20:59 schrieb Wander Lairson Costa:
> tx_queue_lock and stats_lock are declared and initialized, but never
> used. Remove them.
>
> Signed-off-by: Wander Lairson Costa <wander@...hat.com>
It’d be great if you added a Fixes: tag.
> ---
> drivers/net/ethernet/intel/igbvf/igbvf.h | 3 ---
> drivers/net/ethernet/intel/igbvf/netdev.c | 3 ---
> 2 files changed, 6 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/igbvf/igbvf.h b/drivers/net/ethernet/intel/igbvf/igbvf.h
> index 6ad35a00a287..ca6e44245a7b 100644
> --- a/drivers/net/ethernet/intel/igbvf/igbvf.h
> +++ b/drivers/net/ethernet/intel/igbvf/igbvf.h
> @@ -169,8 +169,6 @@ struct igbvf_adapter {
> u16 link_speed;
> u16 link_duplex;
>
> - spinlock_t tx_queue_lock; /* prevent concurrent tail updates */
> -
> /* track device up/down/testing state */
> unsigned long state;
>
> @@ -220,7 +218,6 @@ struct igbvf_adapter {
> /* OS defined structs */
> struct net_device *netdev;
> struct pci_dev *pdev;
> - spinlock_t stats_lock; /* prevent concurrent stats updates */
>
> /* structs defined in e1000_hw.h */
> struct e1000_hw hw;
> diff --git a/drivers/net/ethernet/intel/igbvf/netdev.c b/drivers/net/ethernet/intel/igbvf/netdev.c
> index 925d7286a8ee..02044aa2181b 100644
> --- a/drivers/net/ethernet/intel/igbvf/netdev.c
> +++ b/drivers/net/ethernet/intel/igbvf/netdev.c
> @@ -1656,12 +1656,9 @@ static int igbvf_sw_init(struct igbvf_adapter *adapter)
> if (igbvf_alloc_queues(adapter))
> return -ENOMEM;
>
> - spin_lock_init(&adapter->tx_queue_lock);
> -
> /* Explicitly disable IRQ since the NIC can be in any state. */
> igbvf_irq_disable(adapter);
>
> - spin_lock_init(&adapter->stats_lock);
> spin_lock_init(&adapter->hw.mbx_lock);
>
> set_bit(__IGBVF_DOWN, &adapter->state);
With that addressed:
Reviewed-by: Paul Menzel <pmenzel@...gen.mpg.de>
Kind regards,
Paul
Powered by blists - more mailing lists