From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Dmitry Kravkov" Subject: Re: [PATCHv2-net-3.8 3/3] bnx2x: set gso_type Date: Thu, 7 Feb 2013 15:37:24 +0200 Message-ID: <1360244244.17835.1.camel@lb-tlvb-dmitry> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Eilon Greenstein , John Fastabend , e1000-devel@lists.sourceforge.net, Jesse Brandeburg , Jacob Keller , Sony Chacko , bhutchings@solarflare.com, Jitendra Kalsaria , netdev@vger.kernel.org, Bruce Allan , linux-kernel@vger.kernel.org, John Ronciak , linux-driver@qlogic.com, "David S. Miller" To: "Michael S. Tsirkin" Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: e1000-devel-bounces@lists.sourceforge.net List-Id: netdev.vger.kernel.org On Thu, 2013-02-07 at 15:13 +0200, Michael S. Tsirkin wrote: > In LRO mode, bnx2x set gso_size but not gso type. > This leads to crashes in macvtap. > Commit cbf1de72324a8105ddcc3d9ce9acbc613faea17e > queued for 3.9 includes a more complete fix. > This is a minimal patch to avoid the crash, for 3.8. > > Signed-off-by: Michael S. Tsirkin > --- > drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-) > > diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c > index f771ddf..a5edac8 100644 > --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c > +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c > @@ -504,13 +504,11 @@ static int bnx2x_fill_frag_skb(struct bnx2x *bp, struct bnx2x_fastpath *fp, > skb_shinfo(skb)->gso_size = bnx2x_set_lro_mss(bp, > tpa_info->parsing_flags, len_on_bd); > > - /* set for GRO */ > - if (fp->mode == TPA_MODE_GRO) > - skb_shinfo(skb)->gso_type = > - (GET_FLAG(tpa_info->parsing_flags, > - PARSING_FLAGS_OVER_ETHERNET_PROTOCOL) == > - PRS_FLAG_OVERETH_IPV6) ? > - SKB_GSO_TCPV6 : SKB_GSO_TCPV4; > + skb_shinfo(skb)->gso_type = > + (GET_FLAG(tpa_info->parsing_flags, > + PARSING_FLAGS_OVER_ETHERNET_PROTOCOL) == > + PRS_FLAG_OVERETH_IPV6) ? > + SKB_GSO_TCPV6 : SKB_GSO_TCPV4; > } > > Thanks! Acked-by: Dmitry Kravkov ------------------------------------------------------------------------------ Free Next-Gen Firewall Hardware Offer Buy your Sophos next-gen firewall before the end March 2013 and get the hardware for free! Learn more. http://p.sf.net/sfu/sophos-d2d-feb _______________________________________________ E1000-devel mailing list E1000-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired