On 06/23/2017 11:29 AM, Holger Hoffstätte wrote:
On 06/23/17 16:32, Chris Mason wrote:
[..]
-static inline int calc_reclaim_items_nr(struct btrfs_fs_info *fs_info,
+static inline u64 calc_reclaim_items_nr(struct btrfs_fs_info *fs_info,
                                        u64 to_reclaim)
 {
        u64 bytes;
-       int nr;
+       u64 nr;

        bytes = btrfs_calc_trans_metadata_size(fs_info, 1);
        nr = (int)div64_u64(to_reclaim, bytes);
        ^^^^^^^^^^

Isn't this a bit odd? I can't even tell whether it matters, just randomly
noticed it because I genuinely dislike type casts..


Crud, yes that's broken.  Thanks!

-chris

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to