Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751982AbbBRO1A (ORCPT ); Wed, 18 Feb 2015 09:27:00 -0500 Received: from mx1.redhat.com ([209.132.183.28]:43862 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751644AbbBRO07 (ORCPT ); Wed, 18 Feb 2015 09:26:59 -0500 Date: Wed, 18 Feb 2015 08:26:52 -0600 From: Josh Poimboeuf To: Miroslav Benes Cc: sjenning@redhat.com, jkosina@suse.cz, vojtech@suse.cz, masami.hiramatsu.pt@hitachi.com, pmladek@suse.cz, live-patching@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] livepatch: remove extern specifier from header files Message-ID: <20150218142652.GA8090@treble.redhat.com> References: <1424269267-1671-1-git-send-email-mbenes@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1424269267-1671-1-git-send-email-mbenes@suse.cz> User-Agent: Mutt/1.5.23.1-rc1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2101 Lines: 59 On Wed, Feb 18, 2015 at 03:21:07PM +0100, Miroslav Benes wrote: > Storage-class specifier 'extern' is redundant in front of the function > declaration. According to the C specification it has the same meaning as > if not present at all. So remove it. > > Signed-off-by: Miroslav Benes Acked-by: Josh Poimboeuf > --- > arch/x86/include/asm/livepatch.h | 4 ++-- > include/linux/livepatch.h | 8 ++++---- > 2 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/arch/x86/include/asm/livepatch.h b/arch/x86/include/asm/livepatch.h > index a455a53..2d29197 100644 > --- a/arch/x86/include/asm/livepatch.h > +++ b/arch/x86/include/asm/livepatch.h > @@ -32,8 +32,8 @@ static inline int klp_check_compiler_support(void) > #endif > return 0; > } > -extern int klp_write_module_reloc(struct module *mod, unsigned long type, > - unsigned long loc, unsigned long value); > +int klp_write_module_reloc(struct module *mod, unsigned long type, > + unsigned long loc, unsigned long value); > > static inline void klp_arch_set_pc(struct pt_regs *regs, unsigned long ip) > { > diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h > index 95023fd..ee6dbb3 100644 > --- a/include/linux/livepatch.h > +++ b/include/linux/livepatch.h > @@ -123,10 +123,10 @@ struct klp_patch { > enum klp_state state; > }; > > -extern int klp_register_patch(struct klp_patch *); > -extern int klp_unregister_patch(struct klp_patch *); > -extern int klp_enable_patch(struct klp_patch *); > -extern int klp_disable_patch(struct klp_patch *); > +int klp_register_patch(struct klp_patch *); > +int klp_unregister_patch(struct klp_patch *); > +int klp_enable_patch(struct klp_patch *); > +int klp_disable_patch(struct klp_patch *); > > #endif /* CONFIG_LIVEPATCH */ > > -- > 2.1.4 > -- Josh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/