2017-01-18 16:07, Shreyansh Jain: > +void > +rte_bus_register(struct rte_bus *bus) > +{ > + RTE_VERIFY(bus); > + RTE_VERIFY(bus->name && strlen(bus->name)); > + > + TAILQ_INSERT_TAIL(&rte_bus_list, bus, next); > + RTE_LOG(DEBUG, EAL, "Registered [%s] bus.\n", bus->name); > +} > + > +void > +rte_bus_unregister(struct rte_bus *bus) > +{ > + TAILQ_REMOVE(&rte_bus_list, bus, next); > + RTE_LOG(INFO, EAL, "Unregistered [%s] bus.\n", bus->name); > +}
This log should be DEBUG, consistently with "register" one. > +DPDK_17.02 { > + global: > + > + rte_bus_list; So finally, is it an useful symbol? Could it be removed from this patch?