On 5/9/23 07:35, Xia, Chenbo wrote:
Hi Maxime,
-----Original Message-----
From: Maxime Coquelin <[email protected]>
Sent: Friday, March 31, 2023 11:43 PM
To: [email protected]; [email protected]; Xia, Chenbo
<[email protected]>; [email protected]; [email protected];
[email protected]; Liang, Cunming <[email protected]>; Xie, Yongji
<[email protected]>; [email protected]; [email protected];
[email protected]
Cc: Maxime Coquelin <[email protected]>
Subject: [RFC 25/27] vhost: add support for VDUSE IOTLB update event
This patch adds support for VDUSE_UPDATE_IOTLB event
handling, which consists in invaliding IOTLB entries for
the range specified in the request.
Signed-off-by: Maxime Coquelin <[email protected]>
---
lib/vhost/vduse.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/lib/vhost/vduse.c b/lib/vhost/vduse.c
index 7d59a5f709..b5b9fa2eb1 100644
--- a/lib/vhost/vduse.c
+++ b/lib/vhost/vduse.c
@@ -169,6 +169,12 @@ vduse_events_handler(int fd, void *arg, int *remove
__rte_unused)
dev->status = req.s.status;
resp.result = VDUSE_REQ_RESULT_OK;
break;
+ case VDUSE_UPDATE_IOTLB:
+ VHOST_LOG_CONFIG(dev->ifname, INFO, "\tIOVA range: %" PRIx64 "
- %" PRIx64 "\n",
+ (uint64_t)req.iova.start,
(uint64_t)req.iova.last);
+ vhost_user_iotlb_cache_remove(dev, req.iova.start,
+ req.iova.last - req.iova.start + 1);
+ break;
We don't need to set the response result here?
Good catch! We indeed need to send the reply for this message.
I'm fixing it now.
Thanks,
Maxime
Thanks,
Chenbo
default:
resp.result = VDUSE_REQ_RESULT_FAILED;
break;
--
2.39.2
