On Mon, Feb 15, 2016 at 05:47:46PM +0100, David Sterba wrote:
> > + struct btrfs_super_block *disk_super = NULL;
>
> disk_super is NULL here and unchanged until ...
>
> > + if (!disk_super) {
>
> ... here, so this will always take this branch
>
> > + ret = btrfs_get_bdev_and_sb(device_path,
> > + FMODE_WRITE | FMODE_EXCL,
> > + root->fs_info->bdev_holder, 0,
> > + &bdev, &bh);
> > + if (ret) {
> > + /*
> > + * It could be a failed device ok for clear_super
> > + * to fail. So return success
> > + */
> > + ret = 0;
> > + goto out;
> > + }
> > +
> > + disk_super = (struct btrfs_super_block *)bh->b_data;
> > + }
>
> The patch looks otherwise good so I'm curious if this is a leftover or
> I'm missing some logic behind that.
Never mind, the code gets removed anyway. Bisectability is not broken.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
