Re: [Xen-devel] [PATCH] Add netfornt tx_queue_len

[Tomonari Horikoshi <t.horikoshi@jp.fujitsu.com>]

Hi Herbert-san

Thank you for your comment.

I agreed.
I examine the another way. 

It is likely to go well if something is added to the check 
on netfront_tx_slot_available(). 

Best regards.

Tomonari Horikoshi,

Herbert Xu wrote:----------------------
Sent:    Wed, 24 Jan 2007 13:29:51 +1100
Subject: Re: [Xen-devel] [PATCH] Add netfornt tx_queue_len

> On Wed, Jan 24, 2007 at 01:37:55AM +0000, Tomonari Horikoshi wrote:
> > 
> > When I executed "netperf" by a short message of UDP, 
> > PV domain and PV-on-HVM driver issued Call trace.
> > 
> > I think that GrantEntry was filled with a lot of messages processings.
> > 
> > This problem is generated in IA64 only.
> > Probably, I think that I am the following problems. 
> > 
> >   In IA64
> >     NET_TX_RING_SIZE 1024,  NR_GRANT_ENTRIES 2048
> >   In x86
> >     NET_TX_RING_SIZE  256,  NR_GRANT_ENTRIES 2048
> > 
> > I corrected to check "number of unprocessing queue > tx_queue_len" before Grant was filled.
> > 
> > However, my correction influences x86. 
> > Please teach to me in that when there is a better improvement. 
> 
> Sorry, but this patch looks bogus.  The tx queue is maintained by
> Linux and has nothing to do with the driver.  So limiting its length
> based on internal state of the driver can't be right.
> 
> We need to find out what's really going wrong with the grant table
> entries here.
> 
> 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