From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MPG7w-0007Ys-28 for qemu-devel@nongnu.org; Fri, 10 Jul 2009 09:27:00 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MPG7r-0007V0-0A for qemu-devel@nongnu.org; Fri, 10 Jul 2009 09:26:59 -0400 Received: from [199.232.76.173] (port=41626 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MPG7q-0007Uo-OZ for qemu-devel@nongnu.org; Fri, 10 Jul 2009 09:26:54 -0400 Received: from mx2.redhat.com ([66.187.237.31]:33105) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MPFtc-0005VQ-Tq for qemu-devel@nongnu.org; Fri, 10 Jul 2009 09:12:13 -0400 Message-ID: <4A573DE0.1070604@redhat.com> Date: Fri, 10 Jul 2009 15:10:56 +0200 From: Kevin Wolf MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH] qemu-iotests: make a few more tests generic References: <20090708194143.GA14640@lst.de> <4A55BF62.9070702@redhat.com> <20090709132513.GA13722@lst.de> <4A56F22A.8030406@redhat.com> <4A573908.5020600@us.ibm.com> In-Reply-To: <4A573908.5020600@us.ibm.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Christoph Hellwig , qemu-devel@nongnu.org Anthony Liguori schrieb: > Kevin Wolf wrote: >> This problem has already be found. Avi's patch from almost three weeks >> ago fixes it. >> > > Christoph posted an alternative patch and there didn't seem to be a > consensus on the thread about what solution was the best one. See > http://article.gmane.org/gmane.comp.emulators.qemu/46032/match=block+clean+up+after. I know. I really don't care which one is used, but I want to have that bug fixed. > It's also up to the submitter to keep track of their patches. If they > think one should be applied that hasn't been, they need to follow up on > it. The only way to scale here is to push as much work as possible to > the outer-most nodes. We can try to help here, but we need to know what you want the process to look like. When do you want us to resend patches? When no patches have been submitted for two weeks, I hesitate to resend a patch that is only a week old. You don't want to be flooded with resends of patches that you have already taken care of or that you have planned to act on later, right? >> It really starts to get annoying. How am I supposed to work with commits >> only every other week (which is bad enough) and then patches are >> forgotten and probably won't be merged before another two weeks? I guess >> I should manage some local tree with fixes myself and move away from >> basing my work on git master... > > Complaining is all well and good but it doesn't help the problem. Your > particular compliant is about a patch that fixes a problem another patch > introduced. Greater patch velocity == greater instability because there > isn't an adequate amount of testing going on. It takes time to do > proper testing and review of patches. Well, yes and no. If a few patches come into git master each day and a patch breaks something (and the breakage is really annoying), we'll most probably have the fix on the next day. On the other hand, if the patches are two weeks in your queue first, maybe it's a bit less likely that they break things, but if they do the breakage persists until the next big patch round comes two weeks later. Also with each day of patches not committed we're more and more likely to post patches that apply on master but conflict with your queue. I would really like to see your queue flushed more often (it has been flushed more frequently in the past, so why did this change in the first place?) > It would be very helpful if you proactively tested/reviewed patches on > the mailing list and then commented appropriately. People adding > Tested-by tags to patches on the list would be a great help for instance. I'm trying to do my share here. If some formalized things like Acked-by or Tested-by tags are more helpful than "Looks good to me" comments, let me know. Kevin