Re: [PATCH 10/10] module.h: remove extable.h include now users have migrated

From: kbuild test robot <lkp@intel.com>
To: Paul Gortmaker <paul.gortmaker@windriver.com>
Cc: kbuild-all@01.org, linux-arch@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Paul Gortmaker <paul.gortmaker@windriver.com>,
	Rusty Russell <rusty@rustcorp.com.au>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [PATCH 10/10] module.h: remove extable.h include now users have migrated
Date: Tue, 10 Jan 2017 10:26:22 +0800	[thread overview]
Message-ID: <201701101052.OZVYlwqi%fengguang.wu@intel.com> (raw)
In-Reply-To: <20170109204010.5047-11-paul.gortmaker@windriver.com>

[-- Attachment #1: Type: text/plain, Size: 3285 bytes --]

Hi Paul,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.10-rc3 next-20170106]
[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/Finalize-separation-of-extable-h-from-module-h/20170110-095123
config: frv-defconfig (attached as .config)
compiler: frv-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=frv 

All error/warnings (new ones prefixed by >>):

   arch/frv/mm/extable.c: In function 'search_exception_table':
>> arch/frv/mm/extable.c:43:10: error: implicit declaration of function 'search_exception_tables' [-Werror=implicit-function-declaration]
     extab = search_exception_tables(pc);
             ^~~~~~~~~~~~~~~~~~~~~~~
>> arch/frv/mm/extable.c:43:8: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
     extab = search_exception_tables(pc);
           ^
   cc1: some warnings being treated as errors

vim +/search_exception_tables +43 arch/frv/mm/extable.c

^1da177e Linus Torvalds 2005-04-16  27  		 * - we search for the return address (in LR) instead of the program counter
^1da177e Linus Torvalds 2005-04-16  28  		 * - it was probably during a clear_user()
^1da177e Linus Torvalds 2005-04-16  29  		 */
^1da177e Linus Torvalds 2005-04-16  30  		return (unsigned long) &__memset_user_error_handler;
^1da177e Linus Torvalds 2005-04-16  31  	}
018b8d12 David Howells  2006-01-08  32  
018b8d12 David Howells  2006-01-08  33  	if (__frame->lr == (unsigned long) &__memcpy_user_error_lr &&
^1da177e Linus Torvalds 2005-04-16  34  	    (unsigned long) &memcpy <= pc && pc < (unsigned long) &__memcpy_end
^1da177e Linus Torvalds 2005-04-16  35  	    ) {
^1da177e Linus Torvalds 2005-04-16  36  		/* the fault occurred in a protected memset
^1da177e Linus Torvalds 2005-04-16  37  		 * - we search for the return address (in LR) instead of the program counter
^1da177e Linus Torvalds 2005-04-16  38  		 * - it was probably during a copy_to/from_user()
^1da177e Linus Torvalds 2005-04-16  39  		 */
^1da177e Linus Torvalds 2005-04-16  40  		return (unsigned long) &__memcpy_user_error_handler;
^1da177e Linus Torvalds 2005-04-16  41  	}
^1da177e Linus Torvalds 2005-04-16  42  
018b8d12 David Howells  2006-01-08 @43  	extab = search_exception_tables(pc);
018b8d12 David Howells  2006-01-08  44  	if (extab)
018b8d12 David Howells  2006-01-08  45  		return extab->fixup;
^1da177e Linus Torvalds 2005-04-16  46  
018b8d12 David Howells  2006-01-08  47  	return 0;
^1da177e Linus Torvalds 2005-04-16  48  
^1da177e Linus Torvalds 2005-04-16  49  } /* end search_exception_table() */

:::::: The code at line 43 was first introduced by commit
:::::: 018b8d12bc85f8fb332239b11d919ea0724c49a4 [PATCH] frv: support module exception tables

:::::: TO: David Howells <dhowells@redhat.com>
:::::: CC: Linus Torvalds <torvalds@g5.osdl.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 8536 bytes --]