From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wei Liu Subject: Re: [PATCH 2/4] xen-netfront: drop skb when skb->len > 65535 Date: Thu, 21 Mar 2013 14:20:13 +0000 Message-ID: <1363875613.2160.17.camel__6204.81461887739$1363875794$gmane$org@zion.uk.xensource.com> References: <1363602955-24790-1-git-send-email-wei.liu2@citrix.com> <1363602955-24790-3-git-send-email-wei.liu2@citrix.com> <51471AAC.2050509@citrix.com> <1363614500.30193.47.camel@zakaz.uk.xensource.com> <51471DE0.9060506@citrix.com> <514A15C8.1050209@citrix.com> <1363873208.2160.13.camel@zion.uk.xensource.com> <514B151A.80705@citrix.com> <1363875350.2160.16.camel@zion.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1363875350.2160.16.camel@zion.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: David Vrabel Cc: wei.liu2@citrix.com, Ian Campbell , "konrad.wilk@oracle.com" , "netdev@vger.kernel.org" , "xen-devel@lists.xen.org" , "annie.li@oracle.com" List-Id: xen-devel@lists.xenproject.org On Thu, 2013-03-21 at 14:15 +0000, Wei Liu wrote: > On Thu, 2013-03-21 at 14:11 +0000, David Vrabel wrote: > > On 21/03/13 13:40, Wei Liu wrote: > > > > > > > > > I think the correct action is to just take first txp and loop responding > > > until we consume the whole packet. > > > > Um. This is what the patch is doing. > > > > No. The idx you passed in is the index of the first txp, and idx + fras > doesn't necessary point to last tx requests of the packet. We should use > XEN_NETTXF_more_data to loop through the packet. > Sorry for the noise, this patch already loop through the whole packet. Wei. > > Wei. > > > David > >