From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=52643 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OqZHV-0002wG-Bt for qemu-devel@nongnu.org; Tue, 31 Aug 2010 18:26:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OqZHQ-0003df-Cd for qemu-devel@nongnu.org; Tue, 31 Aug 2010 18:26:17 -0400 Received: from mx1.redhat.com ([209.132.183.28]:48254) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OqZHQ-0003dT-5V for qemu-devel@nongnu.org; Tue, 31 Aug 2010 18:26:12 -0400 From: Alex Williamson In-Reply-To: <4C7D751B.8070705@codemonkey.ws> References: <20100827223659.2696.3589.stgit@s20.home> <20100831202705.GE31570@redhat.com> <4C7D751B.8070705@codemonkey.ws> Content-Type: text/plain; charset="UTF-8" Date: Tue, 31 Aug 2010 16:26:07 -0600 Message-ID: <1283293567.5826.147.camel@x201> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH 0/5] virtio-net: More configurability and bh handling for tx List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: jes.sorensen@redhat.com, kraxel@redhat.com, qemu-devel@nongnu.org, kvm@vger.kernel.org, "Michael S. Tsirkin" On Tue, 2010-08-31 at 16:33 -0500, Anthony Liguori wrote: > On 08/31/2010 03:27 PM, Michael S. Tsirkin wrote: > > On Fri, Aug 27, 2010 at 04:36:59PM -0600, Alex Williamson wrote: > > > >> Add the ability to configure the tx_timer timeout and add a bottom > >> half tx handler that typically shows a nice perf boost over the > >> time based approach. See last patch for perf details. Make this > >> the new default when the iothread is enabled. Thanks, > >> > >> Alex > >> > > As a further thought, maybe it would help to have a > > separate namespace for unsupported, developer-only parameters. > > Thoughts? > > > > We already have an undocumented one, just prefix with 'x-' to indicate > that it's experimental. Any objection then to x-txburst and x-txtimer then? Easy enough to rename. Thanks, Alex