From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=51472 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OqXPN-0008Ga-Tb for qemu-devel@nongnu.org; Tue, 31 Aug 2010 16:26:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OqXPI-0000do-9n for qemu-devel@nongnu.org; Tue, 31 Aug 2010 16:26:17 -0400 Received: from mx1.redhat.com ([209.132.183.28]:61544) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OqXPI-0000db-2W for qemu-devel@nongnu.org; Tue, 31 Aug 2010 16:26:12 -0400 Date: Tue, 31 Aug 2010 23:20:17 +0300 From: "Michael S. Tsirkin" Message-ID: <20100831202017.GB31570@redhat.com> References: <20100827223659.2696.3589.stgit@s20.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100827223659.2696.3589.stgit@s20.home> 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: Alex Williamson Cc: jes.sorensen@redhat.com, qemu-devel@nongnu.org, kvm@vger.kernel.org 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 Looks good overall, but do we really want to make these parameters qdev properties? It really looks like exposing an implementation detail that we'll have hard time supporting in the future and users will have hard time understanding. OTOH developers should have no trouble tweaking a define on top of the file. > --- > > Alex Williamson (5): > virtio-net: Switch default to new bottom half TX handler for iothread > virtio-net: Introduce a new bottom half packet TX > virtio-net: Rename tx_timer_active to tx_waiting > virtio-net: Limit number of packets sent per TX flush > virtio-net: Make tx_timer timeout configurable > > > hw/s390-virtio-bus.c | 6 ++ > hw/s390-virtio-bus.h | 4 ++ > hw/syborg_virtio.c | 10 +++- > hw/virtio-net.c | 129 ++++++++++++++++++++++++++++++++++++++++---------- > hw/virtio-net.h | 2 - > hw/virtio-pci.c | 10 +++- > hw/virtio.h | 9 +++ > 7 files changed, 137 insertions(+), 33 deletions(-) > > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html