From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Dumazet Subject: Re: [PATCH 3/4 net-next] net: make GRO aware of skb->head_frag Date: Mon, 30 Apr 2012 19:54:06 +0200 Message-ID: <1335808446.2296.7.camel@edumazet-glaptop> References: <1335523026.2775.236.camel@edumazet-glaptop> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: netdev , Neal Cardwell , Tom Herbert , Jeff Kirsher , Michael Chan , Matt Carlson , Herbert Xu , Ben Hutchings , Ilpo =?ISO-8859-1?Q?J=E4rvinen?= , Maciej =?UTF-8?Q?=C5=BBenczykowski?= To: David Miller Return-path: Received: from mail-ey0-f174.google.com ([209.85.215.174]:59227 "EHLO mail-ey0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754920Ab2D3RyM (ORCPT ); Mon, 30 Apr 2012 13:54:12 -0400 Received: by eaaq12 with SMTP id q12so757655eaa.19 for ; Mon, 30 Apr 2012 10:54:11 -0700 (PDT) In-Reply-To: <1335523026.2775.236.camel@edumazet-glaptop> Sender: netdev-owner@vger.kernel.org List-ID: On Fri, 2012-04-27 at 12:37 +0200, Eric Dumazet wrote: > From: Eric Dumazet > ... > + first_offset = skb->head - > + (unsigned char *)page_address(page) + > + offset; > + > + pinfo->nr_frags = nr_frags + 1 + skbinfo->nr_frags; > + Oops. I realize this is a previous version of this patch. skb->head should be changed by skb->data here, I'll post a v2 for this GRO change.