[PATCH] drivers: use dedicated logtypes

Maxime Coquelin maxime.coquelin at redhat.com
Fri Jan 12 14:53:32 CET 2024



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.coquelin at redhat.com>

Thanks,
Maxime



More information about the dev mailing list