29/11/2018 08:16, Yahui Cao: > If duplicated vdev name is detected, print out a warning message. > > Signed-off-by: Yahui Cao <yahui....@intel.com> > --- > drivers/bus/vdev/vdev.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c > index 9c66bdc78..ff2db7d3f 100644 > --- a/drivers/bus/vdev/vdev.c > +++ b/drivers/bus/vdev/vdev.c > @@ -462,6 +462,8 @@ vdev_scan(void) > if (find_vdev(devargs->name)) { > rte_spinlock_recursive_unlock(&vdev_device_list_lock); > free(dev); > + VDEV_LOG(WARNING, "duplicated vdev name %s detected!", > + devargs->name); > continue; > }
I'm surprised there is nothing to prevent from creating 2 vdevs with the same name. It should be considered as an error and reject the vdev creation.