Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756157AbcCPVIP (ORCPT ); Wed, 16 Mar 2016 17:08:15 -0400 Received: from mail-wm0-f67.google.com ([74.125.82.67]:34716 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755856AbcCPVIL (ORCPT ); Wed, 16 Mar 2016 17:08:11 -0400 Date: Wed, 16 Mar 2016 22:06:02 +0100 From: Emese Revfy To: Masahiro Yamada Cc: Linux Kbuild mailing list , pageexec@freemail.hu, spender@grsecurity.net, kernel-hardening@lists.openwall.com, Michal Marek , Kees Cook , Rasmus Villemoes , fengguang.wu@intel.com, Dmitry Vyukov , Linux Kernel Mailing List Subject: Re: [PATCH v5 2/5] GCC plugin infrastructure Message-Id: <20160316220602.5269b38c162daa97f3cbe044@gmail.com> In-Reply-To: References: <20160307000208.1bec3e7dc874489d1b4fcbb4@gmail.com> <20160307000427.c82f18670568e1e656fc9532@gmail.com> <20160314215217.7fa20e0c85bfda75dc11b70c@gmail.com> X-Mailer: Sylpheed 3.4.3 (GTK+ 2.24.29; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 887 Lines: 24 On Wed, 16 Mar 2016 16:41:36 +0900 Masahiro Yamada wrote: > > The scripts name also doesn't describe gcc plugins well. > > With my suggestion "scripts/gcc-plugins/", > the sub-directory name describes it very well. > > > > Plugins take part in the image building process > > in a different way than these tools and scripts do. > > Since there doesn't seen to be a good place for compiler plugins, maybe we should create a new toplevel directory > > called "build". Compiler plugins and other existing build tools could live there. What do you think? > > In my understanding, they are plug-in'ed into the cross-compiler that > compiles vmlinux. > > If so, GCC plugins should be happy in "scripts/" > because Kbuild descends into "scripts/" before building any objects for vmlinux. Ok, I'll move them under scripts/ in the next patch set. -- Emese