Re: [PATCH V9 5/9] nvmet: add bio get helper for different backends
From: Chaitanya Kulkarni <hidden>
Date: 2021-01-13 05:04:13
Also in:
linux-nvme
From: Chaitanya Kulkarni <hidden>
Date: 2021-01-13 05:04:13
Also in:
linux-nvme
On 1/11/21 23:33, Christoph Hellwig wrote:
I'm not a huge fan of this helper, especially as it sets an end_io callback only for the allocated case, which is a weird calling convention.
The patch has a right documentation and that end_io is needed for passthru case. To get rid of the weirdness I can remove passthru case and make itend_io assign to inline and non-inline bio to make it non-weired. Since this eliminates exactly identical lines of the code in atleast two backend which we should try to use the helper in non-wried way.