From: kbuild test robot <lkp@intel.com>
To: Kees Cook <keescook@chromium.org>
Cc: kbuild-all@01.org, linux-kernel@vger.kernel.org,
Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: fs/ext4/inode.c:83:1: warning: the frame size of 1712 bytes is larger than 1024 bytes
Date: Wed, 24 Apr 2019 18:14:45 +0800 [thread overview]
Message-ID: <201904241837.YG641LRG%lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 4271 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 12a54b150fb5b6c2f3da932dc0e665355f8a5a48
commit: 81a56f6dcd20325607d6008f4bb560c96f4c821a gcc-plugins: structleak: Generalize to all variable types
date: 7 weeks ago
config: arm64-tizen_tm2_defconfig (attached as .config)
compiler: aarch64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout 81a56f6dcd20325607d6008f4bb560c96f4c821a
# save the attached .config to linux build tree
GCC_VERSION=7.2.0 make.cross ARCH=arm64
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
fs/ext4/inode.c: In function 'ext4_inode_csum.isra.0':
>> fs/ext4/inode.c:83:1: warning: the frame size of 1712 bytes is larger than 1024 bytes [-Wframe-larger-than=]
}
^
--
fs/ext4/xattr.c: In function 'ext4_xattr_block_csum.isra.0':
>> fs/ext4/xattr.c:147:1: warning: the frame size of 1184 bytes is larger than 1024 bytes [-Wframe-larger-than=]
}
^
--
fs/ext4/super.c: In function 'ext4_group_desc_csum.isra.92':
>> fs/ext4/super.c:2381:1: warning: the frame size of 1184 bytes is larger than 1024 bytes [-Wframe-larger-than=]
}
^
vim +83 fs/ext4/inode.c
a1d6cc56 Aneesh Kumar K.V 2008-08-19 52
814525f4 Darrick J. Wong 2012-04-29 53 static __u32 ext4_inode_csum(struct inode *inode, struct ext4_inode *raw,
814525f4 Darrick J. Wong 2012-04-29 54 struct ext4_inode_info *ei)
814525f4 Darrick J. Wong 2012-04-29 55 {
814525f4 Darrick J. Wong 2012-04-29 56 struct ext4_sb_info *sbi = EXT4_SB(inode->i_sb);
814525f4 Darrick J. Wong 2012-04-29 57 __u32 csum;
b47820ed Daeho Jeong 2016-07-03 58 __u16 dummy_csum = 0;
b47820ed Daeho Jeong 2016-07-03 59 int offset = offsetof(struct ext4_inode, i_checksum_lo);
b47820ed Daeho Jeong 2016-07-03 60 unsigned int csum_size = sizeof(dummy_csum);
814525f4 Darrick J. Wong 2012-04-29 61
b47820ed Daeho Jeong 2016-07-03 62 csum = ext4_chksum(sbi, ei->i_csum_seed, (__u8 *)raw, offset);
b47820ed Daeho Jeong 2016-07-03 63 csum = ext4_chksum(sbi, csum, (__u8 *)&dummy_csum, csum_size);
b47820ed Daeho Jeong 2016-07-03 64 offset += csum_size;
b47820ed Daeho Jeong 2016-07-03 65 csum = ext4_chksum(sbi, csum, (__u8 *)raw + offset,
b47820ed Daeho Jeong 2016-07-03 66 EXT4_GOOD_OLD_INODE_SIZE - offset);
814525f4 Darrick J. Wong 2012-04-29 67
b47820ed Daeho Jeong 2016-07-03 68 if (EXT4_INODE_SIZE(inode->i_sb) > EXT4_GOOD_OLD_INODE_SIZE) {
b47820ed Daeho Jeong 2016-07-03 69 offset = offsetof(struct ext4_inode, i_checksum_hi);
b47820ed Daeho Jeong 2016-07-03 70 csum = ext4_chksum(sbi, csum, (__u8 *)raw +
b47820ed Daeho Jeong 2016-07-03 71 EXT4_GOOD_OLD_INODE_SIZE,
b47820ed Daeho Jeong 2016-07-03 72 offset - EXT4_GOOD_OLD_INODE_SIZE);
b47820ed Daeho Jeong 2016-07-03 73 if (EXT4_FITS_IN_INODE(raw, ei, i_checksum_hi)) {
b47820ed Daeho Jeong 2016-07-03 74 csum = ext4_chksum(sbi, csum, (__u8 *)&dummy_csum,
b47820ed Daeho Jeong 2016-07-03 75 csum_size);
b47820ed Daeho Jeong 2016-07-03 76 offset += csum_size;
b47820ed Daeho Jeong 2016-07-03 77 }
05ac5aa1 Daeho Jeong 2016-12-01 78 csum = ext4_chksum(sbi, csum, (__u8 *)raw + offset,
05ac5aa1 Daeho Jeong 2016-12-01 79 EXT4_INODE_SIZE(inode->i_sb) - offset);
814525f4 Darrick J. Wong 2012-04-29 80 }
814525f4 Darrick J. Wong 2012-04-29 81
814525f4 Darrick J. Wong 2012-04-29 82 return csum;
814525f4 Darrick J. Wong 2012-04-29 @83 }
814525f4 Darrick J. Wong 2012-04-29 84
:::::: The code at line 83 was first introduced by commit
:::::: 814525f4df50a196464ce2c7abe91f693203060f ext4: calculate and verify inode checksums
:::::: TO: Darrick J. Wong <djwong@us.ibm.com>
:::::: CC: Theodore Ts'o <tytso@mit.edu>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 39068 bytes --]
reply other threads:[~2019-04-24 10:15 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201904241837.YG641LRG%lkp@intel.com \
--to=lkp@intel.com \
--cc=ard.biesheuvel@linaro.org \
--cc=kbuild-all@01.org \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome