* [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically
@ 2022-09-13 9:08 Hao Chen
2022-09-15 7:52 ` houyl
2022-09-20 2:18 ` Jason Wang
0 siblings, 2 replies; 5+ messages in thread
From: Hao Chen @ 2022-09-13 9:08 UTC (permalink / raw)
To: mst, raphael.norwitz
Cc: kwolf, hreitz, jasowang, qemu-block, qemu-devel, houyl, zy,
Hao Chen
When use dpdk-vdpa tests vdpa device. You need to specify the mac address to
start the virtual machine through libvirt or qemu, but now, the libvirt or
qemu can call dpdk vdpa vendor driver's ops .get_config through vhost_net_get_config
to get the mac address of the vdpa hardware without manual configuration.
Signed-off-by: Hao Chen <chenh@yusur.tech>
---
hw/block/vhost-user-blk.c | 1 -
hw/net/virtio-net.c | 3 ++-
hw/virtio/vhost-user.c | 19 -------------------
3 files changed, 2 insertions(+), 21 deletions(-)
diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c
index 9117222456..5dca4eab09 100644
--- a/hw/block/vhost-user-blk.c
+++ b/hw/block/vhost-user-blk.c
@@ -337,7 +337,6 @@ static int vhost_user_blk_connect(DeviceState *dev, Error **errp)
vhost_dev_set_config_notifier(&s->dev, &blk_ops);
- s->vhost_user.supports_config = true;
ret = vhost_dev_init(&s->dev, &s->vhost_user, VHOST_BACKEND_TYPE_USER, 0,
errp);
if (ret < 0) {
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index dd0d056fde..274ea84644 100644
--- a/hw/net/virtio-net.c
+++ b/hw/net/virtio-net.c
@@ -149,7 +149,8 @@ static void virtio_net_get_config(VirtIODevice *vdev, uint8_t *config)
* Is this VDPA? No peer means not VDPA: there's no way to
* disconnect/reconnect a VDPA peer.
*/
- if (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) {
+ if ((nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) ||
+ (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_USER)) {
ret = vhost_net_get_config(get_vhost_net(nc->peer), (uint8_t *)&netcfg,
n->config_size);
if (ret != -1) {
diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
index bd24741be8..8b01078249 100644
--- a/hw/virtio/vhost-user.c
+++ b/hw/virtio/vhost-user.c
@@ -2013,8 +2013,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
}
if (virtio_has_feature(features, VHOST_USER_F_PROTOCOL_FEATURES)) {
- bool supports_f_config = vus->supports_config ||
- (dev->config_ops && dev->config_ops->vhost_dev_config_notifier);
uint64_t protocol_features;
dev->backend_features |= 1ULL << VHOST_USER_F_PROTOCOL_FEATURES;
@@ -2033,23 +2031,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
*/
protocol_features &= VHOST_USER_PROTOCOL_FEATURE_MASK;
- if (supports_f_config) {
- if (!virtio_has_feature(protocol_features,
- VHOST_USER_PROTOCOL_F_CONFIG)) {
- error_setg(errp, "vhost-user device expecting "
- "VHOST_USER_PROTOCOL_F_CONFIG but the vhost-user backend does "
- "not support it.");
- return -EPROTO;
- }
- } else {
- if (virtio_has_feature(protocol_features,
- VHOST_USER_PROTOCOL_F_CONFIG)) {
- warn_reportf_err(*errp, "vhost-user backend supports "
- "VHOST_USER_PROTOCOL_F_CONFIG but QEMU does not.");
- protocol_features &= ~(1ULL << VHOST_USER_PROTOCOL_F_CONFIG);
- }
- }
-
/* final set of protocol features */
dev->protocol_features = protocol_features;
err = vhost_user_set_protocol_features(dev, dev->protocol_features);
--
2.27.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically
2022-09-13 9:08 [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically Hao Chen
@ 2022-09-15 7:52 ` houyl
2022-09-20 2:18 ` Jason Wang
1 sibling, 0 replies; 5+ messages in thread
From: houyl @ 2022-09-15 7:52 UTC (permalink / raw)
To: 陈浩, mst, raphael.norwitz, jasowang
Cc: kwolf, hreitz, qemu-block, qemu-devel, zy, 陈浩,
houyl
On Tue, Sep 13, 2022 at 17:08 PM Hao Chen <chenh@yusur.tech> wrote:
>From: Hao Chen
>Date: 2022-09-13 17:08
>To: mst; raphael.norwitz
>CC: kwolf; hreitz; jasowang; qemu-block; qemu-devel; houyl; zy; Hao Chen
>Subject: [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically
>When use dpdk-vdpa tests vdpa device. You need to specify the mac address to
>start the virtual machine through libvirt or qemu, but now, the libvirt or
>qemu can call dpdk vdpa vendor driver's ops .get_config through vhost_net_get_config
>to get the mac address of the vdpa hardware without manual configuration.
>Signed-off-by: Hao Chen <chenh@yusur.tech>
>---
>hw/block/vhost-user-blk.c | 1 -
>hw/net/virtio-net.c | 3 ++-
>hw/virtio/vhost-user.c | 19 -------------------
>3 files changed, 2 insertions(+), 21 deletions(-)
>diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c
>index 9117222456..5dca4eab09 100644
>--- a/hw/block/vhost-user-blk.c
>+++ b/hw/block/vhost-user-blk.c
>@@ -337,7 +337,6 @@ static int vhost_user_blk_connect(DeviceState *dev, Error **errp)
> vhost_dev_set_config_notifier(&s->dev, &blk_ops);
>- s->vhost_user.supports_config = true;
> ret = vhost_dev_init(&s->dev, &s->vhost_user, VHOST_BACKEND_TYPE_USER, 0,
> errp);
> if (ret < 0) {
>diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
>index dd0d056fde..274ea84644 100644
>--- a/hw/net/virtio-net.c
>+++ b/hw/net/virtio-net.c
>@@ -149,7 +149,8 @@ static void virtio_net_get_config(VirtIODevice *vdev, uint8_t *config)
> * Is this VDPA? No peer means not VDPA: there's no way to
>
> * disconnect/reconnect a VDPA peer.
> */
>- if (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) {
>+ if ((nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) ||
>+ (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_USER)) {
> ret = vhost_net_get_config(get_vhost_net(nc->peer), (uint8_t *)&netcfg,
n->config_size);
> if (ret != -1) {
>diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
>index bd24741be8..8b01078249 100644
>--- a/hw/virtio/vhost-user.c
>+++ b/hw/virtio/vhost-user.c
>@@ -2013,8 +2013,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
> }
> if (virtio_has_feature(features, VHOST_USER_F_PROTOCOL_FEATURES)) {
>- bool supports_f_config = vus->supports_config ||
>- (dev->config_ops && dev->config_ops->vhost_dev_config_notifier);
> uint64_t protocol_features;
> dev->backend_features |= 1ULL << VHOST_USER_F_PROTOCOL_FEATURES;
>@@ -2033,23 +2031,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
> */
> protocol_features &= VHOST_USER_PROTOCOL_FEATURE_MASK;
>- if (supports_f_config) {
>- if (!virtio_has_feature(protocol_features,
>- VHOST_USER_PROTOCOL_F_CONFIG)) {
>- error_setg(errp, "vhost-user device expecting "
>- "VHOST_USER_PROTOCOL_F_CONFIG but the vhost-user backend does "
>- "not support it.");
>- return -EPROTO;
>- }
>- } else {
>- if (virtio_has_feature(protocol_features,
>- VHOST_USER_PROTOCOL_F_CONFIG)) {
>- warn_reportf_err(*errp, "vhost-user backend supports "
>- "VHOST_USER_PROTOCOL_F_CONFIG but QEMU does not.");
>- protocol_features &= ~(1ULL << VHOST_USER_PROTOCOL_F_CONFIG);
>- }
>- }
>-
> /* final set of protocol features */
> dev->protocol_features = protocol_features;
> err = vhost_user_set_protocol_features(dev, dev->protocol_features);
>--
Yusur (YUSUR Technology Co., Ltd.) focuses on the research and development of special data processors.
The independently developed international leading DPU series products can be widely used in ultra-low
latency network, big data processing, 5G edge computing, high-speed storage and other scenarios,
helping computing power become the new productivity of the digital era.
The company put forward innovative Software Defined Accelerator technology and independently
developed the chip architecture KPU(Kernel Processing Unit) and agile heterogeneous Software stack
(HADOS) for domain-specific computing (DSA).
Now, adding the feature of obtain vdpa device's mac address automatically for DPDK vDPA path.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically
2022-09-13 9:08 [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically Hao Chen
2022-09-15 7:52 ` houyl
@ 2022-09-20 2:18 ` Jason Wang
2022-09-21 5:28 ` Raphael Norwitz
1 sibling, 1 reply; 5+ messages in thread
From: Jason Wang @ 2022-09-20 2:18 UTC (permalink / raw)
To: Hao Chen
Cc: mst, raphael.norwitz, Kevin Wolf, Hanna Reitz,
open list:Block layer core, qemu-devel, houyl, zy, Cindy Lu
On Tue, Sep 13, 2022 at 5:13 PM Hao Chen <chenh@yusur.tech> wrote:
>
> When use dpdk-vdpa tests vdpa device. You need to specify the mac address to
> start the virtual machine through libvirt or qemu, but now, the libvirt or
> qemu can call dpdk vdpa vendor driver's ops .get_config through vhost_net_get_config
> to get the mac address of the vdpa hardware without manual configuration.
>
> Signed-off-by: Hao Chen <chenh@yusur.tech>
Adding Cindy for comments.
Thanks
> ---
> hw/block/vhost-user-blk.c | 1 -
> hw/net/virtio-net.c | 3 ++-
> hw/virtio/vhost-user.c | 19 -------------------
> 3 files changed, 2 insertions(+), 21 deletions(-)
>
> diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c
> index 9117222456..5dca4eab09 100644
> --- a/hw/block/vhost-user-blk.c
> +++ b/hw/block/vhost-user-blk.c
> @@ -337,7 +337,6 @@ static int vhost_user_blk_connect(DeviceState *dev, Error **errp)
>
> vhost_dev_set_config_notifier(&s->dev, &blk_ops);
>
> - s->vhost_user.supports_config = true;
> ret = vhost_dev_init(&s->dev, &s->vhost_user, VHOST_BACKEND_TYPE_USER, 0,
> errp);
> if (ret < 0) {
> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> index dd0d056fde..274ea84644 100644
> --- a/hw/net/virtio-net.c
> +++ b/hw/net/virtio-net.c
> @@ -149,7 +149,8 @@ static void virtio_net_get_config(VirtIODevice *vdev, uint8_t *config)
> * Is this VDPA? No peer means not VDPA: there's no way to
> * disconnect/reconnect a VDPA peer.
> */
> - if (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) {
> + if ((nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) ||
> + (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_USER)) {
> ret = vhost_net_get_config(get_vhost_net(nc->peer), (uint8_t *)&netcfg,
> n->config_size);
> if (ret != -1) {
> diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
> index bd24741be8..8b01078249 100644
> --- a/hw/virtio/vhost-user.c
> +++ b/hw/virtio/vhost-user.c
> @@ -2013,8 +2013,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
> }
>
> if (virtio_has_feature(features, VHOST_USER_F_PROTOCOL_FEATURES)) {
> - bool supports_f_config = vus->supports_config ||
> - (dev->config_ops && dev->config_ops->vhost_dev_config_notifier);
> uint64_t protocol_features;
>
> dev->backend_features |= 1ULL << VHOST_USER_F_PROTOCOL_FEATURES;
> @@ -2033,23 +2031,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
> */
> protocol_features &= VHOST_USER_PROTOCOL_FEATURE_MASK;
>
> - if (supports_f_config) {
> - if (!virtio_has_feature(protocol_features,
> - VHOST_USER_PROTOCOL_F_CONFIG)) {
> - error_setg(errp, "vhost-user device expecting "
> - "VHOST_USER_PROTOCOL_F_CONFIG but the vhost-user backend does "
> - "not support it.");
> - return -EPROTO;
> - }
> - } else {
> - if (virtio_has_feature(protocol_features,
> - VHOST_USER_PROTOCOL_F_CONFIG)) {
> - warn_reportf_err(*errp, "vhost-user backend supports "
> - "VHOST_USER_PROTOCOL_F_CONFIG but QEMU does not.");
> - protocol_features &= ~(1ULL << VHOST_USER_PROTOCOL_F_CONFIG);
> - }
> - }
> -
> /* final set of protocol features */
> dev->protocol_features = protocol_features;
> err = vhost_user_set_protocol_features(dev, dev->protocol_features);
> --
> 2.27.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically
2022-09-20 2:18 ` Jason Wang
@ 2022-09-21 5:28 ` Raphael Norwitz
2022-09-21 6:26 ` 陈浩
0 siblings, 1 reply; 5+ messages in thread
From: Raphael Norwitz @ 2022-09-21 5:28 UTC (permalink / raw)
To: Jason Wang, Hao Chen
Cc: mst, Kevin Wolf, Hanna Reitz, open list:Block layer core,
qemu-devel, houyl@yusur.tech, zy@yusur.tech, Cindy Lu
[-- Attachment #1: Type: text/plain, Size: 4445 bytes --]
I have some concerns from the vhost-user-blk side.
>On Tue, Sep 13, 2022 at 5:13 PM Hao Chen <chenh@yusur.tech> wrote:
>>
>> When use dpdk-vdpa tests vdpa device. You need to specify the mac address to
>> start the virtual machine through libvirt or qemu, but now, the libvirt or
>> qemu can call dpdk vdpa vendor driver's ops .get_config through vhost_net_get_config
>> to get the mac address of the vdpa hardware without manual configuration.
>>
>> Signed-off-by: Hao Chen <chenh@yusur.tech>
>
>Adding Cindy for comments.
>
>Thanks
>
>> ---
>> hw/block/vhost-user-blk.c | 1 -
>> hw/net/virtio-net.c | 3 ++-
>> hw/virtio/vhost-user.c | 19 -------------------
>> 3 files changed, 2 insertions(+), 21 deletions(-)
>>
>> diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c
>> index 9117222456..5dca4eab09 100644
>> --- a/hw/block/vhost-user-blk.c
>> +++ b/hw/block/vhost-user-blk.c
>> @@ -337,7 +337,6 @@ static int vhost_user_blk_connect(DeviceState *dev, Error **errp)
>>
>> vhost_dev_set_config_notifier(&s->dev, &blk_ops);
>>
>> - s->vhost_user.supports_config = true;
vhost-user-blk requires the backend to support VHOST_USER_PROTOCOL_F_CONFIG
and vhost_user.supports_config is used to enforce that.
Why are you removing it here?
>> ret = vhost_dev_init(&s->dev, &s->vhost_user, VHOST_BACKEND_TYPE_USER, 0,
>> errp);
>> if (ret < 0) {
>> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
>> index dd0d056fde..274ea84644 100644
>> --- a/hw/net/virtio-net.c
>> +++ b/hw/net/virtio-net.c
>> @@ -149,7 +149,8 @@ static void virtio_net_get_config(VirtIODevice *vdev, uint8_t *config)
>> * Is this VDPA? No peer means not VDPA: there's no way to
>> * disconnect/reconnect a VDPA peer.
>> */
>> - if (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) {
>> + if ((nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) ||
>> + (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_USER)) {
>> ret = vhost_net_get_config(get_vhost_net(nc->peer), (uint8_t *)&netcfg,
>> n->config_size);
>> if (ret != -1) {
>> diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
>> index bd24741be8..8b01078249 100644
>> --- a/hw/virtio/vhost-user.c
>> +++ b/hw/virtio/vhost-user.c
>> @@ -2013,8 +2013,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
>> }
>>
Why are you removing this? Can you expand on how it helps dpdk-vdpa.
>> if (virtio_has_feature(features, VHOST_USER_F_PROTOCOL_FEATURES)) {
>> - bool supports_f_config = vus->supports_config ||
>> - (dev->config_ops && dev->config_ops->vhost_dev_config_notifier);
>> uint64_t protocol_features;
>>
>> dev->backend_features |= 1ULL << VHOST_USER_F_PROTOCOL_FEATURES;
>> @@ -2033,23 +2031,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
>> */
>> protocol_features &= VHOST_USER_PROTOCOL_FEATURE_MASK;
>>
>> - if (supports_f_config) {
>> - if (!virtio_has_feature(protocol_features,
>> - VHOST_USER_PROTOCOL_F_CONFIG)) {
>> - error_setg(errp, "vhost-user device expecting "
>> - "VHOST_USER_PROTOCOL_F_CONFIG but the vhost-user backend does "
>> - "not support it.");
>> - return -EPROTO;
>> - }
>> - } else {
>> - if (virtio_has_feature(protocol_features,
>> - VHOST_USER_PROTOCOL_F_CONFIG)) {
>> - warn_reportf_err(*errp, "vhost-user backend supports "
>> - "VHOST_USER_PROTOCOL_F_CONFIG but QEMU does not.");
>> - protocol_features &= ~(1ULL << VHOST_USER_PROTOCOL_F_CONFIG);
>> - }
>> - }
>> -
>> /* final set of protocol features */
>> dev->protocol_features = protocol_features;
>> err = vhost_user_set_protocol_features(dev, dev->protocol_features);
>> --
>> 2.27.0
>>
>
[-- Attachment #2: Type: text/html, Size: 15575 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically
2022-09-21 5:28 ` Raphael Norwitz
@ 2022-09-21 6:26 ` 陈浩
0 siblings, 0 replies; 5+ messages in thread
From: 陈浩 @ 2022-09-21 6:26 UTC (permalink / raw)
To: Raphael Norwitz, Jason Wang
Cc: mst, Kevin Wolf, Hanna Reitz, open list:Block layer core,
qemu-devel, houyl@yusur.tech, zy@yusur.tech, Cindy Lu
[-- Attachment #1: Type: text/plain, Size: 5268 bytes --]
在 2022/9/21 13:28, Raphael Norwitz 写道:
>
> I have some concerns from the vhost-user-blk side.
>
> >On Tue, Sep 13, 2022 at 5:13 PM Hao Chen <chenh@yusur.tech> wrote:
>
> >>
>
> >> When use dpdk-vdpa tests vdpa device. You need to specify the mac address to
>
> >> start the virtual machine through libvirt or qemu, but now, the libvirt or
>
> >> qemu can call dpdk vdpa vendor driver's ops .get_config through vhost_net_get_config
>
> >> to get the mac address of the vdpa hardware without manual configuration.
>
> >>
>
> >> Signed-off-by: Hao Chen <chenh@yusur.tech>
>
> >
>
> >Adding Cindy for comments.
>
> >
>
> >Thanks
>
> >
>
> >> ---
>
> >>hw/block/vhost-user-blk.c |1 -
>
> >>hw/net/virtio-net.c |3 ++-
>
> >>hw/virtio/vhost-user.c| 19 -------------------
>
> >>3 files changed, 2 insertions(+), 21 deletions(-)
>
> >>
>
> >> diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c
>
> >> index 9117222456..5dca4eab09 100644
>
> >> --- a/hw/block/vhost-user-blk.c
>
> >> +++ b/hw/block/vhost-user-blk.c
>
> >> @@ -337,7 +337,6 @@ static int vhost_user_blk_connect(DeviceState *dev, Error **errp)
>
> >>
>
> >>vhost_dev_set_config_notifier(&s->dev, &blk_ops);
>
> >>
>
> >> -s->vhost_user.supports_config = true;
>
> vhost-user-blk requires the backend to support
> VHOST_USER_PROTOCOL_F_CONFIG
>
> and vhost_user.supports_config is used to enforce that.
>
> Why are you removing it here?
>
> >>ret = vhost_dev_init(&s->dev, &s->vhost_user, VHOST_BACKEND_TYPE_USER, 0,
>
> >> errp);
>
> >>if (ret < 0) {
>
> >> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
>
> >> index dd0d056fde..274ea84644 100644
>
> >> --- a/hw/net/virtio-net.c
>
> >> +++ b/hw/net/virtio-net.c
>
> >> @@ -149,7 +149,8 @@ static void virtio_net_get_config(VirtIODevice *vdev, uint8_t *config)
>
> >> * Is this VDPA? No peer means not VDPA: there's no way to
>
> >> * disconnect/reconnect a VDPA peer.
>
> >> */
>
> >> -if (nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) {
>
> >> +if ((nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_VDPA) ||
>
> >> +(nc->peer && nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_USER)) {
>
> >>ret = vhost_net_get_config(get_vhost_net(nc->peer), (uint8_t *)&netcfg,
>
> >> n->config_size);
>
> >>if (ret != -1) {
>
> >> diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
>
> >> index bd24741be8..8b01078249 100644
>
> >> --- a/hw/virtio/vhost-user.c
>
> >> +++ b/hw/virtio/vhost-user.c
>
> >> @@ -2013,8 +2013,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
>
> >>}
>
> >>
>
> Why are you removing this? Can you expand on how it helps dpdk-vdpa.
>
I implemented a function by modifying these places in the patch, and
then implementing vhost msg ops '.get_config' in the driver under
'DPDK_SRC/drivers/vdpa/' , qemu can use 'vhost_user_get_config' to send
vhost msg to calls the dpdk vdpa vendor driver's ops 'get_config' to
obtain the mac address information of the vdpa device. In this way, even
if the mac address of the network device parameters configured by the
user for qemu vhost user is inconsistent with the mac address of the
vdpa hardware, qemu can be corrected to be consistent with the mac
address of the vdpa hardware.
When vhost-user interface is used by dpdk-vdpa & qemu,
'vhost_user_get_config' will get the vdpa device's mac address if dpdk
vdpa vendor driver implement 'get_config',otherwise, it won't do anything.
When vhost-user interface is used by dpdk vhost-user, it won't do anything.
I think the functions provided by the VHOST_USER_PROTOCOL_F_CONFIG
feature can be available separately, but the check here caused
VHOST_USER_PROTOCOL_F_CONFIG feature is cleared, which will cause
vhost_user_get_config is not working properly in virtio-net at
hw/net/virtio_net.c.
Maybe the check here should be moved other side?
The new patch v2 has been sent, please reply there.
> >>if (virtio_has_feature(features, VHOST_USER_F_PROTOCOL_FEATURES)) {
>
> >> -bool supports_f_config = vus->supports_config ||
>
> >> -(dev->config_ops && dev->config_ops->vhost_dev_config_notifier);
>
> >>uint64_t protocol_features;
>
> >>
>
> >>dev->backend_features |= 1ULL << VHOST_USER_F_PROTOCOL_FEATURES;
>
> >> @@ -2033,23 +2031,6 @@ static int vhost_user_backend_init(struct vhost_dev *dev, void *opaque,
>
> >> */
>
> >>protocol_features &= VHOST_USER_PROTOCOL_FEATURE_MASK;
>
> >>
>
> >> -if (supports_f_config) {
>
> >> -if (!virtio_has_feature(protocol_features,
>
> >> -VHOST_USER_PROTOCOL_F_CONFIG)) {
>
> >> -error_setg(errp, "vhost-user device expecting "
>
> >> - "VHOST_USER_PROTOCOL_F_CONFIG but the vhost-user backend does "
>
> >> - "not support it.");
>
> >> -return -EPROTO;
>
> >> -}
>
> >> -} else {
>
> >> -if (virtio_has_feature(protocol_features,
>
> >> - VHOST_USER_PROTOCOL_F_CONFIG)) {
>
> >> -warn_reportf_err(*errp, "vhost-user backend supports "
>
> >> - "VHOST_USER_PROTOCOL_F_CONFIG but QEMU does not.");
>
> >> -protocol_features &= ~(1ULL << VHOST_USER_PROTOCOL_F_CONFIG);
>
> >> -}
>
> >> -}
>
> >> -
>
> >>/* final set of protocol features */
>
> >>dev->protocol_features = protocol_features;
>
> >>err = vhost_user_set_protocol_features(dev, dev->protocol_features);
>
> >> --
>
> >> 2.27.0
>
> >>
>
> >
>
[-- Attachment #2: Type: text/html, Size: 19114 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-09-21 6:34 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-13 9:08 [PATCH] hw/virtio/vhost-user: support obtain vdpa device's mac address automatically Hao Chen
2022-09-15 7:52 ` houyl
2022-09-20 2:18 ` Jason Wang
2022-09-21 5:28 ` Raphael Norwitz
2022-09-21 6:26 ` 陈浩
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).