From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37294) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z0D8T-0004L4-Iz for qemu-devel@nongnu.org; Wed, 03 Jun 2015 14:11:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z0D8P-0006LZ-Id for qemu-devel@nongnu.org; Wed, 03 Jun 2015 14:11:29 -0400 Received: from e17.ny.us.ibm.com ([129.33.205.207]:57873) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z0D8P-0006KZ-FX for qemu-devel@nongnu.org; Wed, 03 Jun 2015 14:11:25 -0400 Received: from /spool/local by e17.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 3 Jun 2015 14:11:24 -0400 Received: from b01cxnp22036.gho.pok.ibm.com (b01cxnp22036.gho.pok.ibm.com [9.57.198.26]) by d01dlp02.pok.ibm.com (Postfix) with ESMTP id 8E8E66E804A for ; Wed, 3 Jun 2015 14:03:08 -0400 (EDT) Received: from d01av01.pok.ibm.com (d01av01.pok.ibm.com [9.56.224.215]) by b01cxnp22036.gho.pok.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id t53IBKVY49938502 for ; Wed, 3 Jun 2015 18:11:20 GMT Received: from d01av01.pok.ibm.com (localhost [127.0.0.1]) by d01av01.pok.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id t53IBJkt023101 for ; Wed, 3 Jun 2015 14:11:20 -0400 Message-ID: <556F4343.6070801@linux.vnet.ibm.com> Date: Wed, 03 Jun 2015 14:11:15 -0400 From: "Jason J. Herne" MIME-Version: 1.0 References: <1433267209-9882-1-git-send-email-jjherne@linux.vnet.ibm.com> <1433267209-9882-2-git-send-email-jjherne@linux.vnet.ibm.com> <87pp5drzrl.fsf@neno.neno> <556F412C.9090106@linux.vnet.ibm.com> <20150603180356.GE2129@work-vm> In-Reply-To: <20150603180356.GE2129@work-vm> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 1/3] cpu: Provide vcpu throttling interface Reply-To: jjherne@linux.vnet.ibm.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Dr. David Alan Gilbert" Cc: amit.shah@redhat.com, borntraeger@de.ibm.com, qemu-devel@nongnu.org, afaerber@suse.de, quintela@redhat.com On 06/03/2015 02:03 PM, Dr. David Alan Gilbert wrote: > * Jason J. Herne (jjherne@linux.vnet.ibm.com) wrote: >> On 06/03/2015 03:56 AM, Juan Quintela wrote: >>> "Jason J. Herne" wrote: ... >>> We are checking for throotling on each cpu each 10ms. >>> But on patch 2 we can see that we only change the throotling each >>> time that we call migration_bitmap_sync(), that only happens each round >>> through all the pages. Normally auto-converge only matters for machines >>> with lots of memory, so this is going to happen each more than 10ms (we >>> change it each 4 passes). You changed it to each 2 passes, and you add >>> it a 0.2. I think that I would preffer to just have it each single >>> pass, but add a 0.1 each pass? simpler and end result would be the same? >>> >>> >> >> Well, we certainly could make it run every pass but I think it would get >> a little too aggressive then. The reason is, we do not increment the >> throttle >> rate by adding 0.2 each time. We increment it by multiplying the current >> rate >> by 2. So by doing that every pass we are doubling the exponential growth >> rate. I will admit the numbers I chose are hardly scientific... I chose them >> because they seemed to provide a decent balance of "throttling aggression" >> in >> my workloads. > > That's the advantage of making them parameters. I see your point. Expecting the user to configure these parameters seems a bit much. But I guess, in theory, it is better to have the ability to change them and not need it, than need it and not have it right? So, as you stated earlier these should hook into MigrationParams somehow? I'll admit this is the first I've seen this construct. If this is the optimal location for the two controls (x-throttle-initial, x-throttle-multiplier?) I can add them there. Will keep defaults of 0.2 for initial and 2.0 for multiplier(is there a better name?)? ... -- -- Jason J. Herne (jjherne@linux.vnet.ibm.com)