Atime and btrfs: a bad combination?

So we already do what Boaz suggested. When I did the ->update_time() stuff I specifically ignore errors to atime, so the user should never see ENOSPC when doing reads. Even prior to the ->update_time() work btrfs would just emit a warning to dmesg and nothing else would happen.