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

Re: [PATCH V9 8/9] nvmet: add common I/O length check helper

From: Christoph Hellwig <hch@lst.de>
Date: 2021-01-18 18:35:37
Also in: linux-nvme

On Wed, Jan 13, 2021 at 05:07:16AM +0000, Chaitanya Kulkarni wrote:
On 1/11/21 23:35, Christoph Hellwig wrote:
quoted
I can't say I like this helper.  The semantics are a little confusing,
not helped by the name.
Is it because of the name of the helper ? if so can you please suggest a
name.
Because all it has a same code repeated in the three backends and it really
bothers me to see duplicate code, but that just me :P.
Also because it does two rather unrelated things.  Or in other words:
I don't think the helper helps the readability of the code.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help