From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 986CEA0579 for ; Fri, 9 Apr 2021 14:45:16 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8D24B4068B; Fri, 9 Apr 2021 14:45:16 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 061F74014D for ; Fri, 9 Apr 2021 14:45:15 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id E3A741208AA; Fri, 9 Apr 2021 14:45:14 +0200 (CEST) In-Reply-To: <20210409124334.24479-1-david.marchand@redhat.com> References: <20210409124334.24479-1-david.marchand@redhat.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: David Marchand Message-Id: <20210409124514.E3A741208AA@dpdk.org> Date: Fri, 9 Apr 2021 14:45:14 +0200 (CEST) Subject: [dpdk-test-report] |WARNING| pw90977 [PATCH] log: deduce log type from filename X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/90977 _coding style issues_ WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #87: sed -i -e 's/RTE_LOG_REGISTER(\(.*\), '$pattern'\(\|\..*\),/RTE_LOG_REGISTER(, ,/' $file; ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #226: FILE: drivers/common/dpaax/dpaax_iova_table.c:463: +RTE_LOG_REGISTER(dpaax_logger, , ERR); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #236: FILE: drivers/common/iavf/iavf_impl.c:88: +RTE_LOG_REGISTER(iavf_common_logger, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #246: FILE: drivers/common/mlx5/mlx5_common.c:42: +RTE_LOG_REGISTER(mlx5_common_logtype, , NOTICE) ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #259: FILE: drivers/compress/isal/isal_compress_pmd.c:753: +RTE_LOG_REGISTER(isal_logtype_driver, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #277: FILE: drivers/compress/mlx5/mlx5_compress.c:903: +RTE_LOG_REGISTER(mlx5_compress_logtype, , NOTICE) ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #290: FILE: drivers/compress/octeontx/otx_zip_pmd.c:649: +RTE_LOG_REGISTER(octtx_zip_logtype_driver, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #300: FILE: drivers/compress/zlib/zlib_pmd.c:429: +RTE_LOG_REGISTER(zlib_logtype_driver, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #310: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:978: +RTE_LOG_REGISTER(aesni_gcm_logtype_driver, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #320: FILE: drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c:2232: +RTE_LOG_REGISTER(aesni_mb_logtype_driver, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #330: FILE: drivers/crypto/armv8/rte_armv8_pmd.c:867: +RTE_LOG_REGISTER(crypto_armv8_log_type, , ERR); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #355: FILE: drivers/crypto/kasumi/rte_kasumi_pmd.c:642: +RTE_LOG_REGISTER(kasumi_logtype_driver, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #365: FILE: drivers/crypto/mvsam/rte_mrvl_pmd.c:1022: +RTE_LOG_REGISTER(mrvl_logtype_driver, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #375: FILE: drivers/crypto/nitrox/nitrox_logs.c:7: +RTE_LOG_REGISTER(nitrox_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #385: FILE: drivers/crypto/null/null_crypto_pmd.c:251: +RTE_LOG_REGISTER(null_logtype_driver, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #395: FILE: drivers/crypto/octeontx/otx_cryptodev.c:114: +RTE_LOG_REGISTER(otx_cpt_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #405: FILE: drivers/crypto/octeontx2/otx2_cryptodev.c:179: +RTE_LOG_REGISTER(otx2_cpt_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #415: FILE: drivers/crypto/openssl/rte_openssl_pmd.c:2281: +RTE_LOG_REGISTER(openssl_logtype_driver, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #425: FILE: drivers/crypto/scheduler/rte_cryptodev_scheduler.c:580: +RTE_LOG_REGISTER(scheduler_logtype_driver, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #435: FILE: drivers/crypto/snow3g/rte_snow3g_pmd.c:656: +RTE_LOG_REGISTER(snow3g_logtype_driver, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #465: FILE: drivers/crypto/zuc/rte_zuc_pmd.c:583: +RTE_LOG_REGISTER(zuc_logtype_driver, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #475: FILE: drivers/event/dlb/dlb.c:4085: +RTE_LOG_REGISTER(eventdev_dlb_log_level, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #485: FILE: drivers/event/dlb2/dlb2.c:4035: +RTE_LOG_REGISTER(eventdev_dlb2_log_level, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #495: FILE: drivers/event/dpaa/dpaa_eventdev.c:49: +RTE_LOG_REGISTER(dpaa_logtype_eventdev, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #508: FILE: drivers/event/dpaa2/dpaa2_eventdev.c:1209: +RTE_LOG_REGISTER(dpaa2_logtype_event, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #518: FILE: drivers/event/octeontx/ssovf_evdev.c:25: +RTE_LOG_REGISTER(otx_logtype_ssovf, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #557: FILE: drivers/event/sw/sw_evdev.c:1154: +RTE_LOG_REGISTER(eventdev_sw_log_level, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #567: FILE: drivers/mempool/octeontx/octeontx_fpavf.c:108: +RTE_LOG_REGISTER(octeontx_logtype_fpavf, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #580: FILE: drivers/net/af_packet/rte_eth_af_packet.c:100: +RTE_LOG_REGISTER(af_packet_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #593: FILE: drivers/net/af_xdp/rte_eth_af_xdp.c:64: +RTE_LOG_REGISTER(af_xdp_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #606: FILE: drivers/net/ark/ark_ethdev.c:1018: +RTE_LOG_REGISTER(ark_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #675: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:3778: +RTE_LOG_REGISTER(bond_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #686: FILE: drivers/net/cxgbe/cxgbe_ethdev.c:1477: +RTE_LOG_REGISTER(cxgbe_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #697: FILE: drivers/net/dpaa/dpaa_ethdev.c:2312: +RTE_LOG_REGISTER(dpaa_logtype_pmd, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #707: FILE: drivers/net/dpaa2/dpaa2_ethdev.c:2896: +RTE_LOG_REGISTER(dpaa2_logtype_pmd, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #771: FILE: drivers/net/enetc/enetc_ethdev.c:963: +RTE_LOG_REGISTER(enetc_logtype_pmd, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #781: FILE: drivers/net/enic/enic_ethdev.c:74: +RTE_LOG_REGISTER(enic_pmd_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #794: FILE: drivers/net/failsafe/failsafe.c:394: +RTE_LOG_REGISTER(failsafe_logtype, , NOTICE) ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #828: FILE: drivers/net/hinic/hinic_pmd_ethdev.c:3269: +RTE_LOG_REGISTER(hinic_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #936: FILE: drivers/net/ionic/ionic_ethdev.c:1296: +RTE_LOG_REGISTER(ionic_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #946: FILE: drivers/net/ipn3ke/ipn3ke_ethdev.c:586: +RTE_LOG_REGISTER(ipn3ke_afu_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #981: FILE: drivers/net/kni/rte_eth_kni.c:71: +RTE_LOG_REGISTER(eth_kni_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1006: FILE: drivers/net/memif/rte_eth_memif.c:1881: +RTE_LOG_REGISTER(memif_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1016: FILE: drivers/net/mlx4/mlx4.c:1322: +RTE_LOG_REGISTER(mlx4_logtype, , NOTICE) ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1029: FILE: drivers/net/mlx5/mlx5.c:2252: +RTE_LOG_REGISTER(mlx5_logtype, , NOTICE) ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1042: FILE: drivers/net/mvneta/mvneta_ethdev.c:990: +RTE_LOG_REGISTER(mvneta_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1052: FILE: drivers/net/mvpp2/mrvl_ethdev.c:3312: +RTE_LOG_REGISTER(mrvl_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1092: FILE: drivers/net/null/rte_eth_null.c:76: +RTE_LOG_REGISTER(eth_null_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1122: FILE: drivers/net/octeontx_ep/otx_ep_ethdev.c:466: +RTE_LOG_REGISTER(otx_net_ep_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1132: FILE: drivers/net/pcap/rte_eth_pcap.c:151: +RTE_LOG_REGISTER(eth_pcap_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1145: FILE: drivers/net/pfe/pfe_ethdev.c:1180: +RTE_LOG_REGISTER(pfe_logtype_pmd, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1167: FILE: drivers/net/ring/rte_eth_ring.c:65: +RTE_LOG_REGISTER(eth_ring_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1180: FILE: drivers/net/softnic/rte_eth_softnic.c:83: +RTE_LOG_REGISTER(pmd_softnic_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1205: FILE: drivers/net/tap/rte_eth_tap.c:2571: +RTE_LOG_REGISTER(tap_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1260: FILE: drivers/net/vdev_netvsc/vdev_netvsc.c:51: +RTE_LOG_REGISTER(vdev_netvsc_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1273: FILE: drivers/net/vhost/rte_eth_vhost.c:22: +RTE_LOG_REGISTER(vhost_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1310: FILE: drivers/raw/ifpga/ifpga_rawdev.c:1614: +RTE_LOG_REGISTER(ifpga_rawdev_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1323: FILE: drivers/raw/ioat/ioat_rawdev.c:31: +RTE_LOG_REGISTER(ioat_pmd_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1336: FILE: drivers/raw/ntb/ntb.c:1537: +RTE_LOG_REGISTER(ntb_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1346: FILE: drivers/raw/skeleton/skeleton_rawdev.c:771: +RTE_LOG_REGISTER(skeleton_pmd_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1364: FILE: drivers/regex/mlx5/mlx5_regex.c:303: +RTE_LOG_REGISTER(mlx5_regex_logtype, , NOTICE) ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1377: FILE: drivers/vdpa/mlx5/mlx5_vdpa.c:872: +RTE_LOG_REGISTER(mlx5_vdpa_logtype, , NOTICE) ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1390: FILE: lib/librte_bbdev/rte_bbdev.c:32: +RTE_LOG_REGISTER(bbdev_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1403: FILE: lib/librte_bpf/bpf.c:56: +RTE_LOG_REGISTER(rte_bpf_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1413: FILE: lib/librte_cfgfile/rte_cfgfile.c:30: +RTE_LOG_REGISTER(cfgfile_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1426: FILE: lib/librte_compressdev/rte_compressdev.c:766: +RTE_LOG_REGISTER(compressdev_logtype, , NOTICE); ^ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #1453: FILE: lib/librte_eal/common/eal_common_log.c:280: + if ((pos = strstr(pattern, "drivers/")) != NULL) { ERROR:ASSIGN_IN_IF: do not use assignment in if condition #1464: FILE: lib/librte_eal/common/eal_common_log.c:291: + } else if ((pos = strstr(pattern, "lib/librte_")) != NULL) { ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1572: FILE: lib/librte_ethdev/rte_ethdev.c:5926: +RTE_LOG_REGISTER(rte_eth_dev_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1602: FILE: lib/librte_flow_classify/rte_flow_classify.c:670: +RTE_LOG_REGISTER(librte_flow_classify_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1612: FILE: lib/librte_graph/graph.c:581: +RTE_LOG_REGISTER(rte_graph_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1622: FILE: lib/librte_member/rte_member.c:300: +RTE_LOG_REGISTER(librte_member_logtype, , DEBUG); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1632: FILE: lib/librte_metrics/rte_metrics_telemetry.c:531: +RTE_LOG_REGISTER(metrics_log_level, , ERR); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1645: FILE: lib/librte_net/rte_net_crc.c:79: +RTE_LOG_REGISTER(libnet_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1658: FILE: lib/librte_node/log.c:7: +RTE_LOG_REGISTER(rte_node_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1668: FILE: lib/librte_pdump/rte_pdump.c:15: +RTE_LOG_REGISTER(pdump_logtype, , NOTICE); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1681: FILE: lib/librte_rawdev/rte_rawdev.c:630: +RTE_LOG_REGISTER(librawdev_logtype, , INFO); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1694: FILE: lib/librte_rcu/rte_rcu_qsbr.c:499: +RTE_LOG_REGISTER(rte_rcu_log_type, , ERR); ^ ERROR:SPACING: space prohibited before that ',' (ctx:WxW) #1704: FILE: lib/librte_stack/rte_stack.c:195: +RTE_LOG_REGISTER(stack_logtype, , NOTICE); ^ total: 78 errors, 1 warnings, 951 lines checked