On Mon, Oct 07, 2019 at 09:37:40PM +0200, David Sterba wrote: > The extent_map::bdev is unused and and can be removed, but it's not > straightforward so there are several steps. The first patch decouples > bdev from map_lookup. The remaining patches clean up use of the bdev, > removing a few bio_set_dev on the way. In the end, submit_extent_page is > one parameter lighter. > > This has survived several fstests runs > > David Sterba (5): > btrfs: assert extent_map bdevs and lookup_map and split > btrfs: get bdev from latest_dev for dio bh_result > btrfs: drop bio_set_dev where not needed > btrfs: remove extent_map::bdev > btrfs: drop bdev argument from submit_extent_page > > fs/btrfs/compression.c | 10 ---------- > fs/btrfs/disk-io.c | 3 --- > fs/btrfs/extent_io.c | 15 +++------------ > fs/btrfs/extent_map.c | 6 +++++- > fs/btrfs/extent_map.h | 11 ++--------- > fs/btrfs/file-item.c | 1 - > fs/btrfs/file.c | 3 --- > fs/btrfs/inode.c | 14 ++++---------- > fs/btrfs/relocation.c | 2 -- > 9 files changed, 14 insertions(+), 51 deletions(-) > This series needs to be dropped from misc-next, it causes any box with cgroups enabled to crash on boot. The bio requires having a bio->bi_disk set when we do wbc_init_bio, which we no longer have with this patch. To fix this you would need to do something similar to what we do with compression, save the wbc css in the bbio and then call the associate_blkg at submit time. However, this is problematic right now because we don't get notified when the bbio is free'd. We have no way to free the ref on the css we save, which means infrastructure needs to be added to biosets so we can get called at free time for every bio in a bioset. Or we can add back the bi_css to the bio, but that seems like a less good idea. Either way this needs to be dropped until this is addressed. Thanks, Josef
