Re: linux-next: manual merge of the block tree with the ext4 tree

From: Stephen Rothwell
Date: Fri Oct 22 2010 - 22:47:56 EST


Hi Ted,

On Tue, 21 Sep 2010 14:04:10 +1000 Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
>
> Today's linux-next merge of the block tree got a conflict in
> fs/ext4/mballoc.c between commit 9c543072a9dd7cf60cbf472914a6480ffda50b27
> ("ext4: Check for negative error code from sb_issue_discard") from the
> ext4 tree and commit 2cf6d26a354ab6362e301b5a323832b02867df47 ("block:
> pass gfp_mask and flags to sb_issue_discard") from the block tree.
>
> Just context changes. I fixed it up (see below) and can carry the fix as
> necessary.
>
> diff --cc fs/ext4/mballoc.c
> index e4f4dae,19aa0d4..0000000
> --- a/fs/ext4/mballoc.c
> +++ b/fs/ext4/mballoc.c
> @@@ -2612,8 -2566,8 +2612,8 @@@ static inline void ext4_issue_discard(s
> discard_block = block + ext4_group_first_block_no(sb, block_group);
> trace_ext4_discard_blocks(sb,
> (unsigned long long) discard_block, count);
> - ret = sb_issue_discard(sb, discard_block, count);
> + ret = sb_issue_discard(sb, discard_block, count, GFP_NOFS, 0);
> - if (ret == EOPNOTSUPP) {
> + if (ret == -EOPNOTSUPP) {
> ext4_warning(sb, "discard not supported, disabling");
> clear_opt(EXT4_SB(sb)->s_mount_opt, DISCARD);
> }

This merge fix will now be necessary when merging with Linus' current tree.
--
Cheers,
Stephen Rothwell sfr@xxxxxxxxxxxxxxxx
http://www.canb.auug.org.au/~sfr/

Attachment: pgp00000.pgp
Description: PGP signature