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.