From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755355Ab1KGMTL (ORCPT ); Mon, 7 Nov 2011 07:19:11 -0500 Received: from mx1.redhat.com ([209.132.183.28]:13223 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753065Ab1KGMTJ (ORCPT ); Mon, 7 Nov 2011 07:19:09 -0500 Message-ID: <4EB7CCA9.7010507@redhat.com> Date: Mon, 07 Nov 2011 13:18:49 +0100 From: Gerd Hoffmann User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.23) Gecko/20110927 Red Hat/3.1.15-1.el6_1 Thunderbird/3.1.15 MIME-Version: 1.0 To: Pekka Enberg CC: Paolo Bonzini , Sasha Levin , Blue Swirl , "kvm@vger.kernel.org list" , "linux-kernel@vger.kernel.org List" , Alexander Graf , qemu-devel Developers , Avi Kivity , =?ISO-8859-1?Q?Am=E9rico_Wang?= , Ingo Molnar , Linus Torvalds Subject: Re: [PATCH] KVM: Add wrapper script around QEMU to test kernels 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> <4EB7B1A9.9000409@redhat.com> <4EB7BACA.70006@redhat.com> In-Reply-To: X-Enigmail-Version: 1.1.2 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/07/11 12:44, Pekka Enberg wrote: > On Mon, Nov 7, 2011 at 1:02 PM, Paolo Bonzini wrote: >> Indeed I do not see any advantage, since all the interfaces they use are >> stable anyway (sysfs, msr.ko). >> >> If they had gone in x86info, for example, my distro (F16, not exactly >> conservative) would have likely picked those tools up already, but it >> didn't. > > Distributing userspace tools in the kernel tree is a relatively new > concept so it's not at all surprising distributions don't pick them up > as quickly. That doesn't mean it's a fundamentally flawed approach, > though. tools/ lacks a separation into "kernel hacker's testing+debugging toolbox" and "userspace tools". It lacks proper buildsystem integration for the userspace tools, there is no "make tools" and also no "make tools_install". Silently dropping new stuff into tools/ and expecting the world magically noticing isn't going to work. cheers, Gerd