aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJoe Perches <joe@perches.com>2010-12-13 13:05:14 -0500
committerDavid S. Miller <davem@davemloft.net>2010-12-13 13:05:14 -0500
commit8c4877a4128e7931077b024a891a4b284d8756a3 (patch)
treeeae8569651f3383abfa7c6a3789cd7c563f92346
parent323e126f0c5995f779d7df7fd035f6e8fed8764d (diff)
ehea: Use the standard logging functions
Remove ehea_error, ehea_info and ehea_debug macros. Use pr_fmt, pr_<level>, netdev_<level> and netif_<level> as appropriate. Fix messages to use trailing "\n", some messages had an extra one as the old ehea_<level> macros added a trailing "\n". Coalesced long format strings. Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Breno Leitao <leitao@linux.vnet.ibm.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/ehea/ehea.h13
-rw-r--r--drivers/net/ehea/ehea_ethtool.c18
-rw-r--r--drivers/net/ehea/ehea_main.c415
-rw-r--r--drivers/net/ehea/ehea_phyp.c40
-rw-r--r--drivers/net/ehea/ehea_qmr.c89
5 files changed, 278 insertions, 297 deletions
diff --git a/drivers/net/ehea/ehea.h b/drivers/net/ehea/ehea.h
index 1f2a675649b7..a724a2d14506 100644
--- a/drivers/net/ehea/ehea.h
+++ b/drivers/net/ehea/ehea.h
@@ -130,19 +130,6 @@
130 130
131/* utility functions */ 131/* utility functions */
132 132
133#define ehea_info(fmt, args...) \
134 printk(KERN_INFO DRV_NAME ": " fmt "\n", ## args)
135
136#define ehea_error(fmt, args...) \
137 printk(KERN_ERR DRV_NAME ": Error in %s: " fmt "\n", __func__, ## args)
138
139#ifdef DEBUG
140#define ehea_debug(fmt, args...) \
141 printk(KERN_DEBUG DRV_NAME ": " fmt, ## args)
142#else
143#define ehea_debug(fmt, args...) do {} while (0)
144#endif
145
146void ehea_dump(void *adr, int len, char *msg); 133void ehea_dump(void *adr, int len, char *msg);
147 134
148#define EHEA_BMASK(pos, length) (((pos) << 16) + (length)) 135#define EHEA_BMASK(pos, length) (((pos) << 16) + (length))
diff --git a/drivers/net/ehea/ehea_ethtool.c b/drivers/net/ehea/ehea_ethtool.c
index 1f37ee6b2a26..afebf2075779 100644
--- a/drivers/net/ehea/ehea_ethtool.c
+++ b/drivers/net/ehea/ehea_ethtool.c
@@ -26,6 +26,8 @@
26 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. 26 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
27 */ 27 */
28 28
29#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
30
29#include "ehea.h" 31#include "ehea.h"
30#include "ehea_phyp.h" 32#include "ehea_phyp.h"
31 33
@@ -118,10 +120,10 @@ doit:
118 ret = ehea_set_portspeed(port, sp); 120 ret = ehea_set_portspeed(port, sp);
119 121
120 if (!ret) 122 if (!ret)
121 ehea_info("%s: Port speed successfully set: %dMbps " 123 netdev_info(dev,
122 "%s Duplex", 124 "Port speed successfully set: %dMbps %s Duplex\n",
123 port->netdev->name, port->port_speed, 125 port->port_speed,
124 port->full_duplex == 1 ? "Full" : "Half"); 126 port->full_duplex == 1 ? "Full" : "Half");
125out: 127out:
126 return ret; 128 return ret;
127} 129}
@@ -134,10 +136,10 @@ static int ehea_nway_reset(struct net_device *dev)
134 ret = ehea_set_portspeed(port, EHEA_SPEED_AUTONEG); 136 ret = ehea_set_portspeed(port, EHEA_SPEED_AUTONEG);
135 137
136 if (!ret) 138 if (!ret)
137 ehea_info("%s: Port speed successfully set: %dMbps " 139 netdev_info(port->netdev,
138 "%s Duplex", 140 "Port speed successfully set: %dMbps %s Duplex\n",
139 port->netdev->name, port->port_speed, 141 port->port_speed,
140 port->full_duplex == 1 ? "Full" : "Half"); 142 port->full_duplex == 1 ? "Full" : "Half");
141 return ret; 143 return ret;
142} 144}
143 145
diff --git a/drivers/net/ehea/ehea_main.c b/drivers/net/ehea/ehea_main.c
index 81e5b7b49e9e..0dfef6d76445 100644
--- a/drivers/net/ehea/ehea_main.c
+++ b/drivers/net/ehea/ehea_main.c
@@ -26,6 +26,8 @@
26 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. 26 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
27 */ 27 */
28 28
29#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
30
29#include <linux/in.h> 31#include <linux/in.h>
30#include <linux/ip.h> 32#include <linux/ip.h>
31#include <linux/tcp.h> 33#include <linux/tcp.h>
@@ -135,8 +137,8 @@ void ehea_dump(void *adr, int len, char *msg)
135 int x; 137 int x;
136 unsigned char *deb = adr; 138 unsigned char *deb = adr;
137 for (x = 0; x < len; x += 16) { 139 for (x = 0; x < len; x += 16) {
138 printk(DRV_NAME " %s adr=%p ofs=%04x %016llx %016llx\n", msg, 140 pr_info("%s adr=%p ofs=%04x %016llx %016llx\n",
139 deb, x, *((u64 *)&deb[0]), *((u64 *)&deb[8])); 141 msg, deb, x, *((u64 *)&deb[0]), *((u64 *)&deb[8]));
140 deb += 16; 142 deb += 16;
141 } 143 }
142} 144}
@@ -336,7 +338,7 @@ static struct net_device_stats *ehea_get_stats(struct net_device *dev)
336 338
337 cb2 = (void *)get_zeroed_page(GFP_KERNEL); 339 cb2 = (void *)get_zeroed_page(GFP_KERNEL);
338 if (!cb2) { 340 if (!cb2) {
339 ehea_error("no mem for cb2"); 341 netdev_err(dev, "no mem for cb2\n");
340 goto out; 342 goto out;
341 } 343 }
342 344
@@ -344,7 +346,7 @@ static struct net_device_stats *ehea_get_stats(struct net_device *dev)
344 port->logical_port_id, 346 port->logical_port_id,
345 H_PORT_CB2, H_PORT_CB2_ALL, cb2); 347 H_PORT_CB2, H_PORT_CB2_ALL, cb2);
346 if (hret != H_SUCCESS) { 348 if (hret != H_SUCCESS) {
347 ehea_error("query_ehea_port failed"); 349 netdev_err(dev, "query_ehea_port failed\n");
348 goto out_herr; 350 goto out_herr;
349 } 351 }
350 352
@@ -399,7 +401,7 @@ static void ehea_refill_rq1(struct ehea_port_res *pr, int index, int nr_of_wqes)
399 skb_arr_rq1[index] = netdev_alloc_skb(dev, 401 skb_arr_rq1[index] = netdev_alloc_skb(dev,
400 EHEA_L_PKT_SIZE); 402 EHEA_L_PKT_SIZE);
401 if (!skb_arr_rq1[index]) { 403 if (!skb_arr_rq1[index]) {
402 ehea_info("Unable to allocate enough skb in the array\n"); 404 netdev_info(dev, "Unable to allocate enough skb in the array\n");
403 pr->rq1_skba.os_skbs = fill_wqes - i; 405 pr->rq1_skba.os_skbs = fill_wqes - i;
404 break; 406 break;
405 } 407 }
@@ -423,14 +425,14 @@ static void ehea_init_fill_rq1(struct ehea_port_res *pr, int nr_rq1a)
423 int i; 425 int i;
424 426
425 if (nr_rq1a > pr->rq1_skba.len) { 427 if (nr_rq1a > pr->rq1_skba.len) {
426 ehea_error("NR_RQ1A bigger than skb array len\n"); 428 netdev_err(dev, "NR_RQ1A bigger than skb array len\n");
427 return; 429 return;
428 } 430 }
429 431
430 for (i = 0; i < nr_rq1a; i++) { 432 for (i = 0; i < nr_rq1a; i++) {
431 skb_arr_rq1[i] = netdev_alloc_skb(dev, EHEA_L_PKT_SIZE); 433 skb_arr_rq1[i] = netdev_alloc_skb(dev, EHEA_L_PKT_SIZE);
432 if (!skb_arr_rq1[i]) { 434 if (!skb_arr_rq1[i]) {
433 ehea_info("No enough memory to allocate skb array\n"); 435 netdev_info(dev, "Not enough memory to allocate skb array\n");
434 break; 436 break;
435 } 437 }
436 } 438 }
@@ -468,8 +470,9 @@ static int ehea_refill_rq_def(struct ehea_port_res *pr,
468 if (!skb) { 470 if (!skb) {
469 q_skba->os_skbs = fill_wqes - i; 471 q_skba->os_skbs = fill_wqes - i;
470 if (q_skba->os_skbs == q_skba->len - 2) { 472 if (q_skba->os_skbs == q_skba->len - 2) {
471 ehea_info("%s: rq%i ran dry - no mem for skb", 473 netdev_info(pr->port->netdev,
472 pr->port->netdev->name, rq_nr); 474 "rq%i ran dry - no mem for skb\n",
475 rq_nr);
473 ret = -ENOMEM; 476 ret = -ENOMEM;
474 } 477 }
475 break; 478 break;
@@ -634,8 +637,8 @@ static int ehea_treat_poll_error(struct ehea_port_res *pr, int rq,
634 637
635 if (cqe->status & EHEA_CQE_STAT_FAT_ERR_MASK) { 638 if (cqe->status & EHEA_CQE_STAT_FAT_ERR_MASK) {
636 if (netif_msg_rx_err(pr->port)) { 639 if (netif_msg_rx_err(pr->port)) {
637 ehea_error("Critical receive error for QP %d. " 640 pr_err("Critical receive error for QP %d. Resetting port.\n",
638 "Resetting port.", pr->qp->init_attr.qp_nr); 641 pr->qp->init_attr.qp_nr);
639 ehea_dump(cqe, sizeof(*cqe), "CQE"); 642 ehea_dump(cqe, sizeof(*cqe), "CQE");
640 } 643 }
641 ehea_schedule_port_reset(pr->port); 644 ehea_schedule_port_reset(pr->port);
@@ -737,13 +740,13 @@ static int ehea_proc_rwqes(struct net_device *dev,
737 skb_arr_rq1_len, 740 skb_arr_rq1_len,
738 wqe_index); 741 wqe_index);
739 if (unlikely(!skb)) { 742 if (unlikely(!skb)) {
740 if (netif_msg_rx_err(port)) 743 netif_err(port, rx_err, dev,
741 ehea_error("LL rq1: skb=NULL"); 744 "LL rq1: skb=NULL\n");
742 745
743 skb = netdev_alloc_skb(dev, 746 skb = netdev_alloc_skb(dev,
744 EHEA_L_PKT_SIZE); 747 EHEA_L_PKT_SIZE);
745 if (!skb) { 748 if (!skb) {
746 ehea_info("Not enough memory to allocate skb\n"); 749 netdev_info(dev, "Not enough memory to allocate skb\n");
747 break; 750 break;
748 } 751 }
749 } 752 }
@@ -755,8 +758,8 @@ static int ehea_proc_rwqes(struct net_device *dev,
755 skb = get_skb_by_index(skb_arr_rq2, 758 skb = get_skb_by_index(skb_arr_rq2,
756 skb_arr_rq2_len, cqe); 759 skb_arr_rq2_len, cqe);
757 if (unlikely(!skb)) { 760 if (unlikely(!skb)) {
758 if (netif_msg_rx_err(port)) 761 netif_err(port, rx_err, dev,
759 ehea_error("rq2: skb=NULL"); 762 "rq2: skb=NULL\n");
760 break; 763 break;
761 } 764 }
762 ehea_fill_skb(dev, skb, cqe); 765 ehea_fill_skb(dev, skb, cqe);
@@ -766,8 +769,8 @@ static int ehea_proc_rwqes(struct net_device *dev,
766 skb = get_skb_by_index(skb_arr_rq3, 769 skb = get_skb_by_index(skb_arr_rq3,
767 skb_arr_rq3_len, cqe); 770 skb_arr_rq3_len, cqe);
768 if (unlikely(!skb)) { 771 if (unlikely(!skb)) {
769 if (netif_msg_rx_err(port)) 772 netif_err(port, rx_err, dev,
770 ehea_error("rq3: skb=NULL"); 773 "rq3: skb=NULL\n");
771 break; 774 break;
772 } 775 }
773 ehea_fill_skb(dev, skb, cqe); 776 ehea_fill_skb(dev, skb, cqe);
@@ -839,7 +842,7 @@ static void check_sqs(struct ehea_port *port)
839 msecs_to_jiffies(100)); 842 msecs_to_jiffies(100));
840 843
841 if (!ret) { 844 if (!ret) {
842 ehea_error("HW/SW queues out of sync"); 845 pr_err("HW/SW queues out of sync\n");
843 ehea_schedule_port_reset(pr->port); 846 ehea_schedule_port_reset(pr->port);
844 return; 847 return;
845 } 848 }
@@ -872,14 +875,14 @@ static struct ehea_cqe *ehea_proc_cqes(struct ehea_port_res *pr, int my_quota)
872 } 875 }
873 876
874 if (cqe->status & EHEA_CQE_STAT_ERR_MASK) { 877 if (cqe->status & EHEA_CQE_STAT_ERR_MASK) {
875 ehea_error("Bad send completion status=0x%04X", 878 pr_err("Bad send completion status=0x%04X\n",
876 cqe->status); 879 cqe->status);
877 880
878 if (netif_msg_tx_err(pr->port)) 881 if (netif_msg_tx_err(pr->port))
879 ehea_dump(cqe, sizeof(*cqe), "Send CQE"); 882 ehea_dump(cqe, sizeof(*cqe), "Send CQE");
880 883
881 if (cqe->status & EHEA_CQE_STAT_RESET_MASK) { 884 if (cqe->status & EHEA_CQE_STAT_RESET_MASK) {
882 ehea_error("Resetting port"); 885 pr_err("Resetting port\n");
883 ehea_schedule_port_reset(pr->port); 886 ehea_schedule_port_reset(pr->port);
884 break; 887 break;
885 } 888 }
@@ -997,8 +1000,8 @@ static irqreturn_t ehea_qp_aff_irq_handler(int irq, void *param)
997 1000
998 while (eqe) { 1001 while (eqe) {
999 qp_token = EHEA_BMASK_GET(EHEA_EQE_QP_TOKEN, eqe->entry); 1002 qp_token = EHEA_BMASK_GET(EHEA_EQE_QP_TOKEN, eqe->entry);
1000 ehea_error("QP aff_err: entry=0x%llx, token=0x%x", 1003 pr_err("QP aff_err: entry=0x%llx, token=0x%x\n",
1001 eqe->entry, qp_token); 1004 eqe->entry, qp_token);
1002 1005
1003 qp = port->port_res[qp_token].qp; 1006 qp = port->port_res[qp_token].qp;
1004 1007
@@ -1016,7 +1019,7 @@ static irqreturn_t ehea_qp_aff_irq_handler(int irq, void *param)
1016 } 1019 }
1017 1020
1018 if (reset_port) { 1021 if (reset_port) {
1019 ehea_error("Resetting port"); 1022 pr_err("Resetting port\n");
1020 ehea_schedule_port_reset(port); 1023 ehea_schedule_port_reset(port);
1021 } 1024 }
1022 1025
@@ -1044,7 +1047,7 @@ int ehea_sense_port_attr(struct ehea_port *port)
1044 /* may be called via ehea_neq_tasklet() */ 1047 /* may be called via ehea_neq_tasklet() */
1045 cb0 = (void *)get_zeroed_page(GFP_ATOMIC); 1048 cb0 = (void *)get_zeroed_page(GFP_ATOMIC);
1046 if (!cb0) { 1049 if (!cb0) {
1047 ehea_error("no mem for cb0"); 1050 pr_err("no mem for cb0\n");
1048 ret = -ENOMEM; 1051 ret = -ENOMEM;
1049 goto out; 1052 goto out;
1050 } 1053 }
@@ -1136,7 +1139,7 @@ int ehea_set_portspeed(struct ehea_port *port, u32 port_speed)
1136 1139
1137 cb4 = (void *)get_zeroed_page(GFP_KERNEL); 1140 cb4 = (void *)get_zeroed_page(GFP_KERNEL);
1138 if (!cb4) { 1141 if (!cb4) {
1139 ehea_error("no mem for cb4"); 1142 pr_err("no mem for cb4\n");
1140 ret = -ENOMEM; 1143 ret = -ENOMEM;
1141 goto out; 1144 goto out;
1142 } 1145 }
@@ -1187,16 +1190,16 @@ int ehea_set_portspeed(struct ehea_port *port, u32 port_speed)
1187 break; 1190 break;
1188 } 1191 }
1189 } else { 1192 } else {
1190 ehea_error("Failed sensing port speed"); 1193 pr_err("Failed sensing port speed\n");
1191 ret = -EIO; 1194 ret = -EIO;
1192 } 1195 }
1193 } else { 1196 } else {
1194 if (hret == H_AUTHORITY) { 1197 if (hret == H_AUTHORITY) {
1195 ehea_info("Hypervisor denied setting port speed"); 1198 pr_info("Hypervisor denied setting port speed\n");
1196 ret = -EPERM; 1199 ret = -EPERM;
1197 } else { 1200 } else {
1198 ret = -EIO; 1201 ret = -EIO;
1199 ehea_error("Failed setting port speed"); 1202 pr_err("Failed setting port speed\n");
1200 } 1203 }
1201 } 1204 }
1202 if (!prop_carrier_state || (port->phy_link == EHEA_PHY_LINK_UP)) 1205 if (!prop_carrier_state || (port->phy_link == EHEA_PHY_LINK_UP))
@@ -1213,80 +1216,78 @@ static void ehea_parse_eqe(struct ehea_adapter *adapter, u64 eqe)
1213 u8 ec; 1216 u8 ec;
1214 u8 portnum; 1217 u8 portnum;
1215 struct ehea_port *port; 1218 struct ehea_port *port;
1219 struct net_device *dev;
1216 1220
1217 ec = EHEA_BMASK_GET(NEQE_EVENT_CODE, eqe); 1221 ec = EHEA_BMASK_GET(NEQE_EVENT_CODE, eqe);
1218 portnum = EHEA_BMASK_GET(NEQE_PORTNUM, eqe); 1222 portnum = EHEA_BMASK_GET(NEQE_PORTNUM, eqe);
1219 port = ehea_get_port(adapter, portnum); 1223 port = ehea_get_port(adapter, portnum);
1224 dev = port->netdev;
1220 1225
1221 switch (ec) { 1226 switch (ec) {
1222 case EHEA_EC_PORTSTATE_CHG: /* port state change */ 1227 case EHEA_EC_PORTSTATE_CHG: /* port state change */
1223 1228
1224 if (!port) { 1229 if (!port) {
1225 ehea_error("unknown portnum %x", portnum); 1230 netdev_err(dev, "unknown portnum %x\n", portnum);
1226 break; 1231 break;
1227 } 1232 }
1228 1233
1229 if (EHEA_BMASK_GET(NEQE_PORT_UP, eqe)) { 1234 if (EHEA_BMASK_GET(NEQE_PORT_UP, eqe)) {
1230 if (!netif_carrier_ok(port->netdev)) { 1235 if (!netif_carrier_ok(dev)) {
1231 ret = ehea_sense_port_attr(port); 1236 ret = ehea_sense_port_attr(port);
1232 if (ret) { 1237 if (ret) {
1233 ehea_error("failed resensing port " 1238 netdev_err(dev, "failed resensing port attributes\n");
1234 "attributes");
1235 break; 1239 break;
1236 } 1240 }
1237 1241
1238 if (netif_msg_link(port)) 1242 netif_info(port, link, dev,
1239 ehea_info("%s: Logical port up: %dMbps " 1243 "Logical port up: %dMbps %s Duplex\n",
1240 "%s Duplex", 1244 port->port_speed,
1241 port->netdev->name, 1245 port->full_duplex == 1 ?
1242 port->port_speed, 1246 "Full" : "Half");
1243 port->full_duplex ==
1244 1 ? "Full" : "Half");
1245 1247
1246 netif_carrier_on(port->netdev); 1248 netif_carrier_on(dev);
1247 netif_wake_queue(port->netdev); 1249 netif_wake_queue(dev);
1248 } 1250 }
1249 } else 1251 } else
1250 if (netif_carrier_ok(port->netdev)) { 1252 if (netif_carrier_ok(dev)) {
1251 if (netif_msg_link(port)) 1253 netif_info(port, link, dev,
1252 ehea_info("%s: Logical port down", 1254 "Logical port down\n");
1253 port->netdev->name); 1255 netif_carrier_off(dev);
1254 netif_carrier_off(port->netdev); 1256 netif_stop_queue(dev);
1255 netif_stop_queue(port->netdev);
1256 } 1257 }
1257 1258
1258 if (EHEA_BMASK_GET(NEQE_EXTSWITCH_PORT_UP, eqe)) { 1259 if (EHEA_BMASK_GET(NEQE_EXTSWITCH_PORT_UP, eqe)) {
1259 port->phy_link = EHEA_PHY_LINK_UP; 1260 port->phy_link = EHEA_PHY_LINK_UP;
1260 if (netif_msg_link(port)) 1261 netif_info(port, link, dev,
1261 ehea_info("%s: Physical port up", 1262 "Physical port up\n");
1262 port->netdev->name);
1263 if (prop_carrier_state) 1263 if (prop_carrier_state)
1264 netif_carrier_on(port->netdev); 1264 netif_carrier_on(dev);
1265 } else { 1265 } else {
1266 port->phy_link = EHEA_PHY_LINK_DOWN; 1266 port->phy_link = EHEA_PHY_LINK_DOWN;
1267 if (netif_msg_link(port)) 1267 netif_info(port, link, dev,
1268 ehea_info("%s: Physical port down", 1268 "Physical port down\n");
1269 port->netdev->name);
1270 if (prop_carrier_state) 1269 if (prop_carrier_state)
1271 netif_carrier_off(port->netdev); 1270 netif_carrier_off(dev);
1272 } 1271 }
1273 1272
1274 if (EHEA_BMASK_GET(NEQE_EXTSWITCH_PRIMARY, eqe)) 1273 if (EHEA_BMASK_GET(NEQE_EXTSWITCH_PRIMARY, eqe))
1275 ehea_info("External switch port is primary port"); 1274 netdev_info(dev,
1275 "External switch port is primary port\n");
1276 else 1276 else
1277 ehea_info("External switch port is backup port"); 1277 netdev_info(dev,
1278 "External switch port is backup port\n");
1278 1279
1279 break; 1280 break;
1280 case EHEA_EC_ADAPTER_MALFUNC: 1281 case EHEA_EC_ADAPTER_MALFUNC:
1281 ehea_error("Adapter malfunction"); 1282 netdev_err(dev, "Adapter malfunction\n");
1282 break; 1283 break;
1283 case EHEA_EC_PORT_MALFUNC: 1284 case EHEA_EC_PORT_MALFUNC:
1284 ehea_info("Port malfunction: Device: %s", port->netdev->name); 1285 netdev_info(dev, "Port malfunction\n");
1285 netif_carrier_off(port->netdev); 1286 netif_carrier_off(dev);
1286 netif_stop_queue(port->netdev); 1287 netif_stop_queue(dev);
1287 break; 1288 break;
1288 default: 1289 default:
1289 ehea_error("unknown event code %x, eqe=0x%llX", ec, eqe); 1290 netdev_err(dev, "unknown event code %x, eqe=0x%llX\n", ec, eqe);
1290 break; 1291 break;
1291 } 1292 }
1292} 1293}
@@ -1298,13 +1299,13 @@ static void ehea_neq_tasklet(unsigned long data)
1298 u64 event_mask; 1299 u64 event_mask;
1299 1300
1300 eqe = ehea_poll_eq(adapter->neq); 1301 eqe = ehea_poll_eq(adapter->neq);
1301 ehea_debug("eqe=%p", eqe); 1302 pr_debug("eqe=%p\n", eqe);
1302 1303
1303 while (eqe) { 1304 while (eqe) {
1304 ehea_debug("*eqe=%lx", eqe->entry); 1305 pr_debug("*eqe=%lx\n", (unsigned long) eqe->entry);
1305 ehea_parse_eqe(adapter, eqe->entry); 1306 ehea_parse_eqe(adapter, eqe->entry);
1306 eqe = ehea_poll_eq(adapter->neq); 1307 eqe = ehea_poll_eq(adapter->neq);
1307 ehea_debug("next eqe=%p", eqe); 1308 pr_debug("next eqe=%p\n", eqe);
1308 } 1309 }
1309 1310
1310 event_mask = EHEA_BMASK_SET(NELR_PORTSTATE_CHG, 1) 1311 event_mask = EHEA_BMASK_SET(NELR_PORTSTATE_CHG, 1)
@@ -1353,14 +1354,14 @@ static int ehea_reg_interrupts(struct net_device *dev)
1353 ehea_qp_aff_irq_handler, 1354 ehea_qp_aff_irq_handler,
1354 IRQF_DISABLED, port->int_aff_name, port); 1355 IRQF_DISABLED, port->int_aff_name, port);
1355 if (ret) { 1356 if (ret) {
1356 ehea_error("failed registering irq for qp_aff_irq_handler:" 1357 netdev_err(dev, "failed registering irq for qp_aff_irq_handler:ist=%X\n",
1357 "ist=%X", port->qp_eq->attr.ist1); 1358 port->qp_eq->attr.ist1);
1358 goto out_free_qpeq; 1359 goto out_free_qpeq;
1359 } 1360 }
1360 1361
1361 if (netif_msg_ifup(port)) 1362 netif_info(port, ifup, dev,
1362 ehea_info("irq_handle 0x%X for function qp_aff_irq_handler " 1363 "irq_handle 0x%X for function qp_aff_irq_handler registered\n",
1363 "registered", port->qp_eq->attr.ist1); 1364 port->qp_eq->attr.ist1);
1364 1365
1365 1366
1366 for (i = 0; i < port->num_def_qps + port->num_add_tx_qps; i++) { 1367 for (i = 0; i < port->num_def_qps + port->num_add_tx_qps; i++) {
@@ -1372,14 +1373,13 @@ static int ehea_reg_interrupts(struct net_device *dev)
1372 IRQF_DISABLED, pr->int_send_name, 1373 IRQF_DISABLED, pr->int_send_name,
1373 pr); 1374 pr);
1374 if (ret) { 1375 if (ret) {
1375 ehea_error("failed registering irq for ehea_queue " 1376 netdev_err(dev, "failed registering irq for ehea_queue port_res_nr:%d, ist=%X\n",
1376 "port_res_nr:%d, ist=%X", i, 1377 i, pr->eq->attr.ist1);
1377 pr->eq->attr.ist1);
1378 goto out_free_req; 1378 goto out_free_req;
1379 } 1379 }
1380 if (netif_msg_ifup(port)) 1380 netif_info(port, ifup, dev,
1381 ehea_info("irq_handle 0x%X for function ehea_queue_int " 1381 "irq_handle 0x%X for function ehea_queue_int %d registered\n",
1382 "%d registered", pr->eq->attr.ist1, i); 1382 pr->eq->attr.ist1, i);
1383 } 1383 }
1384out: 1384out:
1385 return ret; 1385 return ret;
@@ -1410,16 +1410,16 @@ static void ehea_free_interrupts(struct net_device *dev)
1410 for (i = 0; i < port->num_def_qps + port->num_add_tx_qps; i++) { 1410 for (i = 0; i < port->num_def_qps + port->num_add_tx_qps; i++) {
1411 pr = &port->port_res[i]; 1411 pr = &port->port_res[i];
1412 ibmebus_free_irq(pr->eq->attr.ist1, pr); 1412 ibmebus_free_irq(pr->eq->attr.ist1, pr);
1413 if (netif_msg_intr(port)) 1413 netif_info(port, intr, dev,
1414 ehea_info("free send irq for res %d with handle 0x%X", 1414 "free send irq for res %d with handle 0x%X\n",
1415 i, pr->eq->attr.ist1); 1415 i, pr->eq->attr.ist1);
1416 } 1416 }
1417 1417
1418 /* associated events */ 1418 /* associated events */
1419 ibmebus_free_irq(port->qp_eq->attr.ist1, port); 1419 ibmebus_free_irq(port->qp_eq->attr.ist1, port);
1420 if (netif_msg_intr(port)) 1420 netif_info(port, intr, dev,
1421 ehea_info("associated event interrupt for handle 0x%X freed", 1421 "associated event interrupt for handle 0x%X freed\n",
1422 port->qp_eq->attr.ist1); 1422 port->qp_eq->attr.ist1);
1423} 1423}
1424 1424
1425static int ehea_configure_port(struct ehea_port *port) 1425static int ehea_configure_port(struct ehea_port *port)
@@ -1488,7 +1488,7 @@ int ehea_gen_smrs(struct ehea_port_res *pr)
1488out_free: 1488out_free:
1489 ehea_rem_mr(&pr->send_mr); 1489 ehea_rem_mr(&pr->send_mr);
1490out: 1490out:
1491 ehea_error("Generating SMRS failed\n"); 1491 pr_err("Generating SMRS failed\n");
1492 return -EIO; 1492 return -EIO;
1493} 1493}
1494 1494
@@ -1543,7 +1543,7 @@ static int ehea_init_port_res(struct ehea_port *port, struct ehea_port_res *pr,
1543 1543
1544 pr->eq = ehea_create_eq(adapter, eq_type, EHEA_MAX_ENTRIES_EQ, 0); 1544 pr->eq = ehea_create_eq(adapter, eq_type, EHEA_MAX_ENTRIES_EQ, 0);
1545 if (!pr->eq) { 1545 if (!pr->eq) {
1546 ehea_error("create_eq failed (eq)"); 1546 pr_err("create_eq failed (eq)\n");
1547 goto out_free; 1547 goto out_free;
1548 } 1548 }
1549 1549
@@ -1551,7 +1551,7 @@ static int ehea_init_port_res(struct ehea_port *port, struct ehea_port_res *pr,
1551 pr->eq->fw_handle, 1551 pr->eq->fw_handle,
1552 port->logical_port_id); 1552 port->logical_port_id);
1553 if (!pr->recv_cq) { 1553 if (!pr->recv_cq) {
1554 ehea_error("create_cq failed (cq_recv)"); 1554 pr_err("create_cq failed (cq_recv)\n");
1555 goto out_free; 1555 goto out_free;
1556 } 1556 }
1557 1557
@@ -1559,19 +1559,19 @@ static int ehea_init_port_res(struct ehea_port *port, struct ehea_port_res *pr,
1559 pr->eq->fw_handle, 1559 pr->eq->fw_handle,
1560 port->logical_port_id); 1560 port->logical_port_id);
1561 if (!pr->send_cq) { 1561 if (!pr->send_cq) {
1562 ehea_error("create_cq failed (cq_send)"); 1562 pr_err("create_cq failed (cq_send)\n");
1563 goto out_free; 1563 goto out_free;
1564 } 1564 }
1565 1565
1566 if (netif_msg_ifup(port)) 1566 if (netif_msg_ifup(port))
1567 ehea_info("Send CQ: act_nr_cqes=%d, Recv CQ: act_nr_cqes=%d", 1567 pr_info("Send CQ: act_nr_cqes=%d, Recv CQ: act_nr_cqes=%d\n",
1568 pr->send_cq->attr.act_nr_of_cqes, 1568 pr->send_cq->attr.act_nr_of_cqes,
1569 pr->recv_cq->attr.act_nr_of_cqes); 1569 pr->recv_cq->attr.act_nr_of_cqes);
1570 1570
1571 init_attr = kzalloc(sizeof(*init_attr), GFP_KERNEL); 1571 init_attr = kzalloc(sizeof(*init_attr), GFP_KERNEL);
1572 if (!init_attr) { 1572 if (!init_attr) {
1573 ret = -ENOMEM; 1573 ret = -ENOMEM;
1574 ehea_error("no mem for ehea_qp_init_attr"); 1574 pr_err("no mem for ehea_qp_init_attr\n");
1575 goto out_free; 1575 goto out_free;
1576 } 1576 }
1577 1577
@@ -1596,18 +1596,18 @@ static int ehea_init_port_res(struct ehea_port *port, struct ehea_port_res *pr,
1596 1596
1597 pr->qp = ehea_create_qp(adapter, adapter->pd, init_attr); 1597 pr->qp = ehea_create_qp(adapter, adapter->pd, init_attr);
1598 if (!pr->qp) { 1598 if (!pr->qp) {
1599 ehea_error("create_qp failed"); 1599 pr_err("create_qp failed\n");
1600 ret = -EIO; 1600 ret = -EIO;
1601 goto out_free; 1601 goto out_free;
1602 } 1602 }
1603 1603
1604 if (netif_msg_ifup(port)) 1604 if (netif_msg_ifup(port))
1605 ehea_info("QP: qp_nr=%d\n act_nr_snd_wqe=%d\n nr_rwqe_rq1=%d\n " 1605 pr_info("QP: qp_nr=%d\n act_nr_snd_wqe=%d\n nr_rwqe_rq1=%d\n nr_rwqe_rq2=%d\n nr_rwqe_rq3=%d\n",
1606 "nr_rwqe_rq2=%d\n nr_rwqe_rq3=%d", init_attr->qp_nr, 1606 init_attr->qp_nr,
1607 init_attr->act_nr_send_wqes, 1607 init_attr->act_nr_send_wqes,
1608 init_attr->act_nr_rwqes_rq1, 1608 init_attr->act_nr_rwqes_rq1,
1609 init_attr->act_nr_rwqes_rq2, 1609 init_attr->act_nr_rwqes_rq2,
1610 init_attr->act_nr_rwqes_rq3); 1610 init_attr->act_nr_rwqes_rq3);
1611 1611
1612 pr->sq_skba_size = init_attr->act_nr_send_wqes + 1; 1612 pr->sq_skba_size = init_attr->act_nr_send_wqes + 1;
1613 1613
@@ -1758,7 +1758,7 @@ static void write_swqe2_TSO(struct sk_buff *skb,
1758 swqe->descriptors++; 1758 swqe->descriptors++;
1759 } 1759 }
1760 } else 1760 } else
1761 ehea_error("cannot handle fragmented headers"); 1761 pr_err("cannot handle fragmented headers\n");
1762} 1762}
1763 1763
1764static void write_swqe2_nonTSO(struct sk_buff *skb, 1764static void write_swqe2_nonTSO(struct sk_buff *skb,
@@ -1854,8 +1854,8 @@ static int ehea_broadcast_reg_helper(struct ehea_port *port, u32 hcallid)
1854 port->logical_port_id, 1854 port->logical_port_id,
1855 reg_type, port->mac_addr, 0, hcallid); 1855 reg_type, port->mac_addr, 0, hcallid);
1856 if (hret != H_SUCCESS) { 1856 if (hret != H_SUCCESS) {
1857 ehea_error("%sregistering bc address failed (tagged)", 1857 pr_err("%sregistering bc address failed (tagged)\n",
1858 hcallid == H_REG_BCMC ? "" : "de"); 1858 hcallid == H_REG_BCMC ? "" : "de");
1859 ret = -EIO; 1859 ret = -EIO;
1860 goto out_herr; 1860 goto out_herr;
1861 } 1861 }
@@ -1866,8 +1866,8 @@ static int ehea_broadcast_reg_helper(struct ehea_port *port, u32 hcallid)
1866 port->logical_port_id, 1866 port->logical_port_id,
1867 reg_type, port->mac_addr, 0, hcallid); 1867 reg_type, port->mac_addr, 0, hcallid);
1868 if (hret != H_SUCCESS) { 1868 if (hret != H_SUCCESS) {
1869 ehea_error("%sregistering bc address failed (vlan)", 1869 pr_err("%sregistering bc address failed (vlan)\n",
1870 hcallid == H_REG_BCMC ? "" : "de"); 1870 hcallid == H_REG_BCMC ? "" : "de");
1871 ret = -EIO; 1871 ret = -EIO;
1872 } 1872 }
1873out_herr: 1873out_herr:
@@ -1889,7 +1889,7 @@ static int ehea_set_mac_addr(struct net_device *dev, void *sa)
1889 1889
1890 cb0 = (void *)get_zeroed_page(GFP_KERNEL); 1890 cb0 = (void *)get_zeroed_page(GFP_KERNEL);
1891 if (!cb0) { 1891 if (!cb0) {
1892 ehea_error("no mem for cb0"); 1892 pr_err("no mem for cb0\n");
1893 ret = -ENOMEM; 1893 ret = -ENOMEM;
1894 goto out; 1894 goto out;
1895 } 1895 }
@@ -1937,11 +1937,11 @@ out:
1937static void ehea_promiscuous_error(u64 hret, int enable) 1937static void ehea_promiscuous_error(u64 hret, int enable)
1938{ 1938{
1939 if (hret == H_AUTHORITY) 1939 if (hret == H_AUTHORITY)
1940 ehea_info("Hypervisor denied %sabling promiscuous mode", 1940 pr_info("Hypervisor denied %sabling promiscuous mode\n",
1941 enable == 1 ? "en" : "dis"); 1941 enable == 1 ? "en" : "dis");
1942 else 1942 else
1943 ehea_error("failed %sabling promiscuous mode", 1943 pr_err("failed %sabling promiscuous mode\n",
1944 enable == 1 ? "en" : "dis"); 1944 enable == 1 ? "en" : "dis");
1945} 1945}
1946 1946
1947static void ehea_promiscuous(struct net_device *dev, int enable) 1947static void ehea_promiscuous(struct net_device *dev, int enable)
@@ -1955,7 +1955,7 @@ static void ehea_promiscuous(struct net_device *dev, int enable)
1955 1955
1956 cb7 = (void *)get_zeroed_page(GFP_ATOMIC); 1956 cb7 = (void *)get_zeroed_page(GFP_ATOMIC);
1957 if (!cb7) { 1957 if (!cb7) {
1958 ehea_error("no mem for cb7"); 1958 pr_err("no mem for cb7\n");
1959 goto out; 1959 goto out;
1960 } 1960 }
1961 1961
@@ -2015,7 +2015,7 @@ static int ehea_drop_multicast_list(struct net_device *dev)
2015 hret = ehea_multicast_reg_helper(port, mc_entry->macaddr, 2015 hret = ehea_multicast_reg_helper(port, mc_entry->macaddr,
2016 H_DEREG_BCMC); 2016 H_DEREG_BCMC);
2017 if (hret) { 2017 if (hret) {
2018 ehea_error("failed deregistering mcast MAC"); 2018 pr_err("failed deregistering mcast MAC\n");
2019 ret = -EIO; 2019 ret = -EIO;
2020 } 2020 }
2021 2021
@@ -2038,7 +2038,8 @@ static void ehea_allmulti(struct net_device *dev, int enable)
2038 if (!hret) 2038 if (!hret)
2039 port->allmulti = 1; 2039 port->allmulti = 1;
2040 else 2040 else
2041 ehea_error("failed enabling IFF_ALLMULTI"); 2041 netdev_err(dev,
2042 "failed enabling IFF_ALLMULTI\n");
2042 } 2043 }
2043 } else 2044 } else
2044 if (!enable) { 2045 if (!enable) {
@@ -2047,7 +2048,8 @@ static void ehea_allmulti(struct net_device *dev, int enable)
2047 if (!hret) 2048 if (!hret)
2048 port->allmulti = 0; 2049 port->allmulti = 0;
2049 else 2050 else
2050 ehea_error("failed disabling IFF_ALLMULTI"); 2051 netdev_err(dev,
2052 "failed disabling IFF_ALLMULTI\n");
2051 } 2053 }
2052} 2054}
2053 2055
@@ -2058,7 +2060,7 @@ static void ehea_add_multicast_entry(struct ehea_port *port, u8 *mc_mac_addr)
2058 2060
2059 ehea_mcl_entry = kzalloc(sizeof(*ehea_mcl_entry), GFP_ATOMIC); 2061 ehea_mcl_entry = kzalloc(sizeof(*ehea_mcl_entry), GFP_ATOMIC);
2060 if (!ehea_mcl_entry) { 2062 if (!ehea_mcl_entry) {
2061 ehea_error("no mem for mcl_entry"); 2063 pr_err("no mem for mcl_entry\n");
2062 return; 2064 return;
2063 } 2065 }
2064 2066
@@ -2071,7 +2073,7 @@ static void ehea_add_multicast_entry(struct ehea_port *port, u8 *mc_mac_addr)
2071 if (!hret) 2073 if (!hret)
2072 list_add(&ehea_mcl_entry->list, &port->mc_list->list); 2074 list_add(&ehea_mcl_entry->list, &port->mc_list->list);
2073 else { 2075 else {
2074 ehea_error("failed registering mcast MAC"); 2076 pr_err("failed registering mcast MAC\n");
2075 kfree(ehea_mcl_entry); 2077 kfree(ehea_mcl_entry);
2076 } 2078 }
2077} 2079}
@@ -2104,9 +2106,8 @@ static void ehea_set_multicast_list(struct net_device *dev)
2104 } 2106 }
2105 2107
2106 if (netdev_mc_count(dev) > port->adapter->max_mc_mac) { 2108 if (netdev_mc_count(dev) > port->adapter->max_mc_mac) {
2107 ehea_info("Mcast registration limit reached (0x%llx). " 2109 pr_info("Mcast registration limit reached (0x%llx). Use ALLMULTI!\n",
2108 "Use ALLMULTI!", 2110 port->adapter->max_mc_mac);
2109 port->adapter->max_mc_mac);
2110 goto out; 2111 goto out;
2111 } 2112 }
2112 2113
@@ -2312,10 +2313,10 @@ static int ehea_start_xmit(struct sk_buff *skb, struct net_device *dev)
2312 } 2313 }
2313 pr->swqe_id_counter += 1; 2314 pr->swqe_id_counter += 1;
2314 2315
2315 if (netif_msg_tx_queued(port)) { 2316 netif_info(port, tx_queued, dev,
2316 ehea_info("post swqe on QP %d", pr->qp->init_attr.qp_nr); 2317 "post swqe on QP %d\n", pr->qp->init_attr.qp_nr);
2318 if (netif_msg_tx_queued(port))
2317 ehea_dump(swqe, 512, "swqe"); 2319 ehea_dump(swqe, 512, "swqe");
2318 }
2319 2320
2320 if (unlikely(test_bit(__EHEA_STOP_XFER, &ehea_driver_flags))) { 2321 if (unlikely(test_bit(__EHEA_STOP_XFER, &ehea_driver_flags))) {
2321 netif_stop_queue(dev); 2322 netif_stop_queue(dev);
@@ -2351,14 +2352,14 @@ static void ehea_vlan_rx_register(struct net_device *dev,
2351 2352
2352 cb1 = (void *)get_zeroed_page(GFP_KERNEL); 2353 cb1 = (void *)get_zeroed_page(GFP_KERNEL);
2353 if (!cb1) { 2354 if (!cb1) {
2354 ehea_error("no mem for cb1"); 2355 pr_err("no mem for cb1\n");
2355 goto out; 2356 goto out;
2356 } 2357 }
2357 2358
2358 hret = ehea_h_modify_ehea_port(adapter->handle, port->logical_port_id, 2359 hret = ehea_h_modify_ehea_port(adapter->handle, port->logical_port_id,
2359 H_PORT_CB1, H_PORT_CB1_ALL, cb1); 2360 H_PORT_CB1, H_PORT_CB1_ALL, cb1);
2360 if (hret != H_SUCCESS) 2361 if (hret != H_SUCCESS)
2361 ehea_error("modify_ehea_port failed"); 2362 pr_err("modify_ehea_port failed\n");
2362 2363
2363 free_page((unsigned long)cb1); 2364 free_page((unsigned long)cb1);
2364out: 2365out:
@@ -2375,14 +2376,14 @@ static void ehea_vlan_rx_add_vid(struct net_device *dev, unsigned short vid)
2375 2376
2376 cb1 = (void *)get_zeroed_page(GFP_KERNEL); 2377 cb1 = (void *)get_zeroed_page(GFP_KERNEL);
2377 if (!cb1) { 2378 if (!cb1) {
2378 ehea_error("no mem for cb1"); 2379 pr_err("no mem for cb1\n");
2379 goto out; 2380 goto out;
2380 } 2381 }
2381 2382
2382 hret = ehea_h_query_ehea_port(adapter->handle, port->logical_port_id, 2383 hret = ehea_h_query_ehea_port(adapter->handle, port->logical_port_id,
2383 H_PORT_CB1, H_PORT_CB1_ALL, cb1); 2384 H_PORT_CB1, H_PORT_CB1_ALL, cb1);
2384 if (hret != H_SUCCESS) { 2385 if (hret != H_SUCCESS) {
2385 ehea_error("query_ehea_port failed"); 2386 pr_err("query_ehea_port failed\n");
2386 goto out; 2387 goto out;
2387 } 2388 }
2388 2389
@@ -2392,7 +2393,7 @@ static void ehea_vlan_rx_add_vid(struct net_device *dev, unsigned short vid)
2392 hret = ehea_h_modify_ehea_port(adapter->handle, port->logical_port_id, 2393 hret = ehea_h_modify_ehea_port(adapter->handle, port->logical_port_id,
2393 H_PORT_CB1, H_PORT_CB1_ALL, cb1); 2394 H_PORT_CB1, H_PORT_CB1_ALL, cb1);
2394 if (hret != H_SUCCESS) 2395 if (hret != H_SUCCESS)
2395 ehea_error("modify_ehea_port failed"); 2396 pr_err("modify_ehea_port failed\n");
2396out: 2397out:
2397 free_page((unsigned long)cb1); 2398 free_page((unsigned long)cb1);
2398 return; 2399 return;
@@ -2410,14 +2411,14 @@ static void ehea_vlan_rx_kill_vid(struct net_device *dev, unsigned short vid)
2410 2411
2411 cb1 = (void *)get_zeroed_page(GFP_KERNEL); 2412 cb1 = (void *)get_zeroed_page(GFP_KERNEL);
2412 if (!cb1) { 2413 if (!cb1) {
2413 ehea_error("no mem for cb1"); 2414 pr_err("no mem for cb1\n");
2414 goto out; 2415 goto out;
2415 } 2416 }
2416 2417
2417 hret = ehea_h_query_ehea_port(adapter->handle, port->logical_port_id, 2418 hret = ehea_h_query_ehea_port(adapter->handle, port->logical_port_id,
2418 H_PORT_CB1, H_PORT_CB1_ALL, cb1); 2419 H_PORT_CB1, H_PORT_CB1_ALL, cb1);
2419 if (hret != H_SUCCESS) { 2420 if (hret != H_SUCCESS) {
2420 ehea_error("query_ehea_port failed"); 2421 pr_err("query_ehea_port failed\n");
2421 goto out; 2422 goto out;
2422 } 2423 }
2423 2424
@@ -2427,7 +2428,7 @@ static void ehea_vlan_rx_kill_vid(struct net_device *dev, unsigned short vid)
2427 hret = ehea_h_modify_ehea_port(adapter->handle, port->logical_port_id, 2428 hret = ehea_h_modify_ehea_port(adapter->handle, port->logical_port_id,
2428 H_PORT_CB1, H_PORT_CB1_ALL, cb1); 2429 H_PORT_CB1, H_PORT_CB1_ALL, cb1);
2429 if (hret != H_SUCCESS) 2430 if (hret != H_SUCCESS)
2430 ehea_error("modify_ehea_port failed"); 2431 pr_err("modify_ehea_port failed\n");
2431out: 2432out:
2432 free_page((unsigned long)cb1); 2433 free_page((unsigned long)cb1);
2433} 2434}
@@ -2449,7 +2450,7 @@ int ehea_activate_qp(struct ehea_adapter *adapter, struct ehea_qp *qp)
2449 hret = ehea_h_query_ehea_qp(adapter->handle, 0, qp->fw_handle, 2450 hret = ehea_h_query_ehea_qp(adapter->handle, 0, qp->fw_handle,
2450 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), cb0); 2451 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), cb0);
2451 if (hret != H_SUCCESS) { 2452 if (hret != H_SUCCESS) {
2452 ehea_error("query_ehea_qp failed (1)"); 2453 pr_err("query_ehea_qp failed (1)\n");
2453 goto out; 2454 goto out;
2454 } 2455 }
2455 2456
@@ -2458,14 +2459,14 @@ int ehea_activate_qp(struct ehea_adapter *adapter, struct ehea_qp *qp)
2458 EHEA_BMASK_SET(H_QPCB0_QP_CTL_REG, 1), cb0, 2459 EHEA_BMASK_SET(H_QPCB0_QP_CTL_REG, 1), cb0,
2459 &dummy64, &dummy64, &dummy16, &dummy16); 2460 &dummy64, &dummy64, &dummy16, &dummy16);
2460 if (hret != H_SUCCESS) { 2461 if (hret != H_SUCCESS) {
2461 ehea_error("modify_ehea_qp failed (1)"); 2462 pr_err("modify_ehea_qp failed (1)\n");
2462 goto out; 2463 goto out;
2463 } 2464 }
2464 2465
2465 hret = ehea_h_query_ehea_qp(adapter->handle, 0, qp->fw_handle, 2466 hret = ehea_h_query_ehea_qp(adapter->handle, 0, qp->fw_handle,
2466 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), cb0); 2467 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), cb0);
2467 if (hret != H_SUCCESS) { 2468 if (hret != H_SUCCESS) {
2468 ehea_error("query_ehea_qp failed (2)"); 2469 pr_err("query_ehea_qp failed (2)\n");
2469 goto out; 2470 goto out;
2470 } 2471 }
2471 2472
@@ -2474,14 +2475,14 @@ int ehea_activate_qp(struct ehea_adapter *adapter, struct ehea_qp *qp)
2474 EHEA_BMASK_SET(H_QPCB0_QP_CTL_REG, 1), cb0, 2475 EHEA_BMASK_SET(H_QPCB0_QP_CTL_REG, 1), cb0,
2475 &dummy64, &dummy64, &dummy16, &dummy16); 2476 &dummy64, &dummy64, &dummy16, &dummy16);
2476 if (hret != H_SUCCESS) { 2477 if (hret != H_SUCCESS) {
2477 ehea_error("modify_ehea_qp failed (2)"); 2478 pr_err("modify_ehea_qp failed (2)\n");
2478 goto out; 2479 goto out;
2479 } 2480 }
2480 2481
2481 hret = ehea_h_query_ehea_qp(adapter->handle, 0, qp->fw_handle, 2482 hret = ehea_h_query_ehea_qp(adapter->handle, 0, qp->fw_handle,
2482 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), cb0); 2483 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), cb0);
2483 if (hret != H_SUCCESS) { 2484 if (hret != H_SUCCESS) {
2484 ehea_error("query_ehea_qp failed (3)"); 2485 pr_err("query_ehea_qp failed (3)\n");
2485 goto out; 2486 goto out;
2486 } 2487 }
2487 2488
@@ -2490,14 +2491,14 @@ int ehea_activate_qp(struct ehea_adapter *adapter, struct ehea_qp *qp)
2490 EHEA_BMASK_SET(H_QPCB0_QP_CTL_REG, 1), cb0, 2491 EHEA_BMASK_SET(H_QPCB0_QP_CTL_REG, 1), cb0,
2491 &dummy64, &dummy64, &dummy16, &dummy16); 2492 &dummy64, &dummy64, &dummy16, &dummy16);
2492 if (hret != H_SUCCESS) { 2493 if (hret != H_SUCCESS) {
2493 ehea_error("modify_ehea_qp failed (3)"); 2494 pr_err("modify_ehea_qp failed (3)\n");
2494 goto out; 2495 goto out;
2495 } 2496 }
2496 2497
2497 hret = ehea_h_query_ehea_qp(adapter->handle, 0, qp->fw_handle, 2498 hret = ehea_h_query_ehea_qp(adapter->handle, 0, qp->fw_handle,
2498 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), cb0); 2499 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), cb0);
2499 if (hret != H_SUCCESS) { 2500 if (hret != H_SUCCESS) {
2500 ehea_error("query_ehea_qp failed (4)"); 2501 pr_err("query_ehea_qp failed (4)\n");
2501 goto out; 2502 goto out;
2502 } 2503 }
2503 2504
@@ -2518,7 +2519,7 @@ static int ehea_port_res_setup(struct ehea_port *port, int def_qps,
2518 EHEA_MAX_ENTRIES_EQ, 1); 2519 EHEA_MAX_ENTRIES_EQ, 1);
2519 if (!port->qp_eq) { 2520 if (!port->qp_eq) {
2520 ret = -EINVAL; 2521 ret = -EINVAL;
2521 ehea_error("ehea_create_eq failed (qp_eq)"); 2522 pr_err("ehea_create_eq failed (qp_eq)\n");
2522 goto out_kill_eq; 2523 goto out_kill_eq;
2523 } 2524 }
2524 2525
@@ -2599,27 +2600,27 @@ static int ehea_up(struct net_device *dev)
2599 ret = ehea_port_res_setup(port, port->num_def_qps, 2600 ret = ehea_port_res_setup(port, port->num_def_qps,
2600 port->num_add_tx_qps); 2601 port->num_add_tx_qps);
2601 if (ret) { 2602 if (ret) {
2602 ehea_error("port_res_failed"); 2603 netdev_err(dev, "port_res_failed\n");
2603 goto out; 2604 goto out;
2604 } 2605 }
2605 2606
2606 /* Set default QP for this port */ 2607 /* Set default QP for this port */
2607 ret = ehea_configure_port(port); 2608 ret = ehea_configure_port(port);
2608 if (ret) { 2609 if (ret) {
2609 ehea_error("ehea_configure_port failed. ret:%d", ret); 2610 netdev_err(dev, "ehea_configure_port failed. ret:%d\n", ret);
2610 goto out_clean_pr; 2611 goto out_clean_pr;
2611 } 2612 }
2612 2613
2613 ret = ehea_reg_interrupts(dev); 2614 ret = ehea_reg_interrupts(dev);
2614 if (ret) { 2615 if (ret) {
2615 ehea_error("reg_interrupts failed. ret:%d", ret); 2616 netdev_err(dev, "reg_interrupts failed. ret:%d\n", ret);
2616 goto out_clean_pr; 2617 goto out_clean_pr;
2617 } 2618 }
2618 2619
2619 for (i = 0; i < port->num_def_qps + port->num_add_tx_qps; i++) { 2620 for (i = 0; i < port->num_def_qps + port->num_add_tx_qps; i++) {
2620 ret = ehea_activate_qp(port->adapter, port->port_res[i].qp); 2621 ret = ehea_activate_qp(port->adapter, port->port_res[i].qp);
2621 if (ret) { 2622 if (ret) {
2622 ehea_error("activate_qp failed"); 2623 netdev_err(dev, "activate_qp failed\n");
2623 goto out_free_irqs; 2624 goto out_free_irqs;
2624 } 2625 }
2625 } 2626 }
@@ -2627,7 +2628,7 @@ static int ehea_up(struct net_device *dev)
2627 for (i = 0; i < port->num_def_qps; i++) { 2628 for (i = 0; i < port->num_def_qps; i++) {
2628 ret = ehea_fill_port_res(&port->port_res[i]); 2629 ret = ehea_fill_port_res(&port->port_res[i]);
2629 if (ret) { 2630 if (ret) {
2630 ehea_error("out_free_irqs"); 2631 netdev_err(dev, "out_free_irqs\n");
2631 goto out_free_irqs; 2632 goto out_free_irqs;
2632 } 2633 }
2633 } 2634 }
@@ -2650,7 +2651,7 @@ out_clean_pr:
2650 ehea_clean_all_portres(port); 2651 ehea_clean_all_portres(port);
2651out: 2652out:
2652 if (ret) 2653 if (ret)
2653 ehea_info("Failed starting %s. ret=%i", dev->name, ret); 2654 netdev_info(dev, "Failed starting. ret=%i\n", ret);
2654 2655
2655 ehea_update_bcmc_registrations(); 2656 ehea_update_bcmc_registrations();
2656 ehea_update_firmware_handles(); 2657 ehea_update_firmware_handles();
@@ -2681,8 +2682,7 @@ static int ehea_open(struct net_device *dev)
2681 2682
2682 mutex_lock(&port->port_lock); 2683 mutex_lock(&port->port_lock);
2683 2684
2684 if (netif_msg_ifup(port)) 2685 netif_info(port, ifup, dev, "enabling port\n");
2685 ehea_info("enabling port %s", dev->name);
2686 2686
2687 ret = ehea_up(dev); 2687 ret = ehea_up(dev);
2688 if (!ret) { 2688 if (!ret) {
@@ -2717,8 +2717,7 @@ static int ehea_down(struct net_device *dev)
2717 2717
2718 ret = ehea_clean_all_portres(port); 2718 ret = ehea_clean_all_portres(port);
2719 if (ret) 2719 if (ret)
2720 ehea_info("Failed freeing resources for %s. ret=%i", 2720 netdev_info(dev, "Failed freeing resources. ret=%i\n", ret);
2721 dev->name, ret);
2722 2721
2723 ehea_update_firmware_handles(); 2722 ehea_update_firmware_handles();
2724 2723
@@ -2730,8 +2729,7 @@ static int ehea_stop(struct net_device *dev)
2730 int ret; 2729 int ret;
2731 struct ehea_port *port = netdev_priv(dev); 2730 struct ehea_port *port = netdev_priv(dev);
2732 2731
2733 if (netif_msg_ifdown(port)) 2732 netif_info(port, ifdown, dev, "disabling port\n");
2734 ehea_info("disabling port %s", dev->name);
2735 2733
2736 set_bit(__EHEA_DISABLE_PORT_RESET, &port->flags); 2734 set_bit(__EHEA_DISABLE_PORT_RESET, &port->flags);
2737 cancel_work_sync(&port->reset_task); 2735 cancel_work_sync(&port->reset_task);
@@ -2772,7 +2770,7 @@ static void ehea_flush_sq(struct ehea_port *port)
2772 msecs_to_jiffies(100)); 2770 msecs_to_jiffies(100));
2773 2771
2774 if (!ret) { 2772 if (!ret) {
2775 ehea_error("WARNING: sq not flushed completely"); 2773 pr_err("WARNING: sq not flushed completely\n");
2776 break; 2774 break;
2777 } 2775 }
2778 } 2776 }
@@ -2808,7 +2806,7 @@ int ehea_stop_qps(struct net_device *dev)
2808 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), 2806 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF),
2809 cb0); 2807 cb0);
2810 if (hret != H_SUCCESS) { 2808 if (hret != H_SUCCESS) {
2811 ehea_error("query_ehea_qp failed (1)"); 2809 pr_err("query_ehea_qp failed (1)\n");
2812 goto out; 2810 goto out;
2813 } 2811 }
2814 2812
@@ -2820,7 +2818,7 @@ int ehea_stop_qps(struct net_device *dev)
2820 1), cb0, &dummy64, 2818 1), cb0, &dummy64,
2821 &dummy64, &dummy16, &dummy16); 2819 &dummy64, &dummy16, &dummy16);
2822 if (hret != H_SUCCESS) { 2820 if (hret != H_SUCCESS) {
2823 ehea_error("modify_ehea_qp failed (1)"); 2821 pr_err("modify_ehea_qp failed (1)\n");
2824 goto out; 2822 goto out;
2825 } 2823 }
2826 2824
@@ -2828,14 +2826,14 @@ int ehea_stop_qps(struct net_device *dev)
2828 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), 2826 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF),
2829 cb0); 2827 cb0);
2830 if (hret != H_SUCCESS) { 2828 if (hret != H_SUCCESS) {
2831 ehea_error("query_ehea_qp failed (2)"); 2829 pr_err("query_ehea_qp failed (2)\n");
2832 goto out; 2830 goto out;
2833 } 2831 }
2834 2832
2835 /* deregister shared memory regions */ 2833 /* deregister shared memory regions */
2836 dret = ehea_rem_smrs(pr); 2834 dret = ehea_rem_smrs(pr);
2837 if (dret) { 2835 if (dret) {
2838 ehea_error("unreg shared memory region failed"); 2836 pr_err("unreg shared memory region failed\n");
2839 goto out; 2837 goto out;
2840 } 2838 }
2841 } 2839 }
@@ -2904,7 +2902,7 @@ int ehea_restart_qps(struct net_device *dev)
2904 2902
2905 ret = ehea_gen_smrs(pr); 2903 ret = ehea_gen_smrs(pr);
2906 if (ret) { 2904 if (ret) {
2907 ehea_error("creation of shared memory regions failed"); 2905 netdev_err(dev, "creation of shared memory regions failed\n");
2908 goto out; 2906 goto out;
2909 } 2907 }
2910 2908
@@ -2915,7 +2913,7 @@ int ehea_restart_qps(struct net_device *dev)
2915 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), 2913 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF),
2916 cb0); 2914 cb0);
2917 if (hret != H_SUCCESS) { 2915 if (hret != H_SUCCESS) {
2918 ehea_error("query_ehea_qp failed (1)"); 2916 netdev_err(dev, "query_ehea_qp failed (1)\n");
2919 goto out; 2917 goto out;
2920 } 2918 }
2921 2919
@@ -2927,7 +2925,7 @@ int ehea_restart_qps(struct net_device *dev)
2927 1), cb0, &dummy64, 2925 1), cb0, &dummy64,
2928 &dummy64, &dummy16, &dummy16); 2926 &dummy64, &dummy16, &dummy16);
2929 if (hret != H_SUCCESS) { 2927 if (hret != H_SUCCESS) {
2930 ehea_error("modify_ehea_qp failed (1)"); 2928 netdev_err(dev, "modify_ehea_qp failed (1)\n");
2931 goto out; 2929 goto out;
2932 } 2930 }
2933 2931
@@ -2935,7 +2933,7 @@ int ehea_restart_qps(struct net_device *dev)
2935 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF), 2933 EHEA_BMASK_SET(H_QPCB0_ALL, 0xFFFF),
2936 cb0); 2934 cb0);
2937 if (hret != H_SUCCESS) { 2935 if (hret != H_SUCCESS) {
2938 ehea_error("query_ehea_qp failed (2)"); 2936 netdev_err(dev, "query_ehea_qp failed (2)\n");
2939 goto out; 2937 goto out;
2940 } 2938 }
2941 2939
@@ -2972,8 +2970,7 @@ static void ehea_reset_port(struct work_struct *work)
2972 2970
2973 ehea_set_multicast_list(dev); 2971 ehea_set_multicast_list(dev);
2974 2972
2975 if (netif_msg_timer(port)) 2973 netif_info(port, timer, dev, "reset successful\n");
2976 ehea_info("Device %s resetted successfully", dev->name);
2977 2974
2978 port_napi_enable(port); 2975 port_napi_enable(port);
2979 2976
@@ -2988,7 +2985,7 @@ static void ehea_rereg_mrs(void)
2988 int ret, i; 2985 int ret, i;
2989 struct ehea_adapter *adapter; 2986 struct ehea_adapter *adapter;
2990 2987
2991 ehea_info("LPAR memory changed - re-initializing driver"); 2988 pr_info("LPAR memory changed - re-initializing driver\n");
2992 2989
2993 list_for_each_entry(adapter, &adapter_list, list) 2990 list_for_each_entry(adapter, &adapter_list, list)
2994 if (adapter->active_ports) { 2991 if (adapter->active_ports) {
@@ -3020,8 +3017,7 @@ static void ehea_rereg_mrs(void)
3020 /* Unregister old memory region */ 3017 /* Unregister old memory region */
3021 ret = ehea_rem_mr(&adapter->mr); 3018 ret = ehea_rem_mr(&adapter->mr);
3022 if (ret) { 3019 if (ret) {
3023 ehea_error("unregister MR failed - driver" 3020 pr_err("unregister MR failed - driver inoperable!\n");
3024 " inoperable!");
3025 goto out; 3021 goto out;
3026 } 3022 }
3027 } 3023 }
@@ -3033,8 +3029,7 @@ static void ehea_rereg_mrs(void)
3033 /* Register new memory region */ 3029 /* Register new memory region */
3034 ret = ehea_reg_kernel_mr(adapter, &adapter->mr); 3030 ret = ehea_reg_kernel_mr(adapter, &adapter->mr);
3035 if (ret) { 3031 if (ret) {
3036 ehea_error("register MR failed - driver" 3032 pr_err("register MR failed - driver inoperable!\n");
3037 " inoperable!");
3038 goto out; 3033 goto out;
3039 } 3034 }
3040 3035
@@ -3057,7 +3052,7 @@ static void ehea_rereg_mrs(void)
3057 } 3052 }
3058 } 3053 }
3059 } 3054 }
3060 ehea_info("re-initializing driver complete"); 3055 pr_info("re-initializing driver complete\n");
3061out: 3056out:
3062 return; 3057 return;
3063} 3058}
@@ -3110,7 +3105,7 @@ int ehea_get_jumboframe_status(struct ehea_port *port, int *jumbo)
3110 /* (Try to) enable *jumbo frames */ 3105 /* (Try to) enable *jumbo frames */
3111 cb4 = (void *)get_zeroed_page(GFP_KERNEL); 3106 cb4 = (void *)get_zeroed_page(GFP_KERNEL);
3112 if (!cb4) { 3107 if (!cb4) {
3113 ehea_error("no mem for cb4"); 3108 pr_err("no mem for cb4\n");
3114 ret = -ENOMEM; 3109 ret = -ENOMEM;
3115 goto out; 3110 goto out;
3116 } else { 3111 } else {
@@ -3172,13 +3167,13 @@ static struct device *ehea_register_port(struct ehea_port *port,
3172 3167
3173 ret = of_device_register(&port->ofdev); 3168 ret = of_device_register(&port->ofdev);
3174 if (ret) { 3169 if (ret) {
3175 ehea_error("failed to register device. ret=%d", ret); 3170 pr_err("failed to register device. ret=%d\n", ret);
3176 goto out; 3171 goto out;
3177 } 3172 }
3178 3173
3179 ret = device_create_file(&port->ofdev.dev, &dev_attr_log_port_id); 3174 ret = device_create_file(&port->ofdev.dev, &dev_attr_log_port_id);
3180 if (ret) { 3175 if (ret) {
3181 ehea_error("failed to register attributes, ret=%d", ret); 3176 pr_err("failed to register attributes, ret=%d\n", ret);
3182 goto out_unreg_of_dev; 3177 goto out_unreg_of_dev;
3183 } 3178 }
3184 3179
@@ -3228,7 +3223,7 @@ struct ehea_port *ehea_setup_single_port(struct ehea_adapter *adapter,
3228 dev = alloc_etherdev(sizeof(struct ehea_port)); 3223 dev = alloc_etherdev(sizeof(struct ehea_port));
3229 3224
3230 if (!dev) { 3225 if (!dev) {
3231 ehea_error("no mem for net_device"); 3226 pr_err("no mem for net_device\n");
3232 ret = -ENOMEM; 3227 ret = -ENOMEM;
3233 goto out_err; 3228 goto out_err;
3234 } 3229 }
@@ -3282,7 +3277,7 @@ struct ehea_port *ehea_setup_single_port(struct ehea_adapter *adapter,
3282 3277
3283 ret = register_netdev(dev); 3278 ret = register_netdev(dev);
3284 if (ret) { 3279 if (ret) {
3285 ehea_error("register_netdev failed. ret=%d", ret); 3280 pr_err("register_netdev failed. ret=%d\n", ret);
3286 goto out_unreg_port; 3281 goto out_unreg_port;
3287 } 3282 }
3288 3283
@@ -3290,11 +3285,10 @@ struct ehea_port *ehea_setup_single_port(struct ehea_adapter *adapter,
3290 3285
3291 ret = ehea_get_jumboframe_status(port, &jumbo); 3286 ret = ehea_get_jumboframe_status(port, &jumbo);
3292 if (ret) 3287 if (ret)
3293 ehea_error("failed determining jumbo frame status for %s", 3288 netdev_err(dev, "failed determining jumbo frame status\n");
3294 port->netdev->name);
3295 3289
3296 ehea_info("%s: Jumbo frames are %sabled", dev->name, 3290 netdev_info(dev, "Jumbo frames are %sabled\n",
3297 jumbo == 1 ? "en" : "dis"); 3291 jumbo == 1 ? "en" : "dis");
3298 3292
3299 adapter->active_ports++; 3293 adapter->active_ports++;
3300 3294
@@ -3310,8 +3304,8 @@ out_free_ethdev:
3310 free_netdev(dev); 3304 free_netdev(dev);
3311 3305
3312out_err: 3306out_err:
3313 ehea_error("setting up logical port with id=%d failed, ret=%d", 3307 pr_err("setting up logical port with id=%d failed, ret=%d\n",
3314 logical_port_id, ret); 3308 logical_port_id, ret);
3315 return NULL; 3309 return NULL;
3316} 3310}
3317 3311
@@ -3341,13 +3335,13 @@ static int ehea_setup_ports(struct ehea_adapter *adapter)
3341 dn_log_port_id = of_get_property(eth_dn, "ibm,hea-port-no", 3335 dn_log_port_id = of_get_property(eth_dn, "ibm,hea-port-no",
3342 NULL); 3336 NULL);
3343 if (!dn_log_port_id) { 3337 if (!dn_log_port_id) {
3344 ehea_error("bad device node: eth_dn name=%s", 3338 pr_err("bad device node: eth_dn name=%s\n",
3345 eth_dn->full_name); 3339 eth_dn->full_name);
3346 continue; 3340 continue;
3347 } 3341 }
3348 3342
3349 if (ehea_add_adapter_mr(adapter)) { 3343 if (ehea_add_adapter_mr(adapter)) {
3350 ehea_error("creating MR failed"); 3344 pr_err("creating MR failed\n");
3351 of_node_put(eth_dn); 3345 of_node_put(eth_dn);
3352 return -EIO; 3346 return -EIO;
3353 } 3347 }
@@ -3356,9 +3350,8 @@ static int ehea_setup_ports(struct ehea_adapter *adapter)
3356 *dn_log_port_id, 3350 *dn_log_port_id,
3357 eth_dn); 3351 eth_dn);
3358 if (adapter->port[i]) 3352 if (adapter->port[i])
3359 ehea_info("%s -> logical port id #%d", 3353 netdev_info(adapter->port[i]->netdev,
3360 adapter->port[i]->netdev->name, 3354 "logical port id #%d\n", *dn_log_port_id);
3361 *dn_log_port_id);
3362 else 3355 else
3363 ehea_remove_adapter_mr(adapter); 3356 ehea_remove_adapter_mr(adapter);
3364 3357
@@ -3403,21 +3396,20 @@ static ssize_t ehea_probe_port(struct device *dev,
3403 port = ehea_get_port(adapter, logical_port_id); 3396 port = ehea_get_port(adapter, logical_port_id);
3404 3397
3405 if (port) { 3398 if (port) {
3406 ehea_info("adding port with logical port id=%d failed. port " 3399 netdev_info(port->netdev, "adding port with logical port id=%d failed: port already configured\n",
3407 "already configured as %s.", logical_port_id, 3400 logical_port_id);
3408 port->netdev->name);
3409 return -EINVAL; 3401 return -EINVAL;
3410 } 3402 }
3411 3403
3412 eth_dn = ehea_get_eth_dn(adapter, logical_port_id); 3404 eth_dn = ehea_get_eth_dn(adapter, logical_port_id);
3413 3405
3414 if (!eth_dn) { 3406 if (!eth_dn) {
3415 ehea_info("no logical port with id %d found", logical_port_id); 3407 pr_info("no logical port with id %d found\n", logical_port_id);
3416 return -EINVAL; 3408 return -EINVAL;
3417 } 3409 }
3418 3410
3419 if (ehea_add_adapter_mr(adapter)) { 3411 if (ehea_add_adapter_mr(adapter)) {
3420 ehea_error("creating MR failed"); 3412 pr_err("creating MR failed\n");
3421 return -EIO; 3413 return -EIO;
3422 } 3414 }
3423 3415
@@ -3432,8 +3424,8 @@ static ssize_t ehea_probe_port(struct device *dev,
3432 break; 3424 break;
3433 } 3425 }
3434 3426
3435 ehea_info("added %s (logical port id=%d)", port->netdev->name, 3427 netdev_info(port->netdev, "added: (logical port id=%d)\n",
3436 logical_port_id); 3428 logical_port_id);
3437 } else { 3429 } else {
3438 ehea_remove_adapter_mr(adapter); 3430 ehea_remove_adapter_mr(adapter);
3439 return -EIO; 3431 return -EIO;
@@ -3456,8 +3448,8 @@ static ssize_t ehea_remove_port(struct device *dev,
3456 port = ehea_get_port(adapter, logical_port_id); 3448 port = ehea_get_port(adapter, logical_port_id);
3457 3449
3458 if (port) { 3450 if (port) {
3459 ehea_info("removed %s (logical port id=%d)", port->netdev->name, 3451 netdev_info(port->netdev, "removed: (logical port id=%d)\n",
3460 logical_port_id); 3452 logical_port_id);
3461 3453
3462 ehea_shutdown_single_port(port); 3454 ehea_shutdown_single_port(port);
3463 3455
@@ -3467,8 +3459,8 @@ static ssize_t ehea_remove_port(struct device *dev,
3467 break; 3459 break;
3468 } 3460 }
3469 } else { 3461 } else {
3470 ehea_error("removing port with logical port id=%d failed. port " 3462 pr_err("removing port with logical port id=%d failed. port not configured.\n",
3471 "not configured.", logical_port_id); 3463 logical_port_id);
3472 return -EINVAL; 3464 return -EINVAL;
3473 } 3465 }
3474 3466
@@ -3505,7 +3497,7 @@ static int __devinit ehea_probe_adapter(struct platform_device *dev,
3505 int ret; 3497 int ret;
3506 3498
3507 if (!dev || !dev->dev.of_node) { 3499 if (!dev || !dev->dev.of_node) {
3508 ehea_error("Invalid ibmebus device probed"); 3500 pr_err("Invalid ibmebus device probed\n");
3509 return -EINVAL; 3501 return -EINVAL;
3510 } 3502 }
3511 3503
@@ -3651,17 +3643,17 @@ static int ehea_mem_notifier(struct notifier_block *nb,
3651 3643
3652 switch (action) { 3644 switch (action) {
3653 case MEM_CANCEL_OFFLINE: 3645 case MEM_CANCEL_OFFLINE:
3654 ehea_info("memory offlining canceled"); 3646 pr_info("memory offlining canceled");
3655 /* Readd canceled memory block */ 3647 /* Readd canceled memory block */
3656 case MEM_ONLINE: 3648 case MEM_ONLINE:
3657 ehea_info("memory is going online"); 3649 pr_info("memory is going online");
3658 set_bit(__EHEA_STOP_XFER, &ehea_driver_flags); 3650 set_bit(__EHEA_STOP_XFER, &ehea_driver_flags);
3659 if (ehea_add_sect_bmap(arg->start_pfn, arg->nr_pages)) 3651 if (ehea_add_sect_bmap(arg->start_pfn, arg->nr_pages))
3660 goto out_unlock; 3652 goto out_unlock;
3661 ehea_rereg_mrs(); 3653 ehea_rereg_mrs();
3662 break; 3654 break;
3663 case MEM_GOING_OFFLINE: 3655 case MEM_GOING_OFFLINE:
3664 ehea_info("memory is going offline"); 3656 pr_info("memory is going offline");
3665 set_bit(__EHEA_STOP_XFER, &ehea_driver_flags); 3657 set_bit(__EHEA_STOP_XFER, &ehea_driver_flags);
3666 if (ehea_rem_sect_bmap(arg->start_pfn, arg->nr_pages)) 3658 if (ehea_rem_sect_bmap(arg->start_pfn, arg->nr_pages))
3667 goto out_unlock; 3659 goto out_unlock;
@@ -3687,7 +3679,7 @@ static int ehea_reboot_notifier(struct notifier_block *nb,
3687 unsigned long action, void *unused) 3679 unsigned long action, void *unused)
3688{ 3680{
3689 if (action == SYS_RESTART) { 3681 if (action == SYS_RESTART) {
3690 ehea_info("Reboot: freeing all eHEA resources"); 3682 pr_info("Reboot: freeing all eHEA resources\n");
3691 ibmebus_unregister_driver(&ehea_driver); 3683 ibmebus_unregister_driver(&ehea_driver);
3692 } 3684 }
3693 return NOTIFY_DONE; 3685 return NOTIFY_DONE;
@@ -3703,22 +3695,22 @@ static int check_module_parm(void)
3703 3695
3704 if ((rq1_entries < EHEA_MIN_ENTRIES_QP) || 3696 if ((rq1_entries < EHEA_MIN_ENTRIES_QP) ||
3705 (rq1_entries > EHEA_MAX_ENTRIES_RQ1)) { 3697 (rq1_entries > EHEA_MAX_ENTRIES_RQ1)) {
3706 ehea_info("Bad parameter: rq1_entries"); 3698 pr_info("Bad parameter: rq1_entries\n");
3707 ret = -EINVAL; 3699 ret = -EINVAL;
3708 } 3700 }
3709 if ((rq2_entries < EHEA_MIN_ENTRIES_QP) || 3701 if ((rq2_entries < EHEA_MIN_ENTRIES_QP) ||
3710 (rq2_entries > EHEA_MAX_ENTRIES_RQ2)) { 3702 (rq2_entries > EHEA_MAX_ENTRIES_RQ2)) {
3711 ehea_info("Bad parameter: rq2_entries"); 3703 pr_info("Bad parameter: rq2_entries\n");
3712 ret = -EINVAL; 3704 ret = -EINVAL;
3713 } 3705 }
3714 if ((rq3_entries < EHEA_MIN_ENTRIES_QP) || 3706 if ((rq3_entries < EHEA_MIN_ENTRIES_QP) ||
3715 (rq3_entries > EHEA_MAX_ENTRIES_RQ3)) { 3707 (rq3_entries > EHEA_MAX_ENTRIES_RQ3)) {
3716 ehea_info("Bad parameter: rq3_entries"); 3708 pr_info("Bad parameter: rq3_entries\n");
3717 ret = -EINVAL; 3709 ret = -EINVAL;
3718 } 3710 }
3719 if ((sq_entries < EHEA_MIN_ENTRIES_QP) || 3711 if ((sq_entries < EHEA_MIN_ENTRIES_QP) ||
3720 (sq_entries > EHEA_MAX_ENTRIES_SQ)) { 3712 (sq_entries > EHEA_MAX_ENTRIES_SQ)) {
3721 ehea_info("Bad parameter: sq_entries"); 3713 pr_info("Bad parameter: sq_entries\n");
3722 ret = -EINVAL; 3714 ret = -EINVAL;
3723 } 3715 }
3724 3716
@@ -3738,8 +3730,7 @@ int __init ehea_module_init(void)
3738{ 3730{
3739 int ret; 3731 int ret;
3740 3732
3741 printk(KERN_INFO "IBM eHEA ethernet device driver (Release %s)\n", 3733 pr_info("IBM eHEA ethernet device driver (Release %s)\n", DRV_VERSION);
3742 DRV_VERSION);
3743 3734
3744 memset(&ehea_fw_handles, 0, sizeof(ehea_fw_handles)); 3735 memset(&ehea_fw_handles, 0, sizeof(ehea_fw_handles));
3745 memset(&ehea_bcmc_regs, 0, sizeof(ehea_bcmc_regs)); 3736 memset(&ehea_bcmc_regs, 0, sizeof(ehea_bcmc_regs));
@@ -3757,27 +3748,27 @@ int __init ehea_module_init(void)
3757 3748
3758 ret = register_reboot_notifier(&ehea_reboot_nb); 3749 ret = register_reboot_notifier(&ehea_reboot_nb);
3759 if (ret) 3750 if (ret)
3760 ehea_info("failed registering reboot notifier"); 3751 pr_info("failed registering reboot notifier\n");
3761 3752
3762 ret = register_memory_notifier(&ehea_mem_nb); 3753 ret = register_memory_notifier(&ehea_mem_nb);
3763 if (ret) 3754 if (ret)
3764 ehea_info("failed registering memory remove notifier"); 3755 pr_info("failed registering memory remove notifier\n");
3765 3756
3766 ret = crash_shutdown_register(ehea_crash_handler); 3757 ret = crash_shutdown_register(ehea_crash_handler);
3767 if (ret) 3758 if (ret)
3768 ehea_info("failed registering crash handler"); 3759 pr_info("failed registering crash handler\n");
3769 3760
3770 ret = ibmebus_register_driver(&ehea_driver); 3761 ret = ibmebus_register_driver(&ehea_driver);
3771 if (ret) { 3762 if (ret) {
3772 ehea_error("failed registering eHEA device driver on ebus"); 3763 pr_err("failed registering eHEA device driver on ebus\n");
3773 goto out2; 3764 goto out2;
3774 } 3765 }
3775 3766
3776 ret = driver_create_file(&ehea_driver.driver, 3767 ret = driver_create_file(&ehea_driver.driver,
3777 &driver_attr_capabilities); 3768 &driver_attr_capabilities);
3778 if (ret) { 3769 if (ret) {
3779 ehea_error("failed to register capabilities attribute, ret=%d", 3770 pr_err("failed to register capabilities attribute, ret=%d\n",
3780 ret); 3771 ret);
3781 goto out3; 3772 goto out3;
3782 } 3773 }
3783 3774
@@ -3802,7 +3793,7 @@ static void __exit ehea_module_exit(void)
3802 unregister_reboot_notifier(&ehea_reboot_nb); 3793 unregister_reboot_notifier(&ehea_reboot_nb);
3803 ret = crash_shutdown_unregister(ehea_crash_handler); 3794 ret = crash_shutdown_unregister(ehea_crash_handler);
3804 if (ret) 3795 if (ret)
3805 ehea_info("failed unregistering crash handler"); 3796 pr_info("failed unregistering crash handler\n");
3806 unregister_memory_notifier(&ehea_mem_nb); 3797 unregister_memory_notifier(&ehea_mem_nb);
3807 kfree(ehea_fw_handles.arr); 3798 kfree(ehea_fw_handles.arr);
3808 kfree(ehea_bcmc_regs.arr); 3799 kfree(ehea_bcmc_regs.arr);
diff --git a/drivers/net/ehea/ehea_phyp.c b/drivers/net/ehea/ehea_phyp.c
index 8fe9dcaa7538..0506967b9044 100644
--- a/drivers/net/ehea/ehea_phyp.c
+++ b/drivers/net/ehea/ehea_phyp.c
@@ -26,6 +26,8 @@
26 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. 26 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
27 */ 27 */
28 28
29#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
30
29#include "ehea_phyp.h" 31#include "ehea_phyp.h"
30 32
31 33
@@ -67,12 +69,11 @@ static long ehea_plpar_hcall_norets(unsigned long opcode,
67 } 69 }
68 70
69 if (ret < H_SUCCESS) 71 if (ret < H_SUCCESS)
70 ehea_error("opcode=%lx ret=%lx" 72 pr_err("opcode=%lx ret=%lx"
71 " arg1=%lx arg2=%lx arg3=%lx arg4=%lx" 73 " arg1=%lx arg2=%lx arg3=%lx arg4=%lx"
72 " arg5=%lx arg6=%lx arg7=%lx ", 74 " arg5=%lx arg6=%lx arg7=%lx\n",
73 opcode, ret, 75 opcode, ret,
74 arg1, arg2, arg3, arg4, arg5, 76 arg1, arg2, arg3, arg4, arg5, arg6, arg7);
75 arg6, arg7);
76 77
77 return ret; 78 return ret;
78 } 79 }
@@ -114,19 +115,18 @@ static long ehea_plpar_hcall9(unsigned long opcode,
114 && (((cb_cat == H_PORT_CB4) && ((arg3 == H_PORT_CB4_JUMBO) 115 && (((cb_cat == H_PORT_CB4) && ((arg3 == H_PORT_CB4_JUMBO)
115 || (arg3 == H_PORT_CB4_SPEED))) || ((cb_cat == H_PORT_CB7) 116 || (arg3 == H_PORT_CB4_SPEED))) || ((cb_cat == H_PORT_CB7)
116 && (arg3 == H_PORT_CB7_DUCQPN))))) 117 && (arg3 == H_PORT_CB7_DUCQPN)))))
117 ehea_error("opcode=%lx ret=%lx" 118 pr_err("opcode=%lx ret=%lx"
118 " arg1=%lx arg2=%lx arg3=%lx arg4=%lx" 119 " arg1=%lx arg2=%lx arg3=%lx arg4=%lx"
119 " arg5=%lx arg6=%lx arg7=%lx arg8=%lx" 120 " arg5=%lx arg6=%lx arg7=%lx arg8=%lx"
120 " arg9=%lx" 121 " arg9=%lx"
121 " out1=%lx out2=%lx out3=%lx out4=%lx" 122 " out1=%lx out2=%lx out3=%lx out4=%lx"
122 " out5=%lx out6=%lx out7=%lx out8=%lx" 123 " out5=%lx out6=%lx out7=%lx out8=%lx"
123 " out9=%lx", 124 " out9=%lx\n",
124 opcode, ret, 125 opcode, ret,
125 arg1, arg2, arg3, arg4, arg5, 126 arg1, arg2, arg3, arg4, arg5,
126 arg6, arg7, arg8, arg9, 127 arg6, arg7, arg8, arg9,
127 outs[0], outs[1], outs[2], outs[3], 128 outs[0], outs[1], outs[2], outs[3], outs[4],
128 outs[4], outs[5], outs[6], outs[7], 129 outs[5], outs[6], outs[7], outs[8]);
129 outs[8]);
130 return ret; 130 return ret;
131 } 131 }
132 132
@@ -515,7 +515,7 @@ u64 ehea_h_register_rpage_mr(const u64 adapter_handle, const u64 mr_handle,
515 const u64 log_pageaddr, const u64 count) 515 const u64 log_pageaddr, const u64 count)
516{ 516{
517 if ((count > 1) && (log_pageaddr & ~PAGE_MASK)) { 517 if ((count > 1) && (log_pageaddr & ~PAGE_MASK)) {
518 ehea_error("not on pageboundary"); 518 pr_err("not on pageboundary\n");
519 return H_PARAMETER; 519 return H_PARAMETER;
520 } 520 }
521 521
diff --git a/drivers/net/ehea/ehea_qmr.c b/drivers/net/ehea/ehea_qmr.c
index 89128b6373e3..cd44bb8017d9 100644
--- a/drivers/net/ehea/ehea_qmr.c
+++ b/drivers/net/ehea/ehea_qmr.c
@@ -26,6 +26,8 @@
26 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. 26 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
27 */ 27 */
28 28
29#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
30
29#include <linux/mm.h> 31#include <linux/mm.h>
30#include <linux/slab.h> 32#include <linux/slab.h>
31#include "ehea.h" 33#include "ehea.h"
@@ -45,7 +47,7 @@ static void *hw_qpageit_get_inc(struct hw_queue *queue)
45 queue->current_q_offset -= queue->pagesize; 47 queue->current_q_offset -= queue->pagesize;
46 retvalue = NULL; 48 retvalue = NULL;
47 } else if (((u64) retvalue) & (EHEA_PAGESIZE-1)) { 49 } else if (((u64) retvalue) & (EHEA_PAGESIZE-1)) {
48 ehea_error("not on pageboundary"); 50 pr_err("not on pageboundary\n");
49 retvalue = NULL; 51 retvalue = NULL;
50 } 52 }
51 return retvalue; 53 return retvalue;
@@ -58,15 +60,15 @@ static int hw_queue_ctor(struct hw_queue *queue, const u32 nr_of_pages,
58 int i, k; 60 int i, k;
59 61
60 if ((pagesize > PAGE_SIZE) || (!pages_per_kpage)) { 62 if ((pagesize > PAGE_SIZE) || (!pages_per_kpage)) {
61 ehea_error("pagesize conflict! kernel pagesize=%d, " 63 pr_err("pagesize conflict! kernel pagesize=%d, ehea pagesize=%d\n",
62 "ehea pagesize=%d", (int)PAGE_SIZE, (int)pagesize); 64 (int)PAGE_SIZE, (int)pagesize);
63 return -EINVAL; 65 return -EINVAL;
64 } 66 }
65 67
66 queue->queue_length = nr_of_pages * pagesize; 68 queue->queue_length = nr_of_pages * pagesize;
67 queue->queue_pages = kmalloc(nr_of_pages * sizeof(void *), GFP_KERNEL); 69 queue->queue_pages = kmalloc(nr_of_pages * sizeof(void *), GFP_KERNEL);
68 if (!queue->queue_pages) { 70 if (!queue->queue_pages) {
69 ehea_error("no mem for queue_pages"); 71 pr_err("no mem for queue_pages\n");
70 return -ENOMEM; 72 return -ENOMEM;
71 } 73 }
72 74
@@ -130,7 +132,7 @@ struct ehea_cq *ehea_create_cq(struct ehea_adapter *adapter,
130 132
131 cq = kzalloc(sizeof(*cq), GFP_KERNEL); 133 cq = kzalloc(sizeof(*cq), GFP_KERNEL);
132 if (!cq) { 134 if (!cq) {
133 ehea_error("no mem for cq"); 135 pr_err("no mem for cq\n");
134 goto out_nomem; 136 goto out_nomem;
135 } 137 }
136 138
@@ -147,7 +149,7 @@ struct ehea_cq *ehea_create_cq(struct ehea_adapter *adapter,
147 hret = ehea_h_alloc_resource_cq(adapter->handle, &cq->attr, 149 hret = ehea_h_alloc_resource_cq(adapter->handle, &cq->attr,
148 &cq->fw_handle, &cq->epas); 150 &cq->fw_handle, &cq->epas);
149 if (hret != H_SUCCESS) { 151 if (hret != H_SUCCESS) {
150 ehea_error("alloc_resource_cq failed"); 152 pr_err("alloc_resource_cq failed\n");
151 goto out_freemem; 153 goto out_freemem;
152 } 154 }
153 155
@@ -159,7 +161,7 @@ struct ehea_cq *ehea_create_cq(struct ehea_adapter *adapter,
159 for (counter = 0; counter < cq->attr.nr_pages; counter++) { 161 for (counter = 0; counter < cq->attr.nr_pages; counter++) {
160 vpage = hw_qpageit_get_inc(&cq->hw_queue); 162 vpage = hw_qpageit_get_inc(&cq->hw_queue);
161 if (!vpage) { 163 if (!vpage) {
162 ehea_error("hw_qpageit_get_inc failed"); 164 pr_err("hw_qpageit_get_inc failed\n");
163 goto out_kill_hwq; 165 goto out_kill_hwq;
164 } 166 }
165 167
@@ -168,9 +170,8 @@ struct ehea_cq *ehea_create_cq(struct ehea_adapter *adapter,
168 0, EHEA_CQ_REGISTER_ORIG, 170 0, EHEA_CQ_REGISTER_ORIG,
169 cq->fw_handle, rpage, 1); 171 cq->fw_handle, rpage, 1);
170 if (hret < H_SUCCESS) { 172 if (hret < H_SUCCESS) {
171 ehea_error("register_rpage_cq failed ehea_cq=%p " 173 pr_err("register_rpage_cq failed ehea_cq=%p hret=%llx counter=%i act_pages=%i\n",
172 "hret=%llx counter=%i act_pages=%i", 174 cq, hret, counter, cq->attr.nr_pages);
173 cq, hret, counter, cq->attr.nr_pages);
174 goto out_kill_hwq; 175 goto out_kill_hwq;
175 } 176 }
176 177
@@ -178,14 +179,14 @@ struct ehea_cq *ehea_create_cq(struct ehea_adapter *adapter,
178 vpage = hw_qpageit_get_inc(&cq->hw_queue); 179 vpage = hw_qpageit_get_inc(&cq->hw_queue);
179 180
180 if ((hret != H_SUCCESS) || (vpage)) { 181 if ((hret != H_SUCCESS) || (vpage)) {
181 ehea_error("registration of pages not " 182 pr_err("registration of pages not complete hret=%llx\n",
182 "complete hret=%llx\n", hret); 183 hret);
183 goto out_kill_hwq; 184 goto out_kill_hwq;
184 } 185 }
185 } else { 186 } else {
186 if (hret != H_PAGE_REGISTERED) { 187 if (hret != H_PAGE_REGISTERED) {
187 ehea_error("CQ: registration of page failed " 188 pr_err("CQ: registration of page failed hret=%llx\n",
188 "hret=%llx\n", hret); 189 hret);
189 goto out_kill_hwq; 190 goto out_kill_hwq;
190 } 191 }
191 } 192 }
@@ -241,7 +242,7 @@ int ehea_destroy_cq(struct ehea_cq *cq)
241 } 242 }
242 243
243 if (hret != H_SUCCESS) { 244 if (hret != H_SUCCESS) {
244 ehea_error("destroy CQ failed"); 245 pr_err("destroy CQ failed\n");
245 return -EIO; 246 return -EIO;
246 } 247 }
247 248
@@ -259,7 +260,7 @@ struct ehea_eq *ehea_create_eq(struct ehea_adapter *adapter,
259 260
260 eq = kzalloc(sizeof(*eq), GFP_KERNEL); 261 eq = kzalloc(sizeof(*eq), GFP_KERNEL);
261 if (!eq) { 262 if (!eq) {
262 ehea_error("no mem for eq"); 263 pr_err("no mem for eq\n");
263 return NULL; 264 return NULL;
264 } 265 }
265 266
@@ -272,21 +273,21 @@ struct ehea_eq *ehea_create_eq(struct ehea_adapter *adapter,
272 hret = ehea_h_alloc_resource_eq(adapter->handle, 273 hret = ehea_h_alloc_resource_eq(adapter->handle,
273 &eq->attr, &eq->fw_handle); 274 &eq->attr, &eq->fw_handle);
274 if (hret != H_SUCCESS) { 275 if (hret != H_SUCCESS) {
275 ehea_error("alloc_resource_eq failed"); 276 pr_err("alloc_resource_eq failed\n");
276 goto out_freemem; 277 goto out_freemem;
277 } 278 }
278 279
279 ret = hw_queue_ctor(&eq->hw_queue, eq->attr.nr_pages, 280 ret = hw_queue_ctor(&eq->hw_queue, eq->attr.nr_pages,
280 EHEA_PAGESIZE, sizeof(struct ehea_eqe)); 281 EHEA_PAGESIZE, sizeof(struct ehea_eqe));
281 if (ret) { 282 if (ret) {
282 ehea_error("can't allocate eq pages"); 283 pr_err("can't allocate eq pages\n");
283 goto out_freeres; 284 goto out_freeres;
284 } 285 }
285 286
286 for (i = 0; i < eq->attr.nr_pages; i++) { 287 for (i = 0; i < eq->attr.nr_pages; i++) {
287 vpage = hw_qpageit_get_inc(&eq->hw_queue); 288 vpage = hw_qpageit_get_inc(&eq->hw_queue);
288 if (!vpage) { 289 if (!vpage) {
289 ehea_error("hw_qpageit_get_inc failed"); 290 pr_err("hw_qpageit_get_inc failed\n");
290 hret = H_RESOURCE; 291 hret = H_RESOURCE;
291 goto out_kill_hwq; 292 goto out_kill_hwq;
292 } 293 }
@@ -370,7 +371,7 @@ int ehea_destroy_eq(struct ehea_eq *eq)
370 } 371 }
371 372
372 if (hret != H_SUCCESS) { 373 if (hret != H_SUCCESS) {
373 ehea_error("destroy EQ failed"); 374 pr_err("destroy EQ failed\n");
374 return -EIO; 375 return -EIO;
375 } 376 }
376 377
@@ -395,7 +396,7 @@ int ehea_qp_alloc_register(struct ehea_qp *qp, struct hw_queue *hw_queue,
395 for (cnt = 0; cnt < nr_pages; cnt++) { 396 for (cnt = 0; cnt < nr_pages; cnt++) {
396 vpage = hw_qpageit_get_inc(hw_queue); 397 vpage = hw_qpageit_get_inc(hw_queue);
397 if (!vpage) { 398 if (!vpage) {
398 ehea_error("hw_qpageit_get_inc failed"); 399 pr_err("hw_qpageit_get_inc failed\n");
399 goto out_kill_hwq; 400 goto out_kill_hwq;
400 } 401 }
401 rpage = virt_to_abs(vpage); 402 rpage = virt_to_abs(vpage);
@@ -403,7 +404,7 @@ int ehea_qp_alloc_register(struct ehea_qp *qp, struct hw_queue *hw_queue,
403 0, h_call_q_selector, 404 0, h_call_q_selector,
404 qp->fw_handle, rpage, 1); 405 qp->fw_handle, rpage, 1);
405 if (hret < H_SUCCESS) { 406 if (hret < H_SUCCESS) {
406 ehea_error("register_rpage_qp failed"); 407 pr_err("register_rpage_qp failed\n");
407 goto out_kill_hwq; 408 goto out_kill_hwq;
408 } 409 }
409 } 410 }
@@ -432,7 +433,7 @@ struct ehea_qp *ehea_create_qp(struct ehea_adapter *adapter,
432 433
433 qp = kzalloc(sizeof(*qp), GFP_KERNEL); 434 qp = kzalloc(sizeof(*qp), GFP_KERNEL);
434 if (!qp) { 435 if (!qp) {
435 ehea_error("no mem for qp"); 436 pr_err("no mem for qp\n");
436 return NULL; 437 return NULL;
437 } 438 }
438 439
@@ -441,7 +442,7 @@ struct ehea_qp *ehea_create_qp(struct ehea_adapter *adapter,
441 hret = ehea_h_alloc_resource_qp(adapter->handle, init_attr, pd, 442 hret = ehea_h_alloc_resource_qp(adapter->handle, init_attr, pd,
442 &qp->fw_handle, &qp->epas); 443 &qp->fw_handle, &qp->epas);
443 if (hret != H_SUCCESS) { 444 if (hret != H_SUCCESS) {
444 ehea_error("ehea_h_alloc_resource_qp failed"); 445 pr_err("ehea_h_alloc_resource_qp failed\n");
445 goto out_freemem; 446 goto out_freemem;
446 } 447 }
447 448
@@ -455,7 +456,7 @@ struct ehea_qp *ehea_create_qp(struct ehea_adapter *adapter,
455 init_attr->act_wqe_size_enc_sq, adapter, 456 init_attr->act_wqe_size_enc_sq, adapter,
456 0); 457 0);
457 if (ret) { 458 if (ret) {
458 ehea_error("can't register for sq ret=%x", ret); 459 pr_err("can't register for sq ret=%x\n", ret);
459 goto out_freeres; 460 goto out_freeres;
460 } 461 }
461 462
@@ -465,7 +466,7 @@ struct ehea_qp *ehea_create_qp(struct ehea_adapter *adapter,
465 init_attr->act_wqe_size_enc_rq1, 466 init_attr->act_wqe_size_enc_rq1,
466 adapter, 1); 467 adapter, 1);
467 if (ret) { 468 if (ret) {
468 ehea_error("can't register for rq1 ret=%x", ret); 469 pr_err("can't register for rq1 ret=%x\n", ret);
469 goto out_kill_hwsq; 470 goto out_kill_hwsq;
470 } 471 }
471 472
@@ -476,7 +477,7 @@ struct ehea_qp *ehea_create_qp(struct ehea_adapter *adapter,
476 init_attr->act_wqe_size_enc_rq2, 477 init_attr->act_wqe_size_enc_rq2,
477 adapter, 2); 478 adapter, 2);
478 if (ret) { 479 if (ret) {
479 ehea_error("can't register for rq2 ret=%x", ret); 480 pr_err("can't register for rq2 ret=%x\n", ret);
480 goto out_kill_hwr1q; 481 goto out_kill_hwr1q;
481 } 482 }
482 } 483 }
@@ -488,7 +489,7 @@ struct ehea_qp *ehea_create_qp(struct ehea_adapter *adapter,
488 init_attr->act_wqe_size_enc_rq3, 489 init_attr->act_wqe_size_enc_rq3,
489 adapter, 3); 490 adapter, 3);
490 if (ret) { 491 if (ret) {
491 ehea_error("can't register for rq3 ret=%x", ret); 492 pr_err("can't register for rq3 ret=%x\n", ret);
492 goto out_kill_hwr2q; 493 goto out_kill_hwr2q;
493 } 494 }
494 } 495 }
@@ -553,7 +554,7 @@ int ehea_destroy_qp(struct ehea_qp *qp)
553 } 554 }
554 555
555 if (hret != H_SUCCESS) { 556 if (hret != H_SUCCESS) {
556 ehea_error("destroy QP failed"); 557 pr_err("destroy QP failed\n");
557 return -EIO; 558 return -EIO;
558 } 559 }
559 560
@@ -842,7 +843,7 @@ static u64 ehea_reg_mr_section(int top, int dir, int idx, u64 *pt,
842 (hret != H_PAGE_REGISTERED)) { 843 (hret != H_PAGE_REGISTERED)) {
843 ehea_h_free_resource(adapter->handle, mr->handle, 844 ehea_h_free_resource(adapter->handle, mr->handle,
844 FORCE_FREE); 845 FORCE_FREE);
845 ehea_error("register_rpage_mr failed"); 846 pr_err("register_rpage_mr failed\n");
846 return hret; 847 return hret;
847 } 848 }
848 } 849 }
@@ -896,7 +897,7 @@ int ehea_reg_kernel_mr(struct ehea_adapter *adapter, struct ehea_mr *mr)
896 897
897 pt = (void *)get_zeroed_page(GFP_KERNEL); 898 pt = (void *)get_zeroed_page(GFP_KERNEL);
898 if (!pt) { 899 if (!pt) {
899 ehea_error("no mem"); 900 pr_err("no mem\n");
900 ret = -ENOMEM; 901 ret = -ENOMEM;
901 goto out; 902 goto out;
902 } 903 }
@@ -906,14 +907,14 @@ int ehea_reg_kernel_mr(struct ehea_adapter *adapter, struct ehea_mr *mr)
906 &mr->handle, &mr->lkey); 907 &mr->handle, &mr->lkey);
907 908
908 if (hret != H_SUCCESS) { 909 if (hret != H_SUCCESS) {
909 ehea_error("alloc_resource_mr failed"); 910 pr_err("alloc_resource_mr failed\n");
910 ret = -EIO; 911 ret = -EIO;
911 goto out; 912 goto out;
912 } 913 }
913 914
914 if (!ehea_bmap) { 915 if (!ehea_bmap) {
915 ehea_h_free_resource(adapter->handle, mr->handle, FORCE_FREE); 916 ehea_h_free_resource(adapter->handle, mr->handle, FORCE_FREE);
916 ehea_error("no busmap available"); 917 pr_err("no busmap available\n");
917 ret = -EIO; 918 ret = -EIO;
918 goto out; 919 goto out;
919 } 920 }
@@ -929,7 +930,7 @@ int ehea_reg_kernel_mr(struct ehea_adapter *adapter, struct ehea_mr *mr)
929 930
930 if (hret != H_SUCCESS) { 931 if (hret != H_SUCCESS) {
931 ehea_h_free_resource(adapter->handle, mr->handle, FORCE_FREE); 932 ehea_h_free_resource(adapter->handle, mr->handle, FORCE_FREE);
932 ehea_error("registering mr failed"); 933 pr_err("registering mr failed\n");
933 ret = -EIO; 934 ret = -EIO;
934 goto out; 935 goto out;
935 } 936 }
@@ -952,7 +953,7 @@ int ehea_rem_mr(struct ehea_mr *mr)
952 hret = ehea_h_free_resource(mr->adapter->handle, mr->handle, 953 hret = ehea_h_free_resource(mr->adapter->handle, mr->handle,
953 FORCE_FREE); 954 FORCE_FREE);
954 if (hret != H_SUCCESS) { 955 if (hret != H_SUCCESS) {
955 ehea_error("destroy MR failed"); 956 pr_err("destroy MR failed\n");
956 return -EIO; 957 return -EIO;
957 } 958 }
958 959
@@ -987,14 +988,14 @@ void print_error_data(u64 *data)
987 length = EHEA_PAGESIZE; 988 length = EHEA_PAGESIZE;
988 989
989 if (type == EHEA_AER_RESTYPE_QP) 990 if (type == EHEA_AER_RESTYPE_QP)
990 ehea_error("QP (resource=%llX) state: AER=0x%llX, AERR=0x%llX, " 991 pr_err("QP (resource=%llX) state: AER=0x%llX, AERR=0x%llX, port=%llX\n",
991 "port=%llX", resource, data[6], data[12], data[22]); 992 resource, data[6], data[12], data[22]);
992 else if (type == EHEA_AER_RESTYPE_CQ) 993 else if (type == EHEA_AER_RESTYPE_CQ)
993 ehea_error("CQ (resource=%llX) state: AER=0x%llX", resource, 994 pr_err("CQ (resource=%llX) state: AER=0x%llX\n",
994 data[6]); 995 resource, data[6]);
995 else if (type == EHEA_AER_RESTYPE_EQ) 996 else if (type == EHEA_AER_RESTYPE_EQ)
996 ehea_error("EQ (resource=%llX) state: AER=0x%llX", resource, 997 pr_err("EQ (resource=%llX) state: AER=0x%llX\n",
997 data[6]); 998 resource, data[6]);
998 999
999 ehea_dump(data, length, "error data"); 1000 ehea_dump(data, length, "error data");
1000} 1001}
@@ -1008,7 +1009,7 @@ u64 ehea_error_data(struct ehea_adapter *adapter, u64 res_handle,
1008 1009
1009 rblock = (void *)get_zeroed_page(GFP_KERNEL); 1010 rblock = (void *)get_zeroed_page(GFP_KERNEL);
1010 if (!rblock) { 1011 if (!rblock) {
1011 ehea_error("Cannot allocate rblock memory."); 1012 pr_err("Cannot allocate rblock memory\n");
1012 goto out; 1013 goto out;
1013 } 1014 }
1014 1015
@@ -1020,9 +1021,9 @@ u64 ehea_error_data(struct ehea_adapter *adapter, u64 res_handle,
1020 *aerr = rblock[12]; 1021 *aerr = rblock[12];
1021 print_error_data(rblock); 1022 print_error_data(rblock);
1022 } else if (ret == H_R_STATE) { 1023 } else if (ret == H_R_STATE) {
1023 ehea_error("No error data available: %llX.", res_handle); 1024 pr_err("No error data available: %llX\n", res_handle);
1024 } else 1025 } else
1025 ehea_error("Error data could not be fetched: %llX", res_handle); 1026 pr_err("Error data could not be fetched: %llX\n", res_handle);
1026 1027
1027 free_page((unsigned long)rblock); 1028 free_page((unsigned long)rblock);
1028out: 1029out: