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: [email protected]
>
> Signed-off-by: Mingjin Ye <[email protected]>
> ---
> +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 <[email protected]>
Applied, thanks.
