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-ia64-devel

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

To: Tomonari Horikoshi <t.horikoshi@xxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Add netfornt tx_queue_len
From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Date: Wed, 24 Jan 2007 13:29:51 +1100
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 23 Jan 2007 18:29:54 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <3DC73F584529FAt.horikoshi@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <3DC73F584529FAt.horikoshi@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.9i
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