On Wed, Aug 30, 2017 at 07:35:42PM +0300, Nikolay Borisov wrote: > This memory barrier was introduced in 817d52f8dba2 ("Btrfs: async block group > caching"), but even at that time it's usage was broken since it didn't pair > with anything. There was one situation where the cached member was set to > BTRFS_CACHE_FINISHED in a spinlock region which *might* have acted as a > pairing > barrier. Since there is no clear semantics how it's supposed to work better > to just remove it. > > Signed-off-by: Nikolay Borisov <nbori...@suse.com>
As discussed offline, this needs a better description that removing the barrier will not break it. -- 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