On 6/26/20 12:30 PM, Adrian Moreno wrote:
>> for (i = 0; i < MAX_VHOST_DEVICE; ++i) {
>> @@ -100,7 +100,7 @@ rte_vdpa_find_device_id(struct rte_vdpa_dev_addr *addr)
>> if (dev->ops == NULL)
>> continue;
>>
>> - if (is_same_vdpa_device(&dev->addr, addr))
>> + if (strcmp(dev->device->name, name) == 0)
>> return i;
>> }
> Considering this function is exposed to the API, I'd consider using
> strncmp(dev->device->name, name, RTE_DEV_NAME_MAX_LEN)
>
>
Right, I'll do that in v3.
Thanks,
Maxime
- [dpdk-dev] [PATCH v2 00/14] vDPA API and framework rework Maxime Coquelin
- [dpdk-dev] [PATCH v2 03/14] vhost: introduce vDPA dev... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v2 03/14] vhost: introduce ... Adrian Moreno
- [dpdk-dev] [PATCH v2 05/14] vhost: replace device ID ... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v2 05/14] vhost: replace de... Adrian Moreno
- [dpdk-dev] [PATCH v2 04/14] vhost: make vDPA framewor... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v2 04/14] vhost: make vDPA ... Adrian Moreno
- Re: [dpdk-dev] [PATCH v2 04/14] vhost: make v... Maxime Coquelin
- [dpdk-dev] [PATCH v2 01/14] bus/dpaa: fix null pointe... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v2 01/14] bus/dpaa: fix nul... Adrian Moreno
- Re: [dpdk-dev] [PATCH v2 01/14] bus/dpaa: fix... Maxime Coquelin
- [dpdk-dev] [PATCH v2 06/14] vhost: replace vDPA devic... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v2 06/14] vhost: replace vD... Adrian Moreno
- [dpdk-dev] [PATCH v2 08/14] vhost: remove useless vDP... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v2 08/14] vhost: remove use... Adrian Moreno
- [dpdk-dev] [PATCH v2 10/12] examples/vdpa: use new wr... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v2 10/12] examples/vdpa: us... Maxime Coquelin
- [dpdk-dev] [PATCH v2 09/14] vhost: use linked-list fo... Maxime Coquelin

