From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sowmini Varadhan Subject: Re: [PATCH v2 net-next] ixgbe: Avoid unaligned access in ixgbe_atr() for LLC packets Date: Mon, 14 Mar 2016 13:59:09 -0400 Message-ID: <20160314175909.GH5084@oracle.com> References: <20160314174624.GG5084@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: intel-wired-lan , Netdev , Jeff Kirsher , "Brandeburg, Jesse" , shannon nelson , Carolyn Wyborny , "Skidmore, Donald C" , Bruce W Allan , John Ronciak , Mitch Williams To: Alexander Duyck Return-path: Received: from userp1040.oracle.com ([156.151.31.81]:34406 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753998AbcCNR7o (ORCPT ); Mon, 14 Mar 2016 13:59:44 -0400 Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On (03/14/16 10:55), Alexander Duyck wrote: > > One other thing I forgot to mention is that we don't support ARP so > that check could be dropped. The ATR code only supports IPv4 or IPv6 > with TCP. I did notice that, but I left it in place because (a) it comes down the stack with the NET_IP_ALIGNment and (b) ARP is only sent over Ethernet II (there is no LLC SAP for ARP, which is a big reason why ipv4 is not sent over llc, despite rfc 1042). I figured it would not hurt to pass it down, in case we decide to do something clever with it in the future. --Sowmini