From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ricardo Neri Subject: Re: [PATCH v4 03/17] x86/mpx, x86/insn: Relocate insn util functions to a new insn-kernel Date: Thu, 23 Feb 2017 14:18:04 -0800 Message-ID: <1487888284.115017.5.camel@ranerica-desktop> References: <201702231831.ALp3TpDE%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <201702231831.ALp3TpDE%fengguang.wu@intel.com> Sender: linux-msdos-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" To: kbuild test robot Cc: kbuild-all@01.org, Ingo Molnar , Thomas Gleixner , "H. Peter Anvin" , Andy Lutomirski , Borislav Petkov , Peter Zijlstra , Andrew Morton , Brian Gerst , Chris Metcalf , Dave Hansen , Paolo Bonzini , Liang Z Li , Masami Hiramatsu , Huang Rui , Jiri Slaby , Jonathan Corbet , "Michael S. Tsirkin" , Paul Gortmaker , Vlastimil Babka , Chen Yucong , A On Thu, 2017-02-23 at 18:54 +0800, kbuild test robot wrote: > >> arch/x86/lib/insn-eval.c:106:21: error: static declaration of > 'insn_get_addr_ref' follows non-static declaration > static void __user *insn_get_addr_ref(struct insn *insn, struct > pt_regs *regs) > ^~~~~~~~~~~~~~~~~ > In file included from arch/x86/lib/insn-eval.c:10:0: > arch/x86/include/asm/insn-eval.h:14:14: note: previous declaration > of 'insn_get_addr_ref' was here > void __user *insn_get_addr_ref(struct insn *insn, struct pt_regs > *regs); > ^~~~~~~~~~~~~~~~~ > arch/x86/lib/insn-eval.c:106:21: warning: 'insn_get_addr_ref' > defined but not used [-Wunused-function] > static void __user *insn_get_addr_ref(struct insn *insn, struct > pt_regs *regs) > ^~~~~~~~~~~~~~~~~ > > vim +/insn_get_addr_ref +106 arch/x86/lib/insn-eval.c Uh! This escaped my tests after many rebases. I will correct it. Thanks and BR, Ricardo