[hch-misc:set_fs-removal 23/85] drivers/md/md-autodetect.c:305:13: warning: no previous prototype for function 'md_run_setup'
by kernel test robot
Hi Christoph,
First bad commit (maybe != root cause):
tree: git://git.infradead.org/users/hch/misc.git set_fs-removal
head: 153bec971d1618bc5abb582c90a6de920fe00e59
commit: c3088068d50e2622ccfffadf9f809a97a87835f6 [23/85] md: move the early init autodetect code to drivers/md/
config: x86_64-allyesconfig (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project bc2b70982be8f5250cd0082a7190f8b417bd4dfe)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
git checkout c3088068d50e2622ccfffadf9f809a97a87835f6
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>, old ones prefixed by <<):
>> drivers/md/md-autodetect.c:305:13: warning: no previous prototype for function 'md_run_setup' [-Wmissing-prototypes]
void __init md_run_setup(void)
^
drivers/md/md-autodetect.c:305:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init md_run_setup(void)
^
static
1 warning generated.
vim +/md_run_setup +305 drivers/md/md-autodetect.c
82cbc11a4146d6 init/do_mounts_md.c Ingo Molnar 2008-08-18 304
82cbc11a4146d6 init/do_mounts_md.c Ingo Molnar 2008-08-18 @305 void __init md_run_setup(void)
:::::: The code at line 305 was first introduced by commit
:::::: 82cbc11a4146d6a8acd81f81f7fe17387668107f warning: fix init do_mounts_md c
:::::: TO: Ingo Molnar <mingo(a)elte.hu>
:::::: CC: Arjan van de Ven <arjan(a)linux.intel.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
[hnaz-linux-mm:master 60/194] include/linux/pgtable.h:74:27: error: redefinition of 'pte_offset_kernel'
by kernel test robot
tree: https://github.com/hnaz/linux-mm master
head: a017b085735719105714da692a82a6eeb8830718
commit: f48b9eb10f8c3f378ea4552fee58db3dea980e62 [60/194] mm: consolidate pte_index() and pte_offset_*() definitions
config: sparc-defconfig (attached as .config)
compiler: sparc-linux-gcc (GCC) 9.3.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
git checkout f48b9eb10f8c3f378ea4552fee58db3dea980e62
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=sparc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All error/warnings (new ones prefixed by >>, old ones prefixed by <<):
In file included from include/linux/mm.h:31,
from include/linux/memblock.h:13,
from arch/sparc/mm/srmmu.c:14:
>> include/linux/pgtable.h:74:27: error: redefinition of 'pte_offset_kernel'
74 | #define pte_offset_kernel pte_offset_kernel
| ^~~~~~~~~~~~~~~~~
>> arch/sparc/mm/srmmu.c:144:8: note: in expansion of macro 'pte_offset_kernel'
144 | pte_t *pte_offset_kernel(pmd_t *dir, unsigned long address)
| ^~~~~~~~~~~~~~~~~
include/linux/pgtable.h:70:22: note: previous definition of 'pte_offset_kernel' was here
70 | static inline pte_t *pte_offset_kernel(pmd_t *pmd, unsigned long address)
| ^~~~~~~~~~~~~~~~~
arch/sparc/mm/srmmu.c: In function 'poke_hypersparc':
arch/sparc/mm/srmmu.c:1093:25: error: variable 'clear' set but not used [-Werror=unused-but-set-variable]
1093 | volatile unsigned long clear;
| ^~~~~
cc1: all warnings being treated as errors
vim +/pte_offset_kernel +74 include/linux/pgtable.h
68
69 #ifndef pte_offset_kernel
70 static inline pte_t *pte_offset_kernel(pmd_t *pmd, unsigned long address)
71 {
72 return (pte_t *)pmd_page_vaddr(*pmd) + pte_index(address);
73 }
> 74 #define pte_offset_kernel pte_offset_kernel
75 #endif
76
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
Re: [PATCH V3 1/9] pinctrl: imx: Support building SCU pinctrl driver as module
by kernel test robot
Hi Anson,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on pinctrl/devel]
[cannot apply to v5.7]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Anson-Huang/Support-i-MX8-SoCs-p...
base: https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git devel
config: arm64-randconfig-r004-20200607 (attached as .config)
compiler: aarch64-linux-gcc (GCC) 9.3.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
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
aarch64-linux-ld: drivers/pinctrl/freescale/pinctrl-scu.o: in function `imx_pinctrl_sc_ipc_init':
>> drivers/pinctrl/freescale/pinctrl-scu.c:43: undefined reference to `imx_scu_get_handle'
aarch64-linux-ld: drivers/pinctrl/freescale/pinctrl-scu.o: in function `imx_pinconf_get_scu':
>> drivers/pinctrl/freescale/pinctrl-scu.c:62: undefined reference to `imx_scu_call_rpc'
aarch64-linux-ld: drivers/pinctrl/freescale/pinctrl-scu.o: in function `imx_pinconf_set_scu':
drivers/pinctrl/freescale/pinctrl-scu.c:100: undefined reference to `imx_scu_call_rpc'
vim +43 drivers/pinctrl/freescale/pinctrl-scu.c
b96eea718bf697 A.s. Dong 2018-10-30 40
b96eea718bf697 A.s. Dong 2018-10-30 41 int imx_pinctrl_sc_ipc_init(struct platform_device *pdev)
b96eea718bf697 A.s. Dong 2018-10-30 42 {
b96eea718bf697 A.s. Dong 2018-10-30 @43 return imx_scu_get_handle(&pinctrl_ipc_handle);
b96eea718bf697 A.s. Dong 2018-10-30 44 }
3c8d42d4fdbf10 Anson Huang 2020-06-09 45 EXPORT_SYMBOL_GPL(imx_pinctrl_sc_ipc_init);
b96eea718bf697 A.s. Dong 2018-10-30 46
b96eea718bf697 A.s. Dong 2018-10-30 47 int imx_pinconf_get_scu(struct pinctrl_dev *pctldev, unsigned pin_id,
b96eea718bf697 A.s. Dong 2018-10-30 48 unsigned long *config)
b96eea718bf697 A.s. Dong 2018-10-30 49 {
b96eea718bf697 A.s. Dong 2018-10-30 50 struct imx_sc_msg_req_pad_get msg;
b96eea718bf697 A.s. Dong 2018-10-30 51 struct imx_sc_msg_resp_pad_get *resp;
b96eea718bf697 A.s. Dong 2018-10-30 52 struct imx_sc_rpc_msg *hdr = &msg.hdr;
b96eea718bf697 A.s. Dong 2018-10-30 53 int ret;
b96eea718bf697 A.s. Dong 2018-10-30 54
b96eea718bf697 A.s. Dong 2018-10-30 55 hdr->ver = IMX_SC_RPC_VERSION;
b96eea718bf697 A.s. Dong 2018-10-30 56 hdr->svc = IMX_SC_RPC_SVC_PAD;
b96eea718bf697 A.s. Dong 2018-10-30 57 hdr->func = IMX_SC_PAD_FUNC_GET;
b96eea718bf697 A.s. Dong 2018-10-30 58 hdr->size = 2;
b96eea718bf697 A.s. Dong 2018-10-30 59
b96eea718bf697 A.s. Dong 2018-10-30 60 msg.pad = pin_id;
b96eea718bf697 A.s. Dong 2018-10-30 61
b96eea718bf697 A.s. Dong 2018-10-30 @62 ret = imx_scu_call_rpc(pinctrl_ipc_handle, &msg, true);
b96eea718bf697 A.s. Dong 2018-10-30 63 if (ret)
b96eea718bf697 A.s. Dong 2018-10-30 64 return ret;
b96eea718bf697 A.s. Dong 2018-10-30 65
b96eea718bf697 A.s. Dong 2018-10-30 66 resp = (struct imx_sc_msg_resp_pad_get *)&msg;
b96eea718bf697 A.s. Dong 2018-10-30 67 *config = resp->val;
b96eea718bf697 A.s. Dong 2018-10-30 68
b96eea718bf697 A.s. Dong 2018-10-30 69 return 0;
b96eea718bf697 A.s. Dong 2018-10-30 70 }
3c8d42d4fdbf10 Anson Huang 2020-06-09 71 EXPORT_SYMBOL_GPL(imx_pinconf_get_scu);
b96eea718bf697 A.s. Dong 2018-10-30 72
:::::: The code at line 43 was first introduced by commit
:::::: b96eea718bf697e4a490c2fabfb89995b9193c21 pinctrl: fsl: add scu based pinctrl support
:::::: TO: A.s. Dong <aisheng.dong(a)nxp.com>
:::::: CC: Linus Walleij <linus.walleij(a)linaro.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
[clk:clk-mediatek 5/6] drivers/clk/mediatek/clk-mt6765.c:537:35: warning: unused variable 'ifr0_cg_regs'
by kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-mediatek
head: 571cfadcc628dd5591444f7289e27445ea732f4c
commit: 1aca9939bf72893887cb7e3455e44c864bada2f9 [5/6] clk: mediatek: Add MT6765 clock support
config: x86_64-allyesconfig (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 2388a096e7865c043e83ece4e26654bd3d1a20d5)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
git checkout 1aca9939bf72893887cb7e3455e44c864bada2f9
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>, old ones prefixed by <<):
>> drivers/clk/mediatek/clk-mt6765.c:537:35: warning: unused variable 'ifr0_cg_regs' [-Wunused-const-variable]
static const struct mtk_gate_regs ifr0_cg_regs = {
^
>> drivers/clk/mediatek/clk-mt6765.c:543:35: warning: unused variable 'ifr1_cg_regs' [-Wunused-const-variable]
static const struct mtk_gate_regs ifr1_cg_regs = {
^
2 warnings generated.
vim +/ifr0_cg_regs +537 drivers/clk/mediatek/clk-mt6765.c
536
> 537 static const struct mtk_gate_regs ifr0_cg_regs = {
538 .set_ofs = 0x200,
539 .clr_ofs = 0x200,
540 .sta_ofs = 0x200,
541 };
542
> 543 static const struct mtk_gate_regs ifr1_cg_regs = {
544 .set_ofs = 0x74,
545 .clr_ofs = 0x74,
546 .sta_ofs = 0x74,
547 };
548
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
Re: [PATCH v4 28/37] memory: tegra: Register as interconnect provider
by kernel test robot
In-Reply-To: <20200609131404.17523-29-digetx(a)gmail.com>
References: <20200609131404.17523-29-digetx(a)gmail.com>
TO: Dmitry Osipenko <digetx(a)gmail.com>
Hi Dmitry,
I love your patch! Yet something to improve:
[auto build test ERROR on tegra/for-next]
[also build test ERROR on clk/clk-next next-20200609]
[cannot apply to robh/for-next tegra-drm/drm/tegra/for-next linus/master v5.7]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Dmitry-Osipenko/Introduce-memory...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git for-next
config: x86_64-randconfig-r023-20200609 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project bc2b70982be8f5250cd0082a7190f8b417bd4dfe)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
>> ld.lld: error: duplicate symbol: icc_node_destroy
>>> defined at tegra30-devfreq.c
>>> devfreq/tegra30-devfreq.o:(icc_node_destroy) in archive drivers/built-in.a
>>> defined at tegra20-devfreq.c
>>> devfreq/tegra20-devfreq.o:(.text+0x0) in archive drivers/built-in.a
--
>> ld.lld: error: duplicate symbol: icc_link_destroy
>>> defined at tegra30-devfreq.c
>>> devfreq/tegra30-devfreq.o:(icc_link_destroy) in archive drivers/built-in.a
>>> defined at tegra20-devfreq.c
>>> devfreq/tegra20-devfreq.o:(.text+0x6) in archive drivers/built-in.a
--
>> ld.lld: error: duplicate symbol: icc_node_add
>>> defined at tegra30-devfreq.c
>>> devfreq/tegra30-devfreq.o:(icc_node_add) in archive drivers/built-in.a
>>> defined at tegra20-devfreq.c
>>> devfreq/tegra20-devfreq.o:(.text+0x11) in archive drivers/built-in.a
--
>> ld.lld: error: duplicate symbol: icc_node_del
>>> defined at tegra30-devfreq.c
>>> devfreq/tegra30-devfreq.o:(icc_node_del) in archive drivers/built-in.a
>>> defined at tegra20-devfreq.c
>>> devfreq/tegra20-devfreq.o:(.text+0x17) in archive drivers/built-in.a
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
[hch-misc:set_fs-removal 85/85] arch/x86/include/asm/processor.h:882:23: error: initializer element is not constant
by kernel test robot
tree: git://git.infradead.org/users/hch/misc.git set_fs-removal
head: 153bec971d1618bc5abb582c90a6de920fe00e59
commit: 153bec971d1618bc5abb582c90a6de920fe00e59 [85/85] x86: remove address space overrides using set_fs()
config: x86_64-randconfig-r036-20200609 (attached as .config)
compiler: gcc-5 (Ubuntu 5.5.0-12ubuntu1) 5.5.0 20171010
reproduce (this is a W=1 build):
git checkout 153bec971d1618bc5abb582c90a6de920fe00e59
# save the attached .config to linux build tree
make W=1 ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All error/warnings (new ones prefixed by >>, old ones prefixed by <<):
In file included from arch/x86/include/asm/cpufeature.h:5:0,
from arch/x86/include/asm/thread_info.h:53,
from include/linux/thread_info.h:38,
from arch/x86/include/asm/preempt.h:7,
from include/linux/preempt.h:78,
from include/linux/spinlock.h:51,
from include/linux/mmzone.h:8,
from include/linux/gfp.h:6,
from arch/x86/mm/init.c:1:
>> arch/x86/include/asm/processor.h:882:23: error: initializer element is not constant
#define TASK_SIZE_MAX ((1UL << __VIRTUAL_MASK_SHIFT) - PAGE_SIZE)
^
>> arch/x86/mm/init.c:38:37: note: in expansion of macro 'TASK_SIZE_MAX'
const unsigned long task_size_max = TASK_SIZE_MAX;
^
arch/x86/mm/init.c:83:6: warning: no previous prototype for 'x86_has_pat_wp' [-Wmissing-prototypes]
bool x86_has_pat_wp(void)
^
arch/x86/mm/init.c:88:22: warning: no previous prototype for 'pgprot2cachemode' [-Wmissing-prototypes]
enum page_cache_mode pgprot2cachemode(pgprot_t pgprot)
^
--
drivers/staging/media/atomisp/pci/atomisp_compat_ioctl32.c: In function 'atomisp_do_compat_ioctl':
>> drivers/staging/media/atomisp/pci/atomisp_compat_ioctl32.c:1009:11: error: implicit declaration of function 'get_fs' [-Werror=implicit-function-declaration]
old_fs = get_fs();
^
>> drivers/staging/media/atomisp/pci/atomisp_compat_ioctl32.c:1009:9: error: incompatible types when assigning to type 'mm_segment_t {aka struct <anonymous>}' from type 'int'
old_fs = get_fs();
^
>> drivers/staging/media/atomisp/pci/atomisp_compat_ioctl32.c:1010:2: error: implicit declaration of function 'set_fs' [-Werror=implicit-function-declaration]
set_fs(KERNEL_DS);
^
>> drivers/staging/media/atomisp/pci/atomisp_compat_ioctl32.c:1010:9: error: 'KERNEL_DS' undeclared (first use in this function)
set_fs(KERNEL_DS);
^
drivers/staging/media/atomisp/pci/atomisp_compat_ioctl32.c:1010:9: note: each undeclared identifier is reported only once for each function it appears in
cc1: some warnings being treated as errors
vim +882 arch/x86/include/asm/processor.h
2f66dcc933f012 include/asm-x86/processor.h Glauber de Oliveira Costa 2008-01-30 862
2f66dcc933f012 include/asm-x86/processor.h Glauber de Oliveira Costa 2008-01-30 863 #else
2f66dcc933f012 include/asm-x86/processor.h Glauber de Oliveira Costa 2008-01-30 864 /*
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 865 * User space process size. This is the first address outside the user range.
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 866 * There are a few constraints that determine this:
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 867 *
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 868 * On Intel CPUs, if a SYSCALL instruction is at the highest canonical
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 869 * address, then that syscall will enter the kernel with a
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 870 * non-canonical return address, and SYSRET will explode dangerously.
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 871 * We avoid this particular problem by preventing anything executable
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 872 * from being mapped at the maximum canonical address.
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 873 *
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 874 * On AMD CPUs in the Ryzen family, there's a nasty bug in which the
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 875 * CPUs malfunction if they execute code from the highest canonical page.
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 876 * They'll speculate right off the end of the canonical space, and
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 877 * bad things happen. This is worked around in the same way as the
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 878 * Intel problem.
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 879 *
f55f0501cbf65e arch/x86/include/asm/processor.h Andy Lutomirski 2017-12-12 880 * With page table isolation enabled, we map the LDT in ... [stay tuned]
2f66dcc933f012 include/asm-x86/processor.h Glauber de Oliveira Costa 2008-01-30 881 */
ee00f4a32a76ef arch/x86/include/asm/processor.h Kirill A. Shutemov 2017-07-17 @882 #define TASK_SIZE_MAX ((1UL << __VIRTUAL_MASK_SHIFT) - PAGE_SIZE)
2f66dcc933f012 include/asm-x86/processor.h Glauber de Oliveira Costa 2008-01-30 883
:::::: The code at line 882 was first introduced by commit
:::::: ee00f4a32a76ef631394f31d5b6028d50462b357 x86/mm: Allow userspace have mappings above 47-bit
:::::: TO: Kirill A. Shutemov <kirill.shutemov(a)linux.intel.com>
:::::: CC: Ingo Molnar <mingo(a)kernel.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
[hch-misc:set_fs-removal 79/85] arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared; did you mean
by kernel test robot
tree: git://git.infradead.org/users/hch/misc.git set_fs-removal
head: 153bec971d1618bc5abb582c90a6de920fe00e59
commit: cf928cce4b6912dc9da1a7706e524cac73df7c30 [79/85] uaccess: add force_uaccess_{begin,end} helpers
config: riscv-allyesconfig (attached as .config)
compiler: riscv64-linux-gcc (GCC) 9.3.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
git checkout cf928cce4b6912dc9da1a7706e524cac73df7c30
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All error/warnings (new ones prefixed by >>, old ones prefixed by <<):
In file included from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/linux/tty.h:5,
from drivers/usb/gadget/function/u_serial.c:21:
include/linux/uaccess.h: In function 'force_uaccess_begin':
>> arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'TASK_SIZE_OF'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
--
In file included from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/linux/poll.h:10,
from include/media/v4l2-dev.h:13,
from include/media/v4l2-common.h:18,
from drivers/media/platform/omap3isp/ispcsi2.c:14:
include/linux/uaccess.h: In function 'force_uaccess_begin':
arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'STATX_SIZE'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
--
In file included from include/linux/uaccess.h:9,
from drivers/gpu/drm/exynos/exynos_drm_ipp.c:19:
include/linux/uaccess.h: In function 'force_uaccess_begin':
>> arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'TASK_SIZE_OF'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
In file included from arch/riscv/include/asm/kgdb.h:109,
from include/linux/kgdb.h:20,
from include/linux/fb.h:5,
from include/drm/drm_crtc.h:31,
from drivers/gpu/drm/exynos/exynos_drm_drv.h:16,
from drivers/gpu/drm/exynos/exynos_drm_ipp.c:26:
At top level:
arch/riscv/include/asm/gdb_xml.h:23:19: warning: 'riscv_gdb_stub_cpuxml' defined but not used [-Wunused-const-variable=]
23 | static const char riscv_gdb_stub_cpuxml[2048] =
| ^~~~~~~~~~~~~~~~~~~~~
arch/riscv/include/asm/gdb_xml.h:16:19: warning: 'riscv_gdb_stub_target_desc' defined but not used [-Wunused-const-variable=]
16 | static const char riscv_gdb_stub_target_desc[256] =
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
arch/riscv/include/asm/gdb_xml.h:13:19: warning: 'gdb_xfer_read_cpuxml' defined but not used [-Wunused-const-variable=]
13 | static const char gdb_xfer_read_cpuxml[39] =
| ^~~~~~~~~~~~~~~~~~~~
arch/riscv/include/asm/gdb_xml.h:10:19: warning: 'gdb_xfer_read_target' defined but not used [-Wunused-const-variable=]
10 | static const char gdb_xfer_read_target[31] = "qXfer:features:read:target.xml:";
| ^~~~~~~~~~~~~~~~~~~~
arch/riscv/include/asm/gdb_xml.h:7:19: warning: 'riscv_gdb_stub_feature' defined but not used [-Wunused-const-variable=]
7 | static const char riscv_gdb_stub_feature[64] =
| ^~~~~~~~~~~~~~~~~~~~~~
--
In file included from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/linux/debugfs.h:15,
from drivers/usb/typec/tcpm/tcpm.c:9:
include/linux/uaccess.h: In function 'force_uaccess_begin':
arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'STATX_SIZE'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
At top level:
drivers/usb/typec/tcpm/tcpm.c:1551:39: warning: 'tcpm_altmode_ops' defined but not used [-Wunused-const-variable=]
1551 | static const struct typec_altmode_ops tcpm_altmode_ops = {
| ^~~~~~~~~~~~~~~~
--
In file included from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/linux/debugfs.h:15,
from drivers/usb/typec/tcpm/fusb302.c:8:
include/linux/uaccess.h: In function 'force_uaccess_begin':
arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'STATX_SIZE'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
drivers/usb/typec/tcpm/fusb302.c: In function 'fusb302_log':
drivers/usb/typec/tcpm/fusb302.c:186:2: warning: function 'fusb302_log' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format]
186 | _fusb302_log(chip, fmt, args);
| ^~~~~~~~~~~~
--
In file included from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/linux/seq_file.h:11,
from include/drm/drm_debugfs.h:36,
from drivers/gpu/drm/qxl/qxl_debugfs.c:31:
include/linux/uaccess.h: In function 'force_uaccess_begin':
arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'STATX_SIZE'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
In file included from arch/riscv/include/asm/kgdb.h:109,
from include/linux/kgdb.h:20,
from include/linux/fb.h:5,
from include/drm/drm_crtc.h:31,
from drivers/gpu/drm/qxl/qxl_drv.h:38,
from drivers/gpu/drm/qxl/qxl_debugfs.c:34:
At top level:
arch/riscv/include/asm/gdb_xml.h:23:19: warning: 'riscv_gdb_stub_cpuxml' defined but not used [-Wunused-const-variable=]
23 | static const char riscv_gdb_stub_cpuxml[2048] =
| ^~~~~~~~~~~~~~~~~~~~~
arch/riscv/include/asm/gdb_xml.h:16:19: warning: 'riscv_gdb_stub_target_desc' defined but not used [-Wunused-const-variable=]
16 | static const char riscv_gdb_stub_target_desc[256] =
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
arch/riscv/include/asm/gdb_xml.h:13:19: warning: 'gdb_xfer_read_cpuxml' defined but not used [-Wunused-const-variable=]
13 | static const char gdb_xfer_read_cpuxml[39] =
| ^~~~~~~~~~~~~~~~~~~~
arch/riscv/include/asm/gdb_xml.h:10:19: warning: 'gdb_xfer_read_target' defined but not used [-Wunused-const-variable=]
10 | static const char gdb_xfer_read_target[31] = "qXfer:features:read:target.xml:";
| ^~~~~~~~~~~~~~~~~~~~
arch/riscv/include/asm/gdb_xml.h:7:19: warning: 'riscv_gdb_stub_feature' defined but not used [-Wunused-const-variable=]
7 | static const char riscv_gdb_stub_feature[64] =
| ^~~~~~~~~~~~~~~~~~~~~~
--
In file included from include/linux/uaccess.h:9,
from drivers/input/input-compat.c:9:
include/linux/uaccess.h: In function 'force_uaccess_begin':
arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'PAGE_SIZE'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
--
In file included from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from drivers/input/rmi4/rmi_driver.c:16:
include/linux/uaccess.h: In function 'force_uaccess_begin':
arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'STATX_SIZE'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
drivers/input/rmi4/rmi_driver.c: In function 'rmi_read_register_desc':
drivers/input/rmi4/rmi_driver.c:583:24: warning: comparison is always false due to limited range of data type [-Wtype-limits]
583 | if (size_presence_reg < 0 || size_presence_reg > 35)
| ^
--
In file included from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from drivers/md/md.c:41:
include/linux/uaccess.h: In function 'force_uaccess_begin':
>> arch/riscv/include/asm/uaccess.h:56:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'TASK_SIZE_OF'?
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/include/asm/uaccess.h:56:30: note: each undeclared identifier is reported only once for each function it appears in
56 | #define USER_DS MAKE_MM_SEG(TASK_SIZE)
| ^~~~~~~~~
arch/riscv/include/asm/uaccess.h:53:43: note: in definition of macro 'MAKE_MM_SEG'
53 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
>> include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
drivers/md/md.c: In function 'bind_rdev_to_array':
drivers/md/md.c:2441:27: warning: suggest braces around empty body in an 'if' statement [-Wempty-body]
2441 | /* failure here is OK */;
| ^
drivers/md/md.c: In function 'slot_store':
drivers/md/md.c:3202:28: warning: suggest braces around empty body in an 'if' statement [-Wempty-body]
3202 | /* failure here is OK */;
| ^
drivers/md/md.c: In function 'remove_and_add_spares':
drivers/md/md.c:9050:29: warning: suggest braces around empty body in an 'if' statement [-Wempty-body]
9050 | /* failure here is OK */;
| ^
..
vim +56 arch/riscv/include/asm/uaccess.h
5cfade5fdcc975 Christoph Hellwig 2019-04-15 54
5cfade5fdcc975 Christoph Hellwig 2019-04-15 55 #define KERNEL_DS MAKE_MM_SEG(~0UL)
5cfade5fdcc975 Christoph Hellwig 2019-04-15 @56 #define USER_DS MAKE_MM_SEG(TASK_SIZE)
5d8544e2d0075a Palmer Dabbelt 2017-07-10 57
:::::: The code at line 56 was first introduced by commit
:::::: 5cfade5fdcc9753c5b3958f65f0716a31268dd3e riscv: turn mm_segment_t into a struct
:::::: TO: Christoph Hellwig <hch(a)lst.de>
:::::: CC: Palmer Dabbelt <palmer(a)sifive.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
[hch-misc:set_fs-removal 79/85] include/asm-generic/uaccess.h:77:30: error: 'TASK_SIZE' undeclared; did you mean
by kernel test robot
tree: git://git.infradead.org/users/hch/misc.git set_fs-removal
head: 153bec971d1618bc5abb582c90a6de920fe00e59
commit: cf928cce4b6912dc9da1a7706e524cac73df7c30 [79/85] uaccess: add force_uaccess_{begin,end} helpers
config: riscv-allnoconfig (attached as .config)
compiler: riscv64-linux-gcc (GCC) 9.3.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
git checkout cf928cce4b6912dc9da1a7706e524cac73df7c30
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
In file included from arch/riscv/include/asm/uaccess.h:478,
from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/linux/seq_file.h:11,
from arch/riscv/kernel/cpu.c:7:
include/linux/uaccess.h: In function 'force_uaccess_begin':
>> include/asm-generic/uaccess.h:77:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'TASK_IDLE'?
77 | #define USER_DS MAKE_MM_SEG(TASK_SIZE - 1)
| ^~~~~~~~~
include/asm-generic/uaccess.h:70:43: note: in definition of macro 'MAKE_MM_SEG'
70 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
include/asm-generic/uaccess.h:77:30: note: each undeclared identifier is reported only once for each function it appears in
77 | #define USER_DS MAKE_MM_SEG(TASK_SIZE - 1)
| ^~~~~~~~~
include/asm-generic/uaccess.h:70:43: note: in definition of macro 'MAKE_MM_SEG'
70 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
--
In file included from arch/riscv/include/asm/uaccess.h:478,
from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/uapi/linux/aio_abi.h:31,
from include/linux/syscalls.h:74,
from arch/riscv/kernel/syscall_table.c:8:
include/linux/uaccess.h: In function 'force_uaccess_begin':
>> include/asm-generic/uaccess.h:77:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'TASK_IDLE'?
77 | #define USER_DS MAKE_MM_SEG(TASK_SIZE - 1)
| ^~~~~~~~~
include/asm-generic/uaccess.h:70:43: note: in definition of macro 'MAKE_MM_SEG'
70 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
include/asm-generic/uaccess.h:77:30: note: each undeclared identifier is reported only once for each function it appears in
77 | #define USER_DS MAKE_MM_SEG(TASK_SIZE - 1)
| ^~~~~~~~~
include/asm-generic/uaccess.h:70:43: note: in definition of macro 'MAKE_MM_SEG'
70 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
include/uapi/asm-generic/unistd.h: At top level:
arch/riscv/kernel/syscall_table.c:14:36: warning: initialized field overwritten [-Woverride-init]
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:29:37: note: in expansion of macro '__SYSCALL'
29 | #define __SC_COMP(_nr, _sys, _comp) __SYSCALL(_nr, _sys)
| ^~~~~~~~~
include/uapi/asm-generic/unistd.h:34:1: note: in expansion of macro '__SC_COMP'
34 | __SC_COMP(__NR_io_setup, sys_io_setup, compat_sys_io_setup)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: note: (near initialization for 'sys_call_table[0]')
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:29:37: note: in expansion of macro '__SYSCALL'
29 | #define __SC_COMP(_nr, _sys, _comp) __SYSCALL(_nr, _sys)
| ^~~~~~~~~
include/uapi/asm-generic/unistd.h:34:1: note: in expansion of macro '__SC_COMP'
34 | __SC_COMP(__NR_io_setup, sys_io_setup, compat_sys_io_setup)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: warning: initialized field overwritten [-Woverride-init]
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:36:1: note: in expansion of macro '__SYSCALL'
36 | __SYSCALL(__NR_io_destroy, sys_io_destroy)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: note: (near initialization for 'sys_call_table[1]')
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:36:1: note: in expansion of macro '__SYSCALL'
36 | __SYSCALL(__NR_io_destroy, sys_io_destroy)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: warning: initialized field overwritten [-Woverride-init]
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:29:37: note: in expansion of macro '__SYSCALL'
29 | #define __SC_COMP(_nr, _sys, _comp) __SYSCALL(_nr, _sys)
| ^~~~~~~~~
include/uapi/asm-generic/unistd.h:38:1: note: in expansion of macro '__SC_COMP'
38 | __SC_COMP(__NR_io_submit, sys_io_submit, compat_sys_io_submit)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: note: (near initialization for 'sys_call_table[2]')
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:29:37: note: in expansion of macro '__SYSCALL'
29 | #define __SC_COMP(_nr, _sys, _comp) __SYSCALL(_nr, _sys)
| ^~~~~~~~~
include/uapi/asm-generic/unistd.h:38:1: note: in expansion of macro '__SC_COMP'
38 | __SC_COMP(__NR_io_submit, sys_io_submit, compat_sys_io_submit)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: warning: initialized field overwritten [-Woverride-init]
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:40:1: note: in expansion of macro '__SYSCALL'
40 | __SYSCALL(__NR_io_cancel, sys_io_cancel)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: note: (near initialization for 'sys_call_table[3]')
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:40:1: note: in expansion of macro '__SYSCALL'
40 | __SYSCALL(__NR_io_cancel, sys_io_cancel)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: warning: initialized field overwritten [-Woverride-init]
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:22:34: note: in expansion of macro '__SYSCALL'
22 | #define __SC_3264(_nr, _32, _64) __SYSCALL(_nr, _64)
| ^~~~~~~~~
include/uapi/asm-generic/unistd.h:43:1: note: in expansion of macro '__SC_3264'
43 | __SC_3264(__NR_io_getevents, sys_io_getevents_time32, sys_io_getevents)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: note: (near initialization for 'sys_call_table[4]')
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:22:34: note: in expansion of macro '__SYSCALL'
22 | #define __SC_3264(_nr, _32, _64) __SYSCALL(_nr, _64)
| ^~~~~~~~~
include/uapi/asm-generic/unistd.h:43:1: note: in expansion of macro '__SC_3264'
43 | __SC_3264(__NR_io_getevents, sys_io_getevents_time32, sys_io_getevents)
| ^~~~~~~~~
arch/riscv/kernel/syscall_table.c:14:36: warning: initialized field overwritten [-Woverride-init]
14 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
include/uapi/asm-generic/unistd.h:48:1: note: in expansion of macro '__SYSCALL'
--
In file included from arch/riscv/include/asm/uaccess.h:478,
from include/linux/uaccess.h:9,
from include/linux/sched/task.h:11,
from include/linux/sched/signal.h:9,
from arch/riscv/kernel/traps.c:11:
include/linux/uaccess.h: In function 'force_uaccess_begin':
>> include/asm-generic/uaccess.h:77:30: error: 'TASK_SIZE' undeclared (first use in this function); did you mean 'TASK_IDLE'?
77 | #define USER_DS MAKE_MM_SEG(TASK_SIZE - 1)
| ^~~~~~~~~
include/asm-generic/uaccess.h:70:43: note: in definition of macro 'MAKE_MM_SEG'
70 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
include/asm-generic/uaccess.h:77:30: note: each undeclared identifier is reported only once for each function it appears in
77 | #define USER_DS MAKE_MM_SEG(TASK_SIZE - 1)
| ^~~~~~~~~
include/asm-generic/uaccess.h:70:43: note: in definition of macro 'MAKE_MM_SEG'
70 | #define MAKE_MM_SEG(s) ((mm_segment_t) { (s) })
| ^
include/linux/uaccess.h:20:9: note: in expansion of macro 'USER_DS'
20 | set_fs(USER_DS);
| ^~~~~~~
arch/riscv/kernel/traps.c: At top level:
arch/riscv/kernel/traps.c:92:15: warning: no previous prototype for 'do_trap_unknown' [-Wmissing-prototypes]
92 | DO_ERROR_INFO(do_trap_unknown,
| ^~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:94:15: warning: no previous prototype for 'do_trap_insn_misaligned' [-Wmissing-prototypes]
94 | DO_ERROR_INFO(do_trap_insn_misaligned,
| ^~~~~~~~~~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:96:15: warning: no previous prototype for 'do_trap_insn_fault' [-Wmissing-prototypes]
96 | DO_ERROR_INFO(do_trap_insn_fault,
| ^~~~~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:98:15: warning: no previous prototype for 'do_trap_insn_illegal' [-Wmissing-prototypes]
98 | DO_ERROR_INFO(do_trap_insn_illegal,
| ^~~~~~~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:100:15: warning: no previous prototype for 'do_trap_load_fault' [-Wmissing-prototypes]
100 | DO_ERROR_INFO(do_trap_load_fault,
| ^~~~~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:111:17: warning: no previous prototype for 'do_trap_load_misaligned' [-Wmissing-prototypes]
111 | asmlinkage void do_trap_load_misaligned(struct pt_regs *regs)
| ^~~~~~~~~~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:119:17: warning: no previous prototype for 'do_trap_store_misaligned' [-Wmissing-prototypes]
119 | asmlinkage void do_trap_store_misaligned(struct pt_regs *regs)
| ^~~~~~~~~~~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:127:15: warning: no previous prototype for 'do_trap_store_fault' [-Wmissing-prototypes]
127 | DO_ERROR_INFO(do_trap_store_fault,
| ^~~~~~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:129:15: warning: no previous prototype for 'do_trap_ecall_u' [-Wmissing-prototypes]
129 | DO_ERROR_INFO(do_trap_ecall_u,
| ^~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:131:15: warning: no previous prototype for 'do_trap_ecall_s' [-Wmissing-prototypes]
131 | DO_ERROR_INFO(do_trap_ecall_s,
| ^~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:133:15: warning: no previous prototype for 'do_trap_ecall_m' [-Wmissing-prototypes]
133 | DO_ERROR_INFO(do_trap_ecall_m,
| ^~~~~~~~~~~~~~~
arch/riscv/kernel/traps.c:87:27: note: in definition of macro 'DO_ERROR_INFO'
87 | asmlinkage __visible void name(struct pt_regs *regs) | ^~~~
arch/riscv/kernel/traps.c:146:27: warning: no previous prototype for 'do_trap_break' [-Wmissing-prototypes]
146 | asmlinkage __visible void do_trap_break(struct pt_regs *regs)
| ^~~~~~~~~~~~~
vim +77 include/asm-generic/uaccess.h
eed417ddd52146 Arnd Bergmann 2009-05-13 75
eed417ddd52146 Arnd Bergmann 2009-05-13 76 #ifndef USER_DS
eed417ddd52146 Arnd Bergmann 2009-05-13 @77 #define USER_DS MAKE_MM_SEG(TASK_SIZE - 1)
eed417ddd52146 Arnd Bergmann 2009-05-13 78 #endif
eed417ddd52146 Arnd Bergmann 2009-05-13 79
:::::: The code at line 77 was first introduced by commit
:::::: eed417ddd52146f446595b5a7d8f21b1814b95b7 asm-generic: add a generic uaccess.h
:::::: TO: Arnd Bergmann <arnd(a)arndb.de>
:::::: CC: Arnd Bergmann <arnd(a)klappe2.(none)>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 3 months
Re: [PATCH v11 5/6] ndctl/papr_scm,uapi: Add support for PAPR nvdimm specific methods
by Dan Williams
On Tue, Jun 9, 2020 at 10:54 AM Vaibhav Jain <vaibhav(a)linux.ibm.com> wrote:
>
> Thanks Dan for the consideration and taking time to look into this.
>
> My responses below:
>
> Dan Williams <dan.j.williams(a)intel.com> writes:
>
> > On Mon, Jun 8, 2020 at 5:16 PM kernel test robot <lkp(a)intel.com> wrote:
> >>
> >> Hi Vaibhav,
> >>
> >> Thank you for the patch! Perhaps something to improve:
> >>
> >> [auto build test WARNING on powerpc/next]
> >> [also build test WARNING on linus/master v5.7 next-20200605]
> >> [cannot apply to linux-nvdimm/libnvdimm-for-next scottwood/next]
> >> [if your patch is applied to the wrong git tree, please drop us a note to help
> >> improve the system. BTW, we also suggest to use '--base' option to specify the
> >> base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
> >>
> >> url: https://github.com/0day-ci/linux/commits/Vaibhav-Jain/powerpc-papr_scm-Ad...
> >> base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
> >> config: powerpc-randconfig-r016-20200607 (attached as .config)
> >> compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project e429cffd4f228f70c1d9df0e5d77c08590dd9766)
> >> 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
> >> # install powerpc cross compiling tool for clang build
> >> # apt-get install binutils-powerpc-linux-gnu
> >> # save the attached .config to linux build tree
> >> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc
> >>
> >> If you fix the issue, kindly add following tag as appropriate
> >> Reported-by: kernel test robot <lkp(a)intel.com>
> >>
> >> All warnings (new ones prefixed by >>, old ones prefixed by <<):
> >>
> >> In file included from <built-in>:1:
> >> >> ./usr/include/asm/papr_pdsm.h:69:20: warning: field 'hdr' with variable sized type 'struct nd_cmd_pkg' not at the end of a struct or class is a GNU extension [-Wgnu-variable-sized-type-not-at-end]
> >> struct nd_cmd_pkg hdr; /* Package header containing sub-cmd */
> >
> > Hi Vaibhav,
> >
> [.]
> > This looks like it's going to need another round to get this fixed. I
> > don't think 'struct nd_pdsm_cmd_pkg' should embed a definition of
> > 'struct nd_cmd_pkg'. An instance of 'struct nd_cmd_pkg' carries a
> > payload that is the 'pdsm' specifics. As the code has it now it's
> > defined as a superset of 'struct nd_cmd_pkg' and the compiler warning
> > is pointing out a real 'struct' organization problem.
> >
> > Given the soak time needed in -next after the code is finalized this
> > there's no time to do another round of updates and still make the v5.8
> > merge window.
>
> Agreed that this looks bad, a solution will probably need some more
> review cycles resulting in this series missing the merge window.
>
> I am investigating into the possible solutions for this reported issue
> and made few observations:
>
> I see command pkg for Intel, Hpe, Msft and Hyperv families using a
> similar layout of embedding nd_cmd_pkg at the head of the
> command-pkg. struct nd_pdsm_cmd_pkg is following the same pattern.
>
> struct nd_pdsm_cmd_pkg {
> struct nd_cmd_pkg hdr;
> /* other members */
> };
>
> struct ndn_pkg_msft {
> struct nd_cmd_pkg gen;
> /* other members */
> };
> struct nd_pkg_intel {
> struct nd_cmd_pkg gen;
> /* other members */
> };
> struct ndn_pkg_hpe1 {
> struct nd_cmd_pkg gen;
> /* other members */
In those cases the other members are a union and there is no second
variable length array. Perhaps that is why those definitions are not
getting flagged? I'm not seeing anything in ndctl build options that
would explicitly disable this warning, but I'm not sure if the ndctl
build environment is missing this build warning by accident.
Those variable size payloads are also not being used in any code paths
that would look at the size of the command payload, like the kernel
ioctl() path. The payload validation code needs static sizes and the
payload parsing code wants to cast the payload to a known type. I
don't think you can use the same struct definition for both those
cases which is why the ndctl parsing code uses the union layout, but
the kernel command marshaling code does strict layering.
> };
>
> Even though other command families implement similar command-package
> layout they were not flagged (yet) as they are (I am guessing) serviced
> in vendor acpi drivers rather than in kernel like in case of papr-scm
> command family.
I sincerely hope there are no vendor acpi kernel drivers outside of
the upstream one.
>
> So, I think this issue is not just specific to papr-scm command family
> introduced in this patch series but rather across all other command
> families. Every other command family assumes 'struct nd_cmd_pkg_hdr' to
> be embeddable and puts it at the beginning of their corresponding
> command-packages. And its only a matter of time when someone tries
> filtering/handling of vendor specific commands in nfit module when they
> hit similar issue.
>
> Possible Solutions:
>
> * One way would be to redefine 'struct nd_cmd_pkg' to mark field
> 'nd_payload[]' from a flexible array to zero sized array as
> 'nd_payload[0]'.
I just went through a round of removing the usage of buf[0] in ndctl
since gcc10 now warns about that too.
> This should make 'struct nd_cmd_pkg' embeddable and
> clang shouldn't report 'gnu-variable-sized-type-not-at-end'
> warning. Also I think this change shouldn't introduce any ABI change.
>
> * Another way to solve this issue might be to redefine 'struct
> nd_pdsm_cmd_pkg' to below removing the 'struct nd_cmd_pkg' member. This
> struct should immediately follow the 'struct nd_cmd_pkg' command package
> when sent to libnvdimm:
>
> struct nd_pdsm_cmd_pkg {
> __s32 cmd_status; /* Out: Sub-cmd status returned back */
> __u16 reserved[2]; /* Ignored and to be used in future */
> __u8 payload[];
> };
>
> This should remove the flexible member nc_cmd_pkg.nd_payload from the
> struct with just one remaining at the end. However this would make
> accessing the [in|out|fw]_size members of 'struct nd_cmd_pkg'
> difficult for the pdsm servicing functions.
>
>
> Any other solution that you think, may solve this issue ?
The union might help, but per the above I think only for parsing the
command at which point I don't think the kernel needs a unified
structure defining both the generic envelope and the end-point
specific payload at once.
2 years, 3 months
[lkp] [+569 bytes kernel size regression] [i386-tinyconfig] [a95efeeeef] x86/speculation: Add Special Register Buffer Data Sampling (SRBDS) mitigation
by kernel test robot
FYI, we noticed a +569 bytes kernel size regression due to commit:
commit: a95efeeeef97c358d14fb3431dfd21510f39cce4 (x86/speculation: Add Special Register Buffer Data Sampling (SRBDS) mitigation)
url: https://github.com/0day-ci/linux/commits/Greg-Kroah-Hartman/5-7-2-rc1-rev...
Details as below (size data is obtained by `nm --size-sort vmlinux`):
89bc331e: x86/cpu: Add table argument to cpu_matches()
a95efeee: x86/speculation: Add Special Register Buffer Data Sampling (SRBDS) mitigation
+--------------------------------------+----------+----------+-------+
| symbol | 89bc331e | a95efeee | delta |
+--------------------------------------+----------+----------+-------+
| bzImage | 439456 | 439744 | 288 |
| nm.t.cpu_vuln_blacklist | 0 | 176 | 176 |
| nm.T.check_bugs | 1321 | 1441 | 120 |
| nm.T.early_cpu_init | 660 | 760 | 100 |
| nm.T.update_srbds_msr | 0 | 89 | 89 |
| nm.t.srbds_parse_cmdline | 0 | 47 | 47 |
| nm.W.cpu_show_srbds | 0 | 14 | 14 |
| nm.t.__setup_srbds_parse_cmdline | 0 | 12 | 12 |
| nm.t.__setup_str_srbds_parse_cmdline | 0 | 6 | 6 |
| nm.T.identify_secondary_cpu | 37 | 42 | 5 |
| nm.r.srbds_mitigation | 0 | 4 | 4 |
| nm.b.srbds_off | 0 | 1 | 1 |
| nm.t.cpu_matches | 34 | 29 | -5 |
+--------------------------------------+----------+----------+-------+
Thanks
2 years, 3 months