diff options
| author | Mikhail Skorzhinskii <mskorzhinskiy@solarflare.com> | 2019-07-08 12:31:29 +0200 |
|---|---|---|
| committer | Christoph Hellwig <hch@lst.de> | 2019-07-09 14:18:09 -0700 |
| commit | 37c15219599f7a4baa73f6e3432afc69ba7cc530 (patch) | |
| tree | 611be0f9375e9b7982bc1ccc42fd0e76ae5f21d7 /drivers/nvme | |
| parent | 958f2a0f8121ae36a5cbff383ab94fadf1fba5eb (diff) | |
nvme-tcp: don't use sendpage for SLAB pages
According to commit a10674bf2406 ("tcp: detecting the misuse of
.sendpage for Slab objects") and previous discussion, tcp_sendpage
should not be used for pages that is managed by SLAB, as SLAB is not
taking page reference counters into consideration.
Signed-off-by: Mikhail Skorzhinskii <mskorzhinskiy@solarflare.com>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/nvme')
| -rw-r--r-- | drivers/nvme/host/tcp.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/drivers/nvme/host/tcp.c b/drivers/nvme/host/tcp.c index 08a2501b9357..606b13d35d16 100644 --- a/drivers/nvme/host/tcp.c +++ b/drivers/nvme/host/tcp.c @@ -860,7 +860,14 @@ static int nvme_tcp_try_send_data(struct nvme_tcp_request *req) else flags |= MSG_MORE; - ret = kernel_sendpage(queue->sock, page, offset, len, flags); + /* can't zcopy slab pages */ + if (unlikely(PageSlab(page))) { + ret = sock_no_sendpage(queue->sock, page, offset, len, + flags); + } else { + ret = kernel_sendpage(queue->sock, page, offset, len, + flags); + } if (ret <= 0) return ret; |