DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype
@ 2020-03-31  4:41 Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 1/7] net/mvneta: do not use " Stephen Hemminger
                   ` (8 more replies)
  0 siblings, 9 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31  4:41 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger

While working on netvsc driver, noticed that there are still some places
in some drivers using the old PMD static logtype.  Would like to
eradicate all of these and even mark it as deprecated in a future
release.

Stephen Hemminger (7):
  net/mvneta: do not use PMD logtype
  net/virtio: do not use PMD logtype
  net/tap: do not use PMD logtype
  net/pfe: do not use PMD logtype
  net/bnxt: don't use PMD logtype
  net/dpaa: use dynamic logtype
  net/thunderx: use dynamic logtype

 drivers/net/bnxt/bnxt_ring.c            |  2 +-
 drivers/net/dpaa/dpaa_ethdev.c          | 24 +++++++++---------------
 drivers/net/mvneta/mvneta_ethdev.c      |  2 +-
 drivers/net/pfe/pfe_ethdev.c            |  4 ++--
 drivers/net/tap/rte_eth_tap.c           |  6 ++++--
 drivers/net/thunderx/nicvf_ethdev.c     | 17 +++++++++--------
 drivers/net/virtio/virtio_rxtx.c        |  6 +++---
 drivers/net/virtio/virtio_user_ethdev.c |  2 +-
 8 files changed, 30 insertions(+), 33 deletions(-)

-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH 1/7] net/mvneta: do not use PMD logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
@ 2020-03-31  4:41 ` Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 2/7] net/virtio: " Stephen Hemminger
                   ` (7 subsequent siblings)
  8 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31  4:41 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, nsamsono

The PMD logtype is legacy and should not be used.

Fixes: 3378383dceab ("net/mvneta: support statistics reset")
Cc: nsamsono@marvell.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/mvneta/mvneta_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/mvneta/mvneta_ethdev.c b/drivers/net/mvneta/mvneta_ethdev.c
index 865ad61aed1c..4aea87648814 100644
--- a/drivers/net/mvneta/mvneta_ethdev.c
+++ b/drivers/net/mvneta/mvneta_ethdev.c
@@ -751,7 +751,7 @@ mvneta_stats_reset(struct rte_eth_dev *dev)
 
 	ret = mvneta_stats_get(dev, &priv->prev_stats);
 	if (unlikely(ret))
-		RTE_LOG(ERR, PMD, "Failed to reset port statistics");
+		MVNETA_LOG(ERR, "Failed to reset port statistics");
 
 	return ret;
 }
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH 2/7] net/virtio: do not use PMD logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 1/7] net/mvneta: do not use " Stephen Hemminger
@ 2020-03-31  4:41 ` Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 3/7] net/tap: " Stephen Hemminger
                   ` (6 subsequent siblings)
  8 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31  4:41 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger

Virtio driver has its own logtype adn should not use legacy
PMD logtype.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/virtio/virtio_rxtx.c        | 6 +++---
 drivers/net/virtio/virtio_user_ethdev.c | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/net/virtio/virtio_rxtx.c b/drivers/net/virtio/virtio_rxtx.c
index 752faa0f6e7e..060410577af4 100644
--- a/drivers/net/virtio/virtio_rxtx.c
+++ b/drivers/net/virtio/virtio_rxtx.c
@@ -1085,7 +1085,7 @@ virtio_dev_tx_queue_setup(struct rte_eth_dev *dev,
 			RTE_MIN(vq->vq_nentries / 4, DEFAULT_TX_FREE_THRESH);
 
 	if (tx_free_thresh >= (vq->vq_nentries - 3)) {
-		RTE_LOG(ERR, PMD, "tx_free_thresh must be less than the "
+		PMD_DRV_LOG(ERR, "tx_free_thresh must be less than the "
 			"number of TX entries minus 3 (%u)."
 			" (tx_free_thresh=%u port=%u queue=%u)\n",
 			vq->vq_nentries - 3,
@@ -1133,7 +1133,7 @@ virtio_discard_rxbuf(struct virtqueue *vq, struct rte_mbuf *m)
 		error = virtqueue_enqueue_recv_refill(vq, &m, 1);
 
 	if (unlikely(error)) {
-		RTE_LOG(ERR, PMD, "cannot requeue discarded mbuf");
+		PMD_DRV_LOG(ERR, "cannot requeue discarded mbuf");
 		rte_pktmbuf_free(m);
 	}
 }
@@ -1145,7 +1145,7 @@ virtio_discard_rxbuf_inorder(struct virtqueue *vq, struct rte_mbuf *m)
 
 	error = virtqueue_enqueue_refill_inorder(vq, &m, 1);
 	if (unlikely(error)) {
-		RTE_LOG(ERR, PMD, "cannot requeue discarded mbuf");
+		PMD_DRV_LOG(ERR, "cannot requeue discarded mbuf");
 		rte_pktmbuf_free(m);
 	}
 }
diff --git a/drivers/net/virtio/virtio_user_ethdev.c b/drivers/net/virtio/virtio_user_ethdev.c
index e61af4068322..4f565f913bf2 100644
--- a/drivers/net/virtio/virtio_user_ethdev.c
+++ b/drivers/net/virtio/virtio_user_ethdev.c
@@ -561,7 +561,7 @@ virtio_user_pmd_probe(struct rte_vdev_device *dev)
 		const char *name = rte_vdev_device_name(dev);
 		eth_dev = rte_eth_dev_attach_secondary(name);
 		if (!eth_dev) {
-			RTE_LOG(ERR, PMD, "Failed to probe %s\n", name);
+			PMD_INIT_LOG(ERR,"Failed to probe %s", name);
 			return -1;
 		}
 
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH 3/7] net/tap: do not use PMD logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 1/7] net/mvneta: do not use " Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 2/7] net/virtio: " Stephen Hemminger
@ 2020-03-31  4:41 ` Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 4/7] net/pfe: " Stephen Hemminger
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31  4:41 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, ophirmu

The PMD logtype is legacy and drivers should use their own logtype.

Fixes: 050316a88313 ("net/tap: support TSO (TCP Segment Offload)")
Cc: ophirmu@mellanox.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/tap/rte_eth_tap.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/net/tap/rte_eth_tap.c b/drivers/net/tap/rte_eth_tap.c
index 05470a211588..2c9fb76b63de 100644
--- a/drivers/net/tap/rte_eth_tap.c
+++ b/drivers/net/tap/rte_eth_tap.c
@@ -778,7 +778,7 @@ tap_ioctl(struct pmd_internals *pmd, unsigned long request,
 	case SIOCSIFMTU:
 		break;
 	default:
-		RTE_LOG(WARNING, PMD, "%s: ioctl() called with wrong arg\n",
+		TAP_LOG(WARNING, "%s: ioctl() called with wrong arg",
 			pmd->name);
 		return -EINVAL;
 	}
@@ -1301,7 +1301,9 @@ tap_gso_ctx_setup(struct rte_gso_ctx *gso_ctx, struct rte_eth_dev *dev)
 			SOCKET_ID_ANY);
 		if (!mp) {
 			struct pmd_internals *pmd = dev->data->dev_private;
-			RTE_LOG(DEBUG, PMD, "%s: failed to create mbuf pool for device %s\n",
+
+			TAP_LOG(ERR,
+				"%s: failed to create mbuf pool for device %s\n",
 				pmd->name, dev->device->name);
 			return -1;
 		}
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH 4/7] net/pfe: do not use PMD logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
                   ` (2 preceding siblings ...)
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 3/7] net/tap: " Stephen Hemminger
@ 2020-03-31  4:41 ` Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 5/7] net/bnxt: don't " Stephen Hemminger
                   ` (4 subsequent siblings)
  8 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31  4:41 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, g.singh

The PMD logtype is a legacy from original DPDK logging.
All drivers must use their own dynamic log type.

Fixes: b1bc1afa4a0e ("net/pfe: support dynamic logging")
Cc: g.singh@nxp.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/pfe/pfe_ethdev.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/pfe/pfe_ethdev.c b/drivers/net/pfe/pfe_ethdev.c
index 940347819875..59b5b8387b9d 100644
--- a/drivers/net/pfe/pfe_ethdev.c
+++ b/drivers/net/pfe/pfe_ethdev.c
@@ -990,7 +990,7 @@ pmd_pfe_probe(struct rte_vdev_device *vdev)
 	if (rc < 0)
 		return -EINVAL;
 
-	RTE_LOG(INFO, PMD, "Initializing pmd_pfe for %s Given gem-id %d\n",
+	PFE_PMD_LOG(INFO, "Initializing pmd_pfe for %s Given gem-id %d",
 		name, init_params.gem_id);
 
 	if (g_pfe) {
@@ -1118,7 +1118,7 @@ pmd_pfe_probe(struct rte_vdev_device *vdev)
 	else
 		gem_id = init_params.gem_id;
 
-	RTE_LOG(INFO, PMD, "Init pmd_pfe for %s gem-id %d(given =%d)\n",
+	PFE_PMD_LOG(INFO, "Init pmd_pfe for %s gem-id %d(given =%d)",
 		name, gem_id, init_params.gem_id);
 
 	rc = pfe_eth_init(vdev, g_pfe, gem_id);
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH 5/7] net/bnxt: don't use PMD logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
                   ` (3 preceding siblings ...)
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 4/7] net/pfe: " Stephen Hemminger
@ 2020-03-31  4:41 ` Stephen Hemminger
  2020-03-31  4:43   ` Somnath Kotur
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 6/7] net/dpaa: use dynamic logtype Stephen Hemminger
                   ` (3 subsequent siblings)
  8 siblings, 1 reply; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31  4:41 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, ajit.khaparde

Accidental use of PMD logtype rather than per-driver logtype.

Fixes: 14255b351537 ("net/bnxt: fix queue start/stop operations")
Cc: ajit.khaparde@broadcom.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/bnxt/bnxt_ring.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/bnxt_ring.c b/drivers/net/bnxt/bnxt_ring.c
index d6e4e8a28138..1999cd7861b8 100644
--- a/drivers/net/bnxt/bnxt_ring.c
+++ b/drivers/net/bnxt/bnxt_ring.c
@@ -598,7 +598,7 @@ int bnxt_alloc_hwrm_rx_ring(struct bnxt *bp, int queue_index)
 
 	if (rxq->rx_started) {
 		if (bnxt_init_one_rx_ring(rxq)) {
-			RTE_LOG(ERR, PMD,
+			PMD_DRV_LOG(ERR,
 				"bnxt_init_one_rx_ring failed!\n");
 			bnxt_rx_queue_release_op(rxq);
 			rc = -ENOMEM;
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH 6/7] net/dpaa: use dynamic logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
                   ` (4 preceding siblings ...)
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 5/7] net/bnxt: don't " Stephen Hemminger
@ 2020-03-31  4:41 ` Stephen Hemminger
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 7/7] net/thunderx: " Stephen Hemminger
                   ` (2 subsequent siblings)
  8 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31  4:41 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, shreyansh.jain

The static PMD logtype should not be used by drivers.
Instead, use existing log macros in this driver.

Also use standard rte_ether routine to format ether address.

Fixes: 37f9b54bd3cf ("net/dpaa: support Tx and Rx queue setup")
Cc: shreyansh.jain@nxp.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/dpaa/dpaa_ethdev.c | 24 +++++++++---------------
 1 file changed, 9 insertions(+), 15 deletions(-)

diff --git a/drivers/net/dpaa/dpaa_ethdev.c b/drivers/net/dpaa/dpaa_ethdev.c
index fce9ce2feb9d..f59c2631a56c 100644
--- a/drivers/net/dpaa/dpaa_ethdev.c
+++ b/drivers/net/dpaa/dpaa_ethdev.c
@@ -881,8 +881,8 @@ dpaa_dev_rx_queue_count(struct rte_eth_dev *dev, uint16_t rx_queue_id)
 	PMD_INIT_FUNC_TRACE();
 
 	if (qman_query_fq_frm_cnt(rxq, &frm_cnt) == 0) {
-		RTE_LOG(DEBUG, PMD, "RX frame count for q(%d) is %u\n",
-			rx_queue_id, frm_cnt);
+		DPAA_PMD_DEBUG("RX frame count for q(%d) is %u",
+			       rx_queue_id, frm_cnt);
 	}
 	return frm_cnt;
 }
@@ -996,8 +996,7 @@ dpaa_dev_add_mac_addr(struct rte_eth_dev *dev,
 	ret = fman_if_add_mac_addr(dpaa_intf->fif, addr->addr_bytes, index);
 
 	if (ret)
-		RTE_LOG(ERR, PMD, "error: Adding the MAC ADDR failed:"
-			" err = %d", ret);
+		DPAA_PMD_ERR("Adding the MAC ADDR failed: err = %d", ret);
 	return 0;
 }
 
@@ -1023,7 +1022,7 @@ dpaa_dev_set_mac_addr(struct rte_eth_dev *dev,
 
 	ret = fman_if_add_mac_addr(dpaa_intf->fif, addr->addr_bytes, 0);
 	if (ret)
-		RTE_LOG(ERR, PMD, "error: Setting the MAC ADDR failed %d", ret);
+		DPAA_PMD_ERR("Setting the MAC ADDR failed %d", ret);
 
 	return ret;
 }
@@ -1306,6 +1305,7 @@ dpaa_dev_init(struct rte_eth_dev *eth_dev)
 	struct fman_if *fman_intf;
 	struct fman_if_bpool *bp, *tmp_bp;
 	uint32_t cgrid[DPAA_MAX_NUM_PCD_QUEUES];
+	char eth_buf[RTE_ETHER_ADDR_FMT_SIZE];
 
 	PMD_INIT_FUNC_TRACE();
 
@@ -1457,15 +1457,9 @@ dpaa_dev_init(struct rte_eth_dev *eth_dev)
 
 	/* copy the primary mac address */
 	rte_ether_addr_copy(&fman_intf->mac_addr, &eth_dev->data->mac_addrs[0]);
+	rte_ether_format_addr(eth_buf, sizeof(eth_buf), &fman_intf->mac_addr);
 
-	RTE_LOG(INFO, PMD, "net: dpaa: %s: %02x:%02x:%02x:%02x:%02x:%02x\n",
-		dpaa_device->name,
-		fman_intf->mac_addr.addr_bytes[0],
-		fman_intf->mac_addr.addr_bytes[1],
-		fman_intf->mac_addr.addr_bytes[2],
-		fman_intf->mac_addr.addr_bytes[3],
-		fman_intf->mac_addr.addr_bytes[4],
-		fman_intf->mac_addr.addr_bytes[5]);
+	DPAA_PMD_INFO("net: dpaa: %s: %s", dpaa_device->name, eth_buf);
 
 	/* Disable RX mode */
 	fman_if_discard_rx_errors(fman_intf);
@@ -1578,8 +1572,8 @@ rte_dpaa_probe(struct rte_dpaa_driver *dpaa_drv __rte_unused,
 
 	if (!is_global_init && (rte_eal_process_type() == RTE_PROC_PRIMARY)) {
 		if (access("/tmp/fmc.bin", F_OK) == -1) {
-			RTE_LOG(INFO, PMD,
-				"* FMC not configured.Enabling default mode\n");
+			DPAA_PMD_INFO(
+				"* FMC not configured.Enabling default mode");
 			default_q = 1;
 		}
 
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH 7/7] net/thunderx: use dynamic logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
                   ` (5 preceding siblings ...)
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 6/7] net/dpaa: use dynamic logtype Stephen Hemminger
@ 2020-03-31  4:41 ` Stephen Hemminger
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
  2020-03-31 17:42 ` [dpdk-dev] [PATCH 0/7] Remove " Ferruh Yigit
  8 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31  4:41 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, jerin.jacob

The PMD static logtype is original DPDK legacy and should not be used.

Fixes: 43362c6a7647 ("net/thunderx: support RSS and RETA query and update")
Cc: jerin.jacob@caviumnetworks.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/thunderx/nicvf_ethdev.c | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/drivers/net/thunderx/nicvf_ethdev.c b/drivers/net/thunderx/nicvf_ethdev.c
index 6f43541a5d42..00e8287fac38 100644
--- a/drivers/net/thunderx/nicvf_ethdev.c
+++ b/drivers/net/thunderx/nicvf_ethdev.c
@@ -496,9 +496,10 @@ nicvf_dev_reta_query(struct rte_eth_dev *dev,
 	int ret, i, j;
 
 	if (reta_size != NIC_MAX_RSS_IDR_TBL_SIZE) {
-		RTE_LOG(ERR, PMD, "The size of hash lookup table configured "
-			"(%d) doesn't match the number hardware can supported "
-			"(%d)", reta_size, NIC_MAX_RSS_IDR_TBL_SIZE);
+		PMD_DRV_LOG(ERR,
+			    "The size of hash lookup table configured "
+			    "(%u) doesn't match the number hardware can supported "
+			    "(%u)", reta_size, NIC_MAX_RSS_IDR_TBL_SIZE);
 		return -EINVAL;
 	}
 
@@ -526,9 +527,9 @@ nicvf_dev_reta_update(struct rte_eth_dev *dev,
 	int ret, i, j;
 
 	if (reta_size != NIC_MAX_RSS_IDR_TBL_SIZE) {
-		RTE_LOG(ERR, PMD, "The size of hash lookup table configured "
-			"(%d) doesn't match the number hardware can supported "
-			"(%d)", reta_size, NIC_MAX_RSS_IDR_TBL_SIZE);
+		PMD_DRV_LOG(ERR,"The size of hash lookup table configured "
+			"(%u) doesn't match the number hardware can supported "
+			"(%u)", reta_size, NIC_MAX_RSS_IDR_TBL_SIZE);
 		return -EINVAL;
 	}
 
@@ -569,8 +570,8 @@ nicvf_dev_rss_hash_update(struct rte_eth_dev *dev,
 
 	if (rss_conf->rss_key &&
 		rss_conf->rss_key_len != RSS_HASH_KEY_BYTE_SIZE) {
-		RTE_LOG(ERR, PMD, "Hash key size mismatch %d",
-				rss_conf->rss_key_len);
+		PMD_DRV_LOG(ERR, "Hash key size mismatch %u",
+			    rss_conf->rss_key_len);
 		return -EINVAL;
 	}
 
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [dpdk-dev] [PATCH 5/7] net/bnxt: don't use PMD logtype
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 5/7] net/bnxt: don't " Stephen Hemminger
@ 2020-03-31  4:43   ` Somnath Kotur
  0 siblings, 0 replies; 20+ messages in thread
From: Somnath Kotur @ 2020-03-31  4:43 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: dev, Ajit Kumar Khaparde

On Tue, Mar 31, 2020 at 10:12 AM Stephen Hemminger
<stephen@networkplumber.org> wrote:
>
> Accidental use of PMD logtype rather than per-driver logtype.
>
> Fixes: 14255b351537 ("net/bnxt: fix queue start/stop operations")
> Cc: ajit.khaparde@broadcom.com
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  drivers/net/bnxt/bnxt_ring.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/bnxt/bnxt_ring.c b/drivers/net/bnxt/bnxt_ring.c
> index d6e4e8a28138..1999cd7861b8 100644
> --- a/drivers/net/bnxt/bnxt_ring.c
> +++ b/drivers/net/bnxt/bnxt_ring.c
> @@ -598,7 +598,7 @@ int bnxt_alloc_hwrm_rx_ring(struct bnxt *bp, int queue_index)
>
>         if (rxq->rx_started) {
>                 if (bnxt_init_one_rx_ring(rxq)) {
> -                       RTE_LOG(ERR, PMD,
> +                       PMD_DRV_LOG(ERR,
>                                 "bnxt_init_one_rx_ring failed!\n");
>                         bnxt_rx_queue_release_op(rxq);
>                         rc = -ENOMEM;
> --
> 2.20.1
>
Acked-by: Somnath Kotur <somnath.kotur@broadcom.com>

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
                   ` (6 preceding siblings ...)
  2020-03-31  4:41 ` [dpdk-dev] [PATCH 7/7] net/thunderx: " Stephen Hemminger
@ 2020-03-31 17:31 ` Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 1/7] net/mvneta: do not use " Stephen Hemminger
                     ` (7 more replies)
  2020-03-31 17:42 ` [dpdk-dev] [PATCH 0/7] Remove " Ferruh Yigit
  8 siblings, 8 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31 17:31 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger

While working on netvsc driver, noticed that there are still some places
in some drivers using the old PMD static logtype.  Would like to
eradicate all of these and even mark it as deprecated in a future
release.

v2 - fix checkpatch whitespace complaints

Stephen Hemminger (7):
  net/mvneta: do not use PMD logtype
  net/virtio: do not use PMD logtype
  net/tap: do not use PMD logtype
  net/pfe: do not use PMD logtype
  net/bnxt: don't use PMD logtype
  net/dpaa: use dynamic logtype
  net/thunderx: use dynamic logtype

 drivers/net/bnxt/bnxt_ring.c            |  2 +-
 drivers/net/dpaa/dpaa_ethdev.c          | 23 ++++++++---------------
 drivers/net/mvneta/mvneta_ethdev.c      |  2 +-
 drivers/net/pfe/pfe_ethdev.c            |  4 ++--
 drivers/net/tap/rte_eth_tap.c           |  6 ++++--
 drivers/net/thunderx/nicvf_ethdev.c     | 17 +++++++++--------
 drivers/net/virtio/virtio_rxtx.c        |  6 +++---
 drivers/net/virtio/virtio_user_ethdev.c |  2 +-
 8 files changed, 29 insertions(+), 33 deletions(-)

-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH v2 1/7] net/mvneta: do not use PMD logtype
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
@ 2020-03-31 17:31   ` Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 2/7] net/virtio: " Stephen Hemminger
                     ` (6 subsequent siblings)
  7 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31 17:31 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, nsamsono

The PMD logtype is legacy and should not be used.

Fixes: 3378383dceab ("net/mvneta: support statistics reset")
Cc: nsamsono@marvell.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/mvneta/mvneta_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/mvneta/mvneta_ethdev.c b/drivers/net/mvneta/mvneta_ethdev.c
index 865ad61aed1c..4aea87648814 100644
--- a/drivers/net/mvneta/mvneta_ethdev.c
+++ b/drivers/net/mvneta/mvneta_ethdev.c
@@ -751,7 +751,7 @@ mvneta_stats_reset(struct rte_eth_dev *dev)
 
 	ret = mvneta_stats_get(dev, &priv->prev_stats);
 	if (unlikely(ret))
-		RTE_LOG(ERR, PMD, "Failed to reset port statistics");
+		MVNETA_LOG(ERR, "Failed to reset port statistics");
 
 	return ret;
 }
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH v2 2/7] net/virtio: do not use PMD logtype
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 1/7] net/mvneta: do not use " Stephen Hemminger
@ 2020-03-31 17:31   ` Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 3/7] net/tap: " Stephen Hemminger
                     ` (5 subsequent siblings)
  7 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31 17:31 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger

Virtio driver has its own logtype adn should not use legacy
PMD logtype.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/virtio/virtio_rxtx.c        | 6 +++---
 drivers/net/virtio/virtio_user_ethdev.c | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/net/virtio/virtio_rxtx.c b/drivers/net/virtio/virtio_rxtx.c
index 752faa0f6e7e..060410577af4 100644
--- a/drivers/net/virtio/virtio_rxtx.c
+++ b/drivers/net/virtio/virtio_rxtx.c
@@ -1085,7 +1085,7 @@ virtio_dev_tx_queue_setup(struct rte_eth_dev *dev,
 			RTE_MIN(vq->vq_nentries / 4, DEFAULT_TX_FREE_THRESH);
 
 	if (tx_free_thresh >= (vq->vq_nentries - 3)) {
-		RTE_LOG(ERR, PMD, "tx_free_thresh must be less than the "
+		PMD_DRV_LOG(ERR, "tx_free_thresh must be less than the "
 			"number of TX entries minus 3 (%u)."
 			" (tx_free_thresh=%u port=%u queue=%u)\n",
 			vq->vq_nentries - 3,
@@ -1133,7 +1133,7 @@ virtio_discard_rxbuf(struct virtqueue *vq, struct rte_mbuf *m)
 		error = virtqueue_enqueue_recv_refill(vq, &m, 1);
 
 	if (unlikely(error)) {
-		RTE_LOG(ERR, PMD, "cannot requeue discarded mbuf");
+		PMD_DRV_LOG(ERR, "cannot requeue discarded mbuf");
 		rte_pktmbuf_free(m);
 	}
 }
@@ -1145,7 +1145,7 @@ virtio_discard_rxbuf_inorder(struct virtqueue *vq, struct rte_mbuf *m)
 
 	error = virtqueue_enqueue_refill_inorder(vq, &m, 1);
 	if (unlikely(error)) {
-		RTE_LOG(ERR, PMD, "cannot requeue discarded mbuf");
+		PMD_DRV_LOG(ERR, "cannot requeue discarded mbuf");
 		rte_pktmbuf_free(m);
 	}
 }
diff --git a/drivers/net/virtio/virtio_user_ethdev.c b/drivers/net/virtio/virtio_user_ethdev.c
index e61af4068322..5637001dfccb 100644
--- a/drivers/net/virtio/virtio_user_ethdev.c
+++ b/drivers/net/virtio/virtio_user_ethdev.c
@@ -561,7 +561,7 @@ virtio_user_pmd_probe(struct rte_vdev_device *dev)
 		const char *name = rte_vdev_device_name(dev);
 		eth_dev = rte_eth_dev_attach_secondary(name);
 		if (!eth_dev) {
-			RTE_LOG(ERR, PMD, "Failed to probe %s\n", name);
+			PMD_INIT_LOG(ERR, "Failed to probe %s", name);
 			return -1;
 		}
 
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH v2 3/7] net/tap: do not use PMD logtype
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 1/7] net/mvneta: do not use " Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 2/7] net/virtio: " Stephen Hemminger
@ 2020-03-31 17:31   ` Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 4/7] net/pfe: " Stephen Hemminger
                     ` (4 subsequent siblings)
  7 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31 17:31 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, ophirmu

The PMD logtype is legacy and drivers should use their own logtype.

Fixes: 050316a88313 ("net/tap: support TSO (TCP Segment Offload)")
Cc: ophirmu@mellanox.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/tap/rte_eth_tap.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/net/tap/rte_eth_tap.c b/drivers/net/tap/rte_eth_tap.c
index 05470a211588..2c9fb76b63de 100644
--- a/drivers/net/tap/rte_eth_tap.c
+++ b/drivers/net/tap/rte_eth_tap.c
@@ -778,7 +778,7 @@ tap_ioctl(struct pmd_internals *pmd, unsigned long request,
 	case SIOCSIFMTU:
 		break;
 	default:
-		RTE_LOG(WARNING, PMD, "%s: ioctl() called with wrong arg\n",
+		TAP_LOG(WARNING, "%s: ioctl() called with wrong arg",
 			pmd->name);
 		return -EINVAL;
 	}
@@ -1301,7 +1301,9 @@ tap_gso_ctx_setup(struct rte_gso_ctx *gso_ctx, struct rte_eth_dev *dev)
 			SOCKET_ID_ANY);
 		if (!mp) {
 			struct pmd_internals *pmd = dev->data->dev_private;
-			RTE_LOG(DEBUG, PMD, "%s: failed to create mbuf pool for device %s\n",
+
+			TAP_LOG(ERR,
+				"%s: failed to create mbuf pool for device %s\n",
 				pmd->name, dev->device->name);
 			return -1;
 		}
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH v2 4/7] net/pfe: do not use PMD logtype
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
                     ` (2 preceding siblings ...)
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 3/7] net/tap: " Stephen Hemminger
@ 2020-03-31 17:31   ` Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 5/7] net/bnxt: don't " Stephen Hemminger
                     ` (3 subsequent siblings)
  7 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31 17:31 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, g.singh

The PMD logtype is a legacy from original DPDK logging.
All drivers must use their own dynamic log type.

Fixes: b1bc1afa4a0e ("net/pfe: support dynamic logging")
Cc: g.singh@nxp.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/pfe/pfe_ethdev.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/pfe/pfe_ethdev.c b/drivers/net/pfe/pfe_ethdev.c
index 940347819875..59b5b8387b9d 100644
--- a/drivers/net/pfe/pfe_ethdev.c
+++ b/drivers/net/pfe/pfe_ethdev.c
@@ -990,7 +990,7 @@ pmd_pfe_probe(struct rte_vdev_device *vdev)
 	if (rc < 0)
 		return -EINVAL;
 
-	RTE_LOG(INFO, PMD, "Initializing pmd_pfe for %s Given gem-id %d\n",
+	PFE_PMD_LOG(INFO, "Initializing pmd_pfe for %s Given gem-id %d",
 		name, init_params.gem_id);
 
 	if (g_pfe) {
@@ -1118,7 +1118,7 @@ pmd_pfe_probe(struct rte_vdev_device *vdev)
 	else
 		gem_id = init_params.gem_id;
 
-	RTE_LOG(INFO, PMD, "Init pmd_pfe for %s gem-id %d(given =%d)\n",
+	PFE_PMD_LOG(INFO, "Init pmd_pfe for %s gem-id %d(given =%d)",
 		name, gem_id, init_params.gem_id);
 
 	rc = pfe_eth_init(vdev, g_pfe, gem_id);
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH v2 5/7] net/bnxt: don't use PMD logtype
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
                     ` (3 preceding siblings ...)
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 4/7] net/pfe: " Stephen Hemminger
@ 2020-03-31 17:31   ` Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 6/7] net/dpaa: use dynamic logtype Stephen Hemminger
                     ` (2 subsequent siblings)
  7 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31 17:31 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, ajit.khaparde

Accidental use of PMD logtype rather than per-driver logtype.

Fixes: 14255b351537 ("net/bnxt: fix queue start/stop operations")
Cc: ajit.khaparde@broadcom.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/bnxt/bnxt_ring.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/bnxt_ring.c b/drivers/net/bnxt/bnxt_ring.c
index d6e4e8a28138..1999cd7861b8 100644
--- a/drivers/net/bnxt/bnxt_ring.c
+++ b/drivers/net/bnxt/bnxt_ring.c
@@ -598,7 +598,7 @@ int bnxt_alloc_hwrm_rx_ring(struct bnxt *bp, int queue_index)
 
 	if (rxq->rx_started) {
 		if (bnxt_init_one_rx_ring(rxq)) {
-			RTE_LOG(ERR, PMD,
+			PMD_DRV_LOG(ERR,
 				"bnxt_init_one_rx_ring failed!\n");
 			bnxt_rx_queue_release_op(rxq);
 			rc = -ENOMEM;
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH v2 6/7] net/dpaa: use dynamic logtype
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
                     ` (4 preceding siblings ...)
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 5/7] net/bnxt: don't " Stephen Hemminger
@ 2020-03-31 17:31   ` Stephen Hemminger
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 7/7] net/thunderx: " Stephen Hemminger
  2020-03-31 17:43   ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Ferruh Yigit
  7 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31 17:31 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, shreyansh.jain

The static PMD logtype should not be used by drivers.
Instead, use existing log macros in this driver.

Also use standard rte_ether routine to format ether address.

Fixes: 37f9b54bd3cf ("net/dpaa: support Tx and Rx queue setup")
Cc: shreyansh.jain@nxp.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/dpaa/dpaa_ethdev.c | 23 ++++++++---------------
 1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/drivers/net/dpaa/dpaa_ethdev.c b/drivers/net/dpaa/dpaa_ethdev.c
index fce9ce2feb9d..5f81968d80e1 100644
--- a/drivers/net/dpaa/dpaa_ethdev.c
+++ b/drivers/net/dpaa/dpaa_ethdev.c
@@ -881,8 +881,8 @@ dpaa_dev_rx_queue_count(struct rte_eth_dev *dev, uint16_t rx_queue_id)
 	PMD_INIT_FUNC_TRACE();
 
 	if (qman_query_fq_frm_cnt(rxq, &frm_cnt) == 0) {
-		RTE_LOG(DEBUG, PMD, "RX frame count for q(%d) is %u\n",
-			rx_queue_id, frm_cnt);
+		DPAA_PMD_DEBUG("RX frame count for q(%d) is %u",
+			       rx_queue_id, frm_cnt);
 	}
 	return frm_cnt;
 }
@@ -996,8 +996,7 @@ dpaa_dev_add_mac_addr(struct rte_eth_dev *dev,
 	ret = fman_if_add_mac_addr(dpaa_intf->fif, addr->addr_bytes, index);
 
 	if (ret)
-		RTE_LOG(ERR, PMD, "error: Adding the MAC ADDR failed:"
-			" err = %d", ret);
+		DPAA_PMD_ERR("Adding the MAC ADDR failed: err = %d", ret);
 	return 0;
 }
 
@@ -1023,7 +1022,7 @@ dpaa_dev_set_mac_addr(struct rte_eth_dev *dev,
 
 	ret = fman_if_add_mac_addr(dpaa_intf->fif, addr->addr_bytes, 0);
 	if (ret)
-		RTE_LOG(ERR, PMD, "error: Setting the MAC ADDR failed %d", ret);
+		DPAA_PMD_ERR("Setting the MAC ADDR failed %d", ret);
 
 	return ret;
 }
@@ -1306,6 +1305,7 @@ dpaa_dev_init(struct rte_eth_dev *eth_dev)
 	struct fman_if *fman_intf;
 	struct fman_if_bpool *bp, *tmp_bp;
 	uint32_t cgrid[DPAA_MAX_NUM_PCD_QUEUES];
+	char eth_buf[RTE_ETHER_ADDR_FMT_SIZE];
 
 	PMD_INIT_FUNC_TRACE();
 
@@ -1457,15 +1457,9 @@ dpaa_dev_init(struct rte_eth_dev *eth_dev)
 
 	/* copy the primary mac address */
 	rte_ether_addr_copy(&fman_intf->mac_addr, &eth_dev->data->mac_addrs[0]);
+	rte_ether_format_addr(eth_buf, sizeof(eth_buf), &fman_intf->mac_addr);
 
-	RTE_LOG(INFO, PMD, "net: dpaa: %s: %02x:%02x:%02x:%02x:%02x:%02x\n",
-		dpaa_device->name,
-		fman_intf->mac_addr.addr_bytes[0],
-		fman_intf->mac_addr.addr_bytes[1],
-		fman_intf->mac_addr.addr_bytes[2],
-		fman_intf->mac_addr.addr_bytes[3],
-		fman_intf->mac_addr.addr_bytes[4],
-		fman_intf->mac_addr.addr_bytes[5]);
+	DPAA_PMD_INFO("net: dpaa: %s: %s", dpaa_device->name, eth_buf);
 
 	/* Disable RX mode */
 	fman_if_discard_rx_errors(fman_intf);
@@ -1578,8 +1572,7 @@ rte_dpaa_probe(struct rte_dpaa_driver *dpaa_drv __rte_unused,
 
 	if (!is_global_init && (rte_eal_process_type() == RTE_PROC_PRIMARY)) {
 		if (access("/tmp/fmc.bin", F_OK) == -1) {
-			RTE_LOG(INFO, PMD,
-				"* FMC not configured.Enabling default mode\n");
+			DPAA_PMD_INFO("* FMC not configured.Enabling default mode");
 			default_q = 1;
 		}
 
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* [dpdk-dev] [PATCH v2 7/7] net/thunderx: use dynamic logtype
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
                     ` (5 preceding siblings ...)
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 6/7] net/dpaa: use dynamic logtype Stephen Hemminger
@ 2020-03-31 17:31   ` Stephen Hemminger
  2020-03-31 17:43   ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Ferruh Yigit
  7 siblings, 0 replies; 20+ messages in thread
From: Stephen Hemminger @ 2020-03-31 17:31 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, jerin.jacob

The PMD static logtype is original DPDK legacy and should not be used.

Fixes: 43362c6a7647 ("net/thunderx: support RSS and RETA query and update")
Cc: jerin.jacob@caviumnetworks.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/thunderx/nicvf_ethdev.c | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/drivers/net/thunderx/nicvf_ethdev.c b/drivers/net/thunderx/nicvf_ethdev.c
index 6f43541a5d42..bfda0ab0ede8 100644
--- a/drivers/net/thunderx/nicvf_ethdev.c
+++ b/drivers/net/thunderx/nicvf_ethdev.c
@@ -496,9 +496,10 @@ nicvf_dev_reta_query(struct rte_eth_dev *dev,
 	int ret, i, j;
 
 	if (reta_size != NIC_MAX_RSS_IDR_TBL_SIZE) {
-		RTE_LOG(ERR, PMD, "The size of hash lookup table configured "
-			"(%d) doesn't match the number hardware can supported "
-			"(%d)", reta_size, NIC_MAX_RSS_IDR_TBL_SIZE);
+		PMD_DRV_LOG(ERR,
+			    "The size of hash lookup table configured "
+			    "(%u) doesn't match the number hardware can supported "
+			    "(%u)", reta_size, NIC_MAX_RSS_IDR_TBL_SIZE);
 		return -EINVAL;
 	}
 
@@ -526,9 +527,9 @@ nicvf_dev_reta_update(struct rte_eth_dev *dev,
 	int ret, i, j;
 
 	if (reta_size != NIC_MAX_RSS_IDR_TBL_SIZE) {
-		RTE_LOG(ERR, PMD, "The size of hash lookup table configured "
-			"(%d) doesn't match the number hardware can supported "
-			"(%d)", reta_size, NIC_MAX_RSS_IDR_TBL_SIZE);
+		PMD_DRV_LOG(ERR, "The size of hash lookup table configured "
+			"(%u) doesn't match the number hardware can supported "
+			"(%u)", reta_size, NIC_MAX_RSS_IDR_TBL_SIZE);
 		return -EINVAL;
 	}
 
@@ -569,8 +570,8 @@ nicvf_dev_rss_hash_update(struct rte_eth_dev *dev,
 
 	if (rss_conf->rss_key &&
 		rss_conf->rss_key_len != RSS_HASH_KEY_BYTE_SIZE) {
-		RTE_LOG(ERR, PMD, "Hash key size mismatch %d",
-				rss_conf->rss_key_len);
+		PMD_DRV_LOG(ERR, "Hash key size mismatch %u",
+			    rss_conf->rss_key_len);
 		return -EINVAL;
 	}
 
-- 
2.20.1


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype
  2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
                   ` (7 preceding siblings ...)
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
@ 2020-03-31 17:42 ` Ferruh Yigit
  8 siblings, 0 replies; 20+ messages in thread
From: Ferruh Yigit @ 2020-03-31 17:42 UTC (permalink / raw)
  To: Stephen Hemminger, dev

On 3/31/2020 5:41 AM, Stephen Hemminger wrote:
> While working on netvsc driver, noticed that there are still some places
> in some drivers using the old PMD static logtype.  Would like to
> eradicate all of these and even mark it as deprecated in a future
> release.
> 
> Stephen Hemminger (7):
>   net/mvneta: do not use PMD logtype
>   net/virtio: do not use PMD logtype
>   net/tap: do not use PMD logtype
>   net/pfe: do not use PMD logtype
>   net/bnxt: don't use PMD logtype
>   net/dpaa: use dynamic logtype
>   net/thunderx: use dynamic logtype
> 

For series,
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Series applied to dpdk-next-net/master, thanks.

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype
  2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
                     ` (6 preceding siblings ...)
  2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 7/7] net/thunderx: " Stephen Hemminger
@ 2020-03-31 17:43   ` Ferruh Yigit
  2020-03-31 17:49     ` Ferruh Yigit
  7 siblings, 1 reply; 20+ messages in thread
From: Ferruh Yigit @ 2020-03-31 17:43 UTC (permalink / raw)
  To: Stephen Hemminger, dev

On 3/31/2020 6:31 PM, Stephen Hemminger wrote:
> While working on netvsc driver, noticed that there are still some places
> in some drivers using the old PMD static logtype.  Would like to
> eradicate all of these and even mark it as deprecated in a future
> release.
> 
> v2 - fix checkpatch whitespace complaints

Hi Stephen,

I have applied the v1 and fixed the whitespace complaints while merging, is
there anything else changed in v2?

> 
> Stephen Hemminger (7):
>   net/mvneta: do not use PMD logtype
>   net/virtio: do not use PMD logtype
>   net/tap: do not use PMD logtype
>   net/pfe: do not use PMD logtype
>   net/bnxt: don't use PMD logtype
>   net/dpaa: use dynamic logtype
>   net/thunderx: use dynamic logtype
> 
>  drivers/net/bnxt/bnxt_ring.c            |  2 +-
>  drivers/net/dpaa/dpaa_ethdev.c          | 23 ++++++++---------------
>  drivers/net/mvneta/mvneta_ethdev.c      |  2 +-
>  drivers/net/pfe/pfe_ethdev.c            |  4 ++--
>  drivers/net/tap/rte_eth_tap.c           |  6 ++++--
>  drivers/net/thunderx/nicvf_ethdev.c     | 17 +++++++++--------
>  drivers/net/virtio/virtio_rxtx.c        |  6 +++---
>  drivers/net/virtio/virtio_user_ethdev.c |  2 +-
>  8 files changed, 29 insertions(+), 33 deletions(-)
> 


^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype
  2020-03-31 17:43   ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Ferruh Yigit
@ 2020-03-31 17:49     ` Ferruh Yigit
  0 siblings, 0 replies; 20+ messages in thread
From: Ferruh Yigit @ 2020-03-31 17:49 UTC (permalink / raw)
  To: Stephen Hemminger, dev

On 3/31/2020 6:43 PM, Ferruh Yigit wrote:
> On 3/31/2020 6:31 PM, Stephen Hemminger wrote:
>> While working on netvsc driver, noticed that there are still some places
>> in some drivers using the old PMD static logtype.  Would like to
>> eradicate all of these and even mark it as deprecated in a future
>> release.
>>
>> v2 - fix checkpatch whitespace complaints
> 
> Hi Stephen,
> 
> I have applied the v1 and fixed the whitespace complaints while merging, is
> there anything else changed in v2?

I double checked, there was only one line merging diff in dpaa and applied that
too, so practically v2 has been merged.

> 
>>
>> Stephen Hemminger (7):
>>   net/mvneta: do not use PMD logtype
>>   net/virtio: do not use PMD logtype
>>   net/tap: do not use PMD logtype
>>   net/pfe: do not use PMD logtype
>>   net/bnxt: don't use PMD logtype
>>   net/dpaa: use dynamic logtype
>>   net/thunderx: use dynamic logtype
>>
>>  drivers/net/bnxt/bnxt_ring.c            |  2 +-
>>  drivers/net/dpaa/dpaa_ethdev.c          | 23 ++++++++---------------
>>  drivers/net/mvneta/mvneta_ethdev.c      |  2 +-
>>  drivers/net/pfe/pfe_ethdev.c            |  4 ++--
>>  drivers/net/tap/rte_eth_tap.c           |  6 ++++--
>>  drivers/net/thunderx/nicvf_ethdev.c     | 17 +++++++++--------
>>  drivers/net/virtio/virtio_rxtx.c        |  6 +++---
>>  drivers/net/virtio/virtio_user_ethdev.c |  2 +-
>>  8 files changed, 29 insertions(+), 33 deletions(-)
>>
> 


^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2020-03-31 17:49 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-31  4:41 [dpdk-dev] [PATCH 0/7] Remove uses of PMD logtype Stephen Hemminger
2020-03-31  4:41 ` [dpdk-dev] [PATCH 1/7] net/mvneta: do not use " Stephen Hemminger
2020-03-31  4:41 ` [dpdk-dev] [PATCH 2/7] net/virtio: " Stephen Hemminger
2020-03-31  4:41 ` [dpdk-dev] [PATCH 3/7] net/tap: " Stephen Hemminger
2020-03-31  4:41 ` [dpdk-dev] [PATCH 4/7] net/pfe: " Stephen Hemminger
2020-03-31  4:41 ` [dpdk-dev] [PATCH 5/7] net/bnxt: don't " Stephen Hemminger
2020-03-31  4:43   ` Somnath Kotur
2020-03-31  4:41 ` [dpdk-dev] [PATCH 6/7] net/dpaa: use dynamic logtype Stephen Hemminger
2020-03-31  4:41 ` [dpdk-dev] [PATCH 7/7] net/thunderx: " Stephen Hemminger
2020-03-31 17:31 ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Stephen Hemminger
2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 1/7] net/mvneta: do not use " Stephen Hemminger
2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 2/7] net/virtio: " Stephen Hemminger
2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 3/7] net/tap: " Stephen Hemminger
2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 4/7] net/pfe: " Stephen Hemminger
2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 5/7] net/bnxt: don't " Stephen Hemminger
2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 6/7] net/dpaa: use dynamic logtype Stephen Hemminger
2020-03-31 17:31   ` [dpdk-dev] [PATCH v2 7/7] net/thunderx: " Stephen Hemminger
2020-03-31 17:43   ` [dpdk-dev] [PATCH v2 0/7] Remove some uses of PMD logtype Ferruh Yigit
2020-03-31 17:49     ` Ferruh Yigit
2020-03-31 17:42 ` [dpdk-dev] [PATCH 0/7] Remove " Ferruh Yigit

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).