From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, linux-kernel@vger.kernel.org,
linux-arch@vger.kernel.org, Rusty Russell <rusty@rustcorp.com.au>,
Andrew Morton <akpm@linux-foundation.org>,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [PATCH 20/20] module.h: remove extable.h include now users have migrated
Date: Sun, 22 Jan 2017 23:16:50 -0500 [thread overview]
Message-ID: <20170123041650.GQ29778@windriver.com> (raw)
In-Reply-To: <201701230645.PYFdx082%fengguang.wu@intel.com>
[Re: [PATCH 20/20] module.h: remove extable.h include now users have migrated] On 23/01/2017 (Mon 06:52) kbuild test robot wrote:
> Hi Paul,
>
> [auto build test ERROR on linus/master]
> [also build test ERROR on v4.10-rc5 next-20170120]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
>
> url: https://github.com/0day-ci/linux/commits/Paul-Gortmaker/ia64-move-ia64_done_with_exception-out-of-asm-uaccess-h/20170123-004155
> config: cris-etraxfs_defconfig (attached as .config)
Thanks (once again) for the build coverage. I'd actually built cris
(allmodconfig) but strangely enough it didn't show this particular
defconfig issue. I'll fix it to ensure this file includes extable.h
Paul.
--
> compiler: cris-linux-gcc (GCC) 6.2.0
> reproduce:
> wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # save the attached .config to linux build tree
> make.cross ARCH=cris
>
> All error/warnings (new ones prefixed by >>):
>
> arch/cris/arch-v32/kernel/traps.c: In function 'fixup_BUG':
> >> arch/cris/arch-v32/kernel/traps.c:159:11: error: implicit declaration of function 'search_exception_tables' [-Werror=implicit-function-declaration]
> fixup = search_exception_tables(instruction_pointer(regs) - 2);
> ^~~~~~~~~~~~~~~~~~~~~~~
> >> arch/cris/arch-v32/kernel/traps.c:159:9: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
> fixup = search_exception_tables(instruction_pointer(regs) - 2);
> ^
> cc1: some warnings being treated as errors
>
> vim +/search_exception_tables +159 arch/cris/arch-v32/kernel/traps.c
>
> 51533b61 Mikael Starvik 2005-07-27 143
> 9ce1ea75 Jesper Nilsson 2007-12-03 144 void fixup_BUG(struct pt_regs *regs)
> 51533b61 Mikael Starvik 2005-07-27 145 {
> 9ce1ea75 Jesper Nilsson 2007-12-03 146 BUG_regs = *regs;
> 9ce1ea75 Jesper Nilsson 2007-12-03 147
> 9ce1ea75 Jesper Nilsson 2007-12-03 148 #ifdef CONFIG_DEBUG_BUGVERBOSE
> 9ce1ea75 Jesper Nilsson 2007-12-03 149 /*
> 9ce1ea75 Jesper Nilsson 2007-12-03 150 * Fixup the BUG arguments through exception handlers.
> 9ce1ea75 Jesper Nilsson 2007-12-03 151 */
> 9ce1ea75 Jesper Nilsson 2007-12-03 152 {
> 9ce1ea75 Jesper Nilsson 2007-12-03 153 const struct exception_table_entry *fixup;
> 9ce1ea75 Jesper Nilsson 2007-12-03 154
> 9ce1ea75 Jesper Nilsson 2007-12-03 155 /*
> 9ce1ea75 Jesper Nilsson 2007-12-03 156 * ERP points at the "break 14" + 2, compensate for the 2
> 9ce1ea75 Jesper Nilsson 2007-12-03 157 * bytes.
> 9ce1ea75 Jesper Nilsson 2007-12-03 158 */
> 9ce1ea75 Jesper Nilsson 2007-12-03 @159 fixup = search_exception_tables(instruction_pointer(regs) - 2);
> 9ce1ea75 Jesper Nilsson 2007-12-03 160 if (fixup) {
> 9ce1ea75 Jesper Nilsson 2007-12-03 161 /* Adjust the instruction pointer in the stackframe. */
> 9ce1ea75 Jesper Nilsson 2007-12-03 162 instruction_pointer(regs) = fixup->fixup;
> 9ce1ea75 Jesper Nilsson 2007-12-03 163 arch_fixup(regs);
> 51533b61 Mikael Starvik 2005-07-27 164 }
> 9ce1ea75 Jesper Nilsson 2007-12-03 165 }
> 9ce1ea75 Jesper Nilsson 2007-12-03 166 #else
> 9ce1ea75 Jesper Nilsson 2007-12-03 167 /* Dont try to lookup the filename + line, just dump regs. */
>
> :::::: The code at line 159 was first introduced by commit
> :::::: 9ce1ea751f7256b2248321c2427612a295f15137 CRIS v32: Update and improve kernel/traps.c
>
> :::::: TO: Jesper Nilsson <jesper.nilsson@axis.com>
> :::::: CC: Jesper Nilsson <jesper.nilsson@axis.com>
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
next prev parent reply other threads:[~2017-01-23 4:16 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-22 16:32 [PATCH v3 00/20] Finalize separation of extable.h from module.h Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 01/20] ia64: move ia64_done_with_exception out of asm/uaccess.h Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 02/20] ia64: ensure exception table search users include extable.h Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 03/20] m32r: migrate exception table users off module.h and onto extable.h Paul Gortmaker
2017-01-22 16:32 ` [PATCH 04/20] arm: " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 05/20] alpha: " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 06/20] mn10300: " Paul Gortmaker
2017-01-22 16:32 ` [PATCH 07/20] xtensa: " Paul Gortmaker
2017-01-22 16:32 ` [PATCH 08/20] sh: " Paul Gortmaker
2017-01-22 16:32 ` [PATCH 09/20] frv: " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 10/20] openrisc: " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 11/20] sparc: migrate exception table users " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 12/20] nios2: migrate exception table users off module.h and " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-25 9:53 ` Tobias Klauser
2017-01-25 9:53 ` Tobias Klauser
2017-01-22 16:32 ` [PATCH 13/20] arc: " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-23 21:14 ` Vineet Gupta
2017-01-22 16:32 ` [PATCH 14/20] metag: " Paul Gortmaker
2017-01-22 16:32 ` [PATCH 15/20] score: " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 16/20] unicore32: " Paul Gortmaker
2017-01-22 16:32 ` [PATCH 17/20] microblaze: " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 18/20] hexagon: " Paul Gortmaker
2017-01-22 16:32 ` Paul Gortmaker
2017-01-22 16:32 ` [PATCH 19/20] core: " Paul Gortmaker
2017-02-09 5:33 ` Jessica Yu
2017-01-22 16:32 ` [PATCH 20/20] module.h: remove extable.h include now users have migrated Paul Gortmaker
2017-01-22 22:52 ` kbuild test robot
2017-01-22 22:52 ` kbuild test robot
2017-01-23 4:16 ` Paul Gortmaker [this message]
2017-01-23 4:16 ` Paul Gortmaker
2017-02-09 5:34 ` Jessica Yu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170123041650.GQ29778@windriver.com \
--to=paul.gortmaker@windriver.com \
--cc=akpm@linux-foundation.org \
--cc=kbuild-all@01.org \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkp@intel.com \
--cc=rusty@rustcorp.com.au \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).