On 12/5/16 10:29 AM, David Sterba wrote:
> On Fri, Dec 02, 2016 at 12:07:30AM -0500, jeffm@xxxxxxxx wrote:
>> -static inline u64 btrfs_calc_trans_metadata_size(struct btrfs_root *root,
>> +static inline u64 btrfs_calc_trans_metadata_size(struct btrfs_fs_info *fs_info,
>> unsigned num_items)
>> {
>> - return root->fs_info->nodesize * BTRFS_MAX_LEVEL * 2 * num_items;
>> + return fs_info->nodesize * BTRFS_MAX_LEVEL * 2 * num_items;
>
> Is there a missing patch that moves 'nodesize' to fs_info? The patch has
> a minor conflict in the original line where it's just 'root->nodesize',
> but thre are many compilation faiures due to lack of fs_info::nodesize.
Yeah, it looks like the list dropped it. It shows up in the thread
posted to me.
I've pushed the series as the for-4.10/misc-4.10 branch in
my repo at git://git.kernel.org/pub/scm/linux/kernel/git/jeffm/linux-btrfs.git
It also contains the fix Omar suggested in his review.
-Jeff
--
Jeff Mahoney
SUSE Labs
Attachment:
signature.asc
Description: OpenPGP digital signature
