| Message ID | 20170726195925.10715-1-hans.van.kranenburg@mendix.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Ah, great, while doing the last git format-patch, my earlier written changes since v1 were lost again: Changes since v1: * Keep ssd_spread behaviour unchanged * Add summary at the beginning of the commit message Thanks, On 07/26/2017 09:59 PM, Hans van Kranenburg wrote: > The purpose of this patch is providing a band aid to improve the > 'out of the box' behaviour of btrfs for disks that are detected as being > an ssd. In a general purpose mixed workload scenario, the current ssd > mode causes overallocation of available raw disk space for data, while > leaving behind increasing amounts of unused fragmented free space. This > situation leads to early ENOSPC problems which are harming user > experience and adoption of btrfs as a general purpose filesystem. > > This patch modifies the data extent allocation behaviour of the ssd mode > to make it behave identical to nossd mode. The metadata behaviour and > additional ssd_spread option stay untouched so far. > > Recommendations for future development are to reconsider the current > oversimplified nossd / ssd distinction and the broken detection > mechanism based on the rotational attribute in sysfs and provide > experienced users with a more flexible way to choose allocator behaviour > for data and metadata, optimized for certain use cases, while keeping > sane 'out of the box' default settings. The internals of the current > btrfs code have more potential than what currently gets exposed to the > user to choose from. > > The SSD story... > > In the first year of btrfs development, around early 2008, btrfs > gained a mount option which enables specific functionality for > filesystems on solid state devices. The first occurance of this > functionality is in commit e18e4809, labeled "Add mount -o ssd, which > includes optimizations for seek free storage". > > The effect on allocating free space for doing (data) writes is to > 'cluster' writes together, writing them out in contiguous space, as > opposed to a 'tetris' way of putting all separate writes into any free > space fragment that fits (which is what the -o nossd behaviour does). > > A somewhat simplified explanation of what happens is that, when for > example, the 'cluster' size is set to 2MiB, when we do some writes, the > data allocator will search for a free space block that is 2MiB big, and > put the writes in there. The ssd mode itself might allow a 2MiB cluster > to be composed of multiple free space extents with some existing data in > between, while the additional ssd_spread mount option kills off this > option and requires fully free space. > > The idea behind this is (commit 536ac8ae): "The [...] clusters make it > more likely a given IO will completely overwrite the ssd block, so it > doesn't have to do an internal rwm cycle."; ssd block meaning nand erase > block. So, effectively this means applying a "locality based algorithm" > and trying to outsmart the actual ssd. > > Since then, various changes have been made to the involved code, but the > basic idea is still present, and gets activated whenever the ssd mount > option is active. This also happens by default, when the rotational flag > as seen at /sys/block/<device>/queue/rotational is set to 0. > > However, there's a number of problems with this approach. > > First, what the optimization is trying to do is outsmart the ssd by > assuming there is a relation between the physical address space of the > block device as seen by btrfs and the actual physical storage of the > ssd, and then adjusting data placement. However, since the introduction > of the Flash Translation Layer (FTL) which is a part of the internal > controller of an ssd, these attempts are futile. The use of good quality > FTL in consumer ssd products might have been limited in 2008, but this > situation has changed drastically soon after that time. Today, even the > flash memory in your automatic cat feeding machine or your grandma's > wheelchair has a full featured one. > > Second, the behaviour as described above results in the filesystem being > filled up with badly fragmented free space extents because of relatively > small pieces of space that are freed up by deletes, but not selected > again as part of a 'cluster'. Since the algorithm prefers allocating a > new chunk over going back to tetris mode, the end result is a filesystem > in which all raw space is allocated, but which is composed of > underutilized chunks with a 'shotgun blast' pattern of fragmented free > space. Usually, the next problematic thing that happens is the > filesystem wanting to allocate new space for metadata, which causes the > filesystem to fail in spectacular ways. > > Third, the default mount options you get for an ssd ('ssd' mode enabled, > 'discard' not enabled), in combination with spreading out writes over > the full address space and ignoring freed up space leads to worst case > behaviour in providing information to the ssd itself, since it will > never learn that all the free space left behind is actually free. There > are two ways to let an ssd know previously written data does not have to > be preserved, which are sending explicit signals using discard or > fstrim, or by simply overwriting the space with new data. The worst > case behaviour is the btrfs ssd_spread mount option in combination with > not having discard enabled. It has a side effect of minimizing the reuse > of free space previously written in. > > Fourth, the rotational flag in /sys/ does not reliably indicate if the > device is a locally attached ssd. For example, iSCSI or NBD displays as > non-rotational, while a loop device on an ssd shows up as rotational. > > The combination of the second and third problem effectively means that > despite all the good intentions, the btrfs ssd mode reliably causes the > ssd hardware and the filesystem structures and performance to be choked > to death. The clickbait version of the title of this story would have > been "Btrfs ssd optimizations considered harmful for ssds". > > The current nossd 'tetris' mode (even still without discard) allows a > pattern of overwriting much more previously used space, causing many > more implicit discards to happen because of the overwrite information > the ssd gets. The actual location in the physical address space, as seen > from the point of view of btrfs is irrelevant, because the actual writes > to the low level flash are reordered anyway thanks to the FTL. > > Changes made in the code > > 1. Make ssd mode data allocation identical to tetris mode, like nossd. > 2. Adjust and clean up filesystem mount messages so that we can easily > identify if a kernel has this patch applied or not, when providing > support to end users. Also, make better use of the *_and_info helpers to > only trigger messages on actual state changes. > > Backporting notes > > Notes for whoever wants to backport this patch to their 4.9 LTS kernel: > * First apply commit 951e7966 "btrfs: drop the nossd flag when > remounting with -o ssd", or fixup the differences manually. > * The rest of the conflicts are because of the fs_info refactoring. So, > for example, instead of using fs_info, it's root->fs_info in > extent-tree.c > > Signed-off-by: Hans van Kranenburg <hans.van.kranenburg@mendix.com> > --- > fs/btrfs/ctree.h | 4 ++-- > fs/btrfs/disk-io.c | 6 ++---- > fs/btrfs/extent-tree.c | 11 ++++++----- > fs/btrfs/super.c | 16 +++++++++------- > 4 files changed, 19 insertions(+), 18 deletions(-) > > diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h > index 4f8f75d9e839..b091dd3f5b38 100644 > --- a/fs/btrfs/ctree.h > +++ b/fs/btrfs/ctree.h > @@ -472,7 +472,7 @@ struct btrfs_block_rsv { > /* > * free clusters are used to claim free space in relatively large chunks, > * allowing us to do less seeky writes. They are used for all metadata > - * allocations and data allocations in ssd mode. > + * allocations and data allocations in ssd_spread mode. > */ > struct btrfs_free_cluster { > spinlock_t lock; > @@ -976,7 +976,7 @@ struct btrfs_fs_info { > > struct reloc_control *reloc_ctl; > > - /* data_alloc_cluster is only used in ssd mode */ > + /* data_alloc_cluster is only used in ssd_spread mode */ > struct btrfs_free_cluster data_alloc_cluster; > > /* all metadata allocations go through this cluster */ > diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c > index 5f678dcb20e6..97ede2de24a3 100644 > --- a/fs/btrfs/disk-io.c > +++ b/fs/btrfs/disk-io.c > @@ -3061,11 +3061,9 @@ int open_ctree(struct super_block *sb, > if (IS_ERR(fs_info->transaction_kthread)) > goto fail_cleaner; > > - if (!btrfs_test_opt(fs_info, SSD) && > - !btrfs_test_opt(fs_info, NOSSD) && > + if (!btrfs_test_opt(fs_info, NOSSD) && > !fs_info->fs_devices->rotating) { > - btrfs_info(fs_info, "detected SSD devices, enabling SSD mode"); > - btrfs_set_opt(fs_info->mount_opt, SSD); > + btrfs_set_and_info(fs_info, SSD, "enabling ssd optimizations"); > } > > /* > diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c > index 33d979e9ea2a..cb1855fede41 100644 > --- a/fs/btrfs/extent-tree.c > +++ b/fs/btrfs/extent-tree.c > @@ -6609,19 +6609,20 @@ fetch_cluster_info(struct btrfs_fs_info *fs_info, > struct btrfs_space_info *space_info, u64 *empty_cluster) > { > struct btrfs_free_cluster *ret = NULL; > - bool ssd = btrfs_test_opt(fs_info, SSD); > > *empty_cluster = 0; > if (btrfs_mixed_space_info(space_info)) > return ret; > > - if (ssd) > - *empty_cluster = SZ_2M; > if (space_info->flags & BTRFS_BLOCK_GROUP_METADATA) { > ret = &fs_info->meta_alloc_cluster; > - if (!ssd) > + if (btrfs_test_opt(fs_info, SSD)) > + *empty_cluster = SZ_2M; > + else > *empty_cluster = SZ_64K; > - } else if ((space_info->flags & BTRFS_BLOCK_GROUP_DATA) && ssd) { > + } else if ((space_info->flags & BTRFS_BLOCK_GROUP_DATA) && > + btrfs_test_opt(fs_info, SSD_SPREAD)) { > + *empty_cluster = SZ_2M; > ret = &fs_info->data_alloc_cluster; > } > > diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c > index 4f1cdd5058f1..63a3bcce6177 100644 > --- a/fs/btrfs/super.c > +++ b/fs/btrfs/super.c > @@ -548,20 +548,22 @@ int btrfs_parse_options(struct btrfs_fs_info *info, char *options, > break; > case Opt_ssd: > btrfs_set_and_info(info, SSD, > - "use ssd allocation scheme"); > + "enabling ssd optimizations"); > btrfs_clear_opt(info->mount_opt, NOSSD); > break; > case Opt_ssd_spread: > + btrfs_set_and_info(info, SSD, > + "enabling ssd optimizations"); > btrfs_set_and_info(info, SSD_SPREAD, > - "use spread ssd allocation scheme"); > - btrfs_set_opt(info->mount_opt, SSD); > + "using spread ssd allocation scheme"); > btrfs_clear_opt(info->mount_opt, NOSSD); > break; > case Opt_nossd: > - btrfs_set_and_info(info, NOSSD, > - "not using ssd allocation scheme"); > - btrfs_clear_opt(info->mount_opt, SSD); > - btrfs_clear_opt(info->mount_opt, SSD_SPREAD); > + btrfs_set_opt(info->mount_opt, NOSSD); > + btrfs_clear_and_info(info, SSD, > + "not using ssd optimizations"); > + btrfs_clear_and_info(info, SSD_SPREAD, > + "not using spread ssd allocation scheme"); > break; > case Opt_barrier: > btrfs_clear_and_info(info, NOBARRIER, >
Hans van Kranenburg posted on Wed, 26 Jul 2017 21:59:25 +0200 as excerpted: > diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h > index 4f8f75d9e839..b091dd3f5b38 100644 > --- a/fs/btrfs/ctree.h > +++ b/fs/btrfs/ctree.h > @@ -472,7 +472,7 @@ struct btrfs_block_rsv { > /* > * free clusters are used to claim free space in relatively large chunks, > * allowing us to do less seeky writes. They are used for all metadata > - * allocations and data allocations in ssd mode. > + * allocations and data allocations in ssd_spread mode. > */ > struct btrfs_free_cluster { > spinlock_t lock; That (post-patch) comment says all metadata and data allocations in ssd_spread mode, but if my understanding is correct, it'll also use free clusters for metadata (only, not data, the patch only changing) the data behavior) in normal ssd mode. Perhaps that (metadata only use of free clusers in ssd mode) should be mentioned as well, since post-patch there's now a distinction to be made?
On 07/27/2017 09:11 PM, Duncan wrote: > Hans van Kranenburg posted on Wed, 26 Jul 2017 21:59:25 +0200 as > excerpted: > >> diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h >> index 4f8f75d9e839..b091dd3f5b38 100644 >> --- a/fs/btrfs/ctree.h >> +++ b/fs/btrfs/ctree.h >> @@ -472,7 +472,7 @@ struct btrfs_block_rsv { >> /* >> * free clusters are used to claim free space in relatively large chunks, >> * allowing us to do less seeky writes. They are used for all metadata >> - * allocations and data allocations in ssd mode. >> + * allocations and data allocations in ssd_spread mode. >> */ >> struct btrfs_free_cluster { >> spinlock_t lock; > > That (post-patch) comment says all metadata and data allocations in > ssd_spread mode, but if my understanding is correct, it'll also use > free clusters for metadata (only, not data, the patch only changing) > the data behavior) in normal ssd mode. > > Perhaps that (metadata only use of free clusers in ssd mode) should be > mentioned as well, since post-patch there's now a distinction to be made? The text is confusing yes. It can be read as... (all metadata allocations and data allocations) in ssd_spread mode or... (all metadata allocations) and (data allocations in ssd_spread mode) ...the second one being correct. Good catch. I changed it in: "They are used for all metadata allocations. In ssd_spread mode they are also used for data allocations."
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index 4f8f75d9e839..b091dd3f5b38 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -472,7 +472,7 @@ struct btrfs_block_rsv { /* * free clusters are used to claim free space in relatively large chunks, * allowing us to do less seeky writes. They are used for all metadata - * allocations and data allocations in ssd mode. + * allocations and data allocations in ssd_spread mode. */ struct btrfs_free_cluster { spinlock_t lock; @@ -976,7 +976,7 @@ struct btrfs_fs_info { struct reloc_control *reloc_ctl; - /* data_alloc_cluster is only used in ssd mode */ + /* data_alloc_cluster is only used in ssd_spread mode */ struct btrfs_free_cluster data_alloc_cluster; /* all metadata allocations go through this cluster */ diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index 5f678dcb20e6..97ede2de24a3 100644 --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c @@ -3061,11 +3061,9 @@ int open_ctree(struct super_block *sb, if (IS_ERR(fs_info->transaction_kthread)) goto fail_cleaner; - if (!btrfs_test_opt(fs_info, SSD) && - !btrfs_test_opt(fs_info, NOSSD) && + if (!btrfs_test_opt(fs_info, NOSSD) && !fs_info->fs_devices->rotating) { - btrfs_info(fs_info, "detected SSD devices, enabling SSD mode"); - btrfs_set_opt(fs_info->mount_opt, SSD); + btrfs_set_and_info(fs_info, SSD, "enabling ssd optimizations"); } /* diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index 33d979e9ea2a..cb1855fede41 100644 --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c @@ -6609,19 +6609,20 @@ fetch_cluster_info(struct btrfs_fs_info *fs_info, struct btrfs_space_info *space_info, u64 *empty_cluster) { struct btrfs_free_cluster *ret = NULL; - bool ssd = btrfs_test_opt(fs_info, SSD); *empty_cluster = 0; if (btrfs_mixed_space_info(space_info)) return ret; - if (ssd) - *empty_cluster = SZ_2M; if (space_info->flags & BTRFS_BLOCK_GROUP_METADATA) { ret = &fs_info->meta_alloc_cluster; - if (!ssd) + if (btrfs_test_opt(fs_info, SSD)) + *empty_cluster = SZ_2M; + else *empty_cluster = SZ_64K; - } else if ((space_info->flags & BTRFS_BLOCK_GROUP_DATA) && ssd) { + } else if ((space_info->flags & BTRFS_BLOCK_GROUP_DATA) && + btrfs_test_opt(fs_info, SSD_SPREAD)) { + *empty_cluster = SZ_2M; ret = &fs_info->data_alloc_cluster; } diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 4f1cdd5058f1..63a3bcce6177 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@ -548,20 +548,22 @@ int btrfs_parse_options(struct btrfs_fs_info *info, char *options, break; case Opt_ssd: btrfs_set_and_info(info, SSD, - "use ssd allocation scheme"); + "enabling ssd optimizations"); btrfs_clear_opt(info->mount_opt, NOSSD); break; case Opt_ssd_spread: + btrfs_set_and_info(info, SSD, + "enabling ssd optimizations"); btrfs_set_and_info(info, SSD_SPREAD, - "use spread ssd allocation scheme"); - btrfs_set_opt(info->mount_opt, SSD); + "using spread ssd allocation scheme"); btrfs_clear_opt(info->mount_opt, NOSSD); break; case Opt_nossd: - btrfs_set_and_info(info, NOSSD, - "not using ssd allocation scheme"); - btrfs_clear_opt(info->mount_opt, SSD); - btrfs_clear_opt(info->mount_opt, SSD_SPREAD); + btrfs_set_opt(info->mount_opt, NOSSD); + btrfs_clear_and_info(info, SSD, + "not using ssd optimizations"); + btrfs_clear_and_info(info, SSD_SPREAD, + "not using spread ssd allocation scheme"); break; case Opt_barrier: btrfs_clear_and_info(info, NOBARRIER,
The purpose of this patch is providing a band aid to improve the 'out of the box' behaviour of btrfs for disks that are detected as being an ssd. In a general purpose mixed workload scenario, the current ssd mode causes overallocation of available raw disk space for data, while leaving behind increasing amounts of unused fragmented free space. This situation leads to early ENOSPC problems which are harming user experience and adoption of btrfs as a general purpose filesystem. This patch modifies the data extent allocation behaviour of the ssd mode to make it behave identical to nossd mode. The metadata behaviour and additional ssd_spread option stay untouched so far. Recommendations for future development are to reconsider the current oversimplified nossd / ssd distinction and the broken detection mechanism based on the rotational attribute in sysfs and provide experienced users with a more flexible way to choose allocator behaviour for data and metadata, optimized for certain use cases, while keeping sane 'out of the box' default settings. The internals of the current btrfs code have more potential than what currently gets exposed to the user to choose from. The SSD story... In the first year of btrfs development, around early 2008, btrfs gained a mount option which enables specific functionality for filesystems on solid state devices. The first occurance of this functionality is in commit e18e4809, labeled "Add mount -o ssd, which includes optimizations for seek free storage". The effect on allocating free space for doing (data) writes is to 'cluster' writes together, writing them out in contiguous space, as opposed to a 'tetris' way of putting all separate writes into any free space fragment that fits (which is what the -o nossd behaviour does). A somewhat simplified explanation of what happens is that, when for example, the 'cluster' size is set to 2MiB, when we do some writes, the data allocator will search for a free space block that is 2MiB big, and put the writes in there. The ssd mode itself might allow a 2MiB cluster to be composed of multiple free space extents with some existing data in between, while the additional ssd_spread mount option kills off this option and requires fully free space. The idea behind this is (commit 536ac8ae): "The [...] clusters make it more likely a given IO will completely overwrite the ssd block, so it doesn't have to do an internal rwm cycle."; ssd block meaning nand erase block. So, effectively this means applying a "locality based algorithm" and trying to outsmart the actual ssd. Since then, various changes have been made to the involved code, but the basic idea is still present, and gets activated whenever the ssd mount option is active. This also happens by default, when the rotational flag as seen at /sys/block/<device>/queue/rotational is set to 0. However, there's a number of problems with this approach. First, what the optimization is trying to do is outsmart the ssd by assuming there is a relation between the physical address space of the block device as seen by btrfs and the actual physical storage of the ssd, and then adjusting data placement. However, since the introduction of the Flash Translation Layer (FTL) which is a part of the internal controller of an ssd, these attempts are futile. The use of good quality FTL in consumer ssd products might have been limited in 2008, but this situation has changed drastically soon after that time. Today, even the flash memory in your automatic cat feeding machine or your grandma's wheelchair has a full featured one. Second, the behaviour as described above results in the filesystem being filled up with badly fragmented free space extents because of relatively small pieces of space that are freed up by deletes, but not selected again as part of a 'cluster'. Since the algorithm prefers allocating a new chunk over going back to tetris mode, the end result is a filesystem in which all raw space is allocated, but which is composed of underutilized chunks with a 'shotgun blast' pattern of fragmented free space. Usually, the next problematic thing that happens is the filesystem wanting to allocate new space for metadata, which causes the filesystem to fail in spectacular ways. Third, the default mount options you get for an ssd ('ssd' mode enabled, 'discard' not enabled), in combination with spreading out writes over the full address space and ignoring freed up space leads to worst case behaviour in providing information to the ssd itself, since it will never learn that all the free space left behind is actually free. There are two ways to let an ssd know previously written data does not have to be preserved, which are sending explicit signals using discard or fstrim, or by simply overwriting the space with new data. The worst case behaviour is the btrfs ssd_spread mount option in combination with not having discard enabled. It has a side effect of minimizing the reuse of free space previously written in. Fourth, the rotational flag in /sys/ does not reliably indicate if the device is a locally attached ssd. For example, iSCSI or NBD displays as non-rotational, while a loop device on an ssd shows up as rotational. The combination of the second and third problem effectively means that despite all the good intentions, the btrfs ssd mode reliably causes the ssd hardware and the filesystem structures and performance to be choked to death. The clickbait version of the title of this story would have been "Btrfs ssd optimizations considered harmful for ssds". The current nossd 'tetris' mode (even still without discard) allows a pattern of overwriting much more previously used space, causing many more implicit discards to happen because of the overwrite information the ssd gets. The actual location in the physical address space, as seen from the point of view of btrfs is irrelevant, because the actual writes to the low level flash are reordered anyway thanks to the FTL. Changes made in the code 1. Make ssd mode data allocation identical to tetris mode, like nossd. 2. Adjust and clean up filesystem mount messages so that we can easily identify if a kernel has this patch applied or not, when providing support to end users. Also, make better use of the *_and_info helpers to only trigger messages on actual state changes. Backporting notes Notes for whoever wants to backport this patch to their 4.9 LTS kernel: * First apply commit 951e7966 "btrfs: drop the nossd flag when remounting with -o ssd", or fixup the differences manually. * The rest of the conflicts are because of the fs_info refactoring. So, for example, instead of using fs_info, it's root->fs_info in extent-tree.c Signed-off-by: Hans van Kranenburg <hans.van.kranenburg@mendix.com> --- fs/btrfs/ctree.h | 4 ++-- fs/btrfs/disk-io.c | 6 ++---- fs/btrfs/extent-tree.c | 11 ++++++----- fs/btrfs/super.c | 16 +++++++++------- 4 files changed, 19 insertions(+), 18 deletions(-)