[sashal-linux-stable:queue-4.4 13/14] fs/squashfs/xattr_id.c:51:12: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git queue-4.4
head: 6a08a9e7fb1525fa296d810d200954f6c18d07a2
commit: d9c7067ceb34ec223012997c1d57f3919528b7ae [13/14] squashfs: add more sanity checks in xattr id lookup
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://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git/c...
git remote add sashal-linux-stable https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git
git fetch --no-tags sashal-linux-stable queue-4.4
git checkout d9c7067ceb34ec223012997c1d57f3919528b7ae
# 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 >>):
In file included from arch/arc/include/asm/bug.h:32,
from include/linux/bug.h:4,
from include/linux/thread_info.h:11,
from include/asm-generic/preempt.h:4,
from arch/arc/include/generated/asm/preempt.h:1,
from include/linux/preempt.h:59,
from include/linux/spinlock.h:50,
from include/linux/wait.h:8,
from include/linux/fs.h:5,
from fs/squashfs/xattr_id.c:29:
include/linux/cpumask.h: In function 'cpumask_check':
include/linux/cpumask.h:117:19: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare]
117 | WARN_ON_ONCE(cpu >= nr_cpumask_bits);
| ^~
include/asm-generic/bug.h:112:27: note: in definition of macro 'WARN_ON_ONCE'
112 | int __ret_warn_once = !!(condition); \
| ^~~~~~~~~
In file included from fs/squashfs/xattr_id.c:33:
fs/squashfs/squashfs_fs.h: In function 'squashfs_block_size':
fs/squashfs/squashfs_fs.h:135:24: warning: operand of ?: changes signedness from 'int' to 'u32' {aka 'unsigned int'} due to unsignedness of other operand [-Wsign-compare]
135 | return (size >> 25) ? -EIO : size;
fs/squashfs/xattr_id.c: In function 'squashfs_xattr_lookup':
>> fs/squashfs/xattr_id.c:51:12: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare]
51 | if (index >= msblk->xattr_ids)
| ^~
fs/squashfs/xattr_id.c: In function 'squashfs_read_xattr_id_table':
>> fs/squashfs/xattr_id.c:121:16: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare]
121 | for (n = 0; n < (indexes - 1); n++) {
| ^
vim +51 fs/squashfs/xattr_id.c
37
38 /*
39 * Map xattr id using the xattr id look up table
40 */
41 int squashfs_xattr_lookup(struct super_block *sb, unsigned int index,
42 int *count, unsigned int *size, unsigned long long *xattr)
43 {
44 struct squashfs_sb_info *msblk = sb->s_fs_info;
45 int block = SQUASHFS_XATTR_BLOCK(index);
46 int offset = SQUASHFS_XATTR_BLOCK_OFFSET(index);
47 u64 start_block;
48 struct squashfs_xattr_id id;
49 int err;
50
> 51 if (index >= msblk->xattr_ids)
52 return -EINVAL;
53
54 start_block = le64_to_cpu(msblk->xattr_id_table[block]);
55
56 err = squashfs_read_metadata(sb, &id, &start_block, &offset,
57 sizeof(id));
58 if (err < 0)
59 return err;
60
61 *xattr = le64_to_cpu(id.xattr);
62 *size = le32_to_cpu(id.size);
63 *count = le32_to_cpu(id.count);
64 return 0;
65 }
66
67
68 /*
69 * Read uncompressed xattr id lookup table indexes from disk into memory
70 */
71 __le64 *squashfs_read_xattr_id_table(struct super_block *sb, u64 table_start,
72 u64 *xattr_table_start, int *xattr_ids)
73 {
74 struct squashfs_sb_info *msblk = sb->s_fs_info;
75 unsigned int len, indexes;
76 struct squashfs_xattr_id_table *id_table;
77 __le64 *table;
78 u64 start, end;
79 int n;
80
81 id_table = squashfs_read_table(sb, table_start, sizeof(*id_table));
82 if (IS_ERR(id_table))
83 return (__le64 *) id_table;
84
85 *xattr_table_start = le64_to_cpu(id_table->xattr_table_start);
86 *xattr_ids = le32_to_cpu(id_table->xattr_ids);
87 kfree(id_table);
88
89 /* Sanity check values */
90
91 /* there is always at least one xattr id */
92 if (*xattr_ids == 0)
93 return ERR_PTR(-EINVAL);
94
95 len = SQUASHFS_XATTR_BLOCK_BYTES(*xattr_ids);
96 indexes = SQUASHFS_XATTR_BLOCKS(*xattr_ids);
97
98 /*
99 * The computed size of the index table (len bytes) should exactly
100 * match the table start and end points
101 */
102 start = table_start + sizeof(*id_table);
103 end = msblk->bytes_used;
104
105 if (len != (end - start))
106 return ERR_PTR(-EINVAL);
107
108 table = squashfs_read_table(sb, start, len);
109 if (IS_ERR(table))
110 return table;
111
112 /* table[0], table[1], ... table[indexes - 1] store the locations
113 * of the compressed xattr id blocks. Each entry should be less than
114 * the next (i.e. table[0] < table[1]), and the difference between them
115 * should be SQUASHFS_METADATA_SIZE or less. table[indexes - 1]
116 * should be less than table_start, and again the difference
117 * shouls be SQUASHFS_METADATA_SIZE or less.
118 *
119 * Finally xattr_table_start should be less than table[0].
120 */
> 121 for (n = 0; n < (indexes - 1); n++) {
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
[sashal-linux-stable:queue-4.4 12/14] fs/squashfs/export.c:63:36: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git queue-4.4
head: 6a08a9e7fb1525fa296d810d200954f6c18d07a2
commit: a128d6484ea6c26f6266bd8d93c62346436bfd4e [12/14] squashfs: add more sanity checks in inode lookup
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://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git/c...
git remote add sashal-linux-stable https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git
git fetch --no-tags sashal-linux-stable queue-4.4
git checkout a128d6484ea6c26f6266bd8d93c62346436bfd4e
# 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 >>):
In file included from arch/arc/include/asm/bug.h:32,
from include/linux/bug.h:4,
from include/linux/thread_info.h:11,
from include/asm-generic/preempt.h:4,
from arch/arc/include/generated/asm/preempt.h:1,
from include/linux/preempt.h:59,
from include/linux/spinlock.h:50,
from include/linux/wait.h:8,
from include/linux/fs.h:5,
from fs/squashfs/export.c:38:
include/linux/cpumask.h: In function 'cpumask_check':
include/linux/cpumask.h:117:19: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare]
117 | WARN_ON_ONCE(cpu >= nr_cpumask_bits);
| ^~
include/asm-generic/bug.h:112:27: note: in definition of macro 'WARN_ON_ONCE'
112 | int __ret_warn_once = !!(condition); \
| ^~~~~~~~~
In file included from fs/squashfs/export.c:44:
fs/squashfs/squashfs_fs.h: In function 'squashfs_block_size':
fs/squashfs/squashfs_fs.h:135:24: warning: operand of ?: changes signedness from 'int' to 'u32' {aka 'unsigned int'} due to unsignedness of other operand [-Wsign-compare]
135 | return (size >> 25) ? -EIO : size;
fs/squashfs/export.c: In function 'squashfs_inode_lookup':
>> fs/squashfs/export.c:63:36: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare]
63 | if (ino_num == 0 || (ino_num - 1) >= msblk->inodes)
| ^~
fs/squashfs/export.c: In function 'squashfs_read_inode_lookup_table':
fs/squashfs/export.c:164:16: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare]
164 | for (n = 0; n < (indexes - 1); n++) {
| ^
vim +63 fs/squashfs/export.c
48
49 /*
50 * Look-up inode number (ino) in table, returning the inode location.
51 */
52 static long long squashfs_inode_lookup(struct super_block *sb, int ino_num)
53 {
54 struct squashfs_sb_info *msblk = sb->s_fs_info;
55 int blk = SQUASHFS_LOOKUP_BLOCK(ino_num - 1);
56 int offset = SQUASHFS_LOOKUP_BLOCK_OFFSET(ino_num - 1);
57 u64 start;
58 __le64 ino;
59 int err;
60
61 TRACE("Entered squashfs_inode_lookup, inode_number = %d\n", ino_num);
62
> 63 if (ino_num == 0 || (ino_num - 1) >= msblk->inodes)
64 return -EINVAL;
65
66 start = le64_to_cpu(msblk->inode_lookup_table[blk]);
67
68 err = squashfs_read_metadata(sb, &ino, &start, &offset, sizeof(ino));
69 if (err < 0)
70 return err;
71
72 TRACE("squashfs_inode_lookup, inode = 0x%llx\n",
73 (u64) le64_to_cpu(ino));
74
75 return le64_to_cpu(ino);
76 }
77
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
[sashal-linux-stable:queue-4.4 13/14] fs/squashfs/xattr_id.c:51:12: warning: comparison between signed and unsigned integer expressions
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git queue-4.4
head: 6a08a9e7fb1525fa296d810d200954f6c18d07a2
commit: d9c7067ceb34ec223012997c1d57f3919528b7ae [13/14] squashfs: add more sanity checks in xattr id lookup
config: mips-allyesconfig (attached as .config)
compiler: mips-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
# https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git/c...
git remote add sashal-linux-stable https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git
git fetch --no-tags sashal-linux-stable queue-4.4
git checkout d9c7067ceb34ec223012997c1d57f3919528b7ae
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-7.5.0 make.cross ARCH=mips
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 >>):
In file included from fs/squashfs/xattr_id.c:33:0:
fs/squashfs/squashfs_fs.h: In function 'squashfs_block_size':
fs/squashfs/squashfs_fs.h:135:29: warning: signed and unsigned type in conditional expression [-Wsign-compare]
return (size >> 25) ? -EIO : size;
^
fs/squashfs/xattr_id.c: In function 'squashfs_xattr_lookup':
>> fs/squashfs/xattr_id.c:51:12: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (index >= msblk->xattr_ids)
^~
fs/squashfs/xattr_id.c: In function 'squashfs_read_xattr_id_table':
fs/squashfs/xattr_id.c:121:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (n = 0; n < (indexes - 1); n++) {
^
vim +51 fs/squashfs/xattr_id.c
37
38 /*
39 * Map xattr id using the xattr id look up table
40 */
41 int squashfs_xattr_lookup(struct super_block *sb, unsigned int index,
42 int *count, unsigned int *size, unsigned long long *xattr)
43 {
44 struct squashfs_sb_info *msblk = sb->s_fs_info;
45 int block = SQUASHFS_XATTR_BLOCK(index);
46 int offset = SQUASHFS_XATTR_BLOCK_OFFSET(index);
47 u64 start_block;
48 struct squashfs_xattr_id id;
49 int err;
50
> 51 if (index >= msblk->xattr_ids)
52 return -EINVAL;
53
54 start_block = le64_to_cpu(msblk->xattr_id_table[block]);
55
56 err = squashfs_read_metadata(sb, &id, &start_block, &offset,
57 sizeof(id));
58 if (err < 0)
59 return err;
60
61 *xattr = le64_to_cpu(id.xattr);
62 *size = le32_to_cpu(id.size);
63 *count = le32_to_cpu(id.count);
64 return 0;
65 }
66
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
[sashal-linux-stable:queue-4.4 11/14] fs/squashfs/id.c:109:16: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git queue-4.4
head: 6a08a9e7fb1525fa296d810d200954f6c18d07a2
commit: 4e3bc622004d182e7038229c8ffc989df2e3e413 [11/14] squashfs: add more sanity checks in id lookup
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://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git/c...
git remote add sashal-linux-stable https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git
git fetch --no-tags sashal-linux-stable queue-4.4
git checkout 4e3bc622004d182e7038229c8ffc989df2e3e413
# 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 >>):
In file included from arch/arc/include/asm/bug.h:32,
from include/linux/bug.h:4,
from include/linux/thread_info.h:11,
from include/asm-generic/preempt.h:4,
from arch/arc/include/generated/asm/preempt.h:1,
from include/linux/preempt.h:59,
from include/linux/spinlock.h:50,
from include/linux/wait.h:8,
from include/linux/fs.h:5,
from fs/squashfs/id.c:34:
include/linux/cpumask.h: In function 'cpumask_check':
include/linux/cpumask.h:117:19: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare]
117 | WARN_ON_ONCE(cpu >= nr_cpumask_bits);
| ^~
include/asm-generic/bug.h:112:27: note: in definition of macro 'WARN_ON_ONCE'
112 | int __ret_warn_once = !!(condition); \
| ^~~~~~~~~
In file included from fs/squashfs/id.c:38:
fs/squashfs/squashfs_fs.h: In function 'squashfs_block_size':
fs/squashfs/squashfs_fs.h:135:24: warning: operand of ?: changes signedness from 'int' to 'u32' {aka 'unsigned int'} due to unsignedness of other operand [-Wsign-compare]
135 | return (size >> 25) ? -EIO : size;
fs/squashfs/id.c: In function 'squashfs_read_id_index_table':
>> fs/squashfs/id.c:109:16: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare]
109 | for (n = 0; n < (indexes - 1); n++) {
| ^
vim +109 fs/squashfs/id.c
68
69
70 /*
71 * Read uncompressed id lookup table indexes from disk into memory
72 */
73 __le64 *squashfs_read_id_index_table(struct super_block *sb,
74 u64 id_table_start, u64 next_table, unsigned short no_ids)
75 {
76 unsigned int length = SQUASHFS_ID_BLOCK_BYTES(no_ids);
77 unsigned int indexes = SQUASHFS_ID_BLOCKS(no_ids);
78 int n;
79 __le64 *table;
80 u64 start, end;
81
82 TRACE("In read_id_index_table, length %d\n", length);
83
84 /* Sanity check values */
85
86 /* there should always be at least one id */
87 if (no_ids == 0)
88 return ERR_PTR(-EINVAL);
89
90 /*
91 * The computed size of the index table (length bytes) should exactly
92 * match the table start and end points
93 */
94 if (length != (next_table - id_table_start))
95 return ERR_PTR(-EINVAL);
96
97 table = squashfs_read_table(sb, id_table_start, length);
98 if (IS_ERR(table))
99 return table;
100
101 /*
102 * table[0], table[1], ... table[indexes - 1] store the locations
103 * of the compressed id blocks. Each entry should be less than
104 * the next (i.e. table[0] < table[1]), and the difference between them
105 * should be SQUASHFS_METADATA_SIZE or less. table[indexes - 1]
106 * should be less than id_table_start, and again the difference
107 * should be SQUASHFS_METADATA_SIZE or less
108 */
> 109 for (n = 0; n < (indexes - 1); n++) {
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
Re: [PATCH] eeprom/optoe: driver to read/write SFP/QSFP/CMIS EEPROMS
by kernel test robot
Hi Don,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on a2ea4e1d9091cd8bc69f1c42c15bedc38618f04c]
url: https://github.com/0day-ci/linux/commits/Don-Bollinger/eeprom-optoe-drive...
base: a2ea4e1d9091cd8bc69f1c42c15bedc38618f04c
config: sparc64-randconfig-p002-20210215 (attached as .config)
compiler: sparc64-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/9089aa757bfb70c473ca54face7625829...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Don-Bollinger/eeprom-optoe-driver-to-read-write-SFP-QSFP-CMIS-EEPROMS/20210215-083817
git checkout 9089aa757bfb70c473ca54face762582908bdd28
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=sparc64
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/misc/eeprom/optoe.c:615:16: warning: no previous prototype for 'optoe_make_regmap' [-Wmissing-prototypes]
615 | struct regmap *optoe_make_regmap(struct i2c_client *client)
| ^~~~~~~~~~~~~~~~~
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for COMPAT_BINFMT_ELF
Depends on COMPAT && BINFMT_ELF
Selected by
- COMPAT && SPARC64
vim +/optoe_make_regmap +615 drivers/misc/eeprom/optoe.c
605
606
607 /*
608 * optoe_make_regmap creates the regmap for the client.
609 * IMPORTANT: Don't call the regmap read/write calls directly
610 * for these devices. These devices are paged, and you have to
611 * set the page register before accessing the data in that page.
612 * Use the nvmem interfaces, those read/write calls use this
613 * driver to manage pages correctly.
614 */
> 615 struct regmap *optoe_make_regmap(struct i2c_client *client)
616 {
617 struct regmap_config regmap_config = { };
618 struct regmap *regmap;
619
620 /* setup a minimal regmap - 8 bits, 8 bit addresses */
621 regmap_config.val_bits = 8;
622 regmap_config.reg_bits = 8;
623
624 /* I'll handle the locking */
625 regmap_config.disable_locking = true;
626 regmap = devm_regmap_init_i2c(client, ®map_config);
627 return regmap;
628 }
629
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
[linux-next:master 5601/11103] fs/zonefs/./trace.h:41:33: warning: format specifies type 'unsigned long' but the argument has type 'ino_t' (aka 'unsigned int')
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: 07f7e57c63aaa2afb4ea31edef05e08699a63a00
commit: 6716b125b33988a1d55ed044e78c856d68b40b7c [5601/11103] zonefs: add tracepoints for file operations
config: s390-randconfig-r001-20210215 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project c9439ca36342fb6013187d0a69aef92736951476)
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 s390 cross compiling tool for clang build
# apt-get install binutils-s390x-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 6716b125b33988a1d55ed044e78c856d68b40b7c
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=s390
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 >>):
In file included from include/linux/iomap.h:11:
In file included from include/linux/blkdev.h:26:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:20:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x0000ff00UL) << 8) | \
^
In file included from fs/zonefs/super.c:10:
In file included from include/linux/iomap.h:11:
In file included from include/linux/blkdev.h:26:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:21:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0x00ff0000UL) >> 8) | \
^
In file included from fs/zonefs/super.c:10:
In file included from include/linux/iomap.h:11:
In file included from include/linux/blkdev.h:26:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:119:21: note: expanded from macro '__swab32'
___constant_swab32(x) : \
^
include/uapi/linux/swab.h:22:12: note: expanded from macro '___constant_swab32'
(((__u32)(x) & (__u32)0xff000000UL) >> 24)))
^
In file included from fs/zonefs/super.c:10:
In file included from include/linux/iomap.h:11:
In file included from include/linux/blkdev.h:26:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
^
include/uapi/linux/swab.h:120:12: note: expanded from macro '__swab32'
__fswab32(x))
^
In file included from fs/zonefs/super.c:10:
In file included from include/linux/iomap.h:11:
In file included from include/linux/blkdev.h:26:
In file included from include/linux/scatterlist.h:9:
In file included from arch/s390/include/asm/io.h:80:
include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writeb(value, PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
__raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsb(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsw(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
readsl(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesb(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesw(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
writesl(PCI_IOBASE + addr, buffer, count);
~~~~~~~~~~ ^
In file included from fs/zonefs/super.c:28:
In file included from fs/zonefs/./trace.h:103:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:401:
>> fs/zonefs/./trace.h:41:33: warning: format specifies type 'unsigned long' but the argument has type 'ino_t' (aka 'unsigned int') [-Wformat]
show_dev(__entry->dev), __entry->ino,
~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~
include/trace/trace_events.h:263:17: note: expanded from macro '__entry'
#define __entry field
^
include/trace/trace_events.h:266:43: note: expanded from macro 'TP_printk'
#define TP_printk(fmt, args...) fmt "\n", args
~~~ ^
include/trace/trace_events.h:80:16: note: expanded from macro 'TRACE_EVENT'
PARAMS(print)); \
~~~~~~~^~~~~~~
include/linux/tracepoint.h:97:25: note: expanded from macro 'PARAMS'
#define PARAMS(args...) args
^~~~
include/trace/trace_events.h:367:22: note: expanded from macro 'DECLARE_EVENT_CLASS'
trace_seq_printf(s, print); \
^~~~~
In file included from fs/zonefs/super.c:28:
In file included from fs/zonefs/./trace.h:103:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:401:
fs/zonefs/./trace.h:67:33: warning: format specifies type 'unsigned long' but the argument has type 'ino_t' (aka 'unsigned int') [-Wformat]
show_dev(__entry->dev), __entry->ino, __entry->sector,
~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/trace/trace_events.h:263:17: note: expanded from macro '__entry'
#define __entry field
^
include/trace/trace_events.h:266:43: note: expanded from macro 'TP_printk'
#define TP_printk(fmt, args...) fmt "\n", args
~~~ ^
include/trace/trace_events.h:80:16: note: expanded from macro 'TRACE_EVENT'
PARAMS(print)); \
~~~~~~~^~~~~~~
include/linux/tracepoint.h:97:25: note: expanded from macro 'PARAMS'
#define PARAMS(args...) args
^~~~
include/trace/trace_events.h:367:22: note: expanded from macro 'DECLARE_EVENT_CLASS'
trace_seq_printf(s, print); \
^~~~~
In file included from fs/zonefs/super.c:28:
In file included from fs/zonefs/./trace.h:103:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:401:
fs/zonefs/./trace.h:90:33: warning: format specifies type 'unsigned long' but the argument has type 'ino_t' (aka 'unsigned int') [-Wformat]
show_dev(__entry->dev), __entry->ino, __entry->addr,
~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/trace/trace_events.h:263:17: note: expanded from macro '__entry'
#define __entry field
^
include/trace/trace_events.h:266:43: note: expanded from macro 'TP_printk'
#define TP_printk(fmt, args...) fmt "\n", args
~~~ ^
include/trace/trace_events.h:80:16: note: expanded from macro 'TRACE_EVENT'
PARAMS(print)); \
~~~~~~~^~~~~~~
include/linux/tracepoint.h:97:25: note: expanded from macro 'PARAMS'
#define PARAMS(args...) args
^~~~
include/trace/trace_events.h:367:22: note: expanded from macro 'DECLARE_EVENT_CLASS'
trace_seq_printf(s, print); \
^~~~~
23 warnings generated.
vim +41 fs/zonefs/./trace.h
21
22 TRACE_EVENT(zonefs_zone_mgmt,
23 TP_PROTO(struct inode *inode, enum req_opf op),
24 TP_ARGS(inode, op),
25 TP_STRUCT__entry(
26 __field(dev_t, dev)
27 __field(ino_t, ino)
28 __field(int, op)
29 __field(sector_t, sector)
30 __field(sector_t, nr_sectors)
31 ),
32 TP_fast_assign(
33 __entry->dev = inode->i_sb->s_dev;
34 __entry->ino = inode->i_ino;
35 __entry->op = op;
36 __entry->sector = ZONEFS_I(inode)->i_zsector;
37 __entry->nr_sectors =
38 ZONEFS_I(inode)->i_zone_size >> SECTOR_SHIFT;
39 ),
40 TP_printk("bdev=(%d,%d), ino=%lu op=%s, sector=%llu, nr_sectors=%llu",
> 41 show_dev(__entry->dev), __entry->ino,
42 blk_op_str(__entry->op), __entry->sector,
43 __entry->nr_sectors
44 )
45 );
46
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
Re: [PATCH] eeprom/optoe: driver to read/write SFP/QSFP/CMIS EEPROMS
by kernel test robot
Hi Don,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on a2ea4e1d9091cd8bc69f1c42c15bedc38618f04c]
url: https://github.com/0day-ci/linux/commits/Don-Bollinger/eeprom-optoe-drive...
base: a2ea4e1d9091cd8bc69f1c42c15bedc38618f04c
config: riscv-randconfig-r001-20210215 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project c9439ca36342fb6013187d0a69aef92736951476)
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://github.com/0day-ci/linux/commit/9089aa757bfb70c473ca54face7625829...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Don-Bollinger/eeprom-optoe-driver-to-read-write-SFP-QSFP-CMIS-EEPROMS/20210215-083817
git checkout 9089aa757bfb70c473ca54face762582908bdd28
# 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 >>):
>> drivers/misc/eeprom/optoe.c:615:16: warning: no previous prototype for function 'optoe_make_regmap' [-Wmissing-prototypes]
struct regmap *optoe_make_regmap(struct i2c_client *client)
^
drivers/misc/eeprom/optoe.c:615:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
struct regmap *optoe_make_regmap(struct i2c_client *client)
^
static
1 warning generated.
vim +/optoe_make_regmap +615 drivers/misc/eeprom/optoe.c
605
606
607 /*
608 * optoe_make_regmap creates the regmap for the client.
609 * IMPORTANT: Don't call the regmap read/write calls directly
610 * for these devices. These devices are paged, and you have to
611 * set the page register before accessing the data in that page.
612 * Use the nvmem interfaces, those read/write calls use this
613 * driver to manage pages correctly.
614 */
> 615 struct regmap *optoe_make_regmap(struct i2c_client *client)
616 {
617 struct regmap_config regmap_config = { };
618 struct regmap *regmap;
619
620 /* setup a minimal regmap - 8 bits, 8 bit addresses */
621 regmap_config.val_bits = 8;
622 regmap_config.reg_bits = 8;
623
624 /* I'll handle the locking */
625 regmap_config.disable_locking = true;
626 regmap = devm_regmap_init_i2c(client, ®map_config);
627 return regmap;
628 }
629
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
[sashal-linux-stable:queue-4.4 12/14] fs/squashfs/export.c:63:36: warning: comparison between signed and unsigned integer expressions
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git queue-4.4
head: 6a08a9e7fb1525fa296d810d200954f6c18d07a2
commit: a128d6484ea6c26f6266bd8d93c62346436bfd4e [12/14] squashfs: add more sanity checks in inode lookup
config: arm64-randconfig-p001-20210215 (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
# https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git/c...
git remote add sashal-linux-stable https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git
git fetch --no-tags sashal-linux-stable queue-4.4
git checkout a128d6484ea6c26f6266bd8d93c62346436bfd4e
# 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 warnings (new ones prefixed by >>):
In file included from arch/arm64/include/asm/hw_breakpoint.h:20:0,
from arch/arm64/include/asm/processor.h:33,
from arch/arm64/include/asm/spinlock.h:21,
from include/linux/spinlock.h:87,
from include/linux/wait.h:8,
from include/linux/fs.h:5,
from fs/squashfs/export.c:38:
arch/arm64/include/asm/cpufeature.h: In function 'cpuid_feature_extract_field':
arch/arm64/include/asm/cpufeature.h:156:55: warning: signed and unsigned type in conditional expression [-Wsign-compare]
cpuid_feature_extract_signed_field(features, field) :
^
In file included from fs/squashfs/export.c:44:0:
fs/squashfs/squashfs_fs.h: In function 'squashfs_block_size':
fs/squashfs/squashfs_fs.h:135:29: warning: signed and unsigned type in conditional expression [-Wsign-compare]
return (size >> 25) ? -EIO : size;
^
fs/squashfs/export.c: In function 'squashfs_inode_lookup':
>> fs/squashfs/export.c:63:36: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (ino_num == 0 || (ino_num - 1) >= msblk->inodes)
^~
fs/squashfs/export.c: In function 'squashfs_read_inode_lookup_table':
fs/squashfs/export.c:164:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (n = 0; n < (indexes - 1); n++) {
^
--
In file included from arch/arm64/include/asm/hw_breakpoint.h:20:0,
from arch/arm64/include/asm/processor.h:33,
from arch/arm64/include/asm/spinlock.h:21,
from include/linux/spinlock.h:87,
from include/linux/wait.h:8,
from include/linux/fs.h:5,
from fs/squashfs/id.c:34:
arch/arm64/include/asm/cpufeature.h: In function 'cpuid_feature_extract_field':
arch/arm64/include/asm/cpufeature.h:156:55: warning: signed and unsigned type in conditional expression [-Wsign-compare]
cpuid_feature_extract_signed_field(features, field) :
^
In file included from fs/squashfs/id.c:38:0:
fs/squashfs/squashfs_fs.h: In function 'squashfs_block_size':
fs/squashfs/squashfs_fs.h:135:29: warning: signed and unsigned type in conditional expression [-Wsign-compare]
return (size >> 25) ? -EIO : size;
^
fs/squashfs/id.c: In function 'squashfs_read_id_index_table':
>> fs/squashfs/id.c:109:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (n = 0; n < (indexes - 1); n++) {
^
vim +63 fs/squashfs/export.c
43
> 44 #include "squashfs_fs.h"
45 #include "squashfs_fs_sb.h"
46 #include "squashfs_fs_i.h"
47 #include "squashfs.h"
48
49 /*
50 * Look-up inode number (ino) in table, returning the inode location.
51 */
52 static long long squashfs_inode_lookup(struct super_block *sb, int ino_num)
53 {
54 struct squashfs_sb_info *msblk = sb->s_fs_info;
55 int blk = SQUASHFS_LOOKUP_BLOCK(ino_num - 1);
56 int offset = SQUASHFS_LOOKUP_BLOCK_OFFSET(ino_num - 1);
57 u64 start;
58 __le64 ino;
59 int err;
60
61 TRACE("Entered squashfs_inode_lookup, inode_number = %d\n", ino_num);
62
> 63 if (ino_num == 0 || (ino_num - 1) >= msblk->inodes)
64 return -EINVAL;
65
66 start = le64_to_cpu(msblk->inode_lookup_table[blk]);
67
68 err = squashfs_read_metadata(sb, &ino, &start, &offset, sizeof(ino));
69 if (err < 0)
70 return err;
71
72 TRACE("squashfs_inode_lookup, inode = 0x%llx\n",
73 (u64) le64_to_cpu(ino));
74
75 return le64_to_cpu(ino);
76 }
77
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
[sashal-linux-stable:queue-5.10 48/49] arch/arm64/kernel/cpufeature.c:1706:1: error: expected identifier or '(' before '}' token
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git queue-5.10
head: 900aea7691ecf9b39e819be55f95a940c75cc3ac
commit: 36c5708fe6f882433a4915efa46f05ad58b52eb3 [48/49] arm64: mte: Allow PTRACE_PEEKMTETAGS access to the zero page
config: arm64-allyesconfig (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
# https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git/c...
git remote add sashal-linux-stable https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git
git fetch --no-tags sashal-linux-stable queue-5.10
git checkout 36c5708fe6f882433a4915efa46f05ad58b52eb3
# 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 >>):
>> arch/arm64/kernel/cpufeature.c:1706:1: error: expected identifier or '(' before '}' token
1706 | }
| ^
vim +1706 arch/arm64/kernel/cpufeature.c
8ef8f360cf30be Dave Martin 2020-03-16 1695
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1696 #ifdef CONFIG_ARM64_MTE
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1697 static void cpu_enable_mte(struct arm64_cpu_capabilities const *cap)
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1698 {
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1699 /*
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1700 * Clear the tags in the zero page. This needs to be done via the
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1701 * linear map which has the Tagged attribute.
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1702 */
36c5708fe6f882 Catalin Marinas 2021-02-10 1703 if (!test_and_set_bit(PG_mte_tagged, &ZERO_PAGE(0)->flags))
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1704 mte_clear_page_tags(lm_alias(empty_zero_page));
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1705 }
34bfeea4a9e9cd Catalin Marinas 2020-05-04 @1706 }
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1707 #endif /* CONFIG_ARM64_MTE */
34bfeea4a9e9cd Catalin Marinas 2020-05-04 1708
:::::: The code at line 1706 was first introduced by commit
:::::: 34bfeea4a9e9cdae713637541f240c3adfdfede3 arm64: mte: Clear the tags when a page is mapped in user-space with PROT_MTE
:::::: TO: Catalin Marinas <catalin.marinas(a)arm.com>
:::::: CC: Catalin Marinas <catalin.marinas(a)arm.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months
[freescale-fslc:pr/256 7470/16397] drivers/net/ethernet/freescale/sdk_dpaa/dpaa_eth.h:506:23: error: type qualifiers ignored on function return type
by kernel test robot
tree: https://github.com/Freescale/linux-fslc pr/256
head: 833dfde20a84e72abea15e51e24b5027091c864c
commit: 7dcadb8d9273dd1ce30f55e8b4489f7c195d44c7 [7470/16397] fsl_qbman: Framework for enabling Link status notification
config: arm-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://github.com/Freescale/linux-fslc/commit/7dcadb8d9273dd1ce30f55e8b4...
git remote add freescale-fslc https://github.com/Freescale/linux-fslc
git fetch --no-tags freescale-fslc pr/256
git checkout 7dcadb8d9273dd1ce30f55e8b4489f7c195d44c7
# 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 errors (new ones prefixed by >>):
In file included from include/linux/fsl_usdpaa.h:17,
from drivers/staging/fsl_qbman/dpa_sys.h:66,
from drivers/staging/fsl_qbman/fsl_usdpaa.c:28:
include/linux/fsl_qman.h:415:1: error: alignment 1 of 'struct qm_eqcr_entry' is less than 8 [-Werror=packed-not-aligned]
415 | } __packed;
| ^
include/linux/fsl_qman.h:469:3: error: alignment 1 of 'struct <anonymous>' is less than 8 [-Werror=packed-not-aligned]
469 | } __packed ern;
| ^
include/linux/fsl_qman.h:468:17: error: 'fd' offset 15 in 'struct <anonymous>' isn't aligned to 8 [-Werror=packed-not-aligned]
468 | struct qm_fd fd;
| ^~
include/linux/fsl_qman.h:486:3: error: alignment 1 of 'struct <anonymous>' is less than 8 [-Werror=packed-not-aligned]
486 | } __packed dcern;
| ^
include/linux/fsl_qman.h:485:17: error: 'fd' offset 15 in 'struct <anonymous>' isn't aligned to 8 [-Werror=packed-not-aligned]
485 | struct qm_fd fd;
| ^~
include/linux/fsl_qman.h:1679:1: error: alignment 1 of 'struct qm_mcr_ceetm_cq_peek_pop_xsfdrread' is less than 8 [-Werror=packed-not-aligned]
1679 | } __packed;
| ^
include/linux/fsl_qman.h:1677:15: error: 'fd' offset 14 in 'struct qm_mcr_ceetm_cq_peek_pop_xsfdrread' isn't aligned to 8 [-Werror=packed-not-aligned]
1677 | struct qm_fd fd;
| ^~
In file included from drivers/staging/fsl_qbman/fsl_usdpaa.c:31:
drivers/staging/fsl_qbman/qman_low.h: In function 'qm_shutdown_fq':
drivers/staging/fsl_qbman/qman_low.h:1291:8: error: variable 'dequeue_wq' set but not used [-Werror=unused-but-set-variable]
1291 | u16 dequeue_wq = 0;
| ^~~~~~~~~~
In file included from drivers/net/ethernet/freescale/sdk_dpaa/dpaa_eth_common.h:40,
from drivers/staging/fsl_qbman/fsl_usdpaa.c:38:
drivers/net/ethernet/freescale/sdk_dpaa/dpaa_eth.h: At top level:
>> drivers/net/ethernet/freescale/sdk_dpaa/dpaa_eth.h:506:23: error: type qualifiers ignored on function return type [-Werror=ignored-qualifiers]
506 | static inline ssize_t __const __must_check __attribute__((nonnull))
| ^~~~~~~
drivers/net/ethernet/freescale/sdk_dpaa/dpaa_eth.h:512:23: error: type qualifiers ignored on function return type [-Werror=ignored-qualifiers]
512 | static inline ssize_t __const __must_check __attribute__((nonnull))
| ^~~~~~~
drivers/staging/fsl_qbman/fsl_usdpaa.c:398:6: error: no previous prototype for 'dbci_portal' [-Werror=missing-prototypes]
398 | void dbci_portal(void *addr)
| ^~~~~~~~~~~
cc1: all warnings being treated as errors
vim +506 drivers/net/ethernet/freescale/sdk_dpaa/dpaa_eth.h
7fab630ae39d23 Madalin Bucur 2017-05-10 505
7fab630ae39d23 Madalin Bucur 2017-05-10 @506 static inline ssize_t __const __must_check __attribute__((nonnull))
7fab630ae39d23 Madalin Bucur 2017-05-10 507 dpa_fd_length(const struct qm_fd *fd)
7fab630ae39d23 Madalin Bucur 2017-05-10 508 {
7fab630ae39d23 Madalin Bucur 2017-05-10 509 return fd->length20;
7fab630ae39d23 Madalin Bucur 2017-05-10 510 }
7fab630ae39d23 Madalin Bucur 2017-05-10 511
:::::: The code at line 506 was first introduced by commit
:::::: 7fab630ae39d2305c71f6ae6311f240ff9348404 dpa: SDK DPAA 1.x Ethernet driver
:::::: TO: Madalin Bucur <madalin.bucur(a)nxp.com>
:::::: CC: Dong Aisheng <aisheng.dong(a)nxp.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 7 months