From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:57073) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TTt8B-0005Eq-1J for qemu-devel@nongnu.org; Thu, 01 Nov 2012 07:40:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TTt89-0007zf-Ve for qemu-devel@nongnu.org; Thu, 01 Nov 2012 07:40:14 -0400 Received: from mx1.redhat.com ([209.132.183.28]:11793) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TTt89-0007qI-Ma for qemu-devel@nongnu.org; Thu, 01 Nov 2012 07:40:13 -0400 Date: Thu, 1 Nov 2012 13:42:23 +0200 From: "Michael S. Tsirkin" Message-ID: <20121101114223.GB3993@redhat.com> References: <20121012173423.27389.27726.malonedeb@chaenomeles.canonical.com> <20121015214606.18996.44940.malone@gac.canonical.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20121015214606.18996.44940.malone@gac.canonical.com> Subject: Re: [Qemu-devel] [Bug 1066055] Re: Network performance regression with vde_switch List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Bug 1066055 <1066055@bugs.launchpad.net> Cc: qemu-devel@nongnu.org On Mon, Oct 15, 2012 at 09:46:06PM -0000, Edivaldo de Araujo Pereira wrote: > Hi Stefan, > > Thank you, very much for taking the time to help me, and excuse me for > not seeing your answer early... > > I've run the procedure you pointed me out, and the result is: > > 0d8d7690850eb0cf2b2b60933cf47669a6b6f18f is the first bad commit > commit 0d8d7690850eb0cf2b2b60933cf47669a6b6f18f > Author: Amit Shah > Date: Tue Sep 25 00:05:15 2012 +0530 > > virtio: Introduce virtqueue_get_avail_bytes() > > The current virtqueue_avail_bytes() is oddly named, and checks if a > particular number of bytes are available in a vq. A better API is to > fetch the number of bytes available in the vq, and let the caller do > what's interesting with the numbers. > > Introduce virtqueue_get_avail_bytes(), which returns the number of bytes > for buffers marked for both, in as well as out. virtqueue_avail_bytes() > is made a wrapper over this new function. > > Signed-off-by: Amit Shah > Signed-off-by: Michael S. Tsirkin > > :040000 040000 1a58b06a228651cf844621d9ee2f49b525e36c93 > e09ea66ce7f6874921670b6aeab5bea921a5227d M hw > > I tried to revert that patch in the latest version, but it obviously > didnt work; I'm trying to figure out the problem, but I don't know very > well the souce code, so I think it's going to take some time. For now, > it's all I could do. > > Thank you, again. > Edivaldo Could you please try reverting the following two patches: git revert 0d8d7690850eb0cf2b2b60933cf47669a6b6f18f git revert ad3005ad8c70a69705149d3ce6d1e51fb76edb15 the result will build and will confirm that this is the problem. Amit, could you please clarify what kind of performance gain if any do you see from ad3005ad8c70a69705149d3ce6d1e51fb76edb15? If it's small maybe we should simply revert? -- MST