| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <202202131837.WoH0aQil-lkp@intel.com>
Date: Sun, 13 Feb 2022 18:49:59 +0800
From: kernel test robot <lkp@...el.com>
To: Christophe Leroy <christophe.leroy@...roup.eu>
Cc: kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
Michael Ellerman <mpe@...erman.id.au>
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@...el.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@...hat.com>
:::::: CC: Mike Snitzer <snitzer@...hat.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
Powered by blists - more mailing lists