[PATCH] nvmet-tcp: bound sgl->length check in nvmet_tcp_map_data()
Christoph Hellwig
hch at lst.de
Fri Mar 20 00:48:25 PDT 2026
On Thu, Mar 19, 2026 at 02:00:27PM -0400, Shivam Kumar wrote:
> Hi Maurizio
>
> Thank you for the review. You're right, I was conflating the per-PDU
> H2CData limit with the total transfer size for the command.
> And the error handling should trigger a fatal transport error rather
> than completing the request with an NVMe status code.
>
> Would setting a sane default for MDTS be the preferred approach here?
> If so, I'm happy to send a v2 implementing that instead.
Yes, if we want to limit the I/O size we have to limit MDTS. Which all
other transports do anyway, so I'm kinda surprised TCP doesn't.
More information about the Linux-nvme
mailing list