From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36152) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ePXEt-0002Sv-Lj for qemu-devel@nongnu.org; Thu, 14 Dec 2017 12:24:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ePXEs-0003IG-W7 for qemu-devel@nongnu.org; Thu, 14 Dec 2017 12:24:07 -0500 References: <20171214005953.8898-1-jsnow@redhat.com> <20171214005953.8898-5-jsnow@redhat.com> <0ac30feb-553b-cfa3-070a-0f703c4e0f10@redhat.com> <1b7494a6-aca5-41e8-469b-e2b9d8aade67@redhat.com> From: Paolo Bonzini Message-ID: Date: Thu, 14 Dec 2017 18:23:50 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 4/7] blockjob: allow block_job_throttle to take delay_ns List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: John Snow , qemu-block@nongnu.org Cc: kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com, mreitz@redhat.com On 14/12/2017 18:22, John Snow wrote: >> Ok, shall we wait for v2 where you look at that BZ as well? > > Sure -- if the only feedback you have on this series is primarily style > and "maybe there are some more wins" I can spin a v2 to try to broaden > the scope if it looks good so far. Yeah, that's pretty much it (except that it isn't "maybe there are some more wins", but rather "going from two to three functions isn't exactly what I was hoping for" :)). Paolo