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 D78E7A0567; Tue, 9 Mar 2021 10:49:14 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A8F1722A4C7; Tue, 9 Mar 2021 10:48:58 +0100 (CET) Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by mails.dpdk.org (Postfix) with ESMTP id 67AE522A4AB for ; Tue, 9 Mar 2021 10:48:57 +0100 (CET) Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailout.nyi.internal (Postfix) with ESMTP id 174F15C00F2; Tue, 9 Mar 2021 04:48:57 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Tue, 09 Mar 2021 04:48:57 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; s=fm3; bh=IwKZ9C6DBww20 Ak9zqcVydk+DfFK5ALvcbSQMYDvjv0=; b=YSEqa6RwehRoUKEgFeF3AQMnNOSxb ChUl3c8biO4/sBKnRH+Nom84iHWz6bCxj42jLZtWFTC8XMUDlBvS31KnIYPaQnzy 8Rv+WI0ALDcKCdWZfjDeLyxJDVKC4VcoCDJidHa72rbh3kcd79xq0SwYCQ3xA84i LG4ReGF+P8G03G+FY0wTgiMKAIkCpxb03V++qGG0W+27r0EcEBHIcKysYaN36I5H LzuIwPL8Om3jTr78BKFe+dwvcMiRpFxapm+AyBwcPK5ZT6kzBYHWrPM9WsNdX9MA MkR+TR+lnGaYMwAASnUz+kbMygGxz8Yucfuq+a5nWQrkIgEQk/1bFobhw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:date:from :in-reply-to:message-id:mime-version:references:subject:to :x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm2; bh=IwKZ9C6DBww20Ak9zqcVydk+DfFK5ALvcbSQMYDvjv0=; b=Van27Rr3 ConLgKhP4JCphxAFzUET4ICycT9Xl6lqBud3iZAaduwRf7qy4zVOk/rIJtyqKo2v hXmc+tRKuygn03MCXxryXwNrKRBZnlmT8twYP9PsWbQ2V0E4JSDSrZuhSFUt4jVA UmZn0Mgv3RyEhMSsFWjuSbpoMGDahEMuFco6uUAaJUF5aTLHYdvchuWaoT6eljQy pOh0l2g5xFDitLs65hFrwnEpdvzljTLHnsn3etgjXf+tffXQQl31xU5oKOyzoyEK 0Ojg5n/GQFBeDx87iS6RKE4D7e8qjWX3FUZgtaLotqJiIXdC8/hkjVC9mH0CR4aQ bgTl8BhC/S4/Hg== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledrudduiedgtdejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkffojghfggfgsedtkeertdertddtnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg ftrfgrthhtvghrnhepffehteduleeugfehveehgefhhfefhefhleefkeekffejfeejudef udevtdfhieetnecuffhomhgrihhnpegurhhivhgvrhdrnhgrmhgvnecukfhppeejjedrud efgedrvddtfedrudekgeenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgr ihhlfhhrohhmpehthhhomhgrshesmhhonhhjrghlohhnrdhnvght X-ME-Proxy: Received: from xps.monjalon.net (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id E5D0A1080064; Tue, 9 Mar 2021 04:48:55 -0500 (EST) From: Thomas Monjalon To: dev@dpdk.org Cc: Matan Azrad , Shahaf Shuler , Viacheslav Ovsiienko , Fiona Trahe , Ashish Gupta Date: Tue, 9 Mar 2021 10:48:35 +0100 Message-Id: <20210309094836.988763-4-thomas@monjalon.net> X-Mailer: git-send-email 2.30.1 In-Reply-To: <20210309094836.988763-1-thomas@monjalon.net> References: <20210308222855.823670-1-thomas@monjalon.net> <20210309094836.988763-1-thomas@monjalon.net> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v2 3/4] common/mlx5: remove extra line feed in log messages X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" The macro DRV_LOG already includes a terminating line feed character defined in PMD_DRV_LOG_. The extra line feeds added in some messages are removed. Signed-off-by: Thomas Monjalon --- drivers/common/mlx5/mlx5_common_pci.c | 4 ++-- drivers/common/mlx5/mlx5_devx_cmds.c | 2 +- drivers/compress/mlx5/mlx5_compress.c | 2 +- drivers/net/mlx5/linux/mlx5_os.c | 8 ++++---- drivers/net/mlx5/mlx5_flow.c | 2 +- drivers/net/mlx5/windows/mlx5_os.c | 2 +- drivers/vdpa/mlx5/mlx5_vdpa.c | 2 +- 7 files changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/common/mlx5/mlx5_common_pci.c b/drivers/common/mlx5/mlx5_common_pci.c index 2b657686d1..a7f541a90c 100644 --- a/drivers/common/mlx5/mlx5_common_pci.c +++ b/drivers/common/mlx5/mlx5_common_pci.c @@ -245,14 +245,14 @@ drivers_probe(struct mlx5_pci_device *dev, struct rte_pci_driver *pci_drv, already_loaded = dev->classes_loaded & driver->driver_class; if (already_loaded && !(driver->pci_driver.drv_flags & RTE_PCI_DRV_PROBE_AGAIN)) { - DRV_LOG(ERR, "Device %s is already probed\n", + DRV_LOG(ERR, "Device %s is already probed", pci_dev->device.name); ret = -EEXIST; goto probe_err; } ret = driver->pci_driver.probe(pci_drv, pci_dev); if (ret < 0) { - DRV_LOG(ERR, "Failed to load driver = %s.\n", + DRV_LOG(ERR, "Failed to load driver %s", driver->pci_driver.driver.name); goto probe_err; } diff --git a/drivers/common/mlx5/mlx5_devx_cmds.c b/drivers/common/mlx5/mlx5_devx_cmds.c index 0060c37fc0..8df14f5b7d 100644 --- a/drivers/common/mlx5/mlx5_devx_cmds.c +++ b/drivers/common/mlx5/mlx5_devx_cmds.c @@ -275,7 +275,7 @@ mlx5_devx_cmd_mkey_create(void *ctx, mkey->obj = mlx5_glue->devx_obj_create(ctx, in, in_size_dw * 4, out, sizeof(out)); if (!mkey->obj) { - DRV_LOG(ERR, "Can't create %sdirect mkey - error %d\n", + DRV_LOG(ERR, "Can't create %sdirect mkey - error %d", klm_num ? "an in" : "a ", errno); rte_errno = errno; mlx5_free(mkey); diff --git a/drivers/compress/mlx5/mlx5_compress.c b/drivers/compress/mlx5/mlx5_compress.c index 46255ab5e0..85238accff 100644 --- a/drivers/compress/mlx5/mlx5_compress.c +++ b/drivers/compress/mlx5/mlx5_compress.c @@ -257,7 +257,7 @@ mlx5_compress_qp_setup(struct rte_compressdev *dev, uint16_t qp_id, DRV_LOG(ERR, "Can't change SQ state to ready."); goto err; } - DRV_LOG(INFO, "QP %u: SQN=0x%X CQN=0x%X entries num = %u\n", + DRV_LOG(INFO, "QP %u: SQN=0x%X CQN=0x%X entries num = %u", (uint32_t)qp_id, qp->sq.sq->id, qp->cq.cq->id, qp->entries_n); return 0; err: diff --git a/drivers/net/mlx5/linux/mlx5_os.c b/drivers/net/mlx5/linux/mlx5_os.c index 81eb2e4b05..b03cafda6a 100644 --- a/drivers/net/mlx5/linux/mlx5_os.c +++ b/drivers/net/mlx5/linux/mlx5_os.c @@ -1233,14 +1233,14 @@ mlx5_dev_spawn(struct rte_device *dpdk_dev, if (config->hca_attr.log_max_ft_sampler_num > 0 && config->dv_flow_en) { priv->sampler_en = 1; - DRV_LOG(DEBUG, "The Sampler enabled!\n"); + DRV_LOG(DEBUG, "Sampler enabled!"); } else { priv->sampler_en = 0; if (!config->hca_attr.log_max_ft_sampler_num) - DRV_LOG(WARNING, "No available register for" - " Sampler."); + DRV_LOG(WARNING, + "No available register for sampler."); else - DRV_LOG(DEBUG, "DV flow is not supported!\n"); + DRV_LOG(DEBUG, "DV flow is not supported!"); } #endif } diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index ab5be3dacc..773f3e63f4 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -7924,7 +7924,7 @@ void mlx5_release_tunnel_hub(struct mlx5_dev_ctx_shared *sh, uint16_t port_id) if (!thub) return; if (!LIST_EMPTY(&thub->tunnels)) - DRV_LOG(WARNING, "port %u tunnels present\n", port_id); + DRV_LOG(WARNING, "port %u tunnels present", port_id); mlx5_hlist_destroy(thub->groups); mlx5_free(thub); } diff --git a/drivers/net/mlx5/windows/mlx5_os.c b/drivers/net/mlx5/windows/mlx5_os.c index e37cc65c17..e7db85b757 100644 --- a/drivers/net/mlx5/windows/mlx5_os.c +++ b/drivers/net/mlx5/windows/mlx5_os.c @@ -165,7 +165,7 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) if (!sh->flow_tbls) err = mlx5_alloc_table_hash_list(priv); else - DRV_LOG(DEBUG, "sh->flow_tbls[%p] already created, reuse\n", + DRV_LOG(DEBUG, "sh->flow_tbls[%p] already created, reuse", (void *)sh->flow_tbls); return err; } diff --git a/drivers/vdpa/mlx5/mlx5_vdpa.c b/drivers/vdpa/mlx5/mlx5_vdpa.c index 4c2d886bd7..5755a68ec4 100644 --- a/drivers/vdpa/mlx5/mlx5_vdpa.c +++ b/drivers/vdpa/mlx5/mlx5_vdpa.c @@ -751,7 +751,7 @@ mlx5_vdpa_pci_probe(struct rte_pci_driver *pci_drv __rte_unused, priv->pci_dev = pci_dev; priv->var = mlx5_glue->dv_alloc_var(ctx, 0); if (!priv->var) { - DRV_LOG(ERR, "Failed to allocate VAR %u.\n", errno); + DRV_LOG(ERR, "Failed to allocate VAR %u.", errno); goto error; } priv->vdev = rte_vdpa_register_device(&pci_dev->device, -- 2.30.1