[v2,00/13] rte_log registration usage improvement
Message ID | 20200701035840.4028056-1-jerinj@marvell.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 6D64EA0350; Wed, 1 Jul 2020 05:59:08 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id AA6871BE95; Wed, 1 Jul 2020 05:59:07 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by dpdk.org (Postfix) with ESMTP id 4EE3FA69 for <dev@dpdk.org>; Wed, 1 Jul 2020 05:59:06 +0200 (CEST) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id 0613uemG018511; Tue, 30 Jun 2020 20:59:04 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding : content-type; s=pfpt0818; bh=SXr3kyYAp+duzD7TxQeeUjXr0dKW4hqdYHo5RZx5Nsc=; b=u1pgdSCreI1cwQN/2DQOwJ5HHKI5jdLggmlWaB7kfcs1VPfrbFmdxUQap4IO5srgAMDF BGELVm1bQw14MDpZFm0oRimM7U+iRM2UvfAD2qxAy+hrdzvn0PqgduQ/t4mbr08/pvqJ VH9uZGPTzZ0+I+EvugPUi1deQW68kqQQcQsYfUyfpX08RkJPOdtKjNdUwojEnT/ug7DZ dVtPTZhRAnzW+3lK19eIJJWD/CRzx4NNGrjRndxpQiQojYVU+J/lYgOmlN9D1Y5cUYuy HpqiEjLs6lggbRw3ddue8bM4+wrtBAjf5qya2s7EHk142pKUfN4x61AZ69QkzbepDZWh kA== Received: from sc-exch02.marvell.com ([199.233.58.182]) by mx0b-0016f401.pphosted.com with ESMTP id 31x5mnps0j-2 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Tue, 30 Jun 2020 20:59:04 -0700 Received: from DC5-EXCH02.marvell.com (10.69.176.39) by SC-EXCH02.marvell.com (10.93.176.82) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Tue, 30 Jun 2020 20:59:03 -0700 Received: from DC5-EXCH02.marvell.com (10.69.176.39) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Tue, 30 Jun 2020 20:59:03 -0700 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Tue, 30 Jun 2020 20:59:03 -0700 Received: from jerin-lab.marvell.com (jerin-lab.marvell.com [10.28.34.14]) by maili.marvell.com (Postfix) with ESMTP id 228663F703F; Tue, 30 Jun 2020 20:59:00 -0700 (PDT) From: <jerinj@marvell.com> To: <dev@dpdk.org> CC: <thomas@monjalon.net>, <olivier.matz@6wind.com>, <david.marchand@redhat.com>, <sachin.saxena@oss.nxp.com>, <adamx.dybkowski@intel.com>, Jerin Jacob <jerinj@marvell.com> Date: Wed, 1 Jul 2020 09:28:27 +0530 Message-ID: <20200701035840.4028056-1-jerinj@marvell.com> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20200617063047.1555518-1-jerinj@marvell.com> References: <20200617063047.1555518-1-jerinj@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.235, 18.0.687 definitions=2020-07-01_01:2020-07-01, 2020-06-30 signatures=0 Subject: [dpdk-dev] [PATCH v2 00/13] rte_log registration usage improvement X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Message
Jerin Jacob Kollanukkaran
July 1, 2020, 3:58 a.m. UTC
From: Jerin Jacob <jerinj@marvell.com>
This patch series improves the rte_log registration code snippet by
avoiding duplication of the code around registration by
introducing RTE_LOG_REGISTER macro.
v2:
- RTE_LOG_REGISTER implemented using rte_log_register_type_and_pick_level()
- Added Sachin Saxena <sachin.saxena@oss.nxp.com> and Adam Dybkowski
<adamx.dybkowski@intel.com> Acks.
- Rebase to ToT
Drivers not adapated to new macro
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
I have converted all existing rte_log_register() to new macro, except
the following (as it has different usage style)
1) drivers/common/mlx5/mlx5_common.c
2) drivers/net/e1000/e1000_logs.c
3) drivers/net/sfc/sfc_ethdev.c
Example usage of new macro
~~~~~~~~~~~~~~~~~~~~~~~~~~
-/**
- * @internal
- */
-int otx2_logtype_base;
-/**
- * @internal
- */
-int otx2_logtype_mbox;
-/**
- * @internal
- */
-int otx2_logtype_npa;
-/**
- * @internal
- */
-int otx2_logtype_nix;
-/**
- * @internal
- */
-int otx2_logtype_npc;
-/**
- * @internal
- */
-int otx2_logtype_tm;
-/**
- * @internal
- */
-int otx2_logtype_sso;
-/**
- * @internal
- */
-int otx2_logtype_tim;
-/**
- * @internal
- */
-int otx2_logtype_dpi;
-/**
- * @internal
- */
-int otx2_logtype_ep;
-
-RTE_INIT(otx2_log_init);
-static void
-otx2_log_init(void)
-{
- otx2_logtype_base = rte_log_register("pmd.octeontx2.base");
- if (otx2_logtype_base >= 0)
- rte_log_set_level(otx2_logtype_base, RTE_LOG_NOTICE);
-
- otx2_logtype_mbox = rte_log_register("pmd.octeontx2.mbox");
- if (otx2_logtype_mbox >= 0)
- rte_log_set_level(otx2_logtype_mbox, RTE_LOG_NOTICE);
-
- otx2_logtype_npa = rte_log_register("pmd.mempool.octeontx2");
- if (otx2_logtype_npa >= 0)
- rte_log_set_level(otx2_logtype_npa, RTE_LOG_NOTICE);
-
- otx2_logtype_nix = rte_log_register("pmd.net.octeontx2");
- if (otx2_logtype_nix >= 0)
- rte_log_set_level(otx2_logtype_nix, RTE_LOG_NOTICE);
-
- otx2_logtype_npc = rte_log_register("pmd.net.octeontx2.flow");
- if (otx2_logtype_npc >= 0)
- rte_log_set_level(otx2_logtype_npc, RTE_LOG_NOTICE);
-
- otx2_logtype_tm = rte_log_register("pmd.net.octeontx2.tm");
- if (otx2_logtype_tm >= 0)
- rte_log_set_level(otx2_logtype_tm, RTE_LOG_NOTICE);
-
- otx2_logtype_sso = rte_log_register("pmd.event.octeontx2");
- if (otx2_logtype_sso >= 0)
- rte_log_set_level(otx2_logtype_sso, RTE_LOG_NOTICE);
-
- otx2_logtype_tim =
rte_log_register("pmd.event.octeontx2.timer");
- if (otx2_logtype_tim >= 0)
- rte_log_set_level(otx2_logtype_tim, RTE_LOG_NOTICE);
-
- otx2_logtype_dpi = rte_log_register("pmd.raw.octeontx2.dpi");
- if (otx2_logtype_dpi >= 0)
- rte_log_set_level(otx2_logtype_dpi, RTE_LOG_NOTICE);
-
- otx2_logtype_ep = rte_log_register("pmd.raw.octeontx2.ep");
- if (otx2_logtype_ep >= 0)
- rte_log_set_level(otx2_logtype_ep, RTE_LOG_NOTICE);
-
-}
+RTE_LOG_REGISTER(otx2_logtype_base, pmd.octeontx2.base, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_mbox, pmd.octeontx2.mbox, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_npa, pmd.mempool.octeontx2, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_nix, pmd.net.octeontx2, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_npc, pmd.net.octeontx2.flow, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_tm, pmd.net.octeontx2.tm, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_sso, pmd.event.octeontx2, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_tim, pmd.event.octeontx2.timer, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_dpi, pmd.raw.octeontx2.dpi, NOTICE);
+RTE_LOG_REGISTER(otx2_logtype_ep, pmd.raw.octeontx2.ep, NOTICE);
Jerin Jacob (13):
eal/log: introduce log register macro
lib: use log register macro
drivers/vdpa: use log register macro
drivers/raw: use log register macro
drivers/net: use log register macro
drivers/mempool: use log register macro
drivers/event: use log register macro
drivers/crypto: use log register macro
drivers/compress: use log register macro
drivers/common: use log register macro
drivers/bus: use log register macro
drivers/baseband: use log register macro
app/test: use log register macro
app/test/test_efd.c | 6 +-
app/test/test_hash.c | 7 +-
app/test/test_timer_racecond.c | 7 +-
.../fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 17 +---
drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 18 +---
drivers/baseband/null/bbdev_null.c | 9 +-
.../baseband/turbo_sw/bbdev_turbo_software.c | 9 +-
drivers/bus/dpaa/dpaa_bus.c | 10 +-
drivers/bus/fslmc/fslmc_bus.c | 11 +--
drivers/bus/ifpga/ifpga_bus.c | 9 +-
drivers/bus/vdev/vdev.c | 10 +-
drivers/bus/vmbus/vmbus_common.c | 9 +-
drivers/common/dpaax/dpaax_iova_table.c | 10 +-
drivers/common/iavf/iavf_impl.c | 9 +-
drivers/common/octeontx/octeontx_mbox.c | 9 +-
drivers/common/octeontx2/otx2_common.c | 96 ++-----------------
drivers/common/qat/qat_logs.c | 17 +---
drivers/compress/isal/isal_compress_pmd.c | 10 +-
drivers/compress/octeontx/otx_zip_pmd.c | 10 +-
drivers/compress/zlib/zlib_pmd.c | 10 +-
drivers/crypto/aesni_gcm/aesni_gcm_pmd.c | 8 +-
drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 8 +-
drivers/crypto/caam_jr/caam_jr.c | 8 +-
drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c | 11 +--
drivers/crypto/dpaa_sec/dpaa_sec.c | 10 +-
drivers/crypto/kasumi/rte_kasumi_pmd.c | 6 +-
drivers/crypto/mvsam/rte_mrvl_pmd.c | 7 +-
drivers/crypto/nitrox/nitrox_logs.c | 9 +-
drivers/crypto/null/null_crypto_pmd.c | 7 +-
drivers/crypto/octeontx/otx_cryptodev.c | 10 +-
drivers/crypto/octeontx2/otx2_cryptodev.c | 12 +--
drivers/crypto/openssl/rte_openssl_pmd.c | 7 +-
.../scheduler/rte_cryptodev_scheduler.c | 8 +-
drivers/crypto/snow3g/rte_snow3g_pmd.c | 7 +-
drivers/crypto/virtio/virtio_cryptodev.c | 39 ++------
drivers/crypto/zuc/rte_zuc_pmd.c | 7 +-
drivers/event/dpaa/dpaa_eventdev.c | 8 +-
drivers/event/dpaa2/dpaa2_eventdev.c | 10 +-
drivers/event/octeontx/ssovf_evdev.c | 8 +-
drivers/event/octeontx/timvf_evdev.c | 9 +-
drivers/event/opdl/opdl_evdev.c | 8 +-
drivers/event/opdl/opdl_ring.c | 2 -
drivers/event/sw/sw_evdev.c | 11 +--
drivers/mempool/dpaa/dpaa_mempool.c | 10 +-
drivers/mempool/dpaa2/dpaa2_hw_mempool.c | 10 +-
drivers/mempool/octeontx/octeontx_fpavf.c | 10 +-
drivers/net/af_packet/rte_eth_af_packet.c | 8 +-
drivers/net/af_xdp/rte_eth_af_xdp.c | 8 +-
drivers/net/ark/ark_ethdev.c | 10 +-
drivers/net/atlantic/atl_ethdev.c | 15 +--
drivers/net/avp/avp_ethdev.c | 10 +-
drivers/net/axgbe/axgbe_ethdev.c | 15 +--
drivers/net/bnx2x/bnx2x_ethdev.c | 15 +--
drivers/net/bnxt/bnxt_ethdev.c | 9 +-
drivers/net/bonding/rte_eth_bond_pmd.c | 9 +-
drivers/net/cxgbe/cxgbe_ethdev.c | 15 +--
drivers/net/dpaa/dpaa_ethdev.c | 9 +-
drivers/net/dpaa2/dpaa2_ethdev.c | 9 +-
drivers/net/ena/ena_ethdev.c | 48 ++--------
drivers/net/enetc/enetc_ethdev.c | 10 +-
drivers/net/enic/enic_ethdev.c | 9 +-
drivers/net/failsafe/failsafe.c | 10 +-
drivers/net/fm10k/fm10k_ethdev.c | 40 +-------
drivers/net/hinic/hinic_pmd_ethdev.c | 11 +--
drivers/net/hns3/hns3_ethdev.c | 15 +--
drivers/net/i40e/i40e_ethdev.c | 38 +-------
drivers/net/iavf/iavf_ethdev.c | 40 +-------
drivers/net/ice/ice_ethdev.c | 38 +-------
drivers/net/igc/igc_logs.c | 16 +---
drivers/net/ionic/ionic_ethdev.c | 10 +-
drivers/net/ipn3ke/ipn3ke_ethdev.c | 10 +-
drivers/net/ixgbe/ixgbe_ethdev.c | 39 ++------
drivers/net/kni/rte_eth_kni.c | 9 +-
drivers/net/liquidio/lio_ethdev.c | 15 +--
drivers/net/memif/rte_eth_memif.c | 9 +-
drivers/net/mlx4/mlx4.c | 11 +--
drivers/net/mlx5/mlx5.c | 10 +-
drivers/net/mvneta/mvneta_ethdev.c | 10 +-
drivers/net/mvpp2/mrvl_ethdev.c | 10 +-
drivers/net/netvsc/hn_ethdev.c | 15 +--
drivers/net/nfp/nfp_net.c | 15 +--
drivers/net/null/rte_eth_null.c | 8 +-
drivers/net/octeontx/octeontx_ethdev.c | 21 +---
drivers/net/pcap/rte_eth_pcap.c | 8 +-
drivers/net/pfe/pfe_ethdev.c | 10 +-
drivers/net/qede/qede_ethdev.c | 16 +---
drivers/net/ring/rte_eth_ring.c | 8 +-
drivers/net/softnic/rte_eth_softnic.c | 10 +-
drivers/net/szedata2/rte_eth_szedata2.c | 15 +--
drivers/net/tap/rte_eth_tap.c | 9 +-
drivers/net/thunderx/nicvf_ethdev.c | 21 +---
drivers/net/vdev_netvsc/vdev_netvsc.c | 13 +--
drivers/net/vhost/rte_eth_vhost.c | 8 +-
drivers/net/virtio/virtio_ethdev.c | 15 +--
drivers/net/vmxnet3/vmxnet3_ethdev.c | 15 +--
drivers/raw/dpaa2_cmdif/dpaa2_cmdif.c | 11 +--
drivers/raw/dpaa2_qdma/dpaa2_qdma.c | 11 +--
drivers/raw/ifpga/ifpga_rawdev.c | 10 +-
drivers/raw/ioat/ioat_rawdev.c | 12 +--
drivers/raw/ioat/rte_ioat_rawdev.h | 2 -
drivers/raw/ntb/ntb.c | 10 +-
drivers/raw/skeleton/skeleton_rawdev.c | 11 +--
drivers/vdpa/ifc/ifcvf_vdpa.c | 10 +-
drivers/vdpa/mlx5/mlx5_vdpa.c | 7 +-
lib/librte_bbdev/rte_bbdev.c | 8 +-
lib/librte_bpf/bpf.c | 9 +-
lib/librte_cfgfile/rte_cfgfile.c | 8 +-
lib/librte_compressdev/rte_compressdev.c | 7 +-
lib/librte_compressdev/rte_compressdev_pmd.c | 2 -
lib/librte_eal/include/rte_log.h | 24 +++++
lib/librte_ethdev/rte_ethdev.c | 9 +-
lib/librte_eventdev/rte_event_timer_adapter.c | 21 +---
lib/librte_flow_classify/rte_flow_classify.c | 10 +-
lib/librte_graph/graph.c | 7 +-
lib/librte_member/rte_member.c | 9 +-
lib/librte_metrics/rte_metrics_telemetry.c | 5 +-
lib/librte_node/log.c | 9 +-
lib/librte_pdump/rte_pdump.c | 9 +-
lib/librte_rawdev/rte_rawdev.c | 7 +-
lib/librte_rcu/rte_rcu_qsbr.c | 9 +-
lib/librte_stack/rte_stack.c | 7 +-
lib/librte_vhost/vhost.c | 15 +--
122 files changed, 228 insertions(+), 1321 deletions(-)