ibmvnic: Remove skb->protocol checks in ibmvnic_xmit

"Linux Kernel Mailing List" <[email protected]>
Newsgroups gmane.linux.kernel.commits.head
Message-ID <[email protected]>
Web:        https://git.kernel.org/torvalds/c/2fa56a494484f19e06bf4f3464b2155a92beafac
Commit:     2fa56a494484f19e06bf4f3464b2155a92beafac
Parent:     941ff6f11c020913f5cddf543a9ec63475d7c082
Refname:    refs/heads/master
Author:     John Allen <[email protected]>
AuthorDate: Fri Feb 9 13:19:46 2018 -0600
Committer:  David S. Miller <[email protected]>
CommitDate: Fri Feb 9 14:47:00 2018 -0500

    ibmvnic: Remove skb->protocol checks in ibmvnic_xmit
    
    Having these checks in ibmvnic_xmit causes problems with VLAN
    tagging and balance-alb/tlb bonding modes. The restriction they
    imposed can be removed.
    
    Signed-off-by: John Allen <[email protected]>
    Signed-off-by: Nathan Fontenot <[email protected]>
    Signed-off-by: David S. Miller <[email protected]>
---
 drivers/net/ethernet/ibm/ibmvnic.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/net/ethernet/ibm/ibmvnic.c b/drivers/net/ethernet/ibm/ibmvnic.c
index ce127a72cda2..27447260215d 100644
--- a/drivers/net/ethernet/ibm/ibmvnic.c
+++ b/drivers/net/ethernet/ibm/ibmvnic.c
@@ -1419,10 +1419,7 @@ static int ibmvnic_xmit(struct sk_buff *skb, struct net_device *netdev)
 		hdrs += 2;
 	}
 	/* determine if l2/3/4 headers are sent to firmware */
-	if ((*hdrs >> 7) & 1 &&
-	    (skb->protocol == htons(ETH_P_IP) ||
-	     skb->protocol == htons(ETH_P_IPV6) ||
-	     skb->protocol == htons(ETH_P_ARP))) {
+	if ((*hdrs >> 7) & 1) {
 		build_hdr_descs_arr(tx_buff, &num_entries, *hdrs);
 		tx_crq.v1.n_crq_elem = num_entries;
 		tx_buff->indir_arr[0] = tx_crq;
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.