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.. -h -- 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