public inbox for [email protected]
 help / color / mirror / Atom feed
* [ammarfaizi2-block:dhowells/linux-fs/netfs-linked-list 56/61] fs/afs/inode.c:927:23: warning: variable 'i_size' is uninitialized when used here
@ 2022-07-02  7:31 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2022-07-02  7:31 UTC (permalink / raw)
  To: David Howells; +Cc: llvm, kbuild-all, GNU/Weeb Mailing List, linux-kernel

tree:   https://github.com/ammarfaizi2/linux-block dhowells/linux-fs/netfs-linked-list
head:   ce4670495468b797b0c5927fcb661bc0da48b9ab
commit: b0af788660145c19754695953b240c9eaa311df8 [56/61] netfs: Implement truncation
config: riscv-randconfig-r001-20220629 (https://download.01.org/0day-ci/archive/20220702/[email protected]/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project bcd153485ebf07fe79e2b843ed5f1cb74997df1b)
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
        # install riscv cross compiling tool for clang build
        # apt-get install binutils-riscv-linux-gnu
        # https://github.com/ammarfaizi2/linux-block/commit/b0af788660145c19754695953b240c9eaa311df8
        git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block
        git fetch --no-tags ammarfaizi2-block dhowells/linux-fs/netfs-linked-list
        git checkout b0af788660145c19754695953b240c9eaa311df8
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash fs/afs/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <[email protected]>

All warnings (new ones prefixed by >>):

>> fs/afs/inode.c:927:23: warning: variable 'i_size' is uninitialized when used here [-Wuninitialized]
                       attr->ia_size < i_size &&
                                       ^~~~~~
   fs/afs/inode.c:892:15: note: initialize the variable 'i_size' to silence this warning
           loff_t i_size;
                        ^
                         = 0
   1 warning generated.


vim +/i_size +927 fs/afs/inode.c

e49c7b2f6de7ff David Howells     2020-04-10  878  
31143d5d515ece David Howells     2007-05-09  879  /*
31143d5d515ece David Howells     2007-05-09  880   * set the attributes of an inode
31143d5d515ece David Howells     2007-05-09  881   */
549c7297717c32 Christian Brauner 2021-01-21  882  int afs_setattr(struct user_namespace *mnt_userns, struct dentry *dentry,
549c7297717c32 Christian Brauner 2021-01-21  883  		struct iattr *attr)
31143d5d515ece David Howells     2007-05-09  884  {
0770bd4187c555 David Howells     2020-07-22  885  	const unsigned int supported =
0770bd4187c555 David Howells     2020-07-22  886  		ATTR_SIZE | ATTR_MODE | ATTR_UID | ATTR_GID |
0770bd4187c555 David Howells     2020-07-22  887  		ATTR_MTIME | ATTR_MTIME_SET | ATTR_TIMES_SET | ATTR_TOUCH;
b0af788660145c David Howells     2021-09-30  888  	struct netfs_io_request *treq = NULL;
e49c7b2f6de7ff David Howells     2020-04-10  889  	struct afs_operation *op;
2b0143b5c986be David Howells     2015-03-17  890  	struct afs_vnode *vnode = AFS_FS_I(d_inode(dentry));
874c8ca1e60b2c David Howells     2022-06-09  891  	struct inode *inode = &vnode->netfs.inode;
0770bd4187c555 David Howells     2020-07-22  892  	loff_t i_size;
793fe82ee33aab David Howells     2020-06-12  893  	int ret;
31143d5d515ece David Howells     2007-05-09  894  
3b6492df4153b8 David Howells     2018-10-20  895  	_enter("{%llx:%llu},{n=%pd},%x",
a455589f181e60 Al Viro           2014-10-21  896  	       vnode->fid.vid, vnode->fid.vnode, dentry,
31143d5d515ece David Howells     2007-05-09  897  	       attr->ia_valid);
31143d5d515ece David Howells     2007-05-09  898  
0770bd4187c555 David Howells     2020-07-22  899  	if (!(attr->ia_valid & supported)) {
31143d5d515ece David Howells     2007-05-09  900  		_leave(" = 0 [unsupported]");
31143d5d515ece David Howells     2007-05-09  901  		return 0;
31143d5d515ece David Howells     2007-05-09  902  	}
31143d5d515ece David Howells     2007-05-09  903  
523d27cda149cf David Howells     2020-02-06  904  	fscache_use_cookie(afs_vnode_cache(vnode), true);
523d27cda149cf David Howells     2020-02-06  905  
b0af788660145c David Howells     2021-09-30  906  	treq = netfs_prepare_to_truncate(dentry, attr);
b0af788660145c David Howells     2021-09-30  907  	if (IS_ERR(treq))
b0af788660145c David Howells     2021-09-30  908  		return PTR_ERR(treq);
b0af788660145c David Howells     2021-09-30  909  
ec0fa0b659144d David Howells     2020-10-07  910  	/* Prevent any new writebacks from starting whilst we do this. */
ec0fa0b659144d David Howells     2020-10-07  911  	down_write(&vnode->validate_lock);
ec0fa0b659144d David Howells     2020-10-07  912  
0770bd4187c555 David Howells     2020-07-22  913  	if ((attr->ia_valid & ATTR_SIZE) && S_ISREG(inode->i_mode)) {
0770bd4187c555 David Howells     2020-07-22  914  		loff_t size = attr->ia_size;
0770bd4187c555 David Howells     2020-07-22  915  
0770bd4187c555 David Howells     2020-07-22  916  		/* Wait for any outstanding writes to the server to complete */
0770bd4187c555 David Howells     2020-07-22  917  		loff_t from = min(size, i_size);
0770bd4187c555 David Howells     2020-07-22  918  		loff_t to = max(size, i_size);
0770bd4187c555 David Howells     2020-07-22  919  		ret = filemap_fdatawait_range(inode->i_mapping, from, to);
0770bd4187c555 David Howells     2020-07-22  920  		if (ret < 0)
0770bd4187c555 David Howells     2020-07-22  921  			goto out_unlock;
0770bd4187c555 David Howells     2020-07-22  922  
0770bd4187c555 David Howells     2020-07-22  923  		/* Don't talk to the server if we're just shortening in-memory
0770bd4187c555 David Howells     2020-07-22  924  		 * writes that haven't gone to the server yet.
0770bd4187c555 David Howells     2020-07-22  925  		 */
0770bd4187c555 David Howells     2020-07-22  926  		if (!(attr->ia_valid & (supported & ~ATTR_SIZE & ~ATTR_MTIME)) &&
0770bd4187c555 David Howells     2020-07-22 @927  		    attr->ia_size < i_size &&

:::::: The code at line 927 was first introduced by commit
:::::: 0770bd4187c555e6df087f7abc252eeacb0842ec afs: Skip truncation on the server of data we haven't written yet

:::::: TO: David Howells <[email protected]>
:::::: CC: David Howells <[email protected]>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-07-02  7:32 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-07-02  7:31 [ammarfaizi2-block:dhowells/linux-fs/netfs-linked-list 56/61] fs/afs/inode.c:927:23: warning: variable 'i_size' is uninitialized when used here kernel test robot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox