Skip to content

Commit

Permalink
nvmet-tcp: pass iov_len instead of sg->length to bvec_set_page()
Browse files Browse the repository at this point in the history
[ Upstream commit 1f0bbf2 ]

iov_len is the valid data length, so pass iov_len instead of sg->length to
bvec_set_page().

Fixes: 5bfaba2 ("nvmet-tcp: don't map pages which can't come from HIGHMEM")
Signed-off-by: Rakshana Sridhar <rakshanas@chelsio.com>
Signed-off-by: Varun Prakash <varun@chelsio.com>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Keith Busch <kbusch@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
  • Loading branch information
Varun Prakash authored and gregkh committed Sep 23, 2023
1 parent 10c160f commit 66797aa
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/nvme/target/tcp.c
Expand Up @@ -348,7 +348,7 @@ static void nvmet_tcp_build_pdu_iovec(struct nvmet_tcp_cmd *cmd)
while (length) {
u32 iov_len = min_t(u32, length, sg->length - sg_offset);

bvec_set_page(iov, sg_page(sg), sg->length,
bvec_set_page(iov, sg_page(sg), iov_len,
sg->offset + sg_offset);

length -= iov_len;
Expand Down

0 comments on commit 66797aa

Please sign in to comment.