> @@ -50,7 +50,7 @@ static void adfs_write_failed(struct address_space *mapping, loff_t to) > struct inode *inode = mapping->host; > > if (to > inode->i_size) > - truncate_pagecache(inode, to, inode->i_size); > + truncate_pagecache(inode, inode->i_size); > } All these _write_failed() boiler plate functions still technically use 'to', so I *guess* they can stay :). > @@ -226,7 +226,7 @@ int btrfs_truncate_free_space_cache(struct btrfs_root *root, > > oldsize = i_size_read(inode); > btrfs_i_size_write(inode, 0); > - truncate_pagecache(inode, oldsize, 0); > + truncate_pagecache(inode, 0); But after this change 'oldsize' is set but never used. That'll generate a warning on some versions of gcc. Can you redo the patch with an eye to removing unused arguments and variables further up the call stack? - z -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>