|
|
|
|
|
|
|
|
|
|
xen-devel
Re: Re: [mdlabriola@xxxxxxxxx: [Xen-devel] Re: Re: [Xen-users] DomU rout
On Tue, 2010-05-11 at 01:43 +0100, Matej Zary wrote:
> On Mon, 2010-05-10 at 16:32 +0100, Ian Campbell wrote:
> > On Mon, 2010-05-10 at 16:18 +0100, Konrad Rzeszutek Wilk wrote:
> > > Ian had a patch that he backported from XCP that might fix this, not
> > > sure thought.
> >
> > If possible can someone who is seeing this issue try this patch:
> >
> > ---
> >
> > xen/netback: correctly setup skb->ip_summed on receive
> >
> > In 2.6.18 CHECKSUM_PARTIAL and CHECKSUM_UNNECESSARY were both synonyms for
> > CHECKSUM_HW. This is no longer the case and we need to select the correct
> > one.
> >
> > data_validated csum_blank -> ip_summed
> > 0 0 CHECKSUM_NONE
> > 0 1 CHECKSUM_PARTIAL
> > 1 0 CHECKSUM_UNNECESSARY
> > 1 1 CHECKSUM_PARTIAL
> >
> > Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> >
> > diff -r 3a1424756f50 drivers/xen/netback/netback.c
> > --- a/drivers/xen/netback/netback.c Fri May 07 14:38:37 2010 +0100
> > +++ b/drivers/xen/netback/netback.c Fri May 07 14:40:31 2010 +0100
> > @@ -1384,14 +1384,10 @@
> > netif_idx_release(pending_idx);
> > }
> >
> > - /*
> > - * Old frontends do not assert data_validated but we
> > - * can infer it from csum_blank so test both flags.
> > - */
> > - if (txp->flags & (NETTXF_data_validated|NETTXF_csum_blank))
> > + if (txp->flags & NETTXF_csum_blank)
> > skb->ip_summed = CHECKSUM_PARTIAL;
> > - else
> > - skb->ip_summed = CHECKSUM_NONE;
> > + else if (txp->flags & NETTXF_data_validated)
> > + skb->ip_summed = CHECKSUM_UNNECESSARY;
> >
> > netbk_fill_frags(skb);
> >
> >
> >
> Works for me!
Thanks everyone who tested.
Jeremy/Konrad,
The following changes since commit 7ec9d1e805338fd18bbce73e8c72e4d09da8f2d5:
Ian Campbell (1):
xen/netback: correctly setup skb->ip_summed on receive
are available in the git repository at:
git://xenbits.xensource.com/people/ianc/linux-2.6.git for-jeremy/netback
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
|
|