Re: [PATCH v6] nvme-tcp: Do not reset transport on data digest errors
From: Daniel Wagner
Date: Mon Aug 30 2021 - 11:50:28 EST
On Mon, Aug 30, 2021 at 06:38:28PM +0300, Sagi Grimberg wrote:
> > + if (req->status == cpu_to_le16(NVME_SC_SUCCESS))
>
> endian conversion a zero enum? looks weird..
I though it makes sense to stay consistent with the rest and don't make
it harder for the reader to figure out why this might not needed here.
And obviously the fear of the static code analyzers...
> But,
> Reviewed-by: Sagi Grimberg <sagi@xxxxxxxxxxx>
Thanks!