On Thu, Aug 29, 2019 at 09:59:51AM +0200, Maxime Coquelin wrote: > + > +RTE_PMD_REGISTER_PCI(net_virtio_vdpa, rte_virtio_vdpa); > +RTE_PMD_REGISTER_PCI_TABLE(net_virtio_vdpa, pci_id_virtio_vdpa_map); > +RTE_PMD_REGISTER_KMOD_DEP(net_virtio_vdpa, "* vfio-pci"); > + > +RTE_INIT(virtio_vdpa_init_log) > +{ > + virtio_vdpa_logtype = rte_log_register("pmd.net.virtio_vdpa");
We need to use pmd.net.virtio.init to enable the logs in virtio_pci.c when using virtio vdpa. Maybe naming above logtype e.g. "pmd.net.virtio.vdpa" will be more consistent? Regards, Tiwei > + if (virtio_vdpa_logtype >= 0) > + rte_log_set_level(virtio_vdpa_logtype, RTE_LOG_NOTICE); > +} > + > -- > 2.21.0 >