FM10K/dpdk-fm10k
FM10K/dpdk-fm10k
1
0
Fork 0
Code Issues Pull requests Releases Activity

vdpa/ifc: support vring update after device config

Browse source 
The device ready state in vhost lib is now defined as the state
that first queue pair is ready. And kick/callfd may be updated
by QEMU when ifc device is configured.

Although now ifc driver only supports one queue pair, it still
has to update callfd when working with QEMU. This patch fixes
this vring update problem by implementing the set_vring_state
callback.

Suggested-by: Maxime Coquelin <maxime.coquelin@redhat.com>
Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
Acked-by: Xiao Wang <xiao.w.wang@intel.com>
Acked-by: Rosen Xu <rosen.xu@intel.com>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
This commit is contained in:
Chenbo Xia 2020-07-28 14:32:24 +00:00 committed by Ferruh Yigit
parent c47463272f
commit e2a1a08a76
2 changed files with 54 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
drivers/vdpa/ifc/base/ifcvf.h
View file

@ -115,6 +115,7 @@ struct vring_info {
u16 size;
u16 last_avail_idx;
u16 last_used_idx;
bool enable;
};
struct ifcvf_hw {

54
drivers/vdpa/ifc/ifcvf_vdpa.c
View file

@ -49,6 +49,7 @@ static const char * const ifcvf_valid_arguments[] = {
struct ifcvf_internal {
struct rte_pci_device *pdev;
struct ifcvf_hw hw;
int configured;
int vfio_container_fd;
int vfio_group_fd;
int vfio_dev_fd;
@ -897,6 +898,7 @@ ifcvf_dev_config(int vid)
DRV_LOG(NOTICE, "vDPA (%s): software relay is used.",
vdev->device->name);
internal->configured = 1;
return 0;
}
@ -935,6 +937,7 @@ ifcvf_dev_close(int vid)
update_datapath(internal);
}
internal->configured = 0;
return 0;
}
@ -1084,13 +1087,61 @@ ifcvf_get_protocol_features(struct rte_vdpa_device *vdev, uint64_t *features)
return 0;
}
static int
ifcvf_set_vring_state(int vid, int vring, int state)
{
struct rte_vdpa_device *vdev;
struct internal_list *list;
struct ifcvf_internal *internal;
struct ifcvf_hw *hw;
struct ifcvf_pci_common_cfg *cfg;
int ret = 0;
vdev = rte_vhost_get_vdpa_device(vid);
list = find_internal_resource_by_vdev(vdev);
if (list == NULL) {
DRV_LOG(ERR, "Invalid vDPA device: %p", vdev);
return -1;
}
internal = list->internal;
if (vring < 0 || vring >= internal->max_queues * 2) {
DRV_LOG(ERR, "Vring index %d not correct", vring);
return -1;
}
hw = &internal->hw;
if (!internal->configured)
goto exit;
cfg = hw->common_cfg;
IFCVF_WRITE_REG16(vring, &cfg->queue_select);
IFCVF_WRITE_REG16(!!state, &cfg->queue_enable);
if (!state && hw->vring[vring].enable) {
ret = vdpa_disable_vfio_intr(internal);
if (ret)
return ret;
}
if (state && !hw->vring[vring].enable) {
ret = vdpa_enable_vfio_intr(internal, 0);
if (ret)
return ret;
}
exit:
hw->vring[vring].enable = !!state;
return 0;
}
static struct rte_vdpa_dev_ops ifcvf_ops = {
.get_queue_num = ifcvf_get_queue_num,
.get_features = ifcvf_get_vdpa_features,
.get_protocol_features = ifcvf_get_protocol_features,
.dev_conf = ifcvf_dev_config,
.dev_close = ifcvf_dev_close,
.set_vring_state = NULL,
.set_vring_state = ifcvf_set_vring_state,
.set_features = ifcvf_set_features,
.migration_done = NULL,
.get_vfio_group_fd = ifcvf_get_vfio_group_fd,
@ -1170,6 +1221,7 @@ ifcvf_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
goto error;
}
internal->configured = 0;
internal->max_queues = IFCVF_MAX_QUEUES;
features = ifcvf_get_features(&internal->hw);
internal->features = (features &