Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754936AbdCGIY6 (ORCPT ); Tue, 7 Mar 2017 03:24:58 -0500 Received: from 92-243-34-74.adsl.nanet.at ([92.243.34.74]:43390 "EHLO mail.osadl.at" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1753449AbdCGIYu (ORCPT ); Tue, 7 Mar 2017 03:24:50 -0500 Date: Tue, 7 Mar 2017 08:13:19 +0000 From: Nicholas Mc Guire To: Josh Poimboeuf Cc: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, Masami Hiramatsu , Adrian Hunter , Arnaldo Carvalho de Melo , linux-kernel@vger.kernel.org Subject: Re: [PATCH] objtool: drop redundant flags generation Message-ID: <20170307081319.GA31027@osadl.at> References: <1488819625-27395-1-git-send-email-der.herr@hofr.at> <20170306172537.uox2f7ru3x72jzea@treble> <20170306175401.GA26026@osadl.at> <20170306214054.tobodci764xvbwr7@treble> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170306214054.tobodci764xvbwr7@treble> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1759 Lines: 39 On Mon, Mar 06, 2017 at 03:40:54PM -0600, Josh Poimboeuf wrote: > On Mon, Mar 06, 2017 at 05:54:01PM +0000, Nicholas Mc Guire wrote: > > On Mon, Mar 06, 2017 at 11:25:37AM -0600, Josh Poimboeuf wrote: > > > > arch/x86/tools/gen-insn-attr-x86.awk | 12 ++++++++++-- > > > > tools/objtool/arch/x86/insn/gen-insn-attr-x86.awk | 12 ++++++++++-- > > > > > > There's actually a third copy of the decoder in: > > > > > > tools/perf/util/intel-pt-decoder/ > > > > > > Yes, the duplication is a pain, but it's part of an effort to keep > > > 'tools/*' source independent from kernel code. > > > > > > Maybe we can at least combine the objtool and perf versions someday. > > > > > Bad - missed that one - did not build perf - the generator seems to > > be the same though only differing by a single blank line - so pulling > > those together should be a non-issue atleast with respect to the > > generator as the x86-opcode-map.txt are all the same ? ...or what > > fun am I missing ? > > In theory, all three copies of the decoder should be identical. That > includes all the files: insn.[ch], inat.[ch], inat_types.h, > gen-insn-attr-x86.awk, x86-opcode-map.txt. > Understood - but this is a different problem that is being addressed with this cleanup - the duplicates make no sense in any case as far as I can see - with or without consolidation of the other files (and x86-opcode-map.txt does seem to be the same in all 3 cases) - the point was that it is causing a quite large number of coccicheck warnings which are iritating and in this case easy to remove. So the question is does it make sense to fix up this one aspect or is this "fixing" an intermediate mess only that will go away, once consolidation happens, anyway. thx! hofrat