[jaegeuk-f2fs-stable:linux-4.19.y 482/951] fs/f2fs/inode.c:307:7: warning: format specifies type 'unsigned long' but the argument has type 'blkcnt_t' (aka 'unsigned long long')
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs-stable.git linux-4.19.y
head: c79c4827e0ed88f2a1c8aae428ccf6da3c104bfe
commit: 0f87f20d0536ae21524ce1c25335aec991229114 [482/951] f2fs: fix to check i_compr_blocks correctly
config: arm-randconfig-r022-20211122 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project c133fb321f7ca6083ce15b6aa5bf89de6600e649)
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/jaegeuk/f2fs-stable.git/c...
git remote add jaegeuk-f2fs-stable https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs-stable.git
git fetch --no-tags jaegeuk-f2fs-stable linux-4.19.y
git checkout 0f87f20d0536ae21524ce1c25335aec991229114
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 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 >>):
>> fs/f2fs/inode.c:307:7: warning: format specifies type 'unsigned long' but the argument has type 'blkcnt_t' (aka 'unsigned long long') [-Wformat]
SECTOR_TO_BLOCK(inode->i_blocks));
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/f2fs/f2fs.h:1949:39: note: expanded from macro 'f2fs_warn'
f2fs_printk(sbi, KERN_WARNING fmt, ##__VA_ARGS__)
~~~ ^~~~~~~~~~~
fs/f2fs/segment.h:119:2: note: expanded from macro 'SECTOR_TO_BLOCK'
((sectors) >> F2FS_LOG_SECTORS_PER_BLOCK)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
vim +307 fs/f2fs/inode.c
198
199 static bool sanity_check_inode(struct inode *inode, struct page *node_page)
200 {
201 struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
202 struct f2fs_inode_info *fi = F2FS_I(inode);
203 struct f2fs_inode *ri = F2FS_INODE(node_page);
204 unsigned long long iblocks;
205
206 iblocks = le64_to_cpu(F2FS_INODE(node_page)->i_blocks);
207 if (!iblocks) {
208 set_sbi_flag(sbi, SBI_NEED_FSCK);
209 f2fs_warn(sbi, "%s: corrupted inode i_blocks i_ino=%lx iblocks=%llu, run fsck to fix.",
210 __func__, inode->i_ino, iblocks);
211 return false;
212 }
213
214 if (ino_of_node(node_page) != nid_of_node(node_page)) {
215 set_sbi_flag(sbi, SBI_NEED_FSCK);
216 f2fs_warn(sbi, "%s: corrupted inode footer i_ino=%lx, ino,nid: [%u, %u] run fsck to fix.",
217 __func__, inode->i_ino,
218 ino_of_node(node_page), nid_of_node(node_page));
219 return false;
220 }
221
222 if (f2fs_sb_has_flexible_inline_xattr(sbi)
223 && !f2fs_has_extra_attr(inode)) {
224 set_sbi_flag(sbi, SBI_NEED_FSCK);
225 f2fs_warn(sbi, "%s: corrupted inode ino=%lx, run fsck to fix.",
226 __func__, inode->i_ino);
227 return false;
228 }
229
230 if (f2fs_has_extra_attr(inode) &&
231 !f2fs_sb_has_extra_attr(sbi)) {
232 set_sbi_flag(sbi, SBI_NEED_FSCK);
233 f2fs_warn(sbi, "%s: inode (ino=%lx) is with extra_attr, but extra_attr feature is off",
234 __func__, inode->i_ino);
235 return false;
236 }
237
238 if (fi->i_extra_isize > F2FS_TOTAL_EXTRA_ATTR_SIZE ||
239 fi->i_extra_isize % sizeof(__le32)) {
240 set_sbi_flag(sbi, SBI_NEED_FSCK);
241 f2fs_warn(sbi, "%s: inode (ino=%lx) has corrupted i_extra_isize: %d, max: %zu",
242 __func__, inode->i_ino, fi->i_extra_isize,
243 F2FS_TOTAL_EXTRA_ATTR_SIZE);
244 return false;
245 }
246
247 if (f2fs_has_extra_attr(inode) &&
248 f2fs_sb_has_flexible_inline_xattr(sbi) &&
249 f2fs_has_inline_xattr(inode) &&
250 (!fi->i_inline_xattr_size ||
251 fi->i_inline_xattr_size > MAX_INLINE_XATTR_SIZE)) {
252 set_sbi_flag(sbi, SBI_NEED_FSCK);
253 f2fs_warn(sbi, "%s: inode (ino=%lx) has corrupted i_inline_xattr_size: %d, max: %zu",
254 __func__, inode->i_ino, fi->i_inline_xattr_size,
255 MAX_INLINE_XATTR_SIZE);
256 return false;
257 }
258
259 if (F2FS_I(inode)->extent_tree) {
260 struct extent_info *ei = &F2FS_I(inode)->extent_tree->largest;
261
262 if (ei->len &&
263 (!f2fs_is_valid_blkaddr(sbi, ei->blk,
264 DATA_GENERIC_ENHANCE) ||
265 !f2fs_is_valid_blkaddr(sbi, ei->blk + ei->len - 1,
266 DATA_GENERIC_ENHANCE))) {
267 set_sbi_flag(sbi, SBI_NEED_FSCK);
268 f2fs_warn(sbi, "%s: inode (ino=%lx) extent info [%u, %u, %u] is incorrect, run fsck to fix",
269 __func__, inode->i_ino,
270 ei->blk, ei->fofs, ei->len);
271 return false;
272 }
273 }
274
275 if (f2fs_has_inline_data(inode) &&
276 (!S_ISREG(inode->i_mode) && !S_ISLNK(inode->i_mode))) {
277 set_sbi_flag(sbi, SBI_NEED_FSCK);
278 f2fs_warn(sbi, "%s: inode (ino=%lx, mode=%u) should not have inline_data, run fsck to fix",
279 __func__, inode->i_ino, inode->i_mode);
280 return false;
281 }
282
283 if (f2fs_has_inline_dentry(inode) && !S_ISDIR(inode->i_mode)) {
284 set_sbi_flag(sbi, SBI_NEED_FSCK);
285 f2fs_warn(sbi, "%s: inode (ino=%lx, mode=%u) should not have inline_dentry, run fsck to fix",
286 __func__, inode->i_ino, inode->i_mode);
287 return false;
288 }
289
290 if (f2fs_has_extra_attr(inode) && f2fs_sb_has_compression(sbi) &&
291 fi->i_flags & F2FS_COMPR_FL &&
292 F2FS_FITS_IN_INODE(ri, fi->i_extra_isize,
293 i_log_cluster_size)) {
294 if (ri->i_compress_algorithm >= COMPRESS_MAX) {
295 f2fs_warn(sbi, "%s: inode (ino=%lx) has unsupported "
296 "compress algorithm: %u, run fsck to fix",
297 __func__, inode->i_ino,
298 ri->i_compress_algorithm);
299 return false;
300 }
301 if (le64_to_cpu(ri->i_compr_blocks) >
302 SECTOR_TO_BLOCK(inode->i_blocks)) {
303 f2fs_warn(sbi, "%s: inode (ino=%lx) has inconsistent "
304 "i_compr_blocks:%llu, i_blocks:%lu, run fsck to fix",
305 __func__, inode->i_ino,
306 le64_to_cpu(ri->i_compr_blocks),
> 307 SECTOR_TO_BLOCK(inode->i_blocks));
308 return false;
309 }
310 if (ri->i_log_cluster_size < MIN_COMPRESS_LOG_SIZE ||
311 ri->i_log_cluster_size > MAX_COMPRESS_LOG_SIZE) {
312 f2fs_warn(sbi, "%s: inode (ino=%lx) has unsupported "
313 "log cluster size: %u, run fsck to fix",
314 __func__, inode->i_ino,
315 ri->i_log_cluster_size);
316 return false;
317 }
318 }
319
320 return true;
321 }
322
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months
net/netlabel/netlabel_domainhash.c:495:21: warning: variable 'old_list6' set but not used
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 136057256686de39cc3a07c2e39ef6bc43003ff6
commit: 010b430d5df556d5d232e3751ac691ba9e88c041 mptcp: MPTCP_IPV6 should depend on IPV6 instead of selecting it
date: 1 year, 1 month ago
config: x86_64-randconfig-a004-20211122 (attached as .config)
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 010b430d5df556d5d232e3751ac691ba9e88c041
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> net/netlabel/netlabel_domainhash.c:495:21: warning: variable 'old_list6' set but not used [-Wunused-but-set-variable]
struct list_head *old_list6;
^
1 warning generated.
vim +/old_list6 +495 net/netlabel/netlabel_domainhash.c
63c41688743760 Paul Moore 2008-10-10 416
6b21e1b77d1a3d Paul Moore 2013-05-17 417 ret_val = netlbl_domhsh_validate(entry);
6b21e1b77d1a3d Paul Moore 2013-05-17 418 if (ret_val != 0)
6b21e1b77d1a3d Paul Moore 2013-05-17 419 return ret_val;
6b21e1b77d1a3d Paul Moore 2013-05-17 420
b914f3a2a35812 Paul Moore 2010-04-01 421 /* XXX - we can remove this RCU read lock as the spinlock protects the
b914f3a2a35812 Paul Moore 2010-04-01 422 * entire function, but before we do we need to fixup the
b914f3a2a35812 Paul Moore 2010-04-01 423 * netlbl_af[4,6]list RCU functions to do "the right thing" with
b914f3a2a35812 Paul Moore 2010-04-01 424 * respect to rcu_dereference() when only a spinlock is held. */
63c41688743760 Paul Moore 2008-10-10 425 rcu_read_lock();
63c41688743760 Paul Moore 2008-10-10 426 spin_lock(&netlbl_domhsh_lock);
63c41688743760 Paul Moore 2008-10-10 427 if (entry->domain != NULL)
8f18e675c3335b Huw Davies 2016-06-27 428 entry_old = netlbl_domhsh_search(entry->domain, entry->family);
63c41688743760 Paul Moore 2008-10-10 429 else
8f18e675c3335b Huw Davies 2016-06-27 430 entry_old = netlbl_domhsh_search_def(entry->domain,
8f18e675c3335b Huw Davies 2016-06-27 431 entry->family);
63c41688743760 Paul Moore 2008-10-10 432 if (entry_old == NULL) {
d15c345fe3b8df Paul Moore 2006-08-03 433 entry->valid = 1;
d15c345fe3b8df Paul Moore 2006-08-03 434
d15c345fe3b8df Paul Moore 2006-08-03 435 if (entry->domain != NULL) {
63c41688743760 Paul Moore 2008-10-10 436 u32 bkt = netlbl_domhsh_hash(entry->domain);
d15c345fe3b8df Paul Moore 2006-08-03 437 list_add_tail_rcu(&entry->list,
3482fd9099e8aa Paul Moore 2007-08-07 438 &rcu_dereference(netlbl_domhsh)->tbl[bkt]);
4be2700fb7b95f Paul Moore 2007-10-26 439 } else {
d15c345fe3b8df Paul Moore 2006-08-03 440 INIT_LIST_HEAD(&entry->list);
8f18e675c3335b Huw Davies 2016-06-27 441 switch (entry->family) {
8f18e675c3335b Huw Davies 2016-06-27 442 case AF_INET:
8f18e675c3335b Huw Davies 2016-06-27 443 rcu_assign_pointer(netlbl_domhsh_def_ipv4,
8f18e675c3335b Huw Davies 2016-06-27 444 entry);
8f18e675c3335b Huw Davies 2016-06-27 445 break;
8f18e675c3335b Huw Davies 2016-06-27 446 case AF_INET6:
8f18e675c3335b Huw Davies 2016-06-27 447 rcu_assign_pointer(netlbl_domhsh_def_ipv6,
8f18e675c3335b Huw Davies 2016-06-27 448 entry);
8f18e675c3335b Huw Davies 2016-06-27 449 break;
8f18e675c3335b Huw Davies 2016-06-27 450 case AF_UNSPEC:
8f18e675c3335b Huw Davies 2016-06-27 451 if (entry->def.type !=
8f18e675c3335b Huw Davies 2016-06-27 452 NETLBL_NLTYPE_UNLABELED) {
8f18e675c3335b Huw Davies 2016-06-27 453 ret_val = -EINVAL;
8f18e675c3335b Huw Davies 2016-06-27 454 goto add_return;
8f18e675c3335b Huw Davies 2016-06-27 455 }
8f18e675c3335b Huw Davies 2016-06-27 456 entry_b = kzalloc(sizeof(*entry_b), GFP_ATOMIC);
8f18e675c3335b Huw Davies 2016-06-27 457 if (entry_b == NULL) {
8f18e675c3335b Huw Davies 2016-06-27 458 ret_val = -ENOMEM;
8f18e675c3335b Huw Davies 2016-06-27 459 goto add_return;
8f18e675c3335b Huw Davies 2016-06-27 460 }
8f18e675c3335b Huw Davies 2016-06-27 461 entry_b->family = AF_INET6;
8f18e675c3335b Huw Davies 2016-06-27 462 entry_b->def.type = NETLBL_NLTYPE_UNLABELED;
8f18e675c3335b Huw Davies 2016-06-27 463 entry_b->valid = 1;
8f18e675c3335b Huw Davies 2016-06-27 464 entry->family = AF_INET;
8f18e675c3335b Huw Davies 2016-06-27 465 rcu_assign_pointer(netlbl_domhsh_def_ipv4,
8f18e675c3335b Huw Davies 2016-06-27 466 entry);
8f18e675c3335b Huw Davies 2016-06-27 467 rcu_assign_pointer(netlbl_domhsh_def_ipv6,
8f18e675c3335b Huw Davies 2016-06-27 468 entry_b);
8f18e675c3335b Huw Davies 2016-06-27 469 break;
8f18e675c3335b Huw Davies 2016-06-27 470 default:
8f18e675c3335b Huw Davies 2016-06-27 471 /* Already checked in
8f18e675c3335b Huw Davies 2016-06-27 472 * netlbl_domhsh_validate(). */
8f18e675c3335b Huw Davies 2016-06-27 473 ret_val = -EINVAL;
8f18e675c3335b Huw Davies 2016-06-27 474 goto add_return;
8f18e675c3335b Huw Davies 2016-06-27 475 }
63c41688743760 Paul Moore 2008-10-10 476 }
63c41688743760 Paul Moore 2008-10-10 477
6a8b7f0c85f1f4 Paul Moore 2013-08-02 478 if (entry->def.type == NETLBL_NLTYPE_ADDRSELECT) {
63c41688743760 Paul Moore 2008-10-10 479 netlbl_af4list_foreach_rcu(iter4,
6a8b7f0c85f1f4 Paul Moore 2013-08-02 480 &entry->def.addrsel->list4)
63c41688743760 Paul Moore 2008-10-10 481 netlbl_domhsh_audit_add(entry, iter4, NULL,
63c41688743760 Paul Moore 2008-10-10 482 ret_val, audit_info);
dfd56b8b38fff3 Eric Dumazet 2011-12-10 483 #if IS_ENABLED(CONFIG_IPV6)
63c41688743760 Paul Moore 2008-10-10 484 netlbl_af6list_foreach_rcu(iter6,
6a8b7f0c85f1f4 Paul Moore 2013-08-02 485 &entry->def.addrsel->list6)
63c41688743760 Paul Moore 2008-10-10 486 netlbl_domhsh_audit_add(entry, NULL, iter6,
63c41688743760 Paul Moore 2008-10-10 487 ret_val, audit_info);
63c41688743760 Paul Moore 2008-10-10 488 #endif /* IPv6 */
63c41688743760 Paul Moore 2008-10-10 489 } else
63c41688743760 Paul Moore 2008-10-10 490 netlbl_domhsh_audit_add(entry, NULL, NULL,
63c41688743760 Paul Moore 2008-10-10 491 ret_val, audit_info);
6a8b7f0c85f1f4 Paul Moore 2013-08-02 492 } else if (entry_old->def.type == NETLBL_NLTYPE_ADDRSELECT &&
6a8b7f0c85f1f4 Paul Moore 2013-08-02 493 entry->def.type == NETLBL_NLTYPE_ADDRSELECT) {
63c41688743760 Paul Moore 2008-10-10 494 struct list_head *old_list4;
63c41688743760 Paul Moore 2008-10-10 @495 struct list_head *old_list6;
63c41688743760 Paul Moore 2008-10-10 496
6a8b7f0c85f1f4 Paul Moore 2013-08-02 497 old_list4 = &entry_old->def.addrsel->list4;
6a8b7f0c85f1f4 Paul Moore 2013-08-02 498 old_list6 = &entry_old->def.addrsel->list6;
63c41688743760 Paul Moore 2008-10-10 499
63c41688743760 Paul Moore 2008-10-10 500 /* we only allow the addition of address selectors if all of
63c41688743760 Paul Moore 2008-10-10 501 * the selectors do not exist in the existing domain map */
6a8b7f0c85f1f4 Paul Moore 2013-08-02 502 netlbl_af4list_foreach_rcu(iter4, &entry->def.addrsel->list4)
63c41688743760 Paul Moore 2008-10-10 503 if (netlbl_af4list_search_exact(iter4->addr,
63c41688743760 Paul Moore 2008-10-10 504 iter4->mask,
63c41688743760 Paul Moore 2008-10-10 505 old_list4)) {
d15c345fe3b8df Paul Moore 2006-08-03 506 ret_val = -EEXIST;
63c41688743760 Paul Moore 2008-10-10 507 goto add_return;
4be2700fb7b95f Paul Moore 2007-10-26 508 }
dfd56b8b38fff3 Eric Dumazet 2011-12-10 509 #if IS_ENABLED(CONFIG_IPV6)
6a8b7f0c85f1f4 Paul Moore 2013-08-02 510 netlbl_af6list_foreach_rcu(iter6, &entry->def.addrsel->list6)
63c41688743760 Paul Moore 2008-10-10 511 if (netlbl_af6list_search_exact(&iter6->addr,
63c41688743760 Paul Moore 2008-10-10 512 &iter6->mask,
63c41688743760 Paul Moore 2008-10-10 513 old_list6)) {
63c41688743760 Paul Moore 2008-10-10 514 ret_val = -EEXIST;
63c41688743760 Paul Moore 2008-10-10 515 goto add_return;
32f50cdee66633 Paul Moore 2006-09-28 516 }
63c41688743760 Paul Moore 2008-10-10 517 #endif /* IPv6 */
63c41688743760 Paul Moore 2008-10-10 518
63c41688743760 Paul Moore 2008-10-10 519 netlbl_af4list_foreach_safe(iter4, tmp4,
6a8b7f0c85f1f4 Paul Moore 2013-08-02 520 &entry->def.addrsel->list4) {
63c41688743760 Paul Moore 2008-10-10 521 netlbl_af4list_remove_entry(iter4);
63c41688743760 Paul Moore 2008-10-10 522 iter4->valid = 1;
63c41688743760 Paul Moore 2008-10-10 523 ret_val = netlbl_af4list_add(iter4, old_list4);
63c41688743760 Paul Moore 2008-10-10 524 netlbl_domhsh_audit_add(entry_old, iter4, NULL,
63c41688743760 Paul Moore 2008-10-10 525 ret_val, audit_info);
63c41688743760 Paul Moore 2008-10-10 526 if (ret_val != 0)
63c41688743760 Paul Moore 2008-10-10 527 goto add_return;
de64688ffb952a Paul Moore 2006-11-17 528 }
dfd56b8b38fff3 Eric Dumazet 2011-12-10 529 #if IS_ENABLED(CONFIG_IPV6)
63c41688743760 Paul Moore 2008-10-10 530 netlbl_af6list_foreach_safe(iter6, tmp6,
6a8b7f0c85f1f4 Paul Moore 2013-08-02 531 &entry->def.addrsel->list6) {
63c41688743760 Paul Moore 2008-10-10 532 netlbl_af6list_remove_entry(iter6);
63c41688743760 Paul Moore 2008-10-10 533 iter6->valid = 1;
63c41688743760 Paul Moore 2008-10-10 534 ret_val = netlbl_af6list_add(iter6, old_list6);
63c41688743760 Paul Moore 2008-10-10 535 netlbl_domhsh_audit_add(entry_old, NULL, iter6,
63c41688743760 Paul Moore 2008-10-10 536 ret_val, audit_info);
63c41688743760 Paul Moore 2008-10-10 537 if (ret_val != 0)
63c41688743760 Paul Moore 2008-10-10 538 goto add_return;
63c41688743760 Paul Moore 2008-10-10 539 }
63c41688743760 Paul Moore 2008-10-10 540 #endif /* IPv6 */
d3b990b7f327e2 Paul Moore 2020-08-21 541 /* cleanup the new entry since we've moved everything over */
d3b990b7f327e2 Paul Moore 2020-08-21 542 netlbl_domhsh_free_entry(&entry->rcu);
63c41688743760 Paul Moore 2008-10-10 543 } else
63c41688743760 Paul Moore 2008-10-10 544 ret_val = -EINVAL;
d15c345fe3b8df Paul Moore 2006-08-03 545
63c41688743760 Paul Moore 2008-10-10 546 add_return:
63c41688743760 Paul Moore 2008-10-10 547 spin_unlock(&netlbl_domhsh_lock);
63c41688743760 Paul Moore 2008-10-10 548 rcu_read_unlock();
d15c345fe3b8df Paul Moore 2006-08-03 549 return ret_val;
d15c345fe3b8df Paul Moore 2006-08-03 550 }
d15c345fe3b8df Paul Moore 2006-08-03 551
:::::: The code at line 495 was first introduced by commit
:::::: 63c41688743760631188cf0f4ae986a6793ccb0a netlabel: Add network address selectors to the NetLabel/LSM domain mapping
:::::: TO: Paul Moore <paul.moore(a)hp.com>
:::::: CC: Paul Moore <paul.moore(a)hp.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months
[cel:topic-xdr-tracepoints 50/169] include/trace/../../fs/lockd/xdrtrace.h:35:19: error: 'NLM_DEADLCK' undeclared here (not in a function)
by kernel test robot
tree: git://git.kernel.org/pub/scm/linux/kernel/git/cel/linux topic-xdr-tracepoints
head: 07f44f67cedcceb30d1fc98899a509d7da3827d1
commit: ccff37ef1fdeb303eab48ff46b0df0e80867c4f0 [50/169] lockd: Infrastructure for lockd server-side XDR tracepoints
config: openrisc-randconfig-r011-20211122 (attached as .config)
compiler: or1k-linux-gcc (GCC) 11.2.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/cel/linux.git/commit/?id=...
git remote add cel git://git.kernel.org/pub/scm/linux/kernel/git/cel/linux
git fetch --no-tags cel topic-xdr-tracepoints
git checkout ccff37ef1fdeb303eab48ff46b0df0e80867c4f0
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=openrisc SHELL=/bin/bash fs/
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/trace/define_trace.h:102,
from fs/lockd/xdrtrace.h:96,
from fs/lockd/xdrtrace.c:31:
>> include/trace/../../fs/lockd/xdrtrace.h:35:19: error: 'NLM_DEADLCK' undeclared here (not in a function)
35 | TRACE_DEFINE_ENUM(NLM_DEADLCK);
| ^~~~~~~~~~~
include/trace/trace_events.h:45:31: note: in definition of macro 'TRACE_DEFINE_ENUM'
45 | .eval_value = a \
| ^
>> include/trace/../../fs/lockd/xdrtrace.h:36:19: error: 'NLM_ROFS' undeclared here (not in a function)
36 | TRACE_DEFINE_ENUM(NLM_ROFS);
| ^~~~~~~~
include/trace/trace_events.h:45:31: note: in definition of macro 'TRACE_DEFINE_ENUM'
45 | .eval_value = a \
| ^
>> include/trace/../../fs/lockd/xdrtrace.h:37:19: error: 'NLM_STALE_FH' undeclared here (not in a function)
37 | TRACE_DEFINE_ENUM(NLM_STALE_FH);
| ^~~~~~~~~~~~
include/trace/trace_events.h:45:31: note: in definition of macro 'TRACE_DEFINE_ENUM'
45 | .eval_value = a \
| ^
>> include/trace/../../fs/lockd/xdrtrace.h:38:19: error: 'NLM_FBIG' undeclared here (not in a function)
38 | TRACE_DEFINE_ENUM(NLM_FBIG);
| ^~~~~~~~
include/trace/trace_events.h:45:31: note: in definition of macro 'TRACE_DEFINE_ENUM'
45 | .eval_value = a \
| ^
>> include/trace/../../fs/lockd/xdrtrace.h:39:19: error: 'NLM_FAILED' undeclared here (not in a function); did you mean 'MF_FAILED'?
39 | TRACE_DEFINE_ENUM(NLM_FAILED);
| ^~~~~~~~~~
include/trace/trace_events.h:45:31: note: in definition of macro 'TRACE_DEFINE_ENUM'
45 | .eval_value = a \
| ^
vim +/NLM_DEADLCK +35 include/trace/../../fs/lockd/xdrtrace.h
25
26 /**
27 ** Helpers
28 **/
29
30 TRACE_DEFINE_ENUM(NLM_LCK_GRANTED);
31 TRACE_DEFINE_ENUM(NLM_LCK_DENIED);
32 TRACE_DEFINE_ENUM(NLM_LCK_DENIED_NOLOCKS);
33 TRACE_DEFINE_ENUM(NLM_LCK_BLOCKED);
34 TRACE_DEFINE_ENUM(NLM_LCK_DENIED_GRACE_PERIOD);
> 35 TRACE_DEFINE_ENUM(NLM_DEADLCK);
> 36 TRACE_DEFINE_ENUM(NLM_ROFS);
> 37 TRACE_DEFINE_ENUM(NLM_STALE_FH);
> 38 TRACE_DEFINE_ENUM(NLM_FBIG);
> 39 TRACE_DEFINE_ENUM(NLM_FAILED);
40
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months
Re: [PATCH] Staging: rtl8712: rtl871x_security: fixed a camel case variable name coding style issue
by kernel test robot
Hi Zoeb,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on staging/staging-testing]
url: https://github.com/0day-ci/linux/commits/Zoeb-Mithaiwala/Staging-rtl8712-...
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 1189d2fb15a4b09b2e8dd01d60a0817d985d933d
config: x86_64-randconfig-a015-20211121 (attached as .config)
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/8c1ab61206da180488d1d32547a732880...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Zoeb-Mithaiwala/Staging-rtl8712-rtl871x_security-fixed-a-camel-case-variable-name-coding-style-issue/20211120-160918
git checkout 8c1ab61206da180488d1d32547a73288052736cd
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash
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/staging/rtl8712/rtl871x_security.c:288:44: error: no member named 'n_bytes_in_m' in 'struct mic_data'; did you mean 'nbytes_in_m'?
pmicdata->M |= ((u32)b) << (8 * pmicdata->n_bytes_in_m);
^~~~~~~~~~~~
nbytes_in_m
drivers/staging/rtl8712/rtl871x_security.h:195:7: note: 'nbytes_in_m' declared here
u32 nbytes_in_m; /* # bytes in M */
^
1 error generated.
vim +288 drivers/staging/rtl8712/rtl871x_security.c
284
285 static void secmicappendbyte(struct mic_data *pmicdata, u8 b)
286 {
287 /* Append the byte to our word-sized buffer */
> 288 pmicdata->M |= ((u32)b) << (8 * pmicdata->n_bytes_in_m);
289 pmicdata->nbytes_in_m++;
290 /* Process the word if it is full. */
291 if (pmicdata->nbytes_in_m >= 4) {
292 pmicdata->L ^= pmicdata->M;
293 pmicdata->R ^= ROL32(pmicdata->L, 17);
294 pmicdata->L += pmicdata->R;
295 pmicdata->R ^= ((pmicdata->L & 0xff00ff00) >> 8) |
296 ((pmicdata->L & 0x00ff00ff) << 8);
297 pmicdata->L += pmicdata->R;
298 pmicdata->R ^= ROL32(pmicdata->L, 3);
299 pmicdata->L += pmicdata->R;
300 pmicdata->R ^= ROR32(pmicdata->L, 2);
301 pmicdata->L += pmicdata->R;
302 /* Clear the buffer */
303 pmicdata->M = 0;
304 pmicdata->nbytes_in_m = 0;
305 }
306 }
307
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months
[chenxing:msc313_mainlining 40/69] drivers/pinctrl/mstar/pinctrl-msc313.c:1339:4: warning: unannotated fall-through between switch labels
by kernel test robot
tree: git://github.com/linux-chenxing/linux.git msc313_mainlining
head: df6df62e60f3a32773b02a41e0d6a4e0f2b353b5
commit: baf52d9889a05e4a516eb56f326b4e6525dc0fa8 [40/69] pinctrl: mstar: msc313 pinctrl driver
config: riscv-randconfig-r013-20211121 (attached as .config)
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/linux-chenxing/linux/commit/baf52d9889a05e4a516eb56f32...
git remote add chenxing git://github.com/linux-chenxing/linux.git
git fetch --no-tags chenxing msc313_mainlining
git checkout baf52d9889a05e4a516eb56f326b4e6525dc0fa8
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 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 >>):
In file included from drivers/pinctrl/mstar/pinctrl-msc313.c:7:
In file included from include/linux/gpio/driver.h:7:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/riscv/include/asm/io.h:136:
include/asm-generic/io.h:464:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __raw_readb(PCI_IOBASE + addr);
~~~~~~~~~~ ^
include/asm-generic/io.h:477:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:36:51: note: expanded from macro '__le16_to_cpu'
#define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
^
In file included from drivers/pinctrl/mstar/pinctrl-msc313.c:7:
In file included from include/linux/gpio/driver.h:7:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/riscv/include/asm/io.h:136:
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/little_endian.h:34:51: note: expanded from macro '__le32_to_cpu'
#define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
^
In file included from drivers/pinctrl/mstar/pinctrl-msc313.c:7:
In file included from include/linux/gpio/driver.h:7:
In file included from include/linux/irq.h:20:
In file included from include/linux/io.h:13:
In file included from arch/riscv/include/asm/io.h:136:
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:1024:55: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
return (port > MMIO_UPPER_LIMIT) ? NULL : PCI_IOBASE + port;
~~~~~~~~~~ ^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:1339:4: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough]
default:
^
drivers/pinctrl/mstar/pinctrl-msc313.c:1339:4: note: insert 'break;' to avoid fall-through
default:
^
break;
>> drivers/pinctrl/mstar/pinctrl-msc313.c:37:27: warning: unused variable 'i2c0_groups' [-Wunused-const-variable]
static const char * const i2c0_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:40:18: warning: unused variable 'i2c0_values' [-Wunused-const-variable]
static const u16 i2c0_values[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:44:27: warning: unused variable 'i2c1_groups' [-Wunused-const-variable]
static const char * const i2c1_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:47:18: warning: unused variable 'i2c1_values' [-Wunused-const-variable]
static const u16 i2c1_values[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:51:27: warning: unused variable 'fuart_groups' [-Wunused-const-variable]
static const char * const fuart_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:55:18: warning: unused variable 'fuart_values' [-Wunused-const-variable]
static const u16 fuart_values[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:60:27: warning: unused variable 'uart0_groups' [-Wunused-const-variable]
static const char * const uart0_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:64:27: warning: unused variable 'uart1_groups' [-Wunused-const-variable]
static const char * const uart1_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:68:18: warning: unused variable 'uart1_values' [-Wunused-const-variable]
static const u16 uart1_values[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:73:27: warning: unused variable 'usb_groups' [-Wunused-const-variable]
static const char * const usb_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:76:27: warning: unused variable 'usb1_groups' [-Wunused-const-variable]
static const char * const usb1_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:80:27: warning: unused variable 'pwm0_groups' [-Wunused-const-variable]
static const char * const pwm0_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:83:18: warning: unused variable 'pwm0_values' [-Wunused-const-variable]
static const u16 pwm0_values[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:87:27: warning: unused variable 'pwm1_groups' [-Wunused-const-variable]
static const char * const pwm1_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:90:18: warning: unused variable 'pwm1_values' [-Wunused-const-variable]
static const u16 pwm1_values[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:94:27: warning: unused variable 'pwm2_groups' [-Wunused-const-variable]
static const char * const pwm2_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:97:18: warning: unused variable 'pwm2_values' [-Wunused-const-variable]
static const u16 pwm2_values[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:101:27: warning: unused variable 'pwm3_groups' [-Wunused-const-variable]
static const char * const pwm3_groups[] = {
^
>> drivers/pinctrl/mstar/pinctrl-msc313.c:104:18: warning: unused variable 'pwm3_values' [-Wunused-const-variable]
static const u16 pwm3_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:108:27: warning: unused variable 'pwm4_groups' [-Wunused-const-variable]
static const char * const pwm4_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:111:18: warning: unused variable 'pwm4_values' [-Wunused-const-variable]
static const u16 pwm4_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:115:27: warning: unused variable 'pwm5_groups' [-Wunused-const-variable]
static const char * const pwm5_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:118:18: warning: unused variable 'pwm5_values' [-Wunused-const-variable]
static const u16 pwm5_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:122:27: warning: unused variable 'pwm6_groups' [-Wunused-const-variable]
static const char * const pwm6_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:125:18: warning: unused variable 'pwm6_values' [-Wunused-const-variable]
static const u16 pwm6_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:129:27: warning: unused variable 'pwm7_groups' [-Wunused-const-variable]
static const char * const pwm7_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:132:18: warning: unused variable 'pwm7_values' [-Wunused-const-variable]
static const u16 pwm7_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:136:27: warning: unused variable 'eth_groups' [-Wunused-const-variable]
static const char * const eth_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:139:18: warning: unused variable 'eth_values' [-Wunused-const-variable]
static const u16 eth_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:143:27: warning: unused variable 'jtag_groups' [-Wunused-const-variable]
static const char * const jtag_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:147:27: warning: unused variable 'spi0_groups' [-Wunused-const-variable]
static const char * const spi0_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:151:18: warning: unused variable 'spi0_values' [-Wunused-const-variable]
static const u16 spi0_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:155:27: warning: unused variable 'spi1_groups' [-Wunused-const-variable]
static const char * const spi1_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:158:18: warning: unused variable 'spi1_values' [-Wunused-const-variable]
static const u16 spi1_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:161:27: warning: unused variable 'sdio_groups' [-Wunused-const-variable]
static const char * const sdio_groups[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:164:18: warning: unused variable 'sdio_values' [-Wunused-const-variable]
static const u16 sdio_values[] = {
^
drivers/pinctrl/mstar/pinctrl-msc313.c:201:27: warning: unused variable 'sd_drivestrengths' [-Wunused-const-variable]
static const unsigned int sd_drivestrengths[] = {4, 8};
^
drivers/pinctrl/mstar/pinctrl-msc313.c:215:27: warning: unused variable 'spi0_drivestrengths' [-Wunused-const-variable]
static const unsigned int spi0_drivestrengths[] = {4, 8, 12, 16};
^
drivers/pinctrl/mstar/pinctrl-msc313.c:225:27: warning: unused variable 'i2c_drivestrengths' [-Wunused-const-variable]
static const unsigned int i2c_drivestrengths[] = {4, 8};
^
drivers/pinctrl/mstar/pinctrl-msc313.c:233:27: warning: unused variable 'sr_drivestrengths' [-Wunused-const-variable]
static const unsigned int sr_drivestrengths[] = {4, 8};
^
48 warnings generated.
vim +1339 drivers/pinctrl/mstar/pinctrl-msc313.c
1318
1319 static int mstar_set_config(struct msc313_pinctrl *pinctrl, int pin, unsigned long config)
1320 {
1321 enum pin_config_param param = pinconf_to_config_param(config);
1322 u32 arg = pinconf_to_config_argument(config);
1323 int i;
1324 unsigned int mask;
1325 const struct msc313_pinctrl_pinconf *confpin;
1326
1327 dev_dbg(pinctrl->dev, "setting %d:%u on pin %d\n", (int)config, (unsigned int)arg, pin);
1328 for (i = 0; i < pinctrl->info->npinconfs; i++) {
1329 if (pinctrl->info->pinconfs[i].pin == pin) {
1330 confpin = &pinctrl->info->pinconfs[i];
1331 switch (param) {
1332 case PIN_CONFIG_BIAS_PULL_UP:
1333 if (confpin->pull_en_reg != -1) {
1334 dev_dbg(pinctrl->dev, "setting pull up %d on pin %d\n", (int) arg, pin);
1335 mask = 1 << confpin->pull_en_bit;
1336 regmap_update_bits(pinctrl->regmap, confpin->pull_en_reg, mask, arg ? mask : 0);
1337 } else
1338 dev_info(pinctrl->dev, "pullup reg/bit isn't known for pin %d\n", pin);
> 1339 default:
1340 break;
1341 }
1342 return 0;
1343 }
1344 }
1345 return 0;
1346 }
1347
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months
[intel-lts:4.19/android_r 21728/25353] sound/soc/soc-core.c:761:27: warning: no previous prototype for function 'soc_find_component_locked'
by kernel test robot
Hi Aditya,
FYI, the error/warning still remains.
tree: https://github.com/intel/linux-intel-lts.git 4.19/android_r
head: 8367d521b6ca9b77d18fbb56f936df6824f0843c
commit: 75fe91933be2dd4b884ca83bd72e1e0f0d05cfc6 [21728/25353] ANDROID: ASoC: core: add locked version of soc_find_component
config: arm-randconfig-r013-20211122 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project c133fb321f7ca6083ce15b6aa5bf89de6600e649)
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/intel/linux-intel-lts/commit/75fe91933be2dd4b884ca83bd...
git remote add intel-lts https://github.com/intel/linux-intel-lts.git
git fetch --no-tags intel-lts 4.19/android_r
git checkout 75fe91933be2dd4b884ca83bd72e1e0f0d05cfc6
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 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 >>):
>> sound/soc/soc-core.c:761:27: warning: no previous prototype for function 'soc_find_component_locked' [-Wmissing-prototypes]
struct snd_soc_component *soc_find_component_locked(
^
sound/soc/soc-core.c:761:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
struct snd_soc_component *soc_find_component_locked(
^
static
sound/soc/soc-core.c:69:27: warning: unused variable 'dmi_blacklist' [-Wunused-const-variable]
static const char * const dmi_blacklist[] = {
^
2 warnings generated.
sound/soc/soc-core.c:2971: warning: Excess function parameter 'legacy_dai_naming' description in 'snd_soc_register_dais'
sound/soc/soc-core.c:3333: warning: Function parameter or member 'online' not described in 'snd_soc_card_change_online_state'
vim +/soc_find_component_locked +761 sound/soc/soc-core.c
750
751 /**
752 * soc_find_component_locked: soc_find_component with client lock acquired
753 *
754 * @of_node: of_node of the component to query.
755 * @name: name of the component to query.
756 *
757 * function to find out if a component is already registered with ASoC core.
758 *
759 * Returns component handle for success, else NULL error.
760 */
> 761 struct snd_soc_component *soc_find_component_locked(
762 const struct device_node *of_node, const char *name)
763 {
764 struct snd_soc_component *component = NULL;
765
766 mutex_lock(&client_mutex);
767 component = soc_find_component(of_node, name);
768 mutex_unlock(&client_mutex);
769 return component;
770 }
771 EXPORT_SYMBOL(soc_find_component_locked);
772
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months
Re: [PATCH v2 2/2] drm/i915: Use to_root_gt() to refer to the root tile
by kernel test robot
Hi Andi,
I love your patch! Yet something to improve:
[auto build test ERROR on drm-tip/drm-tip]
[also build test ERROR on next-20211118]
[cannot apply to drm-intel/for-linux-next drm-exynos/exynos-drm-next drm/drm-next tegra-drm/drm/tegra/for-next airlied/drm-next v5.16-rc2]
[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/Andi-Shyti/More-preparation-for-...
base: git://anongit.freedesktop.org/drm/drm-tip drm-tip
config: x86_64-randconfig-c007-20211121 (attached as .config)
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/068a75571292e317e35752c1b078605dd...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Andi-Shyti/More-preparation-for-multi-gt-patches/20211121-213526
git checkout 068a75571292e317e35752c1b078605dda122741
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from <built-in>:4:
In file included from drivers/gpu/drm/i915/display/intel_de.h:9:
>> drivers/gpu/drm/i915/i915_drv.h:1756:1: error: all paths through this function will call itself [-Werror,-Winfinite-recursion]
{
^
1 error generated.
--
In file included from drivers/gpu/drm/i915/pxp/intel_pxp_debugfs.c:12:
>> drivers/gpu/drm/i915/i915_drv.h:1756:1: error: all paths through this function will call itself [-Werror,-Winfinite-recursion]
{
^
drivers/gpu/drm/i915/pxp/intel_pxp_debugfs.c:59:6: error: no previous prototype for function 'intel_pxp_debugfs_register' [-Werror,-Wmissing-prototypes]
void intel_pxp_debugfs_register(struct intel_pxp *pxp, struct dentry *gt_root)
^
drivers/gpu/drm/i915/pxp/intel_pxp_debugfs.c:59:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void intel_pxp_debugfs_register(struct intel_pxp *pxp, struct dentry *gt_root)
^
static
2 errors generated.
--
In file included from drivers/gpu/drm/i915/pxp/intel_pxp_tee.c:11:
>> drivers/gpu/drm/i915/i915_drv.h:1756:1: error: all paths through this function will call itself [-Werror,-Winfinite-recursion]
{
^
>> drivers/gpu/drm/i915/pxp/intel_pxp_tee.c:19:35: error: no member named 'gt' in 'struct drm_i915_private'
return &kdev_to_i915(i915_kdev)->gt.pxp;
~~~~~~~~~~~~~~~~~~~~~~~ ^
2 errors generated.
--
In file included from drivers/gpu/drm/i915/selftests/igt_reset.c:12:
>> drivers/gpu/drm/i915/selftests/../i915_drv.h:1756:1: error: all paths through this function will call itself [-Werror,-Winfinite-recursion]
{
^
1 error generated.
vim +1756 drivers/gpu/drm/i915/i915_drv.h
1750
1751 /* Only valid when HAS_DISPLAY() is true */
1752 #define INTEL_DISPLAY_ENABLED(dev_priv) \
1753 (drm_WARN_ON(&(dev_priv)->drm, !HAS_DISPLAY(dev_priv)), !(dev_priv)->params.disable_display)
1754
1755 static inline struct intel_gt *to_root_gt(struct drm_i915_private *i915)
> 1756 {
1757 return to_root_gt(i915);
1758 }
1759
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months
Re: [PATCH] brcmfmac: Configure keep-alive packet on suspend
by kernel test robot
Hi Loic,
I love your patch! Perhaps something to improve:
[auto build test WARNING on kvalo-wireless-drivers-next/master]
[also build test WARNING on kvalo-wireless-drivers/master v5.16-rc2 next-20211118]
[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/Loic-Poulain/brcmfmac-Configure-...
base: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-ne... master
config: riscv-randconfig-r002-20211122 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project c133fb321f7ca6083ce15b6aa5bf89de6600e649)
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/13a67e895024e7cdbf0faa409fe3349cb...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Loic-Poulain/brcmfmac-Configure-keep-alive-packet-on-suspend/20211122-165520
git checkout 13a67e895024e7cdbf0faa409fe3349cb0d741fc
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 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/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c:3904:5: warning: no previous prototype for function 'brcmf_keepalive_start' [-Wmissing-prototypes]
int brcmf_keepalive_start(struct brcmf_if *ifp, unsigned int interval)
^
drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c:3904:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int brcmf_keepalive_start(struct brcmf_if *ifp, unsigned int interval)
^
static
1 warning generated.
vim +/brcmf_keepalive_start +3904 drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
3903
> 3904 int brcmf_keepalive_start(struct brcmf_if *ifp, unsigned int interval)
3905 {
3906 struct brcmf_mkeep_alive_pkt_le kalive = {0};
3907 int ret = 0;
3908
3909 /* Configure Null function/data keepalive */
3910 kalive.version = cpu_to_le16(1);
3911 kalive.period_msec = cpu_to_le16(interval * MSEC_PER_SEC);
3912 kalive.len_bytes = cpu_to_le16(0);
3913 kalive.keep_alive_id = cpu_to_le16(0);
3914
3915 ret = brcmf_fil_iovar_data_set(ifp, "mkeep_alive", &kalive, sizeof(kalive));
3916 if (ret)
3917 brcmf_err("keep-alive packet config failed, ret=%d\n", ret);
3918
3919 return ret;
3920 }
3921
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months
Re: [PATCH v0.9 3/6] sched/umcg: implement UMCG syscalls
by kernel test robot
Hi Peter,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on cb0e52b7748737b2cf6481fdd9b920ce7e1ebbdf]
url: https://github.com/0day-ci/linux/commits/Peter-Oskolkov/sched-mm-x86-uacc...
base: cb0e52b7748737b2cf6481fdd9b920ce7e1ebbdf
config: arm64-randconfig-r012-20211121 (attached as .config)
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://github.com/0day-ci/linux/commit/e455791cacec2b140558a717d2b8b07f5...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Peter-Oskolkov/sched-mm-x86-uaccess-implement-User-Managed-Concurrency-Groups/20211122-052209
git checkout e455791cacec2b140558a717d2b8b07f5561a251
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 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 >>):
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:34:1: note: expanded from here
__arm64_sys_recvmsg
^
kernel/sys_ni.c:257:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:263:1: warning: no previous prototype for function '__arm64_sys_mremap' [-Wmissing-prototypes]
COND_SYSCALL(mremap);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:39:1: note: expanded from here
__arm64_sys_mremap
^
kernel/sys_ni.c:263:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:266:1: warning: no previous prototype for function '__arm64_sys_add_key' [-Wmissing-prototypes]
COND_SYSCALL(add_key);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:40:1: note: expanded from here
__arm64_sys_add_key
^
kernel/sys_ni.c:266:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:267:1: warning: no previous prototype for function '__arm64_sys_request_key' [-Wmissing-prototypes]
COND_SYSCALL(request_key);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:41:1: note: expanded from here
__arm64_sys_request_key
^
kernel/sys_ni.c:267:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:268:1: warning: no previous prototype for function '__arm64_sys_keyctl' [-Wmissing-prototypes]
COND_SYSCALL(keyctl);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:42:1: note: expanded from here
__arm64_sys_keyctl
^
kernel/sys_ni.c:268:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:272:1: warning: no previous prototype for function '__arm64_sys_landlock_create_ruleset' [-Wmissing-prototypes]
COND_SYSCALL(landlock_create_ruleset);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:47:1: note: expanded from here
__arm64_sys_landlock_create_ruleset
^
kernel/sys_ni.c:272:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:273:1: warning: no previous prototype for function '__arm64_sys_landlock_add_rule' [-Wmissing-prototypes]
COND_SYSCALL(landlock_add_rule);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:48:1: note: expanded from here
__arm64_sys_landlock_add_rule
^
kernel/sys_ni.c:273:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:274:1: warning: no previous prototype for function '__arm64_sys_landlock_restrict_self' [-Wmissing-prototypes]
COND_SYSCALL(landlock_restrict_self);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:49:1: note: expanded from here
__arm64_sys_landlock_restrict_self
^
kernel/sys_ni.c:274:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
>> kernel/sys_ni.c:277:1: warning: no previous prototype for function '__arm64_sys_umcg_ctl' [-Wmissing-prototypes]
COND_SYSCALL(umcg_ctl);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:50:1: note: expanded from here
__arm64_sys_umcg_ctl
^
kernel/sys_ni.c:277:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
>> kernel/sys_ni.c:278:1: warning: no previous prototype for function '__arm64_sys_umcg_wait' [-Wmissing-prototypes]
COND_SYSCALL(umcg_wait);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:51:1: note: expanded from here
__arm64_sys_umcg_wait
^
kernel/sys_ni.c:278:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:283:1: warning: no previous prototype for function '__arm64_sys_fadvise64_64' [-Wmissing-prototypes]
COND_SYSCALL(fadvise64_64);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:52:1: note: expanded from here
__arm64_sys_fadvise64_64
^
kernel/sys_ni.c:283:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:286:1: warning: no previous prototype for function '__arm64_sys_swapon' [-Wmissing-prototypes]
COND_SYSCALL(swapon);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:53:1: note: expanded from here
__arm64_sys_swapon
^
kernel/sys_ni.c:286:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:287:1: warning: no previous prototype for function '__arm64_sys_swapoff' [-Wmissing-prototypes]
COND_SYSCALL(swapoff);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:54:1: note: expanded from here
__arm64_sys_swapoff
^
kernel/sys_ni.c:287:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:288:1: warning: no previous prototype for function '__arm64_sys_mprotect' [-Wmissing-prototypes]
COND_SYSCALL(mprotect);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:55:1: note: expanded from here
__arm64_sys_mprotect
^
kernel/sys_ni.c:288:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:289:1: warning: no previous prototype for function '__arm64_sys_msync' [-Wmissing-prototypes]
COND_SYSCALL(msync);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:56:1: note: expanded from here
__arm64_sys_msync
^
kernel/sys_ni.c:289:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:290:1: warning: no previous prototype for function '__arm64_sys_mlock' [-Wmissing-prototypes]
COND_SYSCALL(mlock);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:57:1: note: expanded from here
__arm64_sys_mlock
^
kernel/sys_ni.c:290:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
arch/arm64/include/asm/syscall_wrapper.h:76:13: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
kernel/sys_ni.c:291:1: warning: no previous prototype for function '__arm64_sys_munlock' [-Wmissing-prototypes]
COND_SYSCALL(munlock);
^
arch/arm64/include/asm/syscall_wrapper.h:76:25: note: expanded from macro 'COND_SYSCALL'
asmlinkage long __weak __arm64_sys_##name(const struct pt_regs *regs) \
^
<scratch space>:58:1: note: expanded from here
__arm64_sys_munlock
^
kernel/sys_ni.c:291:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
vim +/__arm64_sys_umcg_ctl +277 kernel/sys_ni.c
275
276 /* kernel/sched/umcg.c */
> 277 COND_SYSCALL(umcg_ctl);
> 278 COND_SYSCALL(umcg_wait);
279
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months