From: Stephen Hemminger <stephen@networkplumber.org>
To: dev@dpdk.org
Cc: Stephen Hemminger <stephen@networkplumber.org>,
Andrew Boyer <andrew.boyer@amd.com>
Subject: [PATCH v2 08/13] net/ionic: remove word aborting
Date: Fri, 18 Aug 2023 10:45:32 -0700 [thread overview]
Message-ID: <20230818174537.290222-9-stephen@networkplumber.org> (raw)
In-Reply-To: <20230818174537.290222-1-stephen@networkplumber.org>
Since abort is on the Tier 1 list of words to replace;
just remove the unnecessary part of the log message.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
drivers/net/ionic/ionic_dev_pci.c | 6 +++---
drivers/net/ionic/ionic_ethdev.c | 18 +++++++++---------
drivers/net/ionic/ionic_lif.c | 2 +-
3 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/net/ionic/ionic_dev_pci.c b/drivers/net/ionic/ionic_dev_pci.c
index 5e74a6da7149..aec76b6f0024 100644
--- a/drivers/net/ionic/ionic_dev_pci.c
+++ b/drivers/net/ionic/ionic_dev_pci.c
@@ -43,13 +43,13 @@ ionic_pci_setup(struct ionic_adapter *adapter)
/* BAR0: dev_cmd and interrupts */
if (num_bars < 1) {
- IONIC_PRINT(ERR, "No bars found, aborting\n");
+ IONIC_PRINT(ERR, "No bars found\n");
return -EFAULT;
}
if (bar->len < IONIC_BAR0_SIZE) {
IONIC_PRINT(ERR,
- "Resource bar size %lu too small, aborting\n",
+ "Resource bar size %lu too small\n",
bar->len);
return -EFAULT;
}
@@ -84,7 +84,7 @@ ionic_pci_setup(struct ionic_adapter *adapter)
/* BAR1: doorbells */
bar++;
if (num_bars < 2) {
- IONIC_PRINT(ERR, "Doorbell bar missing, aborting\n");
+ IONIC_PRINT(ERR, "Doorbell bar missing\n");
return -EFAULT;
}
diff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ionic_ethdev.c
index 340fd0cd5923..2eee70d226dd 100644
--- a/drivers/net/ionic/ionic_ethdev.c
+++ b/drivers/net/ionic/ionic_ethdev.c
@@ -1030,14 +1030,14 @@ eth_ionic_dev_init(struct rte_eth_dev *eth_dev, void *init_params)
err = ionic_lif_alloc(lif);
if (err) {
- IONIC_PRINT(ERR, "Cannot allocate LIFs: %d, aborting",
+ IONIC_PRINT(ERR, "Cannot allocate LIFs: %d",
err);
goto err;
}
err = ionic_lif_init(lif);
if (err) {
- IONIC_PRINT(ERR, "Cannot init LIFs: %d, aborting", err);
+ IONIC_PRINT(ERR, "Cannot init LIFs: %d", err);
goto err_free_lif;
}
@@ -1144,48 +1144,48 @@ eth_ionic_dev_probe(void *bus_dev, struct rte_device *rte_dev,
/* Discover ionic dev resources */
err = ionic_setup(adapter);
if (err) {
- IONIC_PRINT(ERR, "Cannot setup device: %d, aborting", err);
+ IONIC_PRINT(ERR, "Cannot setup device: %d", err);
goto err_free_adapter;
}
err = ionic_identify(adapter);
if (err) {
- IONIC_PRINT(ERR, "Cannot identify device: %d, aborting",
+ IONIC_PRINT(ERR, "Cannot identify device: %d",
err);
goto err_free_adapter;
}
err = ionic_init(adapter);
if (err) {
- IONIC_PRINT(ERR, "Cannot init device: %d, aborting", err);
+ IONIC_PRINT(ERR, "Cannot init device: %d", err);
goto err_free_adapter;
}
/* Configure the ports */
err = ionic_port_identify(adapter);
if (err) {
- IONIC_PRINT(ERR, "Cannot identify port: %d, aborting",
+ IONIC_PRINT(ERR, "Cannot identify port: %d",
err);
goto err_free_adapter;
}
err = ionic_port_init(adapter);
if (err) {
- IONIC_PRINT(ERR, "Cannot init port: %d, aborting", err);
+ IONIC_PRINT(ERR, "Cannot init port: %d", err);
goto err_free_adapter;
}
/* Configure LIFs */
err = ionic_lif_identify(adapter);
if (err) {
- IONIC_PRINT(ERR, "Cannot identify lif: %d, aborting", err);
+ IONIC_PRINT(ERR, "Cannot identify lif: %d", err);
goto err_free_adapter;
}
/* Allocate and init LIFs */
err = ionic_lifs_size(adapter);
if (err) {
- IONIC_PRINT(ERR, "Cannot size LIFs: %d, aborting", err);
+ IONIC_PRINT(ERR, "Cannot size LIFs: %d", err);
goto err_free_adapter;
}
diff --git a/drivers/net/ionic/ionic_lif.c b/drivers/net/ionic/ionic_lif.c
index 25b490deb626..1944205428bc 100644
--- a/drivers/net/ionic/ionic_lif.c
+++ b/drivers/net/ionic/ionic_lif.c
@@ -1045,7 +1045,7 @@ ionic_lif_alloc(struct ionic_lif *lif)
lif->kern_dbpage = adapter->idev.db_pages;
if (!lif->kern_dbpage) {
- IONIC_PRINT(ERR, "Cannot map dbpage, aborting");
+ IONIC_PRINT(ERR, "Cannot map dbpage");
return -ENOMEM;
}
--
2.39.2
next prev parent reply other threads:[~2023-08-18 17:47 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-18 2:52 [PATCH 00/14] remove use of term abort Stephen Hemminger
2023-08-18 2:52 ` [PATCH 01/14] jobstats: change jobstats_abort to jobstats_cancel Stephen Hemminger
2023-08-18 9:08 ` Bruce Richardson
2023-08-18 16:10 ` Stephen Hemminger
2023-08-18 2:52 ` [PATCH 02/14] all: remove use of word abort Stephen Hemminger
2023-08-18 4:04 ` Honnappa Nagarahalli
2023-08-18 2:52 ` [PATCH 03/14] pipeline: remove use of term abort Stephen Hemminger
2023-08-18 2:52 ` [PATCH 04/14] net/vmxnet3: replace abort() with rte_panic() Stephen Hemminger
2023-08-18 2:52 ` [PATCH 05/14] event/dlb2: remove word abort in comments Stephen Hemminger
2023-08-18 2:52 ` [PATCH 06/14] net/vdev_netvsc: replace use of term abort Stephen Hemminger
2023-08-18 2:52 ` [PATCH 07/14] net/netvsc: replace abort with cancel Stephen Hemminger
2023-08-18 2:52 ` [PATCH 08/14] net/ionic: remove word aborting Stephen Hemminger
2023-08-18 2:52 ` [PATCH 09/14] net/bnx2x: replace abort with cancel Stephen Hemminger
2023-08-18 2:52 ` [PATCH 10/14] net/mlx5: " Stephen Hemminger
2023-08-18 2:52 ` [PATCH 11/14] net/softnic: " Stephen Hemminger
2023-08-18 2:52 ` [PATCH 12/14] net/mlx4: remove word abort Stephen Hemminger
2023-08-18 2:52 ` [PATCH 13/14] sfc: remove use of term abort Stephen Hemminger
2023-08-18 2:52 ` [PATCH 14/14] net/axgbe: replace word abort Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 00/13] Replace us of term abort Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 01/13] jobstats: change jobstats_abort to jobstats_cancel Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 02/13] all: remove use of word abort Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 03/13] pipeline: remove use of term abort Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 04/13] net/vmxnet3: replace abort() with rte_panic() Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 05/13] event/dlb2: remove word abort in comments Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 06/13] net/vdev_netvsc: replace use of term abort Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 07/13] net/netvsc: replace abort with cancel Stephen Hemminger
2023-08-18 22:26 ` Long Li
2023-08-18 17:45 ` Stephen Hemminger [this message]
2023-08-18 17:45 ` [PATCH v2 09/13] net/bnx2x: " Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 10/13] net/mlx5: " Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 11/13] net/softnic: " Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 12/13] net/mlx4: remove word abort Stephen Hemminger
2023-08-18 17:45 ` [PATCH v2 13/13] net/axgbe: replace " Stephen Hemminger
2023-08-30 16:49 ` [PATCH v2 00/13] Replace us of term abort Thomas Monjalon
2023-09-06 19:39 ` Stephen Hemminger
2023-09-06 21:19 ` Thomas Monjalon
2023-09-06 23:03 ` Stephen Hemminger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230818174537.290222-9-stephen@networkplumber.org \
--to=stephen@networkplumber.org \
--cc=andrew.boyer@amd.com \
--cc=dev@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).