Thread (16 messages) 16 messages, 3 authors, 2021-12-06

Re: [PATCH 2/4] nvme: split command copy into a helper

From: Jens Axboe <axboe@kernel.dk>
Date: 2021-12-06 16:33:06
Also in: linux-block

On 12/6/21 12:38 AM, Christoph Hellwig wrote:
As mentioned last time I really don't like the proliferation of
tiny helpers with just two callers.  See my patch I responded with
for my alternative.
I tend to agree with you, but for this particular case I really don't
think it's better to not have the helper. I did try that version too,
but you end up with several sites duplicating it. So while I agree in
general, I don't think this one applies.

-- 
Jens Axboe

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help