> +static int __stash_bdev_file(struct block_device *bdev) I've said that on the previous version. I think that this is really error prone and seems overall like an unpleasant solution. I would really like to avoid going down that route. I think a chunk of this series is good though specicially simple conversions of individual filesystems where file_inode() or f_mapping makes sense. There's a few exceptions where we might be better of replacing the current apis with something else (I think Al touched on that somewhere further down the thread.). I'd suggest the straightforward bd_inode removals into a separate series that I can take. Thanks for working on all of this. It's certainly a contentious area.
