Re: [RFC PATCH v0 1/3] percpu: CPU hotplug support for alloc_percpu()
by kernel test robot
Hi Bharata,
[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on tip/sched/core]
[also build test ERROR on linux/master linus/master v5.13-rc4 next-20210528]
[cannot apply to dennis-percpu/for-next]
[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/Bharata-B-Rao/CPU-hotplug-awaren...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git a8ea6fc9b089156d9230bfeef964dd9be101a4a9
config: i386-tinyconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
# https://github.com/0day-ci/linux/commit/56c0782cce05ec8801eac57b90a531eb0...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Bharata-B-Rao/CPU-hotplug-awareness-in-percpu-allocator/20210601-145307
git checkout 56c0782cce05ec8801eac57b90a531eb01e025c0
# save the attached .config to linux build tree
make W=1 ARCH=i386
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 >>):
mm/percpu.c: In function '__pcpu_cpuhp_setup':
>> mm/percpu.c:1698:5: error: implicit declaration of function 'pcpu_populate_chunk_cpu'; did you mean 'pcpu_populate_chunk'? [-Werror=implicit-function-declaration]
1698 | pcpu_populate_chunk_cpu(cpu, chunk, rs, re,
| ^~~~~~~~~~~~~~~~~~~~~~~
| pcpu_populate_chunk
mm/percpu.c: In function '__pcpu_cpuhp_destroy':
>> mm/percpu.c:1738:5: error: implicit declaration of function 'pcpu_depopulate_chunk_cpu'; did you mean 'pcpu_depopulate_chunk'? [-Werror=implicit-function-declaration]
1738 | pcpu_depopulate_chunk_cpu(cpu, chunk, rs, re);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
| pcpu_depopulate_chunk
mm/percpu.c: At top level:
>> mm/percpu.c:2124:16: warning: no previous prototype for '__alloc_reserved_percpu_cb' [-Wmissing-prototypes]
2124 | void __percpu *__alloc_reserved_percpu_cb(size_t size, size_t align,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +1698 mm/percpu.c
1682
1683 static void __pcpu_cpuhp_setup(enum pcpu_chunk_type type, unsigned int cpu)
1684 {
1685 int slot;
1686 struct list_head *pcpu_slot = pcpu_chunk_list(type);
1687 struct pcpu_chunk *chunk;
1688
1689 for (slot = 0; slot < pcpu_nr_slots; slot++) {
1690 list_for_each_entry(chunk, &pcpu_slot[slot], list) {
1691 unsigned int rs, re;
1692
1693 if (chunk == pcpu_first_chunk)
1694 continue;
1695
1696 bitmap_for_each_set_region(chunk->populated, rs, re, 0,
1697 chunk->nr_pages)
> 1698 pcpu_populate_chunk_cpu(cpu, chunk, rs, re,
1699 GFP_KERNEL);
1700 }
1701 }
1702 }
1703
1704 /**
1705 * cpu hotplug callback for percpu allocator
1706 * @cpu: cpu that is being hotplugged
1707 *
1708 * Allocates and maps the pages that corresponds to @cpu's unit
1709 * in all chunks.
1710 */
1711 static int percpu_cpuhp_setup(unsigned int cpu)
1712 {
1713 enum pcpu_chunk_type type;
1714
1715 mutex_lock(&pcpu_alloc_mutex);
1716 for (type = 0; type < PCPU_NR_CHUNK_TYPES; type++)
1717 __pcpu_cpuhp_setup(type, cpu);
1718 mutex_unlock(&pcpu_alloc_mutex);
1719
1720 return 0;
1721 }
1722
1723 static void __pcpu_cpuhp_destroy(enum pcpu_chunk_type type, unsigned int cpu)
1724 {
1725 int slot;
1726 struct list_head *pcpu_slot = pcpu_chunk_list(type);
1727 struct pcpu_chunk *chunk;
1728
1729 for (slot = 0; slot < pcpu_nr_slots; slot++) {
1730 list_for_each_entry(chunk, &pcpu_slot[slot], list) {
1731 unsigned int rs, re;
1732
1733 if (chunk == pcpu_first_chunk)
1734 continue;
1735
1736 bitmap_for_each_set_region(chunk->populated, rs, re, 0,
1737 chunk->nr_pages)
> 1738 pcpu_depopulate_chunk_cpu(cpu, chunk, rs, re);
1739 }
1740 }
1741 }
1742
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
[linux-next:master 5596/5886] arch/x86/include/asm/fixmap.h:103:48: error: use of undeclared identifier 'NR_CPUS'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: 3e029760e6f8ce90c122c267a039ae73b3f1f5a4
commit: 366e9aec4a22d77b869f1b7641a74fdd96f02b4c [5596/5886] mm/early_ioremap: add prototype for early_memremap_pgprot_adjust
config: x86_64-randconfig-a006-20210530 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project bc6799f2f79f0ae87e9f1ebf9d25ba799fbd25a9)
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://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 366e9aec4a22d77b869f1b7641a74fdd96f02b4c
# 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 >>):
In file included from drivers/gpu/drm/udl/udl_modeset.c:12:
In file included from include/linux/dma-buf.h:16:
In file included from include/linux/dma-buf-map.h:9:
In file included from include/linux/io.h:13:
In file included from arch/x86/include/asm/io.h:44:
In file included from ./arch/x86/include/generated/asm/early_ioremap.h:1:
In file included from include/asm-generic/early_ioremap.h:6:
>> arch/x86/include/asm/fixmap.h:103:48: error: use of undeclared identifier 'NR_CPUS'
FIX_KMAP_END = FIX_KMAP_BEGIN + (KM_MAX_IDX * NR_CPUS) - 1,
^
1 error generated.
vim +/NR_CPUS +103 arch/x86/include/asm/fixmap.h
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 61
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 62 /*
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 63 * Here we define all the compile-time 'special' virtual
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 64 * addresses. The point is to have a constant address at
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 65 * compile time, but to set the physical address only
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 66 * in the boot process.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 67 * for x86_32: We allocate these special addresses
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 68 * from the end of virtual memory (0xfffff000) backwards.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 69 * Also this lets us do fail-safe vmalloc(), we
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 70 * can guarantee that these special addresses and
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 71 * vmalloc()-ed addresses never overlap.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 72 *
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 73 * These 'compile-time allocated' memory buffers are
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 74 * fixed-size 4k pages (or larger if used with an increment
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 75 * higher than 1). Use set_fixmap(idx,phys) to associate
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 76 * physical memory with fixmap indices.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 77 *
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 78 * TLB entries of such buffers will not be flushed across
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 79 * task switches.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 80 */
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 81 enum fixed_addresses {
96a388de5dc53a include/asm-x86/fixmap.h Thomas Gleixner 2007-10-11 82 #ifdef CONFIG_X86_32
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 83 FIX_HOLE,
96a388de5dc53a include/asm-x86/fixmap.h Thomas Gleixner 2007-10-11 84 #else
1ad83c858c7d4e arch/x86/include/asm/fixmap.h Andy Lutomirski 2014-10-29 85 #ifdef CONFIG_X86_VSYSCALL_EMULATION
f40c330091c7aa arch/x86/include/asm/fixmap.h Andy Lutomirski 2014-05-05 86 VSYSCALL_PAGE = (FIXADDR_TOP - VSYSCALL_ADDR) >> PAGE_SHIFT,
1ad83c858c7d4e arch/x86/include/asm/fixmap.h Andy Lutomirski 2014-10-29 87 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 88 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 89 FIX_DBGP_BASE,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 90 FIX_EARLYCON_MEM_BASE,
ff30a0543e9a6c arch/x86/include/asm/fixmap.h Jan Beulich 2010-03-15 91 #ifdef CONFIG_PROVIDE_OHCI1394_DMA_INIT
ff30a0543e9a6c arch/x86/include/asm/fixmap.h Jan Beulich 2010-03-15 92 FIX_OHCI1394_BASE,
ff30a0543e9a6c arch/x86/include/asm/fixmap.h Jan Beulich 2010-03-15 93 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 94 #ifdef CONFIG_X86_LOCAL_APIC
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 95 FIX_APIC_BASE, /* local (CPU) APIC) -- required for SMP or not */
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 96 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 97 #ifdef CONFIG_X86_IO_APIC
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 98 FIX_IO_APIC_BASE_0,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 99 FIX_IO_APIC_BASE_END = FIX_IO_APIC_BASE_0 + MAX_IO_APICS - 1,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 100 #endif
14df3267029158 arch/x86/include/asm/fixmap.h Thomas Gleixner 2020-11-18 101 #ifdef CONFIG_KMAP_LOCAL
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 102 FIX_KMAP_BEGIN, /* reserved pte's for temporary kernel mappings */
157e118b55113d arch/x86/include/asm/fixmap.h Thomas Gleixner 2020-11-03 @103 FIX_KMAP_END = FIX_KMAP_BEGIN + (KM_MAX_IDX * NR_CPUS) - 1,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 104 #ifdef CONFIG_PCI_MMCONFIG
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 105 FIX_PCIE_MCFG,
96a388de5dc53a include/asm-x86/fixmap.h Thomas Gleixner 2007-10-11 106 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 107 #endif
ecac71816a1829 arch/x86/include/asm/fixmap.h Juergen Gross 2020-08-15 108 #ifdef CONFIG_PARAVIRT_XXL
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 109 FIX_PARAVIRT_BOOTMAP,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 110 #endif
69218e47994da6 arch/x86/include/asm/fixmap.h Thomas Garnier 2017-03-14 111
:::::: The code at line 103 was first introduced by commit
:::::: 157e118b55113d1e6c7f8ddfcec0a1dbf3a69511 x86/mm/highmem: Use generic kmap atomic implementation
:::::: TO: Thomas Gleixner <tglx(a)linutronix.de>
:::::: CC: Thomas Gleixner <tglx(a)linutronix.de>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
Re: [PATCH v3 3/5] media: i2c: add driver for the SK Hynix Hi-846 8M pixel camera
by kernel test robot
Hi Martin,
I love your patch! Yet something to improve:
[auto build test ERROR on linuxtv-media/master]
[also build test ERROR on robh/for-next pavel-linux-leds/for-next v5.13-rc4 next-20210528]
[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/Martin-Kepplinger/Add-support-fo...
base: git://linuxtv.org/media_tree.git master
config: nios2-allyesconfig (attached as .config)
compiler: nios2-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/990f55e1171e0120ffbbb77faac9b7509...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Martin-Kepplinger/Add-support-for-the-Hynix-Hi-846-camera/20210531-200931
git checkout 990f55e1171e0120ffbbb77faac9b75091f105f2
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=nios2
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 >>):
In file included from ./arch/nios2/include/generated/asm/div64.h:1,
from include/linux/math.h:5,
from include/linux/kernel.h:14,
from include/linux/unaligned/packed_struct.h:4,
from include/linux/unaligned/le_struct.h:5,
from include/asm-generic/unaligned.h:18,
from ./arch/nios2/include/generated/asm/unaligned.h:1,
from drivers/media/i2c/hi846.c:4:
drivers/media/i2c/hi846.c: In function 'hi846_set_video_mode':
include/asm-generic/div64.h:228:28: warning: comparison of distinct pointer types lacks a cast
228 | (void)(((typeof((n)) *)0) == ((uint64_t *)0)); \
| ^~
include/asm-generic/div64.h:226:21: note: in definition of macro 'do_div'
226 | uint32_t __base = (base); \
| ^~~~
drivers/media/i2c/hi846.c:1469:35: note: in expansion of macro 'do_div'
1469 | frame_length = do_div(link_freq, do_div(fps, HI846_LINE_LENGTH));
| ^~~~~~
include/asm-generic/div64.h:241:25: warning: right shift count >= width of type [-Wshift-count-overflow]
241 | } else if (likely(((n) >> 32) == 0)) { \
| ^~
include/asm-generic/div64.h:226:21: note: in definition of macro 'do_div'
226 | uint32_t __base = (base); \
| ^~~~
include/asm-generic/div64.h:241:13: note: in expansion of macro 'likely'
241 | } else if (likely(((n) >> 32) == 0)) { \
| ^~~~~~
drivers/media/i2c/hi846.c:1469:35: note: in expansion of macro 'do_div'
1469 | frame_length = do_div(link_freq, do_div(fps, HI846_LINE_LENGTH));
| ^~~~~~
>> include/asm-generic/div64.h:245:22: error: passing argument 1 of '__div64_32' from incompatible pointer type [-Werror=incompatible-pointer-types]
245 | __rem = __div64_32(&(n), __base); \
| ^~~~
| |
| int *
include/asm-generic/div64.h:226:21: note: in definition of macro 'do_div'
226 | uint32_t __base = (base); \
| ^~~~
drivers/media/i2c/hi846.c:1469:35: note: in expansion of macro 'do_div'
1469 | frame_length = do_div(link_freq, do_div(fps, HI846_LINE_LENGTH));
| ^~~~~~
include/asm-generic/div64.h:219:38: note: expected 'uint64_t *' {aka 'long long unsigned int *'} but argument is of type 'int *'
219 | extern uint32_t __div64_32(uint64_t *dividend, uint32_t divisor);
| ~~~~~~~~~~^~~~~~~~
include/asm-generic/div64.h:228:28: warning: comparison of distinct pointer types lacks a cast
228 | (void)(((typeof((n)) *)0) == ((uint64_t *)0)); \
| ^~
drivers/media/i2c/hi846.c:1469:17: note: in expansion of macro 'do_div'
1469 | frame_length = do_div(link_freq, do_div(fps, HI846_LINE_LENGTH));
| ^~~~~~
cc1: some warnings being treated as errors
vim +/__div64_32 +245 include/asm-generic/div64.h
^1da177e4c3f41 Linus Torvalds 2005-04-16 221
^1da177e4c3f41 Linus Torvalds 2005-04-16 222 /* The unnecessary pointer compare is there
^1da177e4c3f41 Linus Torvalds 2005-04-16 223 * to check for type safety (n must be 64bit)
^1da177e4c3f41 Linus Torvalds 2005-04-16 224 */
^1da177e4c3f41 Linus Torvalds 2005-04-16 225 # define do_div(n,base) ({ \
^1da177e4c3f41 Linus Torvalds 2005-04-16 226 uint32_t __base = (base); \
^1da177e4c3f41 Linus Torvalds 2005-04-16 227 uint32_t __rem; \
^1da177e4c3f41 Linus Torvalds 2005-04-16 228 (void)(((typeof((n)) *)0) == ((uint64_t *)0)); \
911918aa7ef6f8 Nicolas Pitre 2015-11-02 229 if (__builtin_constant_p(__base) && \
911918aa7ef6f8 Nicolas Pitre 2015-11-02 230 is_power_of_2(__base)) { \
911918aa7ef6f8 Nicolas Pitre 2015-11-02 231 __rem = (n) & (__base - 1); \
911918aa7ef6f8 Nicolas Pitre 2015-11-02 232 (n) >>= ilog2(__base); \
461a5e51060c93 Nicolas Pitre 2015-10-30 233 } else if (__div64_const32_is_OK && \
461a5e51060c93 Nicolas Pitre 2015-10-30 234 __builtin_constant_p(__base) && \
461a5e51060c93 Nicolas Pitre 2015-10-30 235 __base != 0) { \
461a5e51060c93 Nicolas Pitre 2015-10-30 236 uint32_t __res_lo, __n_lo = (n); \
461a5e51060c93 Nicolas Pitre 2015-10-30 237 (n) = __div64_const32(n, __base); \
461a5e51060c93 Nicolas Pitre 2015-10-30 238 /* the remainder can be computed with 32-bit regs */ \
461a5e51060c93 Nicolas Pitre 2015-10-30 239 __res_lo = (n); \
461a5e51060c93 Nicolas Pitre 2015-10-30 240 __rem = __n_lo - __res_lo * __base; \
911918aa7ef6f8 Nicolas Pitre 2015-11-02 241 } else if (likely(((n) >> 32) == 0)) { \
^1da177e4c3f41 Linus Torvalds 2005-04-16 242 __rem = (uint32_t)(n) % __base; \
^1da177e4c3f41 Linus Torvalds 2005-04-16 243 (n) = (uint32_t)(n) / __base; \
^1da177e4c3f41 Linus Torvalds 2005-04-16 244 } else \
^1da177e4c3f41 Linus Torvalds 2005-04-16 @245 __rem = __div64_32(&(n), __base); \
^1da177e4c3f41 Linus Torvalds 2005-04-16 246 __rem; \
^1da177e4c3f41 Linus Torvalds 2005-04-16 247 })
^1da177e4c3f41 Linus Torvalds 2005-04-16 248
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
drivers/edac/altera_edac.c:541:42: warning: unused variable 'ocramecc_data'
by kernel test robot
Hi Krzysztof,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: c2131f7e73c9e9365613e323d65c7b9e5b910f56
commit: 4a9a1a5602d82c079325bf37466af0b67d6c0b9e arm64: socfpga: merge Agilex and N5X into ARCH_INTEL_SOCFPGA
date: 2 months ago
config: arm64-randconfig-r032-20210601 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project db26cd30b6dd65e88d786e97a1e453af5cd48966)
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 arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 4a9a1a5602d82c079325bf37466af0b67d6c0b9e
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang 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 warnings (new ones prefixed by >>):
>> drivers/edac/altera_edac.c:541:42: warning: unused variable 'ocramecc_data' [-Wunused-const-variable]
static const struct edac_device_prv_data ocramecc_data;
^
>> drivers/edac/altera_edac.c:542:42: warning: unused variable 'l2ecc_data' [-Wunused-const-variable]
static const struct edac_device_prv_data l2ecc_data;
^
>> drivers/edac/altera_edac.c:543:42: warning: unused variable 'a10_ocramecc_data' [-Wunused-const-variable]
static const struct edac_device_prv_data a10_ocramecc_data;
^
>> drivers/edac/altera_edac.c:544:42: warning: unused variable 'a10_l2ecc_data' [-Wunused-const-variable]
static const struct edac_device_prv_data a10_l2ecc_data;
^
>> drivers/edac/altera_edac.c:642:37: warning: unused variable 'altr_edac_device_inject_fops' [-Wunused-const-variable]
static const struct file_operations altr_edac_device_inject_fops = {
^
5 warnings generated.
vim +/ocramecc_data +541 drivers/edac/altera_edac.c
c3eea1942a16db Thor Thayer 2016-02-10 532
c3eea1942a16db Thor Thayer 2016-02-10 533 /*
c3eea1942a16db Thor Thayer 2016-02-10 534 * EDAC Device Functions (shared between various IPs).
c3eea1942a16db Thor Thayer 2016-02-10 535 * The discrete memories use the EDAC Device framework. The probe
c3eea1942a16db Thor Thayer 2016-02-10 536 * and error handling functions are very similar between memories
c3eea1942a16db Thor Thayer 2016-02-10 537 * so they are shared. The memory allocation and freeing for EDAC
c3eea1942a16db Thor Thayer 2016-02-10 538 * trigger testing are different for each memory.
c3eea1942a16db Thor Thayer 2016-02-10 539 */
c3eea1942a16db Thor Thayer 2016-02-10 540
1cf70377247302 Thor Thayer 2016-06-22 @541 static const struct edac_device_prv_data ocramecc_data;
1cf70377247302 Thor Thayer 2016-06-22 @542 static const struct edac_device_prv_data l2ecc_data;
1cf70377247302 Thor Thayer 2016-06-22 @543 static const struct edac_device_prv_data a10_ocramecc_data;
1cf70377247302 Thor Thayer 2016-06-22 @544 static const struct edac_device_prv_data a10_l2ecc_data;
c3eea1942a16db Thor Thayer 2016-02-10 545
c3eea1942a16db Thor Thayer 2016-02-10 546 static irqreturn_t altr_edac_device_handler(int irq, void *dev_id)
c3eea1942a16db Thor Thayer 2016-02-10 547 {
c3eea1942a16db Thor Thayer 2016-02-10 548 irqreturn_t ret_value = IRQ_NONE;
c3eea1942a16db Thor Thayer 2016-02-10 549 struct edac_device_ctl_info *dci = dev_id;
c3eea1942a16db Thor Thayer 2016-02-10 550 struct altr_edac_device_dev *drvdata = dci->pvt_info;
c3eea1942a16db Thor Thayer 2016-02-10 551 const struct edac_device_prv_data *priv = drvdata->data;
c3eea1942a16db Thor Thayer 2016-02-10 552
c3eea1942a16db Thor Thayer 2016-02-10 553 if (irq == drvdata->sb_irq) {
c3eea1942a16db Thor Thayer 2016-02-10 554 if (priv->ce_clear_mask)
c3eea1942a16db Thor Thayer 2016-02-10 555 writel(priv->ce_clear_mask, drvdata->base);
c3eea1942a16db Thor Thayer 2016-02-10 556 edac_device_handle_ce(dci, 0, 0, drvdata->edac_dev_name);
c3eea1942a16db Thor Thayer 2016-02-10 557 ret_value = IRQ_HANDLED;
c3eea1942a16db Thor Thayer 2016-02-10 558 } else if (irq == drvdata->db_irq) {
c3eea1942a16db Thor Thayer 2016-02-10 559 if (priv->ue_clear_mask)
c3eea1942a16db Thor Thayer 2016-02-10 560 writel(priv->ue_clear_mask, drvdata->base);
c3eea1942a16db Thor Thayer 2016-02-10 561 edac_device_handle_ue(dci, 0, 0, drvdata->edac_dev_name);
c3eea1942a16db Thor Thayer 2016-02-10 562 panic("\nEDAC:ECC_DEVICE[Uncorrectable errors]\n");
c3eea1942a16db Thor Thayer 2016-02-10 563 ret_value = IRQ_HANDLED;
c3eea1942a16db Thor Thayer 2016-02-10 564 } else {
c3eea1942a16db Thor Thayer 2016-02-10 565 WARN_ON(1);
c3eea1942a16db Thor Thayer 2016-02-10 566 }
c3eea1942a16db Thor Thayer 2016-02-10 567
c3eea1942a16db Thor Thayer 2016-02-10 568 return ret_value;
c3eea1942a16db Thor Thayer 2016-02-10 569 }
c3eea1942a16db Thor Thayer 2016-02-10 570
c3eea1942a16db Thor Thayer 2016-02-10 571 static ssize_t altr_edac_device_trig(struct file *file,
c3eea1942a16db Thor Thayer 2016-02-10 572 const char __user *user_buf,
c3eea1942a16db Thor Thayer 2016-02-10 573 size_t count, loff_t *ppos)
c3eea1942a16db Thor Thayer 2016-02-10 574
c3eea1942a16db Thor Thayer 2016-02-10 575 {
c3eea1942a16db Thor Thayer 2016-02-10 576 u32 *ptemp, i, error_mask;
c3eea1942a16db Thor Thayer 2016-02-10 577 int result = 0;
c3eea1942a16db Thor Thayer 2016-02-10 578 u8 trig_type;
c3eea1942a16db Thor Thayer 2016-02-10 579 unsigned long flags;
c3eea1942a16db Thor Thayer 2016-02-10 580 struct edac_device_ctl_info *edac_dci = file->private_data;
c3eea1942a16db Thor Thayer 2016-02-10 581 struct altr_edac_device_dev *drvdata = edac_dci->pvt_info;
c3eea1942a16db Thor Thayer 2016-02-10 582 const struct edac_device_prv_data *priv = drvdata->data;
c3eea1942a16db Thor Thayer 2016-02-10 583 void *generic_ptr = edac_dci->dev;
c3eea1942a16db Thor Thayer 2016-02-10 584
c3eea1942a16db Thor Thayer 2016-02-10 585 if (!user_buf || get_user(trig_type, user_buf))
c3eea1942a16db Thor Thayer 2016-02-10 586 return -EFAULT;
c3eea1942a16db Thor Thayer 2016-02-10 587
c3eea1942a16db Thor Thayer 2016-02-10 588 if (!priv->alloc_mem)
c3eea1942a16db Thor Thayer 2016-02-10 589 return -ENOMEM;
c3eea1942a16db Thor Thayer 2016-02-10 590
c3eea1942a16db Thor Thayer 2016-02-10 591 /*
c3eea1942a16db Thor Thayer 2016-02-10 592 * Note that generic_ptr is initialized to the device * but in
c3eea1942a16db Thor Thayer 2016-02-10 593 * some alloc_functions, this is overridden and returns data.
c3eea1942a16db Thor Thayer 2016-02-10 594 */
c3eea1942a16db Thor Thayer 2016-02-10 595 ptemp = priv->alloc_mem(priv->trig_alloc_sz, &generic_ptr);
c3eea1942a16db Thor Thayer 2016-02-10 596 if (!ptemp) {
c3eea1942a16db Thor Thayer 2016-02-10 597 edac_printk(KERN_ERR, EDAC_DEVICE,
c3eea1942a16db Thor Thayer 2016-02-10 598 "Inject: Buffer Allocation error\n");
c3eea1942a16db Thor Thayer 2016-02-10 599 return -ENOMEM;
c3eea1942a16db Thor Thayer 2016-02-10 600 }
c3eea1942a16db Thor Thayer 2016-02-10 601
c3eea1942a16db Thor Thayer 2016-02-10 602 if (trig_type == ALTR_UE_TRIGGER_CHAR)
c3eea1942a16db Thor Thayer 2016-02-10 603 error_mask = priv->ue_set_mask;
c3eea1942a16db Thor Thayer 2016-02-10 604 else
c3eea1942a16db Thor Thayer 2016-02-10 605 error_mask = priv->ce_set_mask;
c3eea1942a16db Thor Thayer 2016-02-10 606
c3eea1942a16db Thor Thayer 2016-02-10 607 edac_printk(KERN_ALERT, EDAC_DEVICE,
c3eea1942a16db Thor Thayer 2016-02-10 608 "Trigger Error Mask (0x%X)\n", error_mask);
c3eea1942a16db Thor Thayer 2016-02-10 609
c3eea1942a16db Thor Thayer 2016-02-10 610 local_irq_save(flags);
c3eea1942a16db Thor Thayer 2016-02-10 611 /* write ECC corrupted data out. */
c3eea1942a16db Thor Thayer 2016-02-10 612 for (i = 0; i < (priv->trig_alloc_sz / sizeof(*ptemp)); i++) {
c3eea1942a16db Thor Thayer 2016-02-10 613 /* Read data so we're in the correct state */
c3eea1942a16db Thor Thayer 2016-02-10 614 rmb();
332efa6374de75 Mark Rutland 2017-10-23 615 if (READ_ONCE(ptemp[i]))
c3eea1942a16db Thor Thayer 2016-02-10 616 result = -1;
c3eea1942a16db Thor Thayer 2016-02-10 617 /* Toggle Error bit (it is latched), leave ECC enabled */
811fce4f2a7aea Thor Thayer 2016-03-21 618 writel(error_mask, (drvdata->base + priv->set_err_ofst));
811fce4f2a7aea Thor Thayer 2016-03-21 619 writel(priv->ecc_enable_mask, (drvdata->base +
811fce4f2a7aea Thor Thayer 2016-03-21 620 priv->set_err_ofst));
c3eea1942a16db Thor Thayer 2016-02-10 621 ptemp[i] = i;
c3eea1942a16db Thor Thayer 2016-02-10 622 }
c3eea1942a16db Thor Thayer 2016-02-10 623 /* Ensure it has been written out */
c3eea1942a16db Thor Thayer 2016-02-10 624 wmb();
c3eea1942a16db Thor Thayer 2016-02-10 625 local_irq_restore(flags);
c3eea1942a16db Thor Thayer 2016-02-10 626
c3eea1942a16db Thor Thayer 2016-02-10 627 if (result)
c3eea1942a16db Thor Thayer 2016-02-10 628 edac_printk(KERN_ERR, EDAC_DEVICE, "Mem Not Cleared\n");
c3eea1942a16db Thor Thayer 2016-02-10 629
c3eea1942a16db Thor Thayer 2016-02-10 630 /* Read out written data. ECC error caused here */
c3eea1942a16db Thor Thayer 2016-02-10 631 for (i = 0; i < ALTR_TRIGGER_READ_WRD_CNT; i++)
332efa6374de75 Mark Rutland 2017-10-23 632 if (READ_ONCE(ptemp[i]) != i)
c3eea1942a16db Thor Thayer 2016-02-10 633 edac_printk(KERN_ERR, EDAC_DEVICE,
c3eea1942a16db Thor Thayer 2016-02-10 634 "Read doesn't match written data\n");
c3eea1942a16db Thor Thayer 2016-02-10 635
c3eea1942a16db Thor Thayer 2016-02-10 636 if (priv->free_mem)
c3eea1942a16db Thor Thayer 2016-02-10 637 priv->free_mem(ptemp, priv->trig_alloc_sz, generic_ptr);
c3eea1942a16db Thor Thayer 2016-02-10 638
c3eea1942a16db Thor Thayer 2016-02-10 639 return count;
c3eea1942a16db Thor Thayer 2016-02-10 640 }
c3eea1942a16db Thor Thayer 2016-02-10 641
c3eea1942a16db Thor Thayer 2016-02-10 @642 static const struct file_operations altr_edac_device_inject_fops = {
c3eea1942a16db Thor Thayer 2016-02-10 643 .open = simple_open,
c3eea1942a16db Thor Thayer 2016-02-10 644 .write = altr_edac_device_trig,
c3eea1942a16db Thor Thayer 2016-02-10 645 .llseek = generic_file_llseek,
c3eea1942a16db Thor Thayer 2016-02-10 646 };
c3eea1942a16db Thor Thayer 2016-02-10 647
:::::: The code at line 541 was first introduced by commit
:::::: 1cf703772473022ac89d815871b20aa4738449b4 EDAC, altera: Make all private data structures static
:::::: TO: Thor Thayer <tthayer(a)opensource.altera.com>
:::::: CC: Borislav Petkov <bp(a)suse.de>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
drivers/scsi/cxgbi/cxgb4i/cxgb4i.c:993:44: warning: variable 'size6' set but not used
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: c2131f7e73c9e9365613e323d65c7b9e5b910f56
commit: b142083b585c2c03af24cca4d274f797796a4064 mptcp: MPTCP_KUNIT_TESTS should depend on MPTCP instead of selecting it
date: 7 months ago
config: ia64-randconfig-r032-20210601 (attached as .config)
compiler: ia64-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://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout b142083b585c2c03af24cca4d274f797796a4064
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=ia64
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/scsi/cxgbi/cxgb4i/cxgb4i.c: In function 'csk_act_open_retry_timer':
>> drivers/scsi/cxgbi/cxgb4i/cxgb4i.c:993:44: warning: variable 'size6' set but not used [-Wunused-but-set-variable]
993 | int t4 = is_t4(lldi->adapter_type), size, size6;
| ^~~~~
drivers/scsi/cxgbi/cxgb4i/cxgb4i.c: In function 'init_act_open':
drivers/scsi/cxgbi/cxgb4i/cxgb4i.c:1774:21: warning: variable 'size6' set but not used [-Wunused-but-set-variable]
1774 | unsigned int size, size6;
| ^~~~~
vim +/size6 +993 drivers/scsi/cxgbi/cxgb4i/cxgb4i.c
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 985
cd07f958e88250 Kees Cook 2017-08-28 986 static void csk_act_open_retry_timer(struct timer_list *t)
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 987 {
001586a737ee8c Anish Bhatt 2014-10-15 988 struct sk_buff *skb = NULL;
cd07f958e88250 Kees Cook 2017-08-28 989 struct cxgbi_sock *csk = from_timer(csk, t, retry_timer);
3bd3e8bf6250f3 Karen Xie 2013-05-29 990 struct cxgb4_lld_info *lldi = cxgbi_cdev_priv(csk->cdev);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 991 void (*send_act_open_func)(struct cxgbi_sock *, struct sk_buff *,
759a0cc5a3e1bc Anish Bhatt 2014-07-17 992 struct l2t_entry *);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 @993 int t4 = is_t4(lldi->adapter_type), size, size6;
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 994
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 995 log_debug(1 << CXGBI_DBG_TOE | 1 << CXGBI_DBG_SOCK,
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 996 "csk 0x%p,%u,0x%lx,%u.\n",
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 997 csk, csk->state, csk->flags, csk->tid);
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 998
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 999 cxgbi_sock_get(csk);
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1000 spin_lock_bh(&csk->lock);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1001
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1002 if (t4) {
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1003 size = sizeof(struct cpl_act_open_req);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1004 size6 = sizeof(struct cpl_act_open_req6);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1005 } else {
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1006 size = sizeof(struct cpl_t5_act_open_req);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1007 size6 = sizeof(struct cpl_t5_act_open_req6);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1008 }
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1009
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1010 if (csk->csk_family == AF_INET) {
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1011 send_act_open_func = send_act_open_req;
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1012 skb = alloc_wr(size, 0, GFP_ATOMIC);
f42bb57c61fd21 Anish Bhatt 2014-10-14 1013 #if IS_ENABLED(CONFIG_IPV6)
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1014 } else {
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1015 send_act_open_func = send_act_open_req6;
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1016 skb = alloc_wr(size6, 0, GFP_ATOMIC);
f42bb57c61fd21 Anish Bhatt 2014-10-14 1017 #endif
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1018 }
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1019
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1020 if (!skb)
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1021 cxgbi_sock_fail_act_open(csk, -ENOMEM);
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1022 else {
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1023 skb->sk = (struct sock *)csk;
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1024 t4_set_arp_err_handler(skb, csk,
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1025 cxgbi_sock_act_open_req_arp_failure);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1026 send_act_open_func(csk, skb, csk->l2t);
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1027 }
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1028
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1029 spin_unlock_bh(&csk->lock);
7b36b6e03b0d6c kxie(a)chelsio.com 2010-08-16 1030 cxgbi_sock_put(csk);
759a0cc5a3e1bc Anish Bhatt 2014-07-17 1031
:::::: The code at line 993 was first introduced by commit
:::::: 759a0cc5a3e1bc2cc48fa3c0b91bdcad8b8f87d6 cxgb4i: Add ipv6 code to driver, call into libcxgbi ipv6 api
:::::: TO: Anish Bhatt <anish(a)chelsio.com>
:::::: CC: David S. Miller <davem(a)davemloft.net>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
[chrome-os:chromeos-4.14 42/46] drivers/net/wireless/ath/ath10k/htt_rx.c:1557:7: error: variable 'more_frags' set but not used
by kernel test robot
tree: https://chromium.googlesource.com/chromiumos/third_party/kernel chromeos-4.14
head: 8e7a868b42c9e1fe9dd8de4347434d0a3289b079
commit: c38550350b9cca9ff303fe691048802716c3be14 [42/46] BACKPORT: FROMGIT: ath10k: add CCMP PN replay protection for fragmented frames for PCIe
config: arm64-chromiumos-qualcomm-customedconfig-chrome-os:chromeos-4.14:cbc1c14057637af41016f8a57efa274fe949bdeb (attached as .config)
compiler: aarch64-linux-gcc (GCC) 7.5.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 remote add chrome-os https://chromium.googlesource.com/chromiumos/third_party/kernel
git fetch --no-tags chrome-os chromeos-4.14
git checkout c38550350b9cca9ff303fe691048802716c3be14
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-7.5.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 >>):
drivers/net/wireless/ath/ath10k/htt_rx.c: In function 'ath10k_htt_rx_h_frag_pn_check':
>> drivers/net/wireless/ath/ath10k/htt_rx.c:1557:7: error: variable 'more_frags' set but not used [-Werror=unused-but-set-variable]
bool more_frags;
^~~~~~~~~~
In file included from drivers/net/wireless/ath/ath10k/core.h:34:0,
from drivers/net/wireless/ath/ath10k/htt_rx.c:19:
At top level:
drivers/net/wireless/ath/ath10k/../ath.h:36:17: error: 'ath_bcast_mac' defined but not used [-Werror=unused-const-variable=]
static const u8 ath_bcast_mac[ETH_ALEN] = {0xff, 0xff, 0xff, 0xff, 0xff, 0xff};
^~~~~~~~~~~~~
cc1: all warnings being treated as errors
vim +/more_frags +1557 drivers/net/wireless/ath/ath10k/htt_rx.c
1547
1548 static bool ath10k_htt_rx_h_frag_pn_check(struct ath10k *ar,
1549 struct sk_buff *skb,
1550 u16 peer_id,
1551 u16 offset,
1552 enum htt_rx_mpdu_encrypt_type enctype)
1553 {
1554 struct ath10k_peer *peer;
1555 union htt_rx_pn_t *last_pn, new_pn = {0};
1556 struct ieee80211_hdr *hdr;
> 1557 bool more_frags;
1558 u8 tid, frag_number;
1559 u32 seq;
1560
1561 peer = ath10k_peer_find_by_id(ar, peer_id);
1562 if (!peer) {
1563 ath10k_dbg(ar, ATH10K_DBG_HTT, "invalid peer for frag pn check\n");
1564 return false;
1565 }
1566
1567 hdr = (struct ieee80211_hdr *)(skb->data + offset);
1568 if (ieee80211_is_data_qos(hdr->frame_control))
1569 tid = ieee80211_get_qos_ctl(hdr)[0] & IEEE80211_QOS_CTL_TID_MASK;
1570 else
1571 tid = ATH10K_TXRX_NON_QOS_TID;
1572
1573 last_pn = &peer->frag_tids_last_pn[tid];
1574 new_pn.pn48 = ath10k_htt_rx_h_get_pn(ar, skb, offset, enctype);
1575 more_frags = ieee80211_has_morefrags(hdr->frame_control);
1576 frag_number = le16_to_cpu(hdr->seq_ctrl) & IEEE80211_SCTL_FRAG;
1577 seq = (__le16_to_cpu(hdr->seq_ctrl) & IEEE80211_SCTL_SEQ) >> 4;
1578
1579 if (frag_number == 0) {
1580 last_pn->pn48 = new_pn.pn48;
1581 peer->frag_tids_seq[tid] = seq;
1582 } else {
1583 if (seq != peer->frag_tids_seq[tid])
1584 return false;
1585
1586 if (new_pn.pn48 != last_pn->pn48 + 1)
1587 return false;
1588
1589 last_pn->pn48 = new_pn.pn48;
1590 }
1591
1592 return true;
1593 }
1594
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
drivers/iio/test/iio-test-format.c:98:1: warning: the frame size of 1520 bytes is larger than 1024 bytes
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: c2131f7e73c9e9365613e323d65c7b9e5b910f56
commit: 0bf49ffbfe949df21e95b8f95b5f308db379ac74 iio: Add basic unit test for iio_format_value()
date: 3 months ago
config: arm-randconfig-r012-20210531 (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/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 0bf49ffbfe949df21e95b8f95b5f308db379ac74
# 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 >>):
drivers/iio/test/iio-test-format.c: In function 'iio_test_iio_format_value_fixedpoint':
>> drivers/iio/test/iio-test-format.c:98:1: warning: the frame size of 1520 bytes is larger than 1024 bytes [-Wframe-larger-than=]
98 | }
| ^
vim +98 drivers/iio/test/iio-test-format.c
41
42 static void iio_test_iio_format_value_fixedpoint(struct kunit *test)
43 {
44 char *buf = kunit_kmalloc(test, PAGE_SIZE, GFP_KERNEL);
45 int values[2];
46 int ret;
47
48 /* positive >= 1 */
49 values[0] = 1;
50 values[1] = 10;
51
52 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_MICRO, 2, values);
53 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "1.000010\n");
54
55 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_MICRO_DB, 2, values);
56 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "1.000010 dB\n");
57
58 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_NANO, 2, values);
59 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "1.000000010\n");
60
61 /* positive < 1 */
62 values[0] = 0;
63 values[1] = 12;
64
65 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_MICRO, 2, values);
66 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "0.000012\n");
67
68 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_MICRO_DB, 2, values);
69 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "0.000012 dB\n");
70
71 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_NANO, 2, values);
72 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "0.000000012\n");
73
74 /* negative <= -1 */
75 values[0] = -1;
76 values[1] = 10;
77
78 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_MICRO, 2, values);
79 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "-1.000010\n");
80
81 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_MICRO_DB, 2, values);
82 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "-1.000010 dB\n");
83
84 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_NANO, 2, values);
85 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "-1.000000010\n");
86
87 /* negative > -1 */
88 values[0] = 0;
89 values[1] = -123;
90 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_MICRO, 2, values);
91 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "-0.000123\n");
92
93 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_MICRO_DB, 2, values);
94 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "-0.000123 dB\n");
95
96 ret = iio_format_value(buf, IIO_VAL_INT_PLUS_NANO, 2, values);
97 IIO_TEST_FORMAT_EXPECT_EQ(test, buf, ret, "-0.000000123\n");
> 98 }
99
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months