Message ID | 20150922210007.GA27786@gmail.com (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
On Tue, Sep 22, 2015 at 09:00:07PM +0000, Alexandru Moise wrote: > rsv_count ultimately gets passed to start_transaction() which > now takes an unsigned int as its num_items parameter. > The value of rsv_count should always be positive so declare it > as being unsigned. > > Signed-off-by: Alexandru Moise <00moses.alexander00@gmail.com> Reviewed-by: David Sterba <dsterba@suse.com> -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/fs/btrfs/file.c b/fs/btrfs/file.c index b823fac..6af5103 100644 --- a/fs/btrfs/file.c +++ b/fs/btrfs/file.c @@ -2266,7 +2266,7 @@ static int btrfs_punch_hole(struct inode *inode, loff_t offset, loff_t len) u64 drop_end; int ret = 0; int err = 0; - int rsv_count; + unsigned int rsv_count; bool same_page; bool no_holes = btrfs_fs_incompat(root->fs_info, NO_HOLES); u64 ino_size;
rsv_count ultimately gets passed to start_transaction() which now takes an unsigned int as its num_items parameter. The value of rsv_count should always be positive so declare it as being unsigned. Signed-off-by: Alexandru Moise <00moses.alexander00@gmail.com> --- v2: followed dave's suggestions of all num_items equivalents to be unsigned ints. fs/btrfs/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)