From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:42375) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QkCdf-000856-8I for qemu-devel@nongnu.org; Fri, 22 Jul 2011 06:07:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QkCde-0000S3-1m for qemu-devel@nongnu.org; Fri, 22 Jul 2011 06:07:23 -0400 Received: from mx1.redhat.com ([209.132.183.28]:17464) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QkCdd-0000Rs-LM for qemu-devel@nongnu.org; Fri, 22 Jul 2011 06:07:22 -0400 Message-ID: <4E294C85.2020005@redhat.com> Date: Fri, 22 Jul 2011 12:10:13 +0200 From: Kevin Wolf MIME-Version: 1.0 References: <1311170189-29734-1-git-send-email-freddy77@gmail.com> <4E2920FE.1060904@redhat.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC PATCH 0/5] qcow: coroutines cleanup List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Frediano Ziglio Cc: qemu-devel@nongnu.org Am 22.07.2011 11:26, schrieb Frediano Ziglio: > 2011/7/22 Kevin Wolf : >> Am 20.07.2011 15:56, schrieb Frediano Ziglio: >>> These patches mostly cleanup some AIO code using coroutines. >>> These patches apply to Kevin's repository, branch coroutine-block. >>> Mostly they use stack instead of allocated AIO structure. >>> >>> Frediano Ziglio (5): >>> qcow: allocate QCowAIOCB structure using stack >>> qcow: QCowAIOCB field cleanup >>> qcow: move some blocks of code to avoid useless variable >>> initialization >>> avoid dandling pointers >>> qcow: small optimization initializing QCowAIOCB >>> >>> block/qcow.c | 210 +++++++++++++++++++++++++-------------------------------- >>> block/qcow2.c | 38 +++------- >>> 2 files changed, 102 insertions(+), 146 deletions(-) >> >> Most of it looks good now. Did you include the "RFC" in the subject just >> because the coroutine work is in RFC state, too, or did you intend to >> tell me that I shouldn't merge yet? >> >> Kevin >> > > As these patches are first quite big patches I send (typo or small > fixes do not counts) I just want to mark that I could write something > really wrong. Just a way to avoid somebody having to send more patches > and get more attention. Some projects are quite prone to merge even > not that fine ones. I prefer to have some (a bit) pedantic comments > and a real fix/improve. > > Now I removed the RFC from last update. The main reason is that I > found your qemu-iotests repository which, I think should be merged to > main repository, but it's just my opinion. > Oh... qcow fails 004 test (even origin/coroutines-block) with a I/O error. Yup, you're right, I must have messed it up. Care to fix it or should I look into it? > I must say there are a lot of small hidden things that a developer > should know about Qemu, for instance > - mailing list follow some LKML rules as CC ML and send to maintainer > to get more attention > - you can use scripts/checkpatch.pl to check your patches before send > > I still have also to understand how to use git format-patch/send-email > correctly and "fluently" :) That's true. Maybe you can update the wiki page with your findings? The one suggestion I have for your use of git format-patch is --subject-prefix="PATCH v3", so I don't get confused by the different versions ;-) Kevin