Re: [PATCH 2/7] nvmet-tcp: Don't set the request's data_len

From: Christoph Hellwig
Date: Wed Oct 23 2019 - 21:15:58 EST


On Wed, Oct 23, 2019 at 10:35:40AM -0600, Logan Gunthorpe wrote:
> It's not apprporiate for the transports to set the data_len
> field of the request which is only used by the core.
>
> In this case, just use a variable on the stack to store the
> length of the sgl for comparison.
>
> Signed-off-by: Logan Gunthorpe <logang@xxxxxxxxxxxx>

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>