9:49 p.m.
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-lk...)
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...
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(a)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(a)redhat.com>
:::::: CC: Mike Snitzer <snitzer(a)redhat.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
12:12 a.m.
On Sun, Feb 13, 2022 at 06:49:59PM +0800, kernel test robot wrote:
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-lk...)
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...
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(a)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=]
Sorry for the false positive, kindly ignore this report.
> 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(a)redhat.com>
> :::::: CC: Mike Snitzer <snitzer(a)redhat.com>
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
> _______________________________________________
> kbuild-all mailing list -- kbuild-all(a)lists.01.org
> To unsubscribe send an email to kbuild-all-leave(a)lists.01.org
219
days inactive
219
days old
1 comments
2 participants
participants (2)
-
kernel test robot -
Philip Li