Thread (49 messages) 49 messages, 3 authors, 2021-02-11

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

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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help