diff mbox

[v2,2/2] btrfs: declare rsv_count as unsigned int instead of int

Message ID 20150922210007.GA27786@gmail.com (mailing list archive)
State Accepted
Headers show

Commit Message

Alexandru Moise Sept. 22, 2015, 9 p.m. UTC
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(-)

Comments

David Sterba Sept. 23, 2015, 12:58 p.m. UTC | #1
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 mbox

Patch

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;