Re: [PATCH v6 3/8] btrfs: add a bio argument to btrfs_csum_one_bio
From: Daniel Vacek
Date: Wed Nov 19 2025 - 04:29:07 EST
On Wed, 19 Nov 2025 at 09:22, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> On Wed, Nov 19, 2025 at 08:34:13AM +0100, Daniel Vacek wrote:
> > That's the case. The bounce bio is created when you submit the
> > original one. The data is encrypted by fscrypt, then the csum hook is
> > called and the new bio submitted instead of the original one. Later
> > the endio frees the new one and calls endio on the original bio. This
> > means we don't have control over the bounce bio and cannot use it
> > asynchronously at the moment. The csum needs to be finished directly
> > in the hook.
>
> And as I told you that can be changed. Please get your entire series
> out of review to allow people to try to review what you're trying to
> do.
It's coming. Stay tuned! I'm just finishing a bit of re-design to
btrfs crypt context metadata storing which was suggested in code
review of matching changes in btrfs-progs. The fscrypt part is mostly
without any changes to the old v5 series from Josef.
--nX