linux-ext4 - Re: [PATCH 4/6] ext4: Provide ext4_issue

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <201510142156.s6DA4V23%fengguang.wu@intel.com>
Date:	Wed, 14 Oct 2015 21:18:38 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Jan Kara <jack@...e.com>
Cc:	kbuild-all@...org, linux-ext4@...r.kernel.org,
	Dan Williams <dan.j.williams@...el.com>,
	Ross Zwisler <ross.zwisler@...ux.intel.com>,
	Matthew Wilcox <matthew.r.wilcox@...el.com>,
	Dave Chinner <david@...morbit.com>, Jan Kara <jack@...e.com>
Subject: Re: [PATCH 4/6] ext4: Provide ext4_issue_zeroout()

Hi Jan,

[auto build test WARNING on v4.3-rc5 -- if it's inappropriate base, please suggest rules for selecting the more suitable base]

url:    https://github.com/0day-ci/linux/commits/Jan-Kara/ext4-Punch-hole-and-DAX-fixes/20151014-193637
reproduce:
        # apt-get install sparse
        make ARCH=x86_64 allmodconfig
        make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

>> fs/ext4/extents.c:3124:44: sparse: incorrect type in argument 2 (different base types)
   fs/ext4/extents.c:3124:44:    expected unsigned int [unsigned] [usertype] lblk
   fs/ext4/extents.c:3124:44:    got restricted __le32 [usertype] ee_block

vim +3124 fs/ext4/extents.c

  3108	
  3109		if (ee_len == 0)
  3110			return 0;
  3111	
  3112		return ext4_es_insert_extent(inode, ee_block, ee_len, ee_pblock,
  3113					     EXTENT_STATUS_WRITTEN);
  3114	}
  3115	
  3116	/* FIXME!! we need to try to merge to left or right after zero-out  */
  3117	static int ext4_ext_zeroout(struct inode *inode, struct ext4_extent *ex)
  3118	{
  3119		ext4_fsblk_t ee_pblock;
  3120		unsigned int ee_len;
  3121	
  3122		ee_len    = ext4_ext_get_actual_len(ex);
  3123		ee_pblock = ext4_ext_pblock(ex);
> 3124		return ext4_issue_zeroout(inode, ex->ee_block, ee_pblock, ee_len);
  3125	}
  3126	
  3127	/*
  3128	 * ext4_split_extent_at() splits an extent at given block.
  3129	 *
  3130	 * @handle: the journal handle
  3131	 * @inode: the file inode
  3132	 * @path: the path to the extent

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html