[PATCH] nvme: use str_enabled_disabled() for digest mismatch messages
Christoph Hellwig
hch at infradead.org
Mon May 11 01:14:57 PDT 2026
On Fri, May 08, 2026 at 03:17:33PM +0530, Rohit Chavan wrote:
> Hi Christoph,
>
> Thank you for the feedback.
>
> My intent was to align the code with the standardized string helpers
> recently introduced in the kernel to reduce open-coding. However,
> I respect your preference for keeping the ternary form in the
> NVMe subsystem.
I guess it's more of a question for the people adding the helpers, but it
should also be in any place converting to it: what's the point?
The helpers don't reduce the code size, but increase cognitive load. Not
a good trade-off as far as I can tell, but maybe I'm missing something?
More information about the Linux-nvme
mailing list