[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH net-next] xen-netback: fix xenvif_count_skb_slots()
> -----Original Message----- > From: Matt Wilson [mailto:mswilson@xxxxxxxxx] On Behalf Of Matt Wilson > Sent: 07 October 2013 01:08 > To: Paul Durrant > Cc: xen-devel@xxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; Xi Xiong; Matt Wilson; > Annie Li; Wei Liu; Ian Campbell; Simon Graham > Subject: Re: [PATCH net-next] xen-netback: fix xenvif_count_skb_slots() > > On Sun, Oct 06, 2013 at 05:06:52PM -0700, Matt Wilson wrote: > > On Fri, Oct 04, 2013 at 05:26:23PM +0100, Paul Durrant wrote: > > > Commit 4f0581d25827d5e864bcf07b05d73d0d12a20a5c introduced an > error into > > > xenvif_count_skb_slots() for skbs with a linear area spanning a page > > > boundary. The alignment of skb->data needs to be taken into account, > not > > > just the head length. This patch fixes the issue by dry-running the code > > > from xenvif_gop_skb() (and adjusting the comment above the function > to note > > > that). > > > > > > Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx> > > > Cc: Xi Xiong <xixiong@xxxxxxxxxx> > > > Cc: Matt Wilson <msw@xxxxxxxxxx> > > > Cc: Annie Li <annie.li@xxxxxxxxxx> > > > Cc: Wei Liu <wei.liu2@xxxxxxxxxx> > > > Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxx> > > > > Paul, can you reconcile this change with the one made by Simon in cs > > 8f985b4f7a5394c8f8725a5109451a541ddb9eea? > > Correction: e26b203ede31fffd52571a5ba607a26c79dc5c0d > The comment is possibly correct with modified ring packing but the problem is that by reducing that count netback now tries to handle more skbs than it has grant copy slots for. Maybe it would be more appropriate to simply revert 4f0581d25827d5e864bcf07b05d73d0d12a20a5c. I see no problem before that patch was applied. Paul _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |