From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752690AbeBFKy4 (ORCPT ); Tue, 6 Feb 2018 05:54:56 -0500 Received: from mail-wr0-f196.google.com ([209.85.128.196]:46606 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752183AbeBFKyt (ORCPT ); Tue, 6 Feb 2018 05:54:49 -0500 X-Google-Smtp-Source: AH8x224YfLTxzpx6F290uLQRsXBge7d1Ech971h8Qjpkemvg+2M9qIMko6KlmWTXzPfufeZkh1dEtg== Date: Tue, 6 Feb 2018 11:54:45 +0100 From: Ingo Molnar To: David Woodhouse Cc: Arnaldo Carvalho de Melo , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Arnaldo Carvalho de Melo , Adrian Hunter , Dave Hansen , David Ahern , Fenghua Yu , Jiri Olsa , Namhyung Kim , Paolo Bonzini , Thomas Gleixner , Wang Nan Subject: Re: [PATCH 05/10] tools headers: Synchoronize x86 features UAPI headers Message-ID: <20180206105445.x3gqpaa3rwbnhugo@gmail.com> References: <20180205195620.9010-1-acme@kernel.org> <20180205195620.9010-6-acme@kernel.org> <1517911838.31953.208.camel@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1517911838.31953.208.camel@infradead.org> User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * David Woodhouse wrote: > > > On Mon, 2018-02-05 at 16:56 -0300, Arnaldo Carvalho de Melo wrote: > > > > None will entail changes in the tools/perf/, synchronizing to elliminate > > these perf build warnings: > > > >   Warning: Kernel ABI header at 'tools/arch/x86/include/asm/disabled-features.h' differs from latest version at 'arch/x86/include/asm/disabled-features.h' > >   Warning: Kernel ABI header at 'tools/arch/x86/include/asm/required-features.h' differs from latest version at 'arch/x86/include/asm/required-features.h' > >   Warning: Kernel ABI header at 'tools/arch/x86/include/asm/cpufeatures.h' differs from latest version at 'arch/x86/include/asm/cpufeatures.h' > > Ick. Have we considered just using a symlink? Why have copies of the > same header file in different places in the tree, and tooling to > complain i̶f̶when they get out of sync? In the past we had incidents where kernel changes broke tooling and vice versa. This is basically a soft-COW mechanism that decouples tooling source code from the kernel source code, while still having a technological mechanism in place that encourages the syncing of header files. Thanks, Ingo