From mboxrd@z Thu Jan 1 00:00:00 1970 Reply-To: kernel-hardening@lists.openwall.com From: "PaX Team" Date: Sat, 17 Dec 2016 00:15:02 +0100 MIME-Version: 1.0 Message-ID: <58547576.17739.2C3ECA62@pageexec.freemail.hu> In-reply-to: References: <1481925984-98605-1-git-send-email-keescook@chromium.org>, <58546E96.16770.2C23F1B4@pageexec.freemail.hu>, Content-type: text/plain; charset=US-ASCII Content-transfer-encoding: 7BIT Content-description: Mail message body Subject: [kernel-hardening] Re: [PATCH v4 1/4] gcc-plugins: Add the initify gcc plugin To: Kees Cook Cc: "kernel-hardening@lists.openwall.com" , Emese Revfy , LKML , Arnd Bergmann , Josh Triplett , Brad Spengler , Michal Marek , Masahiro Yamada , linux-kbuild , minipli@ld-linux.so, Russell King , Catalin Marinas , Rasmus Villemoes , David Brown , "benh@kernel.crashing.org" , Thomas Gleixner , Andrew Morton , Jeff Layton , Sam Ravnborg List-ID: On 16 Dec 2016 at 15:02, Kees Cook wrote: > >> static inline struct cgraph_node_hook_list *cgraph_add_function_insertion_hook(cgraph_node_hook hook, void *data) > >> { > >> return symtab->add_cgraph_insertion_hook(hook, data); > > > > ...this one aren't needed by any plugins upstream so maybe introduce them when > > the needed arises? > > Hrm, sure. I was just going off of Emese's v3. (And this is partially > an artifact of basing off of v4.9-rc2... I'll refresh it to v4.10-rc2 > once it's out.) > > > and the whole patch against gcc-common.h would also conflict > > with the version i maintain and that you said you'd sync to so there's a decision > > to be made regarding how this will is to be maintained... > > What's easiest for you? I'm okay to carry "unused by upstream yet" > functions and macros in gcc-common, though I don't like carrying lots > of commented out stuff. :P well, as i explained it the other day, my version has 'everything but the kitchen sink' because i use it for development which isn't necessarily what other projects need or want for themselves (e.g., consider the consequences of Arnd's recent call to reduce the number of supported gcc versions, if that goes anywhere above 4.5, gcc-common.h in linux can be further trimmed from my version). at the end of the day this is a policy call and i'm not the one to make it for linux ;).