fs/ntfs/mft.c:627:1: warning: the frame size of 2136 bytes is larger than 1024 bytes

From: kernel test robot
Date: Wed Jul 28 2021 - 09:00:39 EST

Next message:
Previous
Messages

Hi Christophe,

FYI, the error/warning
tree: config: powerpc64-randconfig-r026-20210728 compiler: powerpc-linux-gcc reproduce (this is a W=1 build):
wget # If you fix the issue, Reported-by: kernel
All warnings (new ones
fs/ntfs/mft.c: In >> fs/ntfs/mft.c:627:1: 627 | }
| ^
fs/ntfs/mft.c: In fs/ntfs/mft.c:839:1: 839 | }
| ^

vim +627 fs/ntfs/mft.c

^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ab62ef82ea49b8 Kees Cook ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds 78af34f03d33d2 Anton ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ab62ef82ea49b8 Kees Cook ab62ef82ea49b8 Kees Cook ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds 09cbfeaf1a5a67 Kirill ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds 09cbfeaf1a5a67 Kirill ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds 640ab98fb3629c Jens Axboe ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds 14ed109e3f3daa Guoqing Jiang ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds e74589ac250e46 Anton ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ca5de404ff036a Nick Piggin ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds 2a222ca992c35a Mike Christie ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds ^1da177e4c3f41 Linus Torvalds
:::::: The code at line :::::: 1da177e4c3f41524e886b7
:::::: TO: Linus Torvalds :::::: CC: Linus Torvalds
---
0-DAY CI Kernel Test Description: application/gzip David Sterba: "Re: [PATCH v2] btrfs: fix rw device counting in __btrfs_free_extra_devids" message: Shivank Garg: "/proc/pid/sched units" sorted by: [ date ] [ thread ] [ subject ] [ author ] still remains.
href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
75e82563369208bc371
600bd69ff77ff3a45a8 powerpc/fixmap: Fix FIX_EARLY_DEBUG_BASE when page size is 256k
(attached as .config)
(GCC) 10.3.0
href="https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross">https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=03fd42d458fb9cb69e712600bd69ff77ff3a45a8">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=03fd42d458fb9cb69e712600bd69ff77ff3a45a8
rel="nofollow" href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
linus master
03fd42d458fb9cb69e712600bd69ff77ff3a45a8
.config to linux build tree
COMPILER=gcc-10.3.0 make.cross ARCH=powerpc64
kindly add following tag as appropriate
test robot <lkp@xxxxxxxxx>
prefixed by >>):
function 'ntfs_sync_mft_mirror':
warning: the frame size of 2136 bytes is larger than 1024 bytes [-Wframe-larger-than=]
function 'write_mft_record_nolock':
warning: the frame size of 2136 bytes is larger than 1024 bytes [-Wframe-larger-than=]
2005-04-16 435
2005-04-16 436 /**
2005-04-16 437 * ntfs_sync_mft_mirror - synchronize an mft record to the mft mirror
2005-04-16 438 * @vol: ntfs volume on which the mft record to synchronize resides
2005-04-16 439 * @mft_no: mft record number of mft record to synchronize
2005-04-16 440 * @m: mapped, mst protected (extent) mft record to synchronize
2005-04-16 441 * @sync: if true, wait for i/o completion
2005-04-16 442 *
2005-04-16 443 * Write the mapped, mst protected (extent) mft record @m with mft record
2005-04-16 444 * number @mft_no to the mft mirror ($MFTMirr) of the ntfs volume @vol.
2005-04-16 445 *
2005-04-16 446 * On success return 0. On error return -errno and set the volume errors flag
2005-04-16 447 * in the ntfs volume @vol.
2005-04-16 448 *
2005-04-16 449 * NOTE: We always perform synchronous i/o and ignore the @sync parameter.
2005-04-16 450 *
2005-04-16 451 * TODO: If @sync is false, want to do truly asynchronous i/o, i.e. just
2005-04-16 452 * schedule i/o via ->writepage or do it via kntfsd or whatever.
2005-04-16 453 */
2005-04-16 454 int ntfs_sync_mft_mirror(ntfs_volume *vol, const unsigned long mft_no,
2005-04-16 455 MFT_RECORD *m, int sync)
2005-04-16 456 {
2005-04-16 457 struct page *page;
2005-04-16 458 unsigned int blocksize = vol->sb->s_blocksize;
2005-04-16 459 int max_bhs = vol->mft_record_size / blocksize;
2018-08-17 460 struct buffer_head *bhs[MAX_BHS];
2005-04-16 461 struct buffer_head *bh, *head;
2005-04-16 462 u8 *kmirr;
2005-04-16 463 runlist_element *rl;
2005-04-16 464 unsigned int block_start, block_end, m_start, m_end, page_ofs;
2005-04-16 465 int i_bhs, nr_bhs, err = 0;
Altaparmakov 2006-02-24 466 unsigned char blocksize_bits = vol->sb->s_blocksize_bits;
2005-04-16 467
2005-04-16 468 ntfs_debug("Entering for inode 0x%lx.", mft_no);
2005-04-16 469 BUG_ON(!max_bhs);
2018-08-17 470 if (WARN_ON(max_bhs > MAX_BHS))
2018-08-17 471 return -EINVAL;
2005-04-16 472 if (unlikely(!vol->mftmirr_ino)) {
2005-04-16 473 /* This could happen during umount... */
2005-04-16 474 err = ntfs_sync_mft_mirror_umount(vol, mft_no, m);
2005-04-16 475 if (likely(!err))
2005-04-16 476 return err;
2005-04-16 477 goto err_out;
2005-04-16 478 }
2005-04-16 479 /* Get the page containing the mirror copy of the mft record @m. */
2005-04-16 480 page = ntfs_map_page(vol->mftmirr_ino->i_mapping, mft_no >>
A. Shutemov 2016-04-01 481 (PAGE_SHIFT - vol->mft_record_size_bits));
2005-04-16 482 if (IS_ERR(page)) {
2005-04-16 483 ntfs_error(vol->sb, "Failed to map mft mirror page.");
2005-04-16 484 err = PTR_ERR(page);
2005-04-16 485 goto err_out;
2005-04-16 486 }
2005-04-16 487 lock_page(page);
2005-04-16 488 BUG_ON(!PageUptodate(page));
2005-04-16 489 ClearPageUptodate(page);
2005-04-16 490 /* Offset of the mft mirror record inside the page. */
A. Shutemov 2016-04-01 491 page_ofs = (mft_no << vol->mft_record_size_bits) & ~PAGE_MASK;
2005-04-16 492 /* The address in the page of the mirror copy of the mft record @m. */
2005-04-16 493 kmirr = page_address(page) + page_ofs;
2005-04-16 494 /* Copy the mst protected mft record to the mirror. */
2005-04-16 495 memcpy(kmirr, m, vol->mft_record_size);
2005-04-16 496 /* Create uptodate buffers if not present. */
2005-04-16 497 if (unlikely(!page_has_buffers(page))) {
2005-04-16 498 struct buffer_head *tail;
2005-04-16 499
2017-09-27 500 bh = head = alloc_page_buffers(page, blocksize, true);
2005-04-16 501 do {
2005-04-16 502 set_buffer_uptodate(bh);
2005-04-16 503 tail = bh;
2005-04-16 504 bh = bh->b_this_page;
2005-04-16 505 } while (bh);
2005-04-16 506 tail->b_this_page = head;
2020-06-01 507 attach_page_private(page, head);
2005-04-16 508 }
2005-04-16 509 bh = head = page_buffers(page);
2005-04-16 510 BUG_ON(!bh);
2005-04-16 511 rl = NULL;
2005-04-16 512 nr_bhs = 0;
2005-04-16 513 block_start = 0;
2005-04-16 514 m_start = kmirr - (u8*)page_address(page);
2005-04-16 515 m_end = m_start + vol->mft_record_size;
2005-04-16 516 do {
2005-04-16 517 block_end = block_start + blocksize;
2005-04-16 518 /* If the buffer is outside the mft record, skip it. */
2005-04-16 519 if (block_end <= m_start)
2005-04-16 520 continue;
2005-04-16 521 if (unlikely(block_start >= m_end))
2005-04-16 522 break;
2005-04-16 523 /* Need to map the buffer if it is not mapped already. */
2005-04-16 524 if (unlikely(!buffer_mapped(bh))) {
2005-04-16 525 VCN vcn;
2005-04-16 526 LCN lcn;
2005-04-16 527 unsigned int vcn_ofs;
2005-04-16 528
Altaparmakov 2005-08-16 529 bh->b_bdev = vol->sb->s_bdev;
2005-04-16 530 /* Obtain the vcn and offset of the current block. */
2005-04-16 531 vcn = ((VCN)mft_no << vol->mft_record_size_bits) +
2005-04-16 532 (block_start - m_start);
2005-04-16 533 vcn_ofs = vcn & vol->cluster_size_mask;
2005-04-16 534 vcn >>= vol->cluster_size_bits;
2005-04-16 535 if (!rl) {
2005-04-16 536 down_read(&NTFS_I(vol->mftmirr_ino)->
2005-04-16 537 runlist.lock);
2005-04-16 538 rl = NTFS_I(vol->mftmirr_ino)->runlist.rl;
2005-04-16 539 /*
2005-04-16 540 * $MFTMirr always has the whole of its runlist
2005-04-16 541 * in memory.
2005-04-16 542 */
2005-04-16 543 BUG_ON(!rl);
2005-04-16 544 }
2005-04-16 545 /* Seek to element containing target vcn. */
2005-04-16 546 while (rl->length && rl[1].vcn <= vcn)
2005-04-16 547 rl++;
2005-04-16 548 lcn = ntfs_rl_vcn_to_lcn(rl, vcn);
2005-04-16 549 /* For $MFTMirr, only lcn >= 0 is a successful remap. */
2005-04-16 550 if (likely(lcn >= 0)) {
2005-04-16 551 /* Setup buffer head to correct block. */
2005-04-16 552 bh->b_blocknr = ((lcn <<
2005-04-16 553 vol->cluster_size_bits) +
2005-04-16 554 vcn_ofs) >> blocksize_bits;
2005-04-16 555 set_buffer_mapped(bh);
2005-04-16 556 } else {
2005-04-16 557 bh->b_blocknr = -1;
2005-04-16 558 ntfs_error(vol->sb, "Cannot write mft mirror "
2005-04-16 559 "record 0x%lx because its "
2005-04-16 560 "location on disk could not "
2005-04-16 561 "be determined (error code "
2005-04-16 562 "%lli).", mft_no,
2005-04-16 563 (long long)lcn);
2005-04-16 564 err = -EIO;
2005-04-16 565 }
2005-04-16 566 }
2005-04-16 567 BUG_ON(!buffer_uptodate(bh));
2005-04-16 568 BUG_ON(!nr_bhs && (m_start != block_start));
2005-04-16 569 BUG_ON(nr_bhs >= max_bhs);
2005-04-16 570 bhs[nr_bhs++] = bh;
2005-04-16 571 BUG_ON((nr_bhs >= max_bhs) && (m_end != block_end));
2005-04-16 572 } while (block_start = block_end, (bh = bh->b_this_page) != head);
2005-04-16 573 if (unlikely(rl))
2005-04-16 574 up_read(&NTFS_I(vol->mftmirr_ino)->runlist.lock);
2005-04-16 575 if (likely(!err)) {
2005-04-16 576 /* Lock buffers and start synchronous write i/o on them. */
2005-04-16 577 for (i_bhs = 0; i_bhs < nr_bhs; i_bhs++) {
2005-04-16 578 struct buffer_head *tbh = bhs[i_bhs];
2005-04-16 579
2008-08-02 580 if (!trylock_buffer(tbh))
2005-04-16 581 BUG();
2005-04-16 582 BUG_ON(!buffer_uptodate(tbh));
2005-04-16 583 clear_buffer_dirty(tbh);
2005-04-16 584 get_bh(tbh);
2005-04-16 585 tbh->b_end_io = end_buffer_write_sync;
2016-06-05 586 submit_bh(REQ_OP_WRITE, 0, tbh);
2005-04-16 587 }
2005-04-16 588 /* Wait on i/o completion of buffers. */
2005-04-16 589 for (i_bhs = 0; i_bhs < nr_bhs; i_bhs++) {
2005-04-16 590 struct buffer_head *tbh = bhs[i_bhs];
2005-04-16 591
2005-04-16 592 wait_on_buffer(tbh);
2005-04-16 593 if (unlikely(!buffer_uptodate(tbh))) {
2005-04-16 594 err = -EIO;
2005-04-16 595 /*
2005-04-16 596 * Set the buffer uptodate so the page and
2005-04-16 597 * buffer states do not become out of sync.
2005-04-16 598 */
2005-04-16 599 set_buffer_uptodate(tbh);
2005-04-16 600 }
2005-04-16 601 }
2005-04-16 602 } else /* if (unlikely(err)) */ {
2005-04-16 603 /* Clean the buffers. */
2005-04-16 604 for (i_bhs = 0; i_bhs < nr_bhs; i_bhs++)
2005-04-16 605 clear_buffer_dirty(bhs[i_bhs]);
2005-04-16 606 }
2005-04-16 607 /* Current state: all buffers are clean, unlocked, and uptodate. */
2005-04-16 608 /* Remove the mst protection fixups again. */
2005-04-16 609 post_write_mst_fixup((NTFS_RECORD*)kmirr);
2005-04-16 610 flush_dcache_page(page);
2005-04-16 611 SetPageUptodate(page);
2005-04-16 612 unlock_page(page);
2005-04-16 613 ntfs_unmap_page(page);
2005-04-16 614 if (likely(!err)) {
2005-04-16 615 ntfs_debug("Done.");
2005-04-16 616 } else {
2005-04-16 617 ntfs_error(vol->sb, "I/O error while writing mft mirror "
2005-04-16 618 "record 0x%lx!", mft_no);
2005-04-16 619 err_out:
2005-04-16 620 ntfs_error(vol->sb, "Failed to synchronize $MFTMirr (error "
2005-04-16 621 "code %i). Volume will be left marked dirty "
2005-04-16 622 "on umount. Run ntfsfix on the partition "
2005-04-16 623 "after umounting to correct this.", -err);
2005-04-16 624 NVolSetErrors(vol);
2005-04-16 625 }
2005-04-16 626 return err;
2005-04-16 @627 }
2005-04-16 628
627 was first introduced by commit
f1b8a0c1fc7321cac2 Linux-2.6.12-rc2
<torvalds@xxxxxxxxxxxxxxx>
<torvalds@xxxxxxxxxxxxxxx>
Service, Intel Corporation
s://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx">https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx
>.config.gz

Next message: David Sterba: "Re: [PATCH v2] btrfs: fix rw device counting in __btrfs_free_extra_devids"
Previous message: Shivank Garg: "/proc/pid/sched units"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]