Re: [PATCH v1] Bluetooth: btintel: Support Digital(N) + RF(N-1) combination
by kernel test robot
Hi Kiran,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on bluetooth-next/master]
[also build test WARNING on bluetooth/master v5.13-rc6 next-20210616]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Kiran-K/Bluetooth-btintel-Suppor...
base: https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git master
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-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
# https://github.com/0day-ci/linux/commit/6e8c708932770f46284508ca6f027fa39...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Kiran-K/Bluetooth-btintel-Support-Digital-N-RF-N-1-combination/20210616-183512
git checkout 6e8c708932770f46284508ca6f027fa39393389e
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arc
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 >>):
>> drivers/bluetooth/btintel.c:486:6: warning: no previous prototype for 'btintel_parse_version_tlv' [-Wmissing-prototypes]
486 | void btintel_parse_version_tlv(struct hci_dev *hdev, struct sk_buff *skb,
| ^~~~~~~~~~~~~~~~~~~~~~~~~
vim +/btintel_parse_version_tlv +486 drivers/bluetooth/btintel.c
485
> 486 void btintel_parse_version_tlv(struct hci_dev *hdev, struct sk_buff *skb,
487 struct intel_version_tlv *version)
488 {
489 /* Consume Command Complete Status field */
490 skb_pull(skb, sizeof(__u8));
491
492 /* Event parameters contatin multiple TLVs. Read each of them
493 * and only keep the required data. Also, it use existing legacy
494 * version field like hw_platform, hw_variant, and fw_variant
495 * to keep the existing setup flow
496 */
497 while (skb->len) {
498 struct intel_tlv *tlv;
499
500 tlv = (struct intel_tlv *)skb->data;
501 switch (tlv->type) {
502 case INTEL_TLV_CNVI_TOP:
503 version->cnvi_top = get_unaligned_le32(tlv->val);
504 break;
505 case INTEL_TLV_CNVR_TOP:
506 version->cnvr_top = get_unaligned_le32(tlv->val);
507 break;
508 case INTEL_TLV_CNVI_BT:
509 version->cnvi_bt = get_unaligned_le32(tlv->val);
510 break;
511 case INTEL_TLV_CNVR_BT:
512 version->cnvr_bt = get_unaligned_le32(tlv->val);
513 break;
514 case INTEL_TLV_DEV_REV_ID:
515 version->dev_rev_id = get_unaligned_le16(tlv->val);
516 break;
517 case INTEL_TLV_IMAGE_TYPE:
518 version->img_type = tlv->val[0];
519 break;
520 case INTEL_TLV_TIME_STAMP:
521 version->timestamp = get_unaligned_le16(tlv->val);
522 break;
523 case INTEL_TLV_BUILD_TYPE:
524 version->build_type = tlv->val[0];
525 break;
526 case INTEL_TLV_BUILD_NUM:
527 version->build_num = get_unaligned_le32(tlv->val);
528 break;
529 case INTEL_TLV_SECURE_BOOT:
530 version->secure_boot = tlv->val[0];
531 break;
532 case INTEL_TLV_OTP_LOCK:
533 version->otp_lock = tlv->val[0];
534 break;
535 case INTEL_TLV_API_LOCK:
536 version->api_lock = tlv->val[0];
537 break;
538 case INTEL_TLV_DEBUG_LOCK:
539 version->debug_lock = tlv->val[0];
540 break;
541 case INTEL_TLV_MIN_FW:
542 version->min_fw_build_nn = tlv->val[0];
543 version->min_fw_build_cw = tlv->val[1];
544 version->min_fw_build_yy = tlv->val[2];
545 break;
546 case INTEL_TLV_LIMITED_CCE:
547 version->limited_cce = tlv->val[0];
548 break;
549 case INTEL_TLV_SBE_TYPE:
550 version->sbe_type = tlv->val[0];
551 break;
552 case INTEL_TLV_OTP_BDADDR:
553 memcpy(&version->otp_bd_addr, tlv->val, tlv->len);
554 break;
555 default:
556 /* Ignore rest of information */
557 break;
558 }
559 /* consume the current tlv and move to next*/
560 skb_pull(skb, tlv->len + sizeof(*tlv));
561 }
562 }
563 EXPORT_SYMBOL_GPL(btintel_parse_version_tlv);
564
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH 1/3] drm/i915/guc: Add fetch of hwconfig table
by kernel test robot
Hi,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on next-20210615]
[cannot apply to v5.13-rc6]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/John-C-Harrison-Intel-com/Add-su...
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: x86_64-randconfig-a002-20210616 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
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
# https://github.com/0day-ci/linux/commit/0006532f8f4a6f0ff36af11df58e8705a...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review John-C-Harrison-Intel-com/Add-support-for-querying-hw-info-that-UMDs-need/20210616-141805
git checkout 0006532f8f4a6f0ff36af11df58e8705af452e3c
# 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 >>):
>> drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c:19: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* GuC has a blob containing hardware configuration information (hwconfig).
drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c:125: warning: Function parameter or member 'hwconfig' not described in 'intel_guc_hwconfig_init'
drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c:160: warning: Function parameter or member 'hwconfig' not described in 'intel_guc_hwconfig_fini'
vim +19 drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c
17
18 /**
> 19 * GuC has a blob containing hardware configuration information (hwconfig).
20 * This is formatted as a simple and flexible KLV (Key/Length/Value) table.
21 *
22 * For example, a minimal version could be:
23 * enum device_attr {
24 * ATTR_SOME_VALUE = 0,
25 * ATTR_SOME_MASK = 1,
26 * };
27 *
28 * static const u32 hwconfig[] = {
29 * ATTR_SOME_VALUE,
30 * 1, // Value Length in DWords
31 * 8, // Value
32 *
33 * ATTR_SOME_MASK,
34 * 3,
35 * 0x00FFFFFFFF, 0xFFFFFFFF, 0xFF000000,
36 * };
37 *
38 * The attribute ids are defined in a hardware spec. The current list as
39 * known to the i915 driver can be found in i915/gt/intel_guc_hwconfig_types.h
40 */
41
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
[linux-next:master 8930/10489] include/linux/kern_levels.h:5:18: warning: format '%x' expects argument of type 'unsigned int', but argument 2 has type 'phys_addr_t' {aka 'long long unsigned int'}
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: c7d4c1fd91ab4a6d2620497921a9c6bf54650ab8
commit: 6e121df14ccd5ca5142a21759beda7f12db0002b [8930/10489] ARM: 9090/1: Map the lowmem and kernel separately
config: arm-keystone_defconfig (attached as .config)
compiler: arm-linux-gnueabi-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
# https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commi...
git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
git fetch --no-tags linux-next master
git checkout 6e121df14ccd5ca5142a21759beda7f12db0002b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm
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 >>):
arch/arm/mm/mmu.c:118:13: warning: no previous prototype for 'init_default_cache_policy' [-Wmissing-prototypes]
118 | void __init init_default_cache_policy(unsigned long pmd)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
arch/arm/mm/mmu.c:1156:13: warning: no previous prototype for 'adjust_lowmem_bounds' [-Wmissing-prototypes]
1156 | void __init adjust_lowmem_bounds(void)
| ^~~~~~~~~~~~~~~~~~~~
arch/arm/mm/mmu.c:1715:13: warning: no previous prototype for 'paging_init' [-Wmissing-prototypes]
1715 | void __init paging_init(const struct machine_desc *mdesc)
| ^~~~~~~~~~~
In file included from include/linux/kernel.h:17,
from include/linux/list.h:9,
from include/linux/module.h:12,
from arch/arm/mm/mmu.c:7:
arch/arm/mm/mmu.c: In function 'paging_init':
>> include/linux/kern_levels.h:5:18: warning: format '%x' expects argument of type 'unsigned int', but argument 2 has type 'phys_addr_t' {aka 'long long unsigned int'} [-Wformat=]
5 | #define KERN_SOH "\001" /* ASCII Start Of Header */
| ^~~~~~
include/linux/printk.h:140:10: note: in definition of macro 'no_printk'
140 | printk(fmt, ##__VA_ARGS__); \
| ^~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
15 | #define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
| ^~~~~~~~
include/linux/printk.h:430:12: note: in expansion of macro 'KERN_DEBUG'
430 | no_printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~~~~
arch/arm/mm/mmu.c:1719:2: note: in expansion of macro 'pr_debug'
1719 | pr_debug("physical kernel sections: 0x%08x-0x%08x\n",
| ^~~~~~~~
arch/arm/mm/mmu.c:1719:43: note: format string is defined here
1719 | pr_debug("physical kernel sections: 0x%08x-0x%08x\n",
| ~~~^
| |
| unsigned int
| %08llx
In file included from include/linux/kernel.h:17,
from include/linux/list.h:9,
from include/linux/module.h:12,
from arch/arm/mm/mmu.c:7:
include/linux/kern_levels.h:5:18: warning: format '%x' expects argument of type 'unsigned int', but argument 3 has type 'phys_addr_t' {aka 'long long unsigned int'} [-Wformat=]
5 | #define KERN_SOH "\001" /* ASCII Start Of Header */
| ^~~~~~
include/linux/printk.h:140:10: note: in definition of macro 'no_printk'
140 | printk(fmt, ##__VA_ARGS__); \
| ^~~
include/linux/kern_levels.h:15:20: note: in expansion of macro 'KERN_SOH'
15 | #define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
| ^~~~~~~~
include/linux/printk.h:430:12: note: in expansion of macro 'KERN_DEBUG'
430 | no_printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~~~~
arch/arm/mm/mmu.c:1719:2: note: in expansion of macro 'pr_debug'
1719 | pr_debug("physical kernel sections: 0x%08x-0x%08x\n",
| ^~~~~~~~
arch/arm/mm/mmu.c:1719:50: note: format string is defined here
1719 | pr_debug("physical kernel sections: 0x%08x-0x%08x\n",
| ~~~^
| |
| unsigned int
| %08llx
arch/arm/mm/mmu.c: At top level:
arch/arm/mm/mmu.c:1748:13: warning: no previous prototype for 'early_mm_init' [-Wmissing-prototypes]
1748 | void __init early_mm_init(const struct machine_desc *mdesc)
| ^~~~~~~~~~~~~
vim +5 include/linux/kern_levels.h
314ba3520e513a Joe Perches 2012-07-30 4
04d2c8c83d0e3a Joe Perches 2012-07-30 @5 #define KERN_SOH "\001" /* ASCII Start Of Header */
04d2c8c83d0e3a Joe Perches 2012-07-30 6 #define KERN_SOH_ASCII '\001'
04d2c8c83d0e3a Joe Perches 2012-07-30 7
:::::: The code at line 5 was first introduced by commit
:::::: 04d2c8c83d0e3ac5f78aeede51babb3236200112 printk: convert the format for KERN_<LEVEL> to a 2 byte pattern
:::::: TO: Joe Perches <joe(a)perches.com>
:::::: CC: Linus Torvalds <torvalds(a)linux-foundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
[linux-next:master 8712/10489] arch/riscv/mm/init.c:707:13: warning: no previous prototype for function 'protect_kernel_linear_mapping_text_rodata'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: c7d4c1fd91ab4a6d2620497921a9c6bf54650ab8
commit: 6b456a82fa767f62c1329378c6795410fa9b46e6 [8712/10489] Merge branch 'riscv-wx-mappings' into for-next
config: riscv-randconfig-r026-20210615 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
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 riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commi...
git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
git fetch --no-tags linux-next master
git checkout 6b456a82fa767f62c1329378c6795410fa9b46e6
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang 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 warnings (new ones prefixed by >>):
>> arch/riscv/mm/init.c:707:13: warning: no previous prototype for function 'protect_kernel_linear_mapping_text_rodata' [-Wmissing-prototypes]
void __init protect_kernel_linear_mapping_text_rodata(void)
^
arch/riscv/mm/init.c:707:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init protect_kernel_linear_mapping_text_rodata(void)
^
static
1 warning generated.
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for LOCKDEP
Depends on DEBUG_KERNEL && LOCK_DEBUGGING_SUPPORT && (FRAME_POINTER || MIPS || PPC || S390 || MICROBLAZE || ARM || ARC || X86)
Selected by
- LOCK_STAT && DEBUG_KERNEL && LOCK_DEBUGGING_SUPPORT
- DEBUG_LOCK_ALLOC && DEBUG_KERNEL && LOCK_DEBUGGING_SUPPORT
vim +/protect_kernel_linear_mapping_text_rodata +707 arch/riscv/mm/init.c
671f9a3e2e24cd Anup Patel 2019-06-28 705
8d91b097335892 Geert Uytterhoeven 2021-04-29 706 #if defined(CONFIG_64BIT) && defined(CONFIG_STRICT_KERNEL_RWX)
010623568222bd Jisheng Zhang 2021-05-16 @707 void __init protect_kernel_linear_mapping_text_rodata(void)
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 708 {
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 709 unsigned long text_start = (unsigned long)lm_alias(_start);
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 710 unsigned long init_text_start = (unsigned long)lm_alias(__init_text_begin);
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 711 unsigned long rodata_start = (unsigned long)lm_alias(__start_rodata);
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 712 unsigned long data_start = (unsigned long)lm_alias(_data);
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 713
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 714 set_memory_ro(text_start, (init_text_start - text_start) >> PAGE_SHIFT);
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 715 set_memory_nx(text_start, (init_text_start - text_start) >> PAGE_SHIFT);
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 716
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 717 set_memory_ro(rodata_start, (data_start - rodata_start) >> PAGE_SHIFT);
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 718 set_memory_nx(rodata_start, (data_start - rodata_start) >> PAGE_SHIFT);
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 719 }
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 720 #endif
2bfc6cd81bd17e Alexandre Ghiti 2021-04-11 721
:::::: The code at line 707 was first introduced by commit
:::::: 010623568222bd144eb73aa9f3b46c79b63d7676 riscv: mm: init: Consolidate vars, functions
:::::: TO: Jisheng Zhang <jszhang(a)kernel.org>
:::::: CC: Palmer Dabbelt <palmerdabbelt(a)google.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH nf 1/2] netfilter: nft_exthdr: check for IPv6 packet before further processing
by kernel test robot
Hi Pablo,
I love your patch! Yet something to improve:
[auto build test ERROR on nf/master]
url: https://github.com/0day-ci/linux/commits/Pablo-Neira-Ayuso/netfilter-nft_...
base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf.git master
config: x86_64-randconfig-a013-20210615 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
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
# https://github.com/0day-ci/linux/commit/3e2206f7e73972c0cb8a3b6b8a8e0b636...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Pablo-Neira-Ayuso/netfilter-nft_exthdr-check-for-IPv6-packet-before-further-processing/20210616-144640
git checkout 3e2206f7e73972c0cb8a3b6b8a8e0b636a959c96
# 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 >>):
>> net/netfilter/nft_exthdr.c:45:6: error: use of undeclared identifier 'skb'
if (skb->protocol != htons(ETH_P_IPV6))
^
1 error generated.
vim +/skb +45 net/netfilter/nft_exthdr.c
35
36 static void nft_exthdr_ipv6_eval(const struct nft_expr *expr,
37 struct nft_regs *regs,
38 const struct nft_pktinfo *pkt)
39 {
40 struct nft_exthdr *priv = nft_expr_priv(expr);
41 u32 *dest = ®s->data[priv->dreg];
42 unsigned int offset = 0;
43 int err;
44
> 45 if (skb->protocol != htons(ETH_P_IPV6))
46 goto err;
47
48 err = ipv6_find_hdr(pkt->skb, &offset, priv->type, NULL, NULL);
49 if (priv->flags & NFT_EXTHDR_F_PRESENT) {
50 nft_reg_store8(dest, err >= 0);
51 return;
52 } else if (err < 0) {
53 goto err;
54 }
55 offset += priv->offset;
56
57 dest[priv->len / NFT_REG32_SIZE] = 0;
58 if (skb_copy_bits(pkt->skb, offset, dest, priv->len) < 0)
59 goto err;
60 return;
61 err:
62 regs->verdict.code = NFT_BREAK;
63 }
64
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
[luto:x86/membarrier 4/8] kernel/kthread.c:1328:30: error: argument type 'void' is incomplete
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/luto/linux.git x86/membarrier
head: 07a8b963002cb955b7516e61bad19514a3acaa82
commit: f184d013a255a523116b692db4996c5db2569e86 [4/8] membarrier: Make the post-switch-mm barrier explicit
config: arm-randconfig-r032-20210615 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
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 arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://git.kernel.org/pub/scm/linux/kernel/git/luto/linux.git/commit/?id...
git remote add luto https://git.kernel.org/pub/scm/linux/kernel/git/luto/linux.git
git fetch --no-tags luto x86/membarrier
git checkout f184d013a255a523116b692db4996c5db2569e86
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
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 >>):
compiletime_assert_rwonce_type(x); \
^
include/asm-generic/rwonce.h:36:35: note: expanded from macro 'compiletime_assert_rwonce_type'
compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long long), \
^
note: (skipping 1 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:328:22: note: expanded from macro 'compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
^~~~~~~~~
include/linux/compiler_types.h:316:23: note: expanded from macro '_compiletime_assert'
__compiletime_assert(condition, msg, prefix, suffix)
^~~~~~~~~
include/linux/compiler_types.h:308:9: note: expanded from macro '__compiletime_assert'
if (!(condition)) \
^~~~~~~~~
kernel/kthread.c:1328:47: error: no member named 'membarrier_state' in 'struct mm_struct'
membarrier_finish_switch_mm(atomic_read(&mm->membarrier_state));
~~ ^
arch/arm/include/asm/atomic.h:25:35: note: expanded from macro 'atomic_read'
#define atomic_read(v) READ_ONCE((v)->counter)
^
include/asm-generic/rwonce.h:49:33: note: expanded from macro 'READ_ONCE'
compiletime_assert_rwonce_type(x); \
^
include/asm-generic/rwonce.h:36:35: note: expanded from macro 'compiletime_assert_rwonce_type'
compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long long), \
^
note: (skipping 1 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:328:22: note: expanded from macro 'compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
^~~~~~~~~
include/linux/compiler_types.h:316:23: note: expanded from macro '_compiletime_assert'
__compiletime_assert(condition, msg, prefix, suffix)
^~~~~~~~~
include/linux/compiler_types.h:308:9: note: expanded from macro '__compiletime_assert'
if (!(condition)) \
^~~~~~~~~
kernel/kthread.c:1328:47: error: no member named 'membarrier_state' in 'struct mm_struct'
membarrier_finish_switch_mm(atomic_read(&mm->membarrier_state));
~~ ^
arch/arm/include/asm/atomic.h:25:35: note: expanded from macro 'atomic_read'
#define atomic_read(v) READ_ONCE((v)->counter)
^
include/asm-generic/rwonce.h:49:33: note: expanded from macro 'READ_ONCE'
compiletime_assert_rwonce_type(x); \
^
include/asm-generic/rwonce.h:36:48: note: expanded from macro 'compiletime_assert_rwonce_type'
compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long long), \
^
include/linux/compiler_types.h:328:22: note: expanded from macro 'compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
^~~~~~~~~
include/linux/compiler_types.h:316:23: note: expanded from macro '_compiletime_assert'
__compiletime_assert(condition, msg, prefix, suffix)
^~~~~~~~~
include/linux/compiler_types.h:308:9: note: expanded from macro '__compiletime_assert'
if (!(condition)) \
^~~~~~~~~
kernel/kthread.c:1328:47: error: no member named 'membarrier_state' in 'struct mm_struct'
membarrier_finish_switch_mm(atomic_read(&mm->membarrier_state));
~~ ^
arch/arm/include/asm/atomic.h:25:35: note: expanded from macro 'atomic_read'
#define atomic_read(v) READ_ONCE((v)->counter)
^
include/asm-generic/rwonce.h:50:14: note: expanded from macro 'READ_ONCE'
__READ_ONCE(x); \
^
include/asm-generic/rwonce.h:44:65: note: expanded from macro '__READ_ONCE'
#define __READ_ONCE(x) (*(const volatile __unqual_scalar_typeof(x) *)&(x))
^
include/linux/compiler_types.h:279:13: note: expanded from macro '__unqual_scalar_typeof'
_Generic((x), \
^
kernel/kthread.c:1328:47: error: no member named 'membarrier_state' in 'struct mm_struct'
membarrier_finish_switch_mm(atomic_read(&mm->membarrier_state));
~~ ^
arch/arm/include/asm/atomic.h:25:35: note: expanded from macro 'atomic_read'
#define atomic_read(v) READ_ONCE((v)->counter)
^
include/asm-generic/rwonce.h:50:14: note: expanded from macro 'READ_ONCE'
__READ_ONCE(x); \
^
include/asm-generic/rwonce.h:44:65: note: expanded from macro '__READ_ONCE'
#define __READ_ONCE(x) (*(const volatile __unqual_scalar_typeof(x) *)&(x))
^
include/linux/compiler_types.h:286:15: note: expanded from macro '__unqual_scalar_typeof'
default: (x)))
^
kernel/kthread.c:1328:47: error: no member named 'membarrier_state' in 'struct mm_struct'
membarrier_finish_switch_mm(atomic_read(&mm->membarrier_state));
~~ ^
arch/arm/include/asm/atomic.h:25:35: note: expanded from macro 'atomic_read'
#define atomic_read(v) READ_ONCE((v)->counter)
^
include/asm-generic/rwonce.h:50:14: note: expanded from macro 'READ_ONCE'
__READ_ONCE(x); \
^
include/asm-generic/rwonce.h:44:72: note: expanded from macro '__READ_ONCE'
#define __READ_ONCE(x) (*(const volatile __unqual_scalar_typeof(x) *)&(x))
^
>> kernel/kthread.c:1328:30: error: argument type 'void' is incomplete
membarrier_finish_switch_mm(atomic_read(&mm->membarrier_state));
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
arch/arm/include/asm/atomic.h:25:24: note: expanded from macro 'atomic_read'
#define atomic_read(v) READ_ONCE((v)->counter)
^~~~~~~~~~~~~~~~~~~~~~~
include/asm-generic/rwonce.h:47:28: note: expanded from macro 'READ_ONCE'
#define READ_ONCE(x) \
^
10 errors generated.
vim +/void +1328 kernel/kthread.c
1304
1305 /**
1306 * kthread_use_mm - make the calling kthread operate on an address space
1307 * @mm: address space to operate on
1308 */
1309 void kthread_use_mm(struct mm_struct *mm)
1310 {
1311 struct mm_struct *active_mm;
1312 struct task_struct *tsk = current;
1313
1314 WARN_ON_ONCE(!(tsk->flags & PF_KTHREAD));
1315 WARN_ON_ONCE(tsk->mm);
1316
1317 task_lock(tsk);
1318 /* Hold off tlb flush IPIs while switching mm's */
1319 local_irq_disable();
1320 active_mm = tsk->active_mm;
1321 if (active_mm != mm) {
1322 mmgrab(mm);
1323 tsk->active_mm = mm;
1324 }
1325 tsk->mm = mm;
1326 membarrier_update_current_mm(mm);
1327 switch_mm_irqs_off(active_mm, mm, tsk);
> 1328 membarrier_finish_switch_mm(atomic_read(&mm->membarrier_state));
1329 local_irq_enable();
1330 task_unlock(tsk);
1331 #ifdef finish_arch_post_lock_switch
1332 finish_arch_post_lock_switch();
1333 #endif
1334
1335 if (active_mm != mm)
1336 mmdrop(active_mm);
1337
1338 to_kthread(tsk)->oldfs = force_uaccess_begin();
1339 }
1340 EXPORT_SYMBOL_GPL(kthread_use_mm);
1341
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH nf 1/2] netfilter: nft_exthdr: check for IPv6 packet before further processing
by kernel test robot
Hi Pablo,
I love your patch! Yet something to improve:
[auto build test ERROR on nf/master]
url: https://github.com/0day-ci/linux/commits/Pablo-Neira-Ayuso/netfilter-nft_...
base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf.git master
config: arm-randconfig-r001-20210615 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
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 arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://github.com/0day-ci/linux/commit/3e2206f7e73972c0cb8a3b6b8a8e0b636...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Pablo-Neira-Ayuso/netfilter-nft_exthdr-check-for-IPv6-packet-before-further-processing/20210616-144640
git checkout 3e2206f7e73972c0cb8a3b6b8a8e0b636a959c96
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
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 >>):
>> net/netfilter/nft_exthdr.c:45:6: error: use of undeclared identifier 'skb'
if (skb->protocol != htons(ETH_P_IPV6))
^
1 error generated.
vim +/skb +45 net/netfilter/nft_exthdr.c
35
36 static void nft_exthdr_ipv6_eval(const struct nft_expr *expr,
37 struct nft_regs *regs,
38 const struct nft_pktinfo *pkt)
39 {
40 struct nft_exthdr *priv = nft_expr_priv(expr);
41 u32 *dest = ®s->data[priv->dreg];
42 unsigned int offset = 0;
43 int err;
44
> 45 if (skb->protocol != htons(ETH_P_IPV6))
46 goto err;
47
48 err = ipv6_find_hdr(pkt->skb, &offset, priv->type, NULL, NULL);
49 if (priv->flags & NFT_EXTHDR_F_PRESENT) {
50 nft_reg_store8(dest, err >= 0);
51 return;
52 } else if (err < 0) {
53 goto err;
54 }
55 offset += priv->offset;
56
57 dest[priv->len / NFT_REG32_SIZE] = 0;
58 if (skb_copy_bits(pkt->skb, offset, dest, priv->len) < 0)
59 goto err;
60 return;
61 err:
62 regs->verdict.code = NFT_BREAK;
63 }
64
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH v2 1/2] mm: rename pud_page_vaddr to pud_pgtable and make it return pmd_t *
by kernel test robot
Hi "Aneesh,
I love your patch! Perhaps something to improve:
[auto build test WARNING on powerpc/next]
[also build test WARNING on asm-generic/master tip/x86/mm linus/master sparc/master v5.13-rc6 next-20210615]
[cannot apply to hnaz-linux-mm/master sparc-next/master]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Aneesh-Kumar-K-V/mm-rename-pud_p...
base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: alpha-randconfig-r026-20210615 (attached as .config)
compiler: alpha-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
# https://github.com/0day-ci/linux/commit/8af9180cf61ec19f32f0594a59277d0ca...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Aneesh-Kumar-K-V/mm-rename-pud_page_vaddr-to-pud_pgtable-and-make-it-return-pmd_t/20210616-162847
git checkout 8af9180cf61ec19f32f0594a59277d0cabff5317
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=alpha
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 >>):
arch/alpha/kernel/asm-offsets.c:15:6: warning: no previous prototype for 'foo' [-Wmissing-prototypes]
15 | void foo(void)
| ^~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from include/linux/pid_namespace.h:7,
from include/linux/ptrace.h:10,
from arch/alpha/kernel/asm-offsets.c:11:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/include/asm/io.h:8,
from include/linux/io.h:13,
from include/linux/irq.h:20,
from arch/alpha/kernel/irq_srm.c:8:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/traps.c:212:1: warning: no previous prototype for 'do_entArith' [-Wmissing-prototypes]
212 | do_entArith(unsigned long summary, unsigned long write_mask,
| ^~~~~~~~~~~
arch/alpha/kernel/traps.c:234:1: warning: no previous prototype for 'do_entIF' [-Wmissing-prototypes]
234 | do_entIF(unsigned long type, struct pt_regs *regs)
| ^~~~~~~~
arch/alpha/kernel/traps.c:401:1: warning: no previous prototype for 'do_entDbg' [-Wmissing-prototypes]
401 | do_entDbg(struct pt_regs *regs)
| ^~~~~~~~~
arch/alpha/kernel/traps.c:437:1: warning: no previous prototype for 'do_entUna' [-Wmissing-prototypes]
437 | do_entUna(void * va, unsigned long opcode, unsigned long reg,
| ^~~~~~~~~
arch/alpha/kernel/traps.c:722:1: warning: no previous prototype for 'do_entUnaUser' [-Wmissing-prototypes]
722 | do_entUnaUser(void __user * va, unsigned long opcode,
| ^~~~~~~~~~~~~
arch/alpha/kernel/traps.c:977:1: warning: no previous prototype for 'trap_init' [-Wmissing-prototypes]
977 | trap_init(void)
| ^~~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/kernel/traps.c:13:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/process.c:57:6: warning: no previous prototype for 'arch_cpu_idle' [-Wmissing-prototypes]
57 | void arch_cpu_idle(void)
| ^~~~~~~~~~~~~
arch/alpha/kernel/process.c:63:6: warning: no previous prototype for 'arch_cpu_idle_dead' [-Wmissing-prototypes]
63 | void arch_cpu_idle_dead(void)
| ^~~~~~~~~~~~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/kernel/process.c:19:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/irq.c:96:1: warning: no previous prototype for 'handle_irq' [-Wmissing-prototypes]
96 | handle_irq(int irq)
| ^~~~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/include/asm/io.h:8,
from include/linux/io.h:13,
from include/linux/irq.h:20,
from include/asm-generic/hardirq.h:17,
from arch/alpha/include/asm/hardirq.h:8,
from include/linux/hardirq.h:11,
from include/linux/interrupt.h:11,
from include/linux/kernel_stat.h:9,
from arch/alpha/kernel/irq.c:17:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/irq_alpha.c:45:1: warning: no previous prototype for 'do_entInt' [-Wmissing-prototypes]
45 | do_entInt(unsigned long type, unsigned long vector,
| ^~~~~~~~~
arch/alpha/kernel/irq_alpha.c:103:1: warning: no previous prototype for 'init_IRQ' [-Wmissing-prototypes]
103 | init_IRQ(void)
| ^~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/include/asm/io.h:8,
from include/linux/io.h:13,
from include/linux/irq.h:20,
from arch/alpha/kernel/irq_alpha.c:8:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/signal.c:204:1: warning: no previous prototype for 'do_sigreturn' [-Wmissing-prototypes]
204 | do_sigreturn(struct sigcontext __user *sc)
| ^~~~~~~~~~~~
arch/alpha/kernel/signal.c:232:1: warning: no previous prototype for 'do_rt_sigreturn' [-Wmissing-prototypes]
232 | do_rt_sigreturn(struct rt_sigframe __user *frame)
| ^~~~~~~~~~~~~~~
arch/alpha/kernel/signal.c:522:1: warning: no previous prototype for 'do_work_pending' [-Wmissing-prototypes]
522 | do_work_pending(struct pt_regs *regs, unsigned long thread_flags,
| ^~~~~~~~~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from include/linux/pid_namespace.h:7,
from include/linux/ptrace.h:10,
from arch/alpha/kernel/signal.c:16:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/setup.c:289:1: warning: no previous prototype for 'move_initrd' [-Wmissing-prototypes]
289 | move_initrd(unsigned long mem_limit)
| ^~~~~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/kernel/setup.c:16:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/ptrace.c:321:26: warning: no previous prototype for 'syscall_trace_enter' [-Wmissing-prototypes]
321 | asmlinkage unsigned long syscall_trace_enter(void)
| ^~~~~~~~~~~~~~~~~~~
arch/alpha/kernel/ptrace.c:333:1: warning: no previous prototype for 'syscall_trace_leave' [-Wmissing-prototypes]
333 | syscall_trace_leave(void)
| ^~~~~~~~~~~~~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/kernel/ptrace.c:11:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/time.c:64:6: warning: no previous prototype for 'arch_irq_work_raise' [-Wmissing-prototypes]
64 | void arch_irq_work_raise(void)
| ^~~~~~~~~~~~~~~~~~~
arch/alpha/kernel/time.c:390:1: warning: no previous prototype for 'time_init' [-Wmissing-prototypes]
390 | time_init(void)
| ^~~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/kernel/time.c:28:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
--
arch/alpha/kernel/io.c:637:1: warning: no previous prototype for 'scr_memcpyw' [-Wmissing-prototypes]
637 | scr_memcpyw(u16 *d, const u16 *s, unsigned int count)
| ^~~~~~~~~~~
In file included from include/linux/pgtable.h:6,
from include/linux/mm.h:33,
from arch/alpha/include/asm/io.h:8,
from arch/alpha/kernel/io.c:10:
>> arch/alpha/include/asm/pgtable.h:295:15: warning: 'pud_pgtable' is static but used in inline function 'pmd_offset' which is not static
295 | pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
| ^~~~~~~~~~~
..
vim +295 arch/alpha/include/asm/pgtable.h
278
279 /*
280 * The smp_rmb() in the following functions are required to order the load of
281 * *dir (the pointer in the top level page table) with any subsequent load of
282 * the returned pmd_t *ret (ret is data dependent on *dir).
283 *
284 * If this ordering is not enforced, the CPU might load an older value of
285 * *ret, which may be uninitialized data. See mm/memory.c:__pte_alloc for
286 * more details.
287 *
288 * Note that we never change the mm->pgd pointer after the task is running, so
289 * pgd_offset does not require such a barrier.
290 */
291
292 /* Find an entry in the second-level page table.. */
293 extern inline pmd_t * pmd_offset(pud_t * dir, unsigned long address)
294 {
> 295 pmd_t *ret = pud_pgtable(*dir) + ((address >> PMD_SHIFT) & (PTRS_PER_PAGE - 1));
296 smp_rmb(); /* see above */
297 return ret;
298 }
299 #define pmd_offset pmd_offset
300
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH v2 2/2] mm: rename p4d_page_vaddr to p4d_pgtable and make it return pud_t *
by kernel test robot
Hi "Aneesh,
I love your patch! Perhaps something to improve:
[auto build test WARNING on powerpc/next]
[also build test WARNING on asm-generic/master tip/x86/mm linus/master sparc/master v5.13-rc6 next-20210615]
[cannot apply to hnaz-linux-mm/master]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Aneesh-Kumar-K-V/mm-rename-pud_p...
base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: m68k-allmodconfig (attached as .config)
compiler: m68k-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
# https://github.com/0day-ci/linux/commit/3eb7b4e26cc0ea2cf27fc7b23daa95f1f...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Aneesh-Kumar-K-V/mm-rename-pud_page_vaddr-to-pud_pgtable-and-make-it-return-pmd_t/20210616-162847
git checkout 3eb7b4e26cc0ea2cf27fc7b23daa95f1f681cfd8
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k
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 >>):
arch/m68k/mm/motorola.c: In function 'kernel_ptr_table':
>> arch/m68k/mm/motorola.c:265:8: warning: assignment to 'long unsigned int' from 'pud_t *' {aka 'struct <anonymous> *'} makes integer from pointer without a cast [-Wint-conversion]
265 | pmd = pgd_page_vaddr(kernel_pg_dir[i]);
| ^
arch/m68k/mm/motorola.c: At top level:
arch/m68k/mm/motorola.c:390:13: warning: no previous prototype for 'paging_init' [-Wmissing-prototypes]
390 | void __init paging_init(void)
| ^~~~~~~~~~~
vim +265 arch/m68k/mm/motorola.c
^1da177e4c3f415 Linus Torvalds 2005-04-16 248
^1da177e4c3f415 Linus Torvalds 2005-04-16 249 static pmd_t * __init kernel_ptr_table(void)
^1da177e4c3f415 Linus Torvalds 2005-04-16 250 {
ef9285f69f0efbc Peter Zijlstra 2020-01-31 251 if (!last_pmd_table) {
^1da177e4c3f415 Linus Torvalds 2005-04-16 252 unsigned long pmd, last;
^1da177e4c3f415 Linus Torvalds 2005-04-16 253 int i;
^1da177e4c3f415 Linus Torvalds 2005-04-16 254
^1da177e4c3f415 Linus Torvalds 2005-04-16 255 /* Find the last ptr table that was used in head.S and
^1da177e4c3f415 Linus Torvalds 2005-04-16 256 * reuse the remaining space in that page for further
^1da177e4c3f415 Linus Torvalds 2005-04-16 257 * ptr tables.
^1da177e4c3f415 Linus Torvalds 2005-04-16 258 */
^1da177e4c3f415 Linus Torvalds 2005-04-16 259 last = (unsigned long)kernel_pg_dir;
^1da177e4c3f415 Linus Torvalds 2005-04-16 260 for (i = 0; i < PTRS_PER_PGD; i++) {
60e50f34b13e9e4 Mike Rapoport 2019-12-04 261 pud_t *pud = (pud_t *)(&kernel_pg_dir[i]);
60e50f34b13e9e4 Mike Rapoport 2019-12-04 262
60e50f34b13e9e4 Mike Rapoport 2019-12-04 263 if (!pud_present(*pud))
^1da177e4c3f415 Linus Torvalds 2005-04-16 264 continue;
60e50f34b13e9e4 Mike Rapoport 2019-12-04 @265 pmd = pgd_page_vaddr(kernel_pg_dir[i]);
^1da177e4c3f415 Linus Torvalds 2005-04-16 266 if (pmd > last)
^1da177e4c3f415 Linus Torvalds 2005-04-16 267 last = pmd;
^1da177e4c3f415 Linus Torvalds 2005-04-16 268 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 269
ef9285f69f0efbc Peter Zijlstra 2020-01-31 270 last_pmd_table = (pmd_t *)last;
^1da177e4c3f415 Linus Torvalds 2005-04-16 271 #ifdef DEBUG
ef9285f69f0efbc Peter Zijlstra 2020-01-31 272 printk("kernel_ptr_init: %p\n", last_pmd_table);
^1da177e4c3f415 Linus Torvalds 2005-04-16 273 #endif
^1da177e4c3f415 Linus Torvalds 2005-04-16 274 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 275
ef9285f69f0efbc Peter Zijlstra 2020-01-31 276 last_pmd_table += PTRS_PER_PMD;
41f1bf37a63ecdb Geert Uytterhoeven 2020-08-26 277 if (PAGE_ALIGNED(last_pmd_table)) {
7e158826564fbbb Geert Uytterhoeven 2020-08-26 278 last_pmd_table = memblock_alloc_low(PAGE_SIZE, PAGE_SIZE);
ef9285f69f0efbc Peter Zijlstra 2020-01-31 279 if (!last_pmd_table)
8a7f97b902f4fb0 Mike Rapoport 2019-03-11 280 panic("%s: Failed to allocate %lu bytes align=%lx\n",
8a7f97b902f4fb0 Mike Rapoport 2019-03-11 281 __func__, PAGE_SIZE, PAGE_SIZE);
^1da177e4c3f415 Linus Torvalds 2005-04-16 282
ef9285f69f0efbc Peter Zijlstra 2020-01-31 283 clear_page(last_pmd_table);
ef9285f69f0efbc Peter Zijlstra 2020-01-31 284 mmu_page_ctor(last_pmd_table);
^1da177e4c3f415 Linus Torvalds 2005-04-16 285 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 286
ef9285f69f0efbc Peter Zijlstra 2020-01-31 287 return last_pmd_table;
^1da177e4c3f415 Linus Torvalds 2005-04-16 288 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 289
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH v5] sched/fair: Correctly insert cfs_rq's to list on unthrottle
by kernel test robot
Hi Odin,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on tip/sched/core]
[also build test ERROR on tip/master linux/master linus/master v5.13-rc6 next-20210615]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Odin-Ugedal/sched-fair-Correctly...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git 0159bb020ca9a43b17aa9149f1199643c1d49426
config: m68k-randconfig-r021-20210615 (attached as .config)
compiler: m68k-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
# https://github.com/0day-ci/linux/commit/670fb8556a2d3594c67bf4ab2983ba4f9...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Odin-Ugedal/sched-fair-Correctly-insert-cfs_rq-s-to-list-on-unthrottle/20210616-151742
git checkout 670fb8556a2d3594c67bf4ab2983ba4f91f5686b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k
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 >>):
kernel/sched/fair.c: In function 'tg_unthrottle_up':
>> kernel/sched/fair.c:4724:8: error: implicit declaration of function 'cfs_rq_is_decayed' [-Werror=implicit-function-declaration]
4724 | if (!cfs_rq_is_decayed(cfs_rq) || cfs_rq->nr_running)
| ^~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/cfs_rq_is_decayed +4724 kernel/sched/fair.c
4712
4713 static int tg_unthrottle_up(struct task_group *tg, void *data)
4714 {
4715 struct rq *rq = data;
4716 struct cfs_rq *cfs_rq = tg->cfs_rq[cpu_of(rq)];
4717
4718 cfs_rq->throttle_count--;
4719 if (!cfs_rq->throttle_count) {
4720 cfs_rq->throttled_clock_task_time += rq_clock_task(rq) -
4721 cfs_rq->throttled_clock_task;
4722
4723 /* Add cfs_rq with load or one or more already running entities to the list */
> 4724 if (!cfs_rq_is_decayed(cfs_rq) || cfs_rq->nr_running)
4725 list_add_leaf_cfs_rq(cfs_rq);
4726 }
4727
4728 return 0;
4729 }
4730
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months