From: Carolyn Wyborny Date: Wed, 24 Aug 2016 18:33:51 +0000 (-0700) Subject: i40e: Fix for extra byte swap in tunnel setup X-Git-Tag: v4.1.12-98.0.20170517_2143~42^2~183 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=381fb752b89c2b3a168e7b42ae4d1c78cf03a3e7;p=users%2Fjedix%2Flinux-maple.git i40e: Fix for extra byte swap in tunnel setup Orabug: 24568124 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 Tested-by: Andrew Bowers Signed-off-by: Jeff Kirsher (cherry picked from commit b3f5c7bc88bab134e9649e42d30be15e3775f00d) Signed-off-by: Brian Maly Signed-off-by: Dhaval Giani --- diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index 1e9e701953268..1bbcbef7440c8 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -7059,9 +7059,9 @@ static void i40e_sync_udp_filters_subtask(struct i40e_pf *pf) pf->pending_udp_bitmap &= ~BIT_ULL(i); port = pf->udp_ports[i].index; if (port) - ret = i40e_aq_add_udp_tunnel(hw, ntohs(port), - pf->udp_ports[i].type, - NULL, NULL); + ret = i40e_aq_add_udp_tunnel(hw, port, + pf->udp_ports[i].type, + NULL, NULL); else ret = i40e_aq_del_udp_tunnel(hw, i, NULL);