Re: [PATCH] drivers: use dedicated logtypes

[Maxime Coquelin]

On 12/18/23 14:59, David Marchand wrote:

diff --git a/drivers/net/virtio/virtio_logs.h b/drivers/net/virtio/virtio_logs.h
index 9b1b1defc5..01911514bd 100644
--- a/drivers/net/virtio/virtio_logs.h
+++ b/drivers/net/virtio/virtio_logs.h
@@ -16,19 +16,20 @@ extern int virtio_logtype_init;

#ifdef RTE_LIBRTE_VIRTIO_DEBUG_RX

  #define PMD_RX_LOG(level, fmt, args...) \
-       RTE_LOG(level, PMD, "%s() rx: " fmt "\n", __func__, ## args)
+       RTE_LOG(level, VIRTIO_DRIVER, "%s() rx: " fmt "\n", __func__, ## args)
  #else
  #define PMD_RX_LOG(level, fmt, args...) do { } while(0)
  #endif

#ifdef RTE_LIBRTE_VIRTIO_DEBUG_TX

  #define PMD_TX_LOG(level, fmt, args...) \
-       RTE_LOG(level, PMD, "%s() tx: " fmt "\n", __func__, ## args)
+       RTE_LOG(level, VIRTIO_DRIVER, "%s() tx: " fmt "\n", __func__, ## args)
  #else
  #define PMD_TX_LOG(level, fmt, args...) do { } while(0)
  #endif

extern int virtio_logtype_driver;

+#define RTE_LOGTYPE_VIRTIO_DRIVER virtio_logtype_driver
  #define PMD_DRV_LOG(level, fmt, args...) \
        rte_log(RTE_LOG_ ## level, virtio_logtype_driver, \
                "%s(): " fmt "\n", __func__, ## args)

For Virtio part:

Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com>

Thanks,
Maxime