WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH 2/2] netfront skb padding

On Wednesday 09 February 2005 11:26 am, Ian Pratt wrote:
> > > > It appears that when alloc'ing a skb, it is bring padded by 
> > > > an arbitrarily 
> > > > (and excessive) long value.  The value for this padding 
> > > > really only needs to 
> > > > be 24.  24 = 14 for the ethernet header + 2 for the cache 
> > > > alignment + 4 for 
> > > > the CRC + 4 for the VLAN flags.
> > > 
> > > Given that we're allocating page sized buffers the current situation
> > > doesn't cost us anything.
> > 
> > Unless it starts using larger packets, e.g. Jumbo Frames 
> > (hint hint).  Then
> > the unnecessary room can be a problem, as the unnecessary pad 
> > could cause the
> > unnecessary allocation of an extra page.
> 
> How? Ethernet packets are circa 1500 bytes, and we allocate a 4K page
> for each so that we can page flip them.

Jumbo frames can be 16k (or possibly larger), that would be 4 pages + 1 
unnecessary page containing the 200 bytes.
 
> > > Infact, what happens if the packet gets encapsulated e.g. by etherip
> > > etc? Is Linux smart enough to be able to put the extra headers on
> > > in-place if there is enough head room?
> > 
> > I would assume that they would have to be there.
> 
> They wouldn't have to be, because Linux could just copy the packet, into
> a new skb, but I believe it could use skb pull etc to see if it could
> allocate space for headers on the front of the packet. In which case,
> having more headroom would be a good thing.

Sure, but we only have 1 layer (not the 9 that we have room for).  All other 
layers would be encapsulated inside the data of the skb (or am I not 
understanding the problem correctly).


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/xen-devel