01/09/2023 09:24, Mingjin Ye:
> When a device is created by a secondary process, an empty devargs is
> temporarily generated and bound to it. This causes the device to not
> be associated with the correct devargs, and the empty devargs are not
> released when the resource is freed.
> 
> This patch fixes the issue by matching the devargs when inserting a
> device in secondary process.
> 
> Fixes: dda987315ca2 ("vdev: make virtual bus use its device struct")
> Fixes: a16040453968 ("eal: extract vdev infra")
> Cc: sta...@dpdk.org
> 
> Signed-off-by: Mingjin Ye <mingjinx...@intel.com>
> ---
> +static struct rte_devargs *
> +vdev_devargs_lookup(const char *name)
> +{
> +     struct rte_devargs *devargs;
> +     char dev_name[32];
> +
> +     RTE_EAL_DEVARGS_FOREACH("vdev", devargs) {
> +             devargs->bus->parse(devargs->name, &dev_name);
> +             if (strcmp(dev_name, name) == 0) {
> +                     VDEV_LOG(INFO, "**Devargs matched %s", dev_name);
The beginning of the log does not need such attention characters.
Removing while merging.

With Acked-by: Anatoly Burakov <anatoly.bura...@intel.com>

Applied, thanks.



Reply via email to