On Fri, Jul 14, 2017 at 09:55:41AM +0300, Nikolay Borisov wrote: > find_raid56_stripe_len statically returns SZ_64K which equals > BTRFS_STRIPE_LEN. > It's sole caller is __btrfs_alloc_chunk and it assigns the return value to ai > variable which is already set to BTRFS_STRIPE_LEN. So remove the function > invocation altogether and remove the function itself. Also remove the variable > since it's only aliasing BTRFS_STRIPE_LEN and use the define directly. Use > the occassion to simplify the rounding down of stripe_size now that the value > we want it to align is a power of 2. > > Signed-off-by: Nikolay Borisov <nbori...@suse.com> > --- > > V2: > * Incorporate Qu's suggestion to use round_down macro. Updated commit > message accordingly.
The custom stripe length support would need to start from zero anyway, so we don't lose anything if the function gets removed. All the alignment, rounding etc will have to be checked. Reviewed-by: David Sterba <dste...@suse.com> -- 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
