From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:37825) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UCaSS-0004Ea-FW for qemu-devel@nongnu.org; Mon, 04 Mar 2013 13:49:57 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UCaSQ-0008BG-6G for qemu-devel@nongnu.org; Mon, 04 Mar 2013 13:49:56 -0500 Received: from mx1.redhat.com ([209.132.183.28]:64036) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UCaSP-00089b-VF for qemu-devel@nongnu.org; Mon, 04 Mar 2013 13:49:54 -0500 Message-ID: <5134ED1C.5030202@redhat.com> Date: Mon, 04 Mar 2013 20:51:08 +0200 From: Orit Wasserman MIME-Version: 1.0 References: <5130ADC0.9070402@dlhnet.de> <5130B29C.3060301@redhat.com> <5130B5D8.7030308@dlhnet.de> <5130B677.1060604@redhat.com> <5130B789.7050709@dlhnet.de> <5130B9D7.2050108@redhat.com> <5130C04F.9090803@dlhnet.de> <5130D179.8070207@redhat.com> <54451010-DFA7-416A-9B8A-0762FB71DF36@dlhnet.de> In-Reply-To: <54451010-DFA7-416A-9B8A-0762FB71DF36@dlhnet.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] migration: use XBZRLE only after bulk stage List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Lieven Cc: Paolo Bonzini , "qemu-devel@nongnu.org" On 03/04/2013 07:10 PM, Peter Lieven wrote: > > Am 01.03.2013 um 17:04 schrieb Eric Blake : > >> On 03/01/2013 07:50 AM, Peter Lieven wrote: >> >>> just a quick test on my desktop: >>> >>> ~/git/qemu$ x86_64-softmmu/qemu-system-x86_64 -k de -enable-kvm -m 1024 >>> -drive >>> if=virtio,file=iscsi://172.21.200.31/iqn.2001-05.com.equallogic:0-8a0906-16470e107-713001aa6de511e0-001-test/0 >>> -vnc :1 -boot dc -monitor stdio >>> >>> using ubuntu 12.04.1 desktop and the example from docs/xbzrle.txt >> >> Thanks. Reformatting a bit: >> >>> >>> a) with the patch >> >> designated with '+' >> >>> b) without the patch >> >> designated with '-' >> >> + total time: 22185 milliseconds >> - total time: 22410 milliseconds >> >> Shaved 0.3 seconds, better than 1%! >> >> + downtime: 29 milliseconds >> - downtime: 21 milliseconds >> >> Not sure why downtime seemed worse, but probably not the end of the world. >> >> + transferred ram: 706034 kbytes >> - transferred ram: 721318 kbytes >> >> Fewer bytes sent - good. >> >> + remaining ram: 0 kbytes >> - remaining ram: 0 kbytes >> + total ram: 1057216 kbytes >> - total ram: 1057216 kbytes >> + duplicate: 108556 pages >> - duplicate: 105553 pages >> + normal: 175146 pages >> - normal: 179589 pages >> + normal bytes: 700584 kbytes >> - normal bytes: 718356 kbytes >> >> Fewer normal bytes... >> >> + cache size: 67108864 bytes >> - cache size: 67108864 bytes >> + xbzrle transferred: 3127 kbytes >> - xbzrle transferred: 630 kbytes >> >> ...and more compressed pages sent - good. >> >> + xbzrle pages: 117811 pages >> - xbzrle pages: 21527 pages >> + xbzrle cache miss: 18750 >> - xbzrle cache miss: 179589 >> >> And very good improvement on the cache miss rate. >> >> + xbzrle overflow : 0 >> - xbzrle overflow : 0 >> >> Thanks, this proves it's a good patch. > > At least for the artificially generated load. > > @Paolo: Have you seen my other question? Can the same page be transferred in the same round > more than once? If yes, I have to improve the patch for that case. The same page can't be transferred more than once in the same round. Orit > > Peter >