* [PATCH net-next] i40e: remove redundant assigment
@ 2020-09-08 12:34 Marek Majtyka
2020-09-09 13:56 ` [Intel-wired-lan] " Björn Töpel
2020-10-13 6:34 ` Kuruvinakunnel, George
0 siblings, 2 replies; 3+ messages in thread
From: Marek Majtyka @ 2020-09-08 12:34 UTC (permalink / raw)
To: magnus.karlsson, bjorn.topel, intel-wired-lan, jeffrey.t.kirsher
Cc: Marek Majtyka, maciej.fijalkowski, maciejromanfijalkowski, netdev
Remove a redundant assigment of the software ring pointer in the i40e
driver. The variable is assigned twice with no use in between, so just
get rid of the first occurrence.
Fixes: 3b4f0b66c2b3 ("i40e, xsk: Migrate to new MEM_TYPE_XSK_BUFF_POOL")
Signed-off-by: Marek Majtyka <marekx.majtyka@intel.com>
---
drivers/net/ethernet/intel/i40e/i40e_xsk.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/intel/i40e/i40e_xsk.c b/drivers/net/ethernet/intel/i40e/i40e_xsk.c
index 2a1153d8957b..8661f461f620 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_xsk.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_xsk.c
@@ -306,7 +306,6 @@ int i40e_clean_rx_irq_zc(struct i40e_ring *rx_ring, int budget)
continue;
}
- bi = i40e_rx_bi(rx_ring, rx_ring->next_to_clean);
size = (qword & I40E_RXD_QW1_LENGTH_PBUF_MASK) >>
I40E_RXD_QW1_LENGTH_PBUF_SHIFT;
if (!size)
--
2.20.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Intel-wired-lan] [PATCH net-next] i40e: remove redundant assigment
2020-09-08 12:34 [PATCH net-next] i40e: remove redundant assigment Marek Majtyka
@ 2020-09-09 13:56 ` Björn Töpel
2020-10-13 6:34 ` Kuruvinakunnel, George
1 sibling, 0 replies; 3+ messages in thread
From: Björn Töpel @ 2020-09-09 13:56 UTC (permalink / raw)
To: Marek Majtyka
Cc: Karlsson, Magnus, Björn Töpel, intel-wired-lan,
Jeff Kirsher, Netdev, Fijalkowski, Maciej, Marek Majtyka,
maciejromanfijalkowski
On Tue, 8 Sep 2020 at 15:53, Marek Majtyka <alardam@gmail.com> wrote:
>
> Remove a redundant assigment of the software ring pointer in the i40e
> driver. The variable is assigned twice with no use in between, so just
> get rid of the first occurrence.
>
> Fixes: 3b4f0b66c2b3 ("i40e, xsk: Migrate to new MEM_TYPE_XSK_BUFF_POOL")
> Signed-off-by: Marek Majtyka <marekx.majtyka@intel.com>
Nice!
Acked-by: Björn Töpel <bjorn.topel@intel.com>
Hmm, this could actually go to the net tree.
Björn
> ---
> drivers/net/ethernet/intel/i40e/i40e_xsk.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_xsk.c b/drivers/net/ethernet/intel/i40e/i40e_xsk.c
> index 2a1153d8957b..8661f461f620 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_xsk.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_xsk.c
> @@ -306,7 +306,6 @@ int i40e_clean_rx_irq_zc(struct i40e_ring *rx_ring, int budget)
> continue;
> }
>
> - bi = i40e_rx_bi(rx_ring, rx_ring->next_to_clean);
> size = (qword & I40E_RXD_QW1_LENGTH_PBUF_MASK) >>
> I40E_RXD_QW1_LENGTH_PBUF_SHIFT;
> if (!size)
> --
> 2.20.1
>
> _______________________________________________
> Intel-wired-lan mailing list
> Intel-wired-lan@osuosl.org
> https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [Intel-wired-lan] [PATCH net-next] i40e: remove redundant assigment
2020-09-08 12:34 [PATCH net-next] i40e: remove redundant assigment Marek Majtyka
2020-09-09 13:56 ` [Intel-wired-lan] " Björn Töpel
@ 2020-10-13 6:34 ` Kuruvinakunnel, George
1 sibling, 0 replies; 3+ messages in thread
From: Kuruvinakunnel, George @ 2020-10-13 6:34 UTC (permalink / raw)
To: Marek Majtyka, Karlsson, Magnus, Topel, Bjorn, intel-wired-lan,
Kirsher, Jeffrey T
Cc: netdev, Fijalkowski, Maciej, Majtyka, MarekX, maciejromanfijalkowski
From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of Marek Majtyka
Sent: Tuesday, September 8, 2020 6:05 PM
To: Karlsson, Magnus <magnus.karlsson@intel.com>; Topel, Bjorn <bjorn.topel@intel.com>; intel-wired-lan@lists.osuosl.org; Kirsher, Jeffrey T <jeffrey.t.kirsher@intel.com>
Cc: netdev@vger.kernel.org; Fijalkowski, Maciej <maciej.fijalkowski@intel.com>; Majtyka, MarekX <marekx.majtyka@intel.com>; maciejromanfijalkowski@gmain.com
Subject: [Intel-wired-lan] [PATCH net-next] i40e: remove redundant assigment
Remove a redundant assigment of the software ring pointer in the i40e driver. The variable is assigned twice with no use in between, so just get rid of the first occurrence.
Fixes: 3b4f0b66c2b3 ("i40e, xsk: Migrate to new MEM_TYPE_XSK_BUFF_POOL")
Signed-off-by: Marek Majtyka <marekx.majtyka@intel.com>
---
drivers/net/ethernet/intel/i40e/i40e_xsk.c | 1 -
1 file changed, 1 deletion(-)
Tested-by: George Kuruvinakunnel <george.kuruvinakunnel@intel.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-10-13 6:35 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-08 12:34 [PATCH net-next] i40e: remove redundant assigment Marek Majtyka
2020-09-09 13:56 ` [Intel-wired-lan] " Björn Töpel
2020-10-13 6:34 ` Kuruvinakunnel, George
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).