On 7/16/2024 11:53 AM, Mingjin Ye wrote:
In secondary processes, insert_vdev() may be called multiple times on the same device due to multi-process hot-plugging of the vdev bus and EAL parameters to add the same vdev.In this case, when rte_devargs_insert() is called, the devargs->name reference will be invalidated because rte_devargs_insert() destroys the just-allocated devargs and replaces the pointer from the devargs list. As a result, the reference to devargs->name stored in dev->device.name will be invalid. This patch fixes the issue by setting the device name after calling rte_devargs_insert(). Fixes: cdb068f031c6 ("bus/vdev: scan by multi-process channel") Cc: [email protected] Signed-off-by: Mingjin Ye <[email protected]> --- v2: Modify commit log. ---
Forgot to add my review tag: Reviewed-by: Anatoly Burakov <[email protected]> -- Thanks, Anatoly
