Xen devel - Mar 2007 - Xen netfront fixes for changed skbuff in net-2.6.22.git

Hi Herbert,

I wonder if you''ve got a chance to look at netfront in light of the new
stuff in davem''s network tree  (the stuff that''s in
http://git.kernel.org/?p=linux/kernel/git/davem/net-2.6.22.git).

In particular, struct sk_buff has been changed so that "nh" has gone,
and the replacement can be just an offset rather than a full pointer. 
This breaks the netfront because it tries to stash a page * in nh.raw. 
I had a quick look at it and couldn''t see an easy fix, but I
don''t
really understand what''s going on in there.

But you do.  Any chance you could have a look at it, and at least give
me some pointers about how to proceed?

Thanks,
    J

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Hi Jeremy:

On Wed, Mar 28, 2007 at 11:36:17AM -0700, Jeremy Fitzhardinge
wrote:
> 
> I wonder if you''ve got a chance to look at netfront in light of
the new
> stuff in davem''s network tree  (the stuff that''s in
> http://git.kernel.org/?p=linux/kernel/git/davem/net-2.6.22.git).
I''ve had a look at now and you can just stuff it into one of the other
pointers that''s still there.  We just need to make sure that it is
reset properly before we feed the packet into the stack.  The pointer
skb->dev is one candidate but there are plenty of others.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Herbert Xu wrote:
> I''ve had a look at now and you can just stuff it into one of the
other
> pointers that''s still there.  We just need to make sure that it is
> reset properly before we feed the packet into the stack.  The pointer
> skb->dev is one candidate but there are plenty of others.
>   
Hm, I was wondering if there''s a nicer way of getting the same result. 
Does it need to be done that way?

Thanks,
    J

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Wed, Mar 28, 2007 at 02:55:56PM -0700, Jeremy Fitzhardinge
wrote:
> 
> Hm, I was wondering if there''s a nicer way of getting the same
result.
> Does it need to be done that way?
Actually you could use the skb->cb buffer which can store anything and
doesn''t need to be cleaned up.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel