Excerpts from Mitch Harder's message of 2011-05-03 11:42:56 -0400: > On Tue, May 3, 2011 at 9:41 AM, Daniel J Blueman > <daniel.blue...@gmail.com> wrote: > > > > It does seem the case generally; on 2.6.39-rc5, writing to a fresh > > filesystem using rsync with BTRFS compression enabled, 128KB extents > > seem very common [1] (filefrag inconsistency noted). > > > > Defragmenting with compression gives a nice linear extent [2]. It > > looks like it'll be a good win to prevent extents being split at > > writeout for the read case on rotational media. > > > > Yes, 128KB extents are hardcoded in Btrfs right now. > > There are two reasons cited in the comments for this: > > (1) Ease the RAM required when spreading compression across several CPUs. > (2) Make sure the amount of IO required to do a random read is > reasonably small. > > For about 4 months, I've been playing locally with 2 patches to > increase the extent size to 512KB. > > I haven't noticed any issues running with these patches. However, I > only have a Core2duo with 2 CPUs, so I'm probably not running into > issues that someone with more CPUs might encounter. > > I'll submit these patches to the list as an RFC so more people can at > least see where this is done. But with my limited hardware, I can't > assert this change is the best for everyone.
The problem is just that any random read into the file will require reading the full 512KB in order to decompress the extent. And you need to make sure you have enough room in ram to represent the decompressed bytes in order to find the pages you care about. The alternative is to keep the smaller compressed extent size and make the allocator work harder to find contiguous 128KB extents to store all of the file bytes. This will work out much better ;) -chris -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
