Messages in this thread | | | Date | Sun, 13 Feb 2022 18:49:59 +0800 | From | kernel test robot <> | Subject | drivers/md/dm-integrity.c:2567:1: warning: the frame size of 1040 bytes is larger than 1024 bytes |
| |
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: b81b1829e7e39f6cebdf6e4d5484eacbceda8554 commit: db87a7199229b75c9996bf78117eceb81854fce2 powerpc/bug: Remove specific powerpc BUG_ON() and WARN_ON() on PPC32 date: 6 months ago config: powerpc-randconfig-r035-20220213 (https://download.01.org/0day-ci/archive/20220213/202202131837.WoH0aQil-lkp@intel.com/config) compiler: powerpc-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/torvalds/linux.git/commit/?id=db87a7199229b75c9996bf78117eceb81854fce2 git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout db87a7199229b75c9996bf78117eceb81854fce2 # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=powerpc SHELL=/bin/bash drivers/md/ lib/zstd/
If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/md/dm-integrity.c: In function 'do_journal_write': >> drivers/md/dm-integrity.c:2567:1: warning: the frame size of 1040 bytes is larger than 1024 bytes [-Wframe-larger-than=] 2567 | } | ^
vim +2567 drivers/md/dm-integrity.c
9d609f85b7eb96 Mikulas Patocka 2017-04-18 2425 7eada909bfd7ac Mikulas Patocka 2017-01-04 2426 static void do_journal_write(struct dm_integrity_c *ic, unsigned write_start, 7eada909bfd7ac Mikulas Patocka 2017-01-04 2427 unsigned write_sections, bool from_replay) 7eada909bfd7ac Mikulas Patocka 2017-01-04 2428 { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2429 unsigned i, j, n; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2430 struct journal_completion comp; a7c3e62bdc71d3 Mikulas Patocka 2017-07-19 2431 struct blk_plug plug; a7c3e62bdc71d3 Mikulas Patocka 2017-07-19 2432 a7c3e62bdc71d3 Mikulas Patocka 2017-07-19 2433 blk_start_plug(&plug); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2434 7eada909bfd7ac Mikulas Patocka 2017-01-04 2435 comp.ic = ic; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2436 comp.in_flight = (atomic_t)ATOMIC_INIT(1); b5e8ad92c3ac0b Arnd Bergmann 2017-08-15 2437 init_completion(&comp.comp); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2438 7eada909bfd7ac Mikulas Patocka 2017-01-04 2439 i = write_start; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2440 for (n = 0; n < write_sections; n++, i++, wraparound_section(ic, &i)) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2441 #ifndef INTERNAL_VERIFY 7eada909bfd7ac Mikulas Patocka 2017-01-04 2442 if (unlikely(from_replay)) 7eada909bfd7ac Mikulas Patocka 2017-01-04 2443 #endif 7eada909bfd7ac Mikulas Patocka 2017-01-04 2444 rw_section_mac(ic, i, false); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2445 for (j = 0; j < ic->journal_section_entries; j++) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2446 struct journal_entry *je = access_journal_entry(ic, i, j); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2447 sector_t sec, area, offset; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2448 unsigned k, l, next_loop; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2449 sector_t metadata_block; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2450 unsigned metadata_offset; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2451 struct journal_io *io; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2452 7eada909bfd7ac Mikulas Patocka 2017-01-04 2453 if (journal_entry_is_unused(je)) 7eada909bfd7ac Mikulas Patocka 2017-01-04 2454 continue; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2455 BUG_ON(unlikely(journal_entry_is_inprogress(je)) && !from_replay); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2456 sec = journal_entry_get_sector(je); 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2457 if (unlikely(from_replay)) { 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2458 if (unlikely(sec & (unsigned)(ic->sectors_per_block - 1))) { 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2459 dm_integrity_io_error(ic, "invalid sector in journal", -EIO); 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2460 sec &= ~(sector_t)(ic->sectors_per_block - 1); 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2461 } 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2462 } f6f72f32c22c0b Mikulas Patocka 2020-03-22 2463 if (unlikely(sec >= ic->provided_data_sectors)) f6f72f32c22c0b Mikulas Patocka 2020-03-22 2464 continue; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2465 get_area_and_offset(ic, sec, &area, &offset); 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2466 restore_last_bytes(ic, access_journal_data(ic, i, j), je); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2467 for (k = j + 1; k < ic->journal_section_entries; k++) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2468 struct journal_entry *je2 = access_journal_entry(ic, i, k); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2469 sector_t sec2, area2, offset2; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2470 if (journal_entry_is_unused(je2)) 7eada909bfd7ac Mikulas Patocka 2017-01-04 2471 break; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2472 BUG_ON(unlikely(journal_entry_is_inprogress(je2)) && !from_replay); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2473 sec2 = journal_entry_get_sector(je2); f6f72f32c22c0b Mikulas Patocka 2020-03-22 2474 if (unlikely(sec2 >= ic->provided_data_sectors)) f6f72f32c22c0b Mikulas Patocka 2020-03-22 2475 break; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2476 get_area_and_offset(ic, sec2, &area2, &offset2); 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2477 if (area2 != area || offset2 != offset + ((k - j) << ic->sb->log2_sectors_per_block)) 7eada909bfd7ac Mikulas Patocka 2017-01-04 2478 break; 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2479 restore_last_bytes(ic, access_journal_data(ic, i, k), je2); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2480 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2481 next_loop = k - 1; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2482 6f1c819c219f78 Kent Overstreet 2018-05-20 2483 io = mempool_alloc(&ic->journal_io_mempool, GFP_NOIO); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2484 io->comp = ∁ 7eada909bfd7ac Mikulas Patocka 2017-01-04 2485 io->range.logical_sector = sec; 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2486 io->range.n_sectors = (k - j) << ic->sb->log2_sectors_per_block; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2487 7eada909bfd7ac Mikulas Patocka 2017-01-04 2488 spin_lock_irq(&ic->endio_wait.lock); 8b3bbd490d880d Mikulas Patocka 2019-04-29 2489 add_new_range_and_wait(ic, &io->range); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2490 7eada909bfd7ac Mikulas Patocka 2017-01-04 2491 if (likely(!from_replay)) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2492 struct journal_node *section_node = &ic->journal_tree[i * ic->journal_section_entries]; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2493 7eada909bfd7ac Mikulas Patocka 2017-01-04 2494 /* don't write if there is newer committed sector */ 7eada909bfd7ac Mikulas Patocka 2017-01-04 2495 while (j < k && find_newer_committed_node(ic, §ion_node[j])) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2496 struct journal_entry *je2 = access_journal_entry(ic, i, j); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2497 7eada909bfd7ac Mikulas Patocka 2017-01-04 2498 journal_entry_set_unused(je2); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2499 remove_journal_node(ic, §ion_node[j]); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2500 j++; 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2501 sec += ic->sectors_per_block; 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2502 offset += ic->sectors_per_block; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2503 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2504 while (j < k && find_newer_committed_node(ic, §ion_node[k - 1])) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2505 struct journal_entry *je2 = access_journal_entry(ic, i, k - 1); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2506 7eada909bfd7ac Mikulas Patocka 2017-01-04 2507 journal_entry_set_unused(je2); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2508 remove_journal_node(ic, §ion_node[k - 1]); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2509 k--; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2510 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2511 if (j == k) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2512 remove_range_unlocked(ic, &io->range); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2513 spin_unlock_irq(&ic->endio_wait.lock); 6f1c819c219f78 Kent Overstreet 2018-05-20 2514 mempool_free(io, &ic->journal_io_mempool); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2515 goto skip_io; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2516 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2517 for (l = j; l < k; l++) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2518 remove_journal_node(ic, §ion_node[l]); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2519 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2520 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2521 spin_unlock_irq(&ic->endio_wait.lock); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2522 7eada909bfd7ac Mikulas Patocka 2017-01-04 2523 metadata_block = get_metadata_sector_and_offset(ic, area, offset, &metadata_offset); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2524 for (l = j; l < k; l++) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2525 int r; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2526 struct journal_entry *je2 = access_journal_entry(ic, i, l); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2527 7eada909bfd7ac Mikulas Patocka 2017-01-04 2528 if ( 7eada909bfd7ac Mikulas Patocka 2017-01-04 2529 #ifndef INTERNAL_VERIFY 7eada909bfd7ac Mikulas Patocka 2017-01-04 2530 unlikely(from_replay) && 7eada909bfd7ac Mikulas Patocka 2017-01-04 2531 #endif 7eada909bfd7ac Mikulas Patocka 2017-01-04 2532 ic->internal_hash) { 6d39a1241ebc3b Kees Cook 2018-08-07 2533 char test_tag[max_t(size_t, HASH_MAX_DIGESTSIZE, MAX_TAG_SIZE)]; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2534 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2535 integrity_sector_checksum(ic, sec + ((l - j) << ic->sb->log2_sectors_per_block), 7eada909bfd7ac Mikulas Patocka 2017-01-04 2536 (char *)access_journal_data(ic, i, l), test_tag); 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2537 if (unlikely(memcmp(test_tag, journal_entry_tag(ic, je2), ic->tag_size))) 7eada909bfd7ac Mikulas Patocka 2017-01-04 2538 dm_integrity_io_error(ic, "tag mismatch when replaying journal", -EILSEQ); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2539 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2540 7eada909bfd7ac Mikulas Patocka 2017-01-04 2541 journal_entry_set_unused(je2); 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2542 r = dm_integrity_rw_tag(ic, journal_entry_tag(ic, je2), &metadata_block, &metadata_offset, 7eada909bfd7ac Mikulas Patocka 2017-01-04 2543 ic->tag_size, TAG_WRITE); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2544 if (unlikely(r)) { 7eada909bfd7ac Mikulas Patocka 2017-01-04 2545 dm_integrity_io_error(ic, "reading tags", r); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2546 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2547 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2548 7eada909bfd7ac Mikulas Patocka 2017-01-04 2549 atomic_inc(&comp.in_flight); 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2550 copy_from_journal(ic, i, j << ic->sb->log2_sectors_per_block, 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2551 (k - j) << ic->sb->log2_sectors_per_block, 9d609f85b7eb96 Mikulas Patocka 2017-04-18 2552 get_data_sector(ic, area, offset), 7eada909bfd7ac Mikulas Patocka 2017-01-04 2553 complete_copy_from_journal, io); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2554 skip_io: 7eada909bfd7ac Mikulas Patocka 2017-01-04 2555 j = next_loop; 7eada909bfd7ac Mikulas Patocka 2017-01-04 2556 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2557 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2558 7eada909bfd7ac Mikulas Patocka 2017-01-04 2559 dm_bufio_write_dirty_buffers_async(ic->bufio); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2560 a7c3e62bdc71d3 Mikulas Patocka 2017-07-19 2561 blk_finish_plug(&plug); a7c3e62bdc71d3 Mikulas Patocka 2017-07-19 2562 7eada909bfd7ac Mikulas Patocka 2017-01-04 2563 complete_journal_op(&comp); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2564 wait_for_completion_io(&comp.comp); 7eada909bfd7ac Mikulas Patocka 2017-01-04 2565 9b5948267adc9e Mikulas Patocka 2021-01-08 2566 dm_integrity_flush_buffers(ic, true); 7eada909bfd7ac Mikulas Patocka 2017-01-04 @2567 } 7eada909bfd7ac Mikulas Patocka 2017-01-04 2568
:::::: The code at line 2567 was first introduced by commit :::::: 7eada909bfd7ac90a4522e56aa3179d1fd68cd14 dm: add integrity target
:::::: TO: Mikulas Patocka <mpatocka@redhat.com> :::::: CC: Mike Snitzer <snitzer@redhat.com>
--- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
| |