Skip to content

Commit b3f5c7b

Browse files
Carolyn WybornyJeff Kirsher
authored andcommitted
i40e: Fix for extra byte swap in tunnel setup
This patch fixes an issue where we were byte swapping the port parameter, then byte swapping it again in function execution. Obviously, that's unnecessary, so take it out of the function calls. Without this patch, the udp based tunnel configuration would not be correct. Change-ID: I788d83c5bd5732170f1a81dbfa0b1ac3ca8ea5b7 Signed-off-by: Carolyn Wyborny <carolyn.wyborny@intel.com> Tested-by: Andrew Bowers <andrewx.bowers@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
1 parent d4a0658 commit b3f5c7b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

drivers/net/ethernet/intel/i40e/i40e_main.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7154,9 +7154,9 @@ static void i40e_sync_udp_filters_subtask(struct i40e_pf *pf)
71547154
pf->pending_udp_bitmap &= ~BIT_ULL(i);
71557155
port = pf->udp_ports[i].index;
71567156
if (port)
7157-
ret = i40e_aq_add_udp_tunnel(hw, ntohs(port),
7158-
pf->udp_ports[i].type,
7159-
NULL, NULL);
7157+
ret = i40e_aq_add_udp_tunnel(hw, port,
7158+
pf->udp_ports[i].type,
7159+
NULL, NULL);
71607160
else
71617161
ret = i40e_aq_del_udp_tunnel(hw, i, NULL);
71627162

0 commit comments

Comments
 (0)