From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 115E6C433F5 for ; Fri, 14 Jan 2022 07:49:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232992AbiANHtS (ORCPT ); Fri, 14 Jan 2022 02:49:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42810 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230300AbiANHtR (ORCPT ); Fri, 14 Jan 2022 02:49:17 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C8B7CC061574 for ; Thu, 13 Jan 2022 23:49:16 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 9C633B8240D for ; Fri, 14 Jan 2022 07:49:15 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 950DFC36AE9; Fri, 14 Jan 2022 07:49:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1642146554; bh=++CerXvsh28EzjN8gsbkvoXEZkr6ayVTZ6s66y20ueE=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=VvjiLtJJtuUIaMVep1nSu2mMMJ6sThJ+DZo3fjWX/r5zn2NROfEWo+F5Wyxn5MKoW Sd9AAbOVdI5Ww6vUliRHgoVUlBv2E7u+JErjOmL9m5MzUtM5aTICoZlDpg/297NuJV yZpMFHiBtbHJz8cd9+U+gIAm19mdzETUvCW1Fzx0PEnDigWPb9Cf00voNBumjv+tUV 59oJcgzZc7IWkZKX/JHQEVkXMjbcgZ1UPaIzDyT0Uz86HufX80rjAyxKTQrIJ2lagh afzvmSY7vHmTWnRyA09b9sZePW5sxL3Tcm6HSgsyfIBPDKchNat+1YJ5bkK0xXjGm+ wh/tt5G88KxUQ== Date: Fri, 14 Jan 2022 16:49:09 +0900 From: Masami Hiramatsu To: Greg Kroah-Hartman Cc: kernel test robot , kbuild-all@lists.01.org, linux-kernel@vger.kernel.org, Ingo Molnar , Sasha Levin Subject: Re: [linux-stable-rc:linux-5.4.y 2563/9999] arch/ia64/kernel/kprobes.c:401:24: error: implicit declaration of function '__kretprobe_trampoline_handler'; did you mean 'kretprobe_trampoline'? Message-Id: <20220114164909.d273e73490f7ea62c751c26c@kernel.org> In-Reply-To: References: <202201130408.tdm9swhk-lkp@intel.com> <20220113182533.15c04b3a16f02af34f2cb9e4@kernel.org> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.32; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 14 Jan 2022 08:09:44 +0100 Greg Kroah-Hartman wrote: > On Fri, Jan 14, 2022 at 08:04:44AM +0100, Greg Kroah-Hartman wrote: > > On Thu, Jan 13, 2022 at 06:25:33PM +0900, Masami Hiramatsu wrote: > > > On Thu, 13 Jan 2022 04:15:43 +0800 > > > kernel test robot wrote: > > > > > > > Hi Masami, > > > > > > > > FYI, the error/warning still remains. > > > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.4.y > > > > head: 0a4ce4977bbeea4560a1f32632650b388c834c8a > > > > commit: 77fa5e15c933a1ec812de61ad709c00aa51e96ae [2563/9999] ia64: kprobes: Use generic kretprobe trampoline handler > > > > config: ia64-randconfig-r002-20220112 (https://download.01.org/0day-ci/archive/20220113/202201130408.tdm9swhk-lkp@intel.com/config) > > > > compiler: ia64-linux-gcc (GCC) 11.2.0 > > > > reproduce (this is a W=1 build): > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > > chmod +x ~/bin/make.cross > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?id=77fa5e15c933a1ec812de61ad709c00aa51e96ae > > > > git remote add linux-stable-rc https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git > > > > git fetch --no-tags linux-stable-rc linux-5.4.y > > > > git checkout 77fa5e15c933a1ec812de61ad709c00aa51e96ae > > > > # save the config file to linux build tree > > > > mkdir build_dir > > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=ia64 SHELL=/bin/bash > > > > > > > > If you fix the issue, kindly add following tag as appropriate > > > > Reported-by: kernel test robot > > > > > > > > All errors (new ones prefixed by >>): > > > > > > > > arch/ia64/kernel/kprobes.c: In function 'get_kprobe_inst': > > > > arch/ia64/kernel/kprobes.c:325:22: warning: variable 'template' set but not used [-Wunused-but-set-variable] > > > > 325 | unsigned int template; > > > > | ^~~~~~~~ > > > > > > OK, this is another issue, which is in the upstream master too. > > > > > > > arch/ia64/kernel/kprobes.c: At top level: > > > > arch/ia64/kernel/kprobes.c:399:15: warning: no previous prototype for 'trampoline_probe_handler' [-Wmissing-prototypes]> > > 399 | int __kprobes trampoline_probe_handler(struct kprobe *p, struct pt_regs *regs) > > > > | ^~~~~~~~~~~~~~~~~~~~~~~~ > > > > arch/ia64/kernel/kprobes.c: In function 'trampoline_probe_handler': > > > > >> arch/ia64/kernel/kprobes.c:401:24: error: implicit declaration of function '__kretprobe_trampoline_handler'; did you mean 'kretprobe_trampoline'? [-Werror=implicit-function-declaration] > > > > 401 | regs->cr_iip = __kretprobe_trampoline_handler(regs, kretprobe_trampoline, NULL); > > > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > > | kretprobe_trampoline > > > > cc1: some warnings being treated as errors > > > > > > This seems that the upstream commit e792ff804f49720ce003b3e4c618b5d996256a18 > > > has been picked to the stable-5.4.y tree accidentally. > > > That is a part of lockless kretprobe series and should not be picked to > > > the stable tree. > > > > > > Greg, can you revert commit 77fa5e15c933a1ec812de61ad709c00aa51e96ae ? > > > (Or should I send revert patch?) > > > > I can revert it, thanks. > > No, it does not revert cleanly, it looks like it was needed for another > patch. So a series of reverts if you could create them would be great. OK, thanks for trying. Let me check what happened. Thanks, -- Masami Hiramatsu From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============2116552779468664659==" MIME-Version: 1.0 From: Masami Hiramatsu To: kbuild-all@lists.01.org Subject: Re: [linux-stable-rc:linux-5.4.y 2563/9999] arch/ia64/kernel/kprobes.c:401:24: error: implicit declaration of function '__kretprobe_trampoline_handler'; did you mean 'kretprobe_trampoline'? Date: Fri, 14 Jan 2022 16:49:09 +0900 Message-ID: <20220114164909.d273e73490f7ea62c751c26c@kernel.org> In-Reply-To: List-Id: --===============2116552779468664659== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Fri, 14 Jan 2022 08:09:44 +0100 Greg Kroah-Hartman wrote: > On Fri, Jan 14, 2022 at 08:04:44AM +0100, Greg Kroah-Hartman wrote: > > On Thu, Jan 13, 2022 at 06:25:33PM +0900, Masami Hiramatsu wrote: > > > On Thu, 13 Jan 2022 04:15:43 +0800 > > > kernel test robot wrote: > > > = > > > > Hi Masami, > > > > = > > > > FYI, the error/warning still remains. > > > > = > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linu= x-stable-rc.git linux-5.4.y > > > > head: 0a4ce4977bbeea4560a1f32632650b388c834c8a > > > > commit: 77fa5e15c933a1ec812de61ad709c00aa51e96ae [2563/9999] ia64: = kprobes: Use generic kretprobe trampoline handler > > > > config: ia64-randconfig-r002-20220112 (https://download.01.org/0day= -ci/archive/20220113/202201130408.tdm9swhk-lkp(a)intel.com/config) > > > > compiler: ia64-linux-gcc (GCC) 11.2.0 > > > > reproduce (this is a W=3D1 build): > > > > wget https://raw.githubusercontent.com/intel/lkp-tests/mast= er/sbin/make.cross -O ~/bin/make.cross > > > > chmod +x ~/bin/make.cross > > > > # https://git.kernel.org/pub/scm/linux/kernel/git/stable/li= nux-stable-rc.git/commit/?id=3D77fa5e15c933a1ec812de61ad709c00aa51e96ae > > > > git remote add linux-stable-rc https://git.kernel.org/pub/s= cm/linux/kernel/git/stable/linux-stable-rc.git > > > > git fetch --no-tags linux-stable-rc linux-5.4.y > > > > git checkout 77fa5e15c933a1ec812de61ad709c00aa51e96ae > > > > # save the config file to linux build tree > > > > mkdir build_dir > > > > COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dgcc-11.2.0 ma= ke.cross O=3Dbuild_dir ARCH=3Dia64 SHELL=3D/bin/bash > > > > = > > > > If you fix the issue, kindly add following tag as appropriate > > > > Reported-by: kernel test robot > > > > = > > > > All errors (new ones prefixed by >>): > > > > = > > > > arch/ia64/kernel/kprobes.c: In function 'get_kprobe_inst': > > > > arch/ia64/kernel/kprobes.c:325:22: warning: variable 'template' = set but not used [-Wunused-but-set-variable] > > > > 325 | unsigned int template; > > > > | ^~~~~~~~ > > > = > > > OK, this is another issue, which is in the upstream master too. > > > = > > > > arch/ia64/kernel/kprobes.c: At top level: > > > > arch/ia64/kernel/kprobes.c:399:15: warning: no previous prototyp= e for 'trampoline_probe_handler' [-Wmissing-prototypes]> > > 399 | int= __kprobes trampoline_probe_handler(struct kprobe *p, struct pt_regs *regs) > > > > | ^~~~~~~~~~~~~~~~~~~~~~~~ > > > > arch/ia64/kernel/kprobes.c: In function 'trampoline_probe_handle= r': > > > > >> arch/ia64/kernel/kprobes.c:401:24: error: implicit declaration o= f function '__kretprobe_trampoline_handler'; did you mean 'kretprobe_trampo= line'? [-Werror=3Dimplicit-function-declaration] > > > > 401 | regs->cr_iip =3D __kretprobe_trampoline_handler(= regs, kretprobe_trampoline, NULL); > > > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > > | kretprobe_trampoline > > > > cc1: some warnings being treated as errors > > > = > > > This seems that the upstream commit e792ff804f49720ce003b3e4c618b5d99= 6256a18 > > > has been picked to the stable-5.4.y tree accidentally. > > > That is a part of lockless kretprobe series and should not be picked = to > > > the stable tree. > > > = > > > Greg, can you revert commit 77fa5e15c933a1ec812de61ad709c00aa51e96ae ? > > > (Or should I send revert patch?) > > = > > I can revert it, thanks. > = > No, it does not revert cleanly, it looks like it was needed for another > patch. So a series of reverts if you could create them would be great. OK, thanks for trying. Let me check what happened. Thanks, -- = Masami Hiramatsu --===============2116552779468664659==--