[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-ia64-devel] Re: [Xen-devel] [PATCH] Add netfornt tx_queue_len
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@xxxxxxxxxxxxxxxxxxx> > Home Page: http://gondor.apana.org.au/~herbert/ > PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt > > _______________________________________________ > Xen-devel mailing list > Xen-devel@xxxxxxxxxxxxxxxxxxx > http://lists.xensource.com/xen-devel _______________________________________________ Xen-ia64-devel mailing list Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-ia64-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |