From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:51377) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RNK87-0004Xn-Vn for qemu-devel@nongnu.org; Mon, 07 Nov 2011 03:00:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RNK87-00051j-0i for qemu-devel@nongnu.org; Mon, 07 Nov 2011 03:00:31 -0500 Received: from mx1.redhat.com ([209.132.183.28]:1028) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RNK86-00050q-N6 for qemu-devel@nongnu.org; Mon, 07 Nov 2011 03:00:30 -0500 Message-ID: <4EB79000.5040308@redhat.com> Date: Mon, 07 Nov 2011 09:00:00 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1320543320-32728-1-git-send-email-agraf@suse.de> <4EB65C5B.8070709@redhat.com> <4EB66036.4080102@redhat.com> <1320577728.1428.73.camel@jaguar> <4EB67486.1070105@redhat.com> <4EB67D17.7000701@redhat.com> <4EB680D9.2070706@redhat.com> <877C82F4-F07C-44AA-8722-3AF57CFC4597@suse.de> <4EB6DBC6.2010404@redhat.com> <4EB6E798.3060000@redhat.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] KVM: Add wrapper script around QEMU to test kernels List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Pekka Enberg Cc: "kvm@vger.kernel.org list" , qemu-devel Developers , "linux-kernel@vger.kernel.org List" , Alexander Graf , Blue Swirl , Avi Kivity , =?ISO-8859-1?Q?Am=E9rico_Wang?= , Ingo Molnar , Linus Torvalds On 11/06/2011 09:17 PM, Pekka Enberg wrote: > > No. I want to try new tool/old kernel and old tool/new kernel (kernel can > > be either guest or host, depending on the nature of the bug), and then > > bisect just one. (*) And that's the exceptional case, and only KVM tool > > developers really should have the need to do that. > > Exactly - having the source code in Linux kernel tree covers the > "exceptional case" where we're unsure which part of the equation broke > things (which are btw the nasties issues we've had so far). No, having the source code in Linux kernel tree is perfectly useless for the exceptional case, and forces you to go through extra hoops to build only one component. Small hoops such as adding "-- tools/kvm" to "git bisect start" perhaps, but still hoops that aren't traded for a practical advantage. You keep saying "oh things have been so much better" because "it's so close to the kernel" and "it worked so great for perf", but you haven't brought any practical example that we can stare at in admiration. (BTW, I'm also convinced like Ted that not having a defined perf ABI might have made sense in the beginning, but it has now devolved into bad software engineering practice). > I have no idea why you're trying to convince me that it doesn't matter. I'm not trying to convince you that it doesn't matter, I'm trying to convince you that it doesn't *make sense*. > It's a hypervisor that implements virtio drivers, serial > emulation, and mini-BIOS. ... all of which have a spec against which you should be working. Save perhaps for the mini-BIOS, if you develop against the kernel source rather than the spec you're doing it *wrong*. Very wrong. But you've been told this many times already. Paolo