From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH 0/5] virtio-net: More configurability and bh handling for tx Date: Tue, 31 Aug 2010 23:27:16 +0300 Message-ID: <20100831202705.GE31570@redhat.com> References: <20100827223659.2696.3589.stgit@s20.home> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org, anthony@codemonkey.ws, jes.sorensen@redhat.com, kraxel@redhat.com To: Alex Williamson Return-path: Received: from mx1.redhat.com ([209.132.183.28]:39217 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753443Ab0HaUdK (ORCPT ); Tue, 31 Aug 2010 16:33:10 -0400 Content-Disposition: inline In-Reply-To: <20100827223659.2696.3589.stgit@s20.home> Sender: kvm-owner@vger.kernel.org List-ID: 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? > --- > > 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