7 files changed, 41 insertions, 22 deletions

diff --git a/drivers/infiniband/hw/ehca/ehca_eq.c b/drivers/infiniband/hw/ehca/ehca_eq.c
index 3b87589b8ea0..d9b1bb40f480 100644
--- a/drivers/infiniband/hw/ehca/ehca_eq.c
+++ b/drivers/infiniband/hw/ehca/ehca_eq.c
@@ -122,21 +122,21 @@ int ehca_create_eq(struct ehca_shca *shca,
 
         /* register interrupt handlers and initialize work queues */
         if (type == EHCA_EQ) {
+                tasklet_init(&eq->interrupt_task, ehca_tasklet_eq, (long)shca);
+
                 ret = ibmebus_request_irq(eq->ist, ehca_interrupt_eq,
                                           IRQF_DISABLED, "ehca_eq",
                                           (void *)shca);
                 if (ret < 0)
                         ehca_err(ib_dev, "Can't map interrupt handler.");
-
-                tasklet_init(&eq->interrupt_task, ehca_tasklet_eq, (long)shca);
         } else if (type == EHCA_NEQ) {
+                tasklet_init(&eq->interrupt_task, ehca_tasklet_neq, (long)shca);
+
                 ret = ibmebus_request_irq(eq->ist, ehca_interrupt_neq,
                                           IRQF_DISABLED, "ehca_neq",
                                           (void *)shca);
                 if (ret < 0)
                         ehca_err(ib_dev, "Can't map interrupt handler.");
-
-                tasklet_init(&eq->interrupt_task, ehca_tasklet_neq, (long)shca);
         }
 
         eq->is_initialized = 1;
diff --git a/drivers/infiniband/hw/ehca/ehca_main.c b/drivers/infiniband/hw/ehca/ehca_main.c
index ecb51b396c42..cfc4de7a5da4 100644
--- a/drivers/infiniband/hw/ehca/ehca_main.c
+++ b/drivers/infiniband/hw/ehca/ehca_main.c
@@ -360,7 +360,8 @@ static int ehca_sense_attributes(struct ehca_shca *shca)
          * a firmware property, so it's valid across all adapters
          */
         if (ehca_lock_hcalls == -1)
-                ehca_lock_hcalls = !(shca->hca_cap & HCA_CAP_H_ALLOC_RES_SYNC);
+                ehca_lock_hcalls = !EHCA_BMASK_GET(HCA_CAP_H_ALLOC_RES_SYNC,
+                                        shca->hca_cap);
 
         /* translate supported MR page sizes; always support 4K */
         shca->hca_cap_mr_pgsize = EHCA_PAGESIZE;
diff --git a/drivers/infiniband/hw/ehca/ehca_mrmw.c b/drivers/infiniband/hw/ehca/ehca_mrmw.c
index 31a68b9c52d0..53f4cd4fc19a 100644
--- a/drivers/infiniband/hw/ehca/ehca_mrmw.c
+++ b/drivers/infiniband/hw/ehca/ehca_mrmw.c
@@ -933,11 +933,6 @@ int ehca_unmap_fmr(struct list_head *fmr_list)
         /* check all FMR belong to same SHCA, and check internal flag */
         list_for_each_entry(ib_fmr, fmr_list, list) {
                 prev_shca = shca;
-                if (!ib_fmr) {
-                        ehca_gen_err("bad fmr=%p in list", ib_fmr);
-                        ret = -EINVAL;
-                        goto unmap_fmr_exit0;
-                }
                 shca = container_of(ib_fmr->device, struct ehca_shca,
                                     ib_device);
                 e_fmr = container_of(ib_fmr, struct ehca_mr, ib.ib_fmr);
diff --git a/drivers/infiniband/hw/ehca/ehca_qp.c b/drivers/infiniband/hw/ehca/ehca_qp.c
index 47d388ec1cde..32fb34201aba 100644
--- a/drivers/infiniband/hw/ehca/ehca_qp.c
+++ b/drivers/infiniband/hw/ehca/ehca_qp.c
@@ -251,7 +251,7 @@ static inline int ibqptype2servicetype(enum ib_qp_type ibqptype)
                 return ST_UD;
         case IB_QPT_RAW_IPV6:
                 return -EINVAL;
-        case IB_QPT_RAW_ETY:
+        case IB_QPT_RAW_ETHERTYPE:
                 return -EINVAL;
         default:
                 ehca_gen_err("Invalid ibqptype=%x", ibqptype);
diff --git a/drivers/infiniband/hw/ehca/hcp_if.c b/drivers/infiniband/hw/ehca/hcp_if.c
index 4d5dc3304d42..e6f9cdd94c7a 100644
--- a/drivers/infiniband/hw/ehca/hcp_if.c
+++ b/drivers/infiniband/hw/ehca/hcp_if.c
@@ -269,6 +269,7 @@ u64 hipz_h_alloc_resource_cq(const struct ipz_adapter_handle adapter_handle,
                              struct ehca_cq *cq,
                              struct ehca_alloc_cq_parms *param)
 {
+        int rc;
         u64 ret;
         unsigned long outs[PLPAR_HCALL9_BUFSIZE];
 
@@ -283,8 +284,19 @@ u64 hipz_h_alloc_resource_cq(const struct ipz_adapter_handle adapter_handle,
         param->act_nr_of_entries = (u32)outs[3];
         param->act_pages = (u32)outs[4];
 
-        if (ret == H_SUCCESS)
-                hcp_galpas_ctor(&cq->galpas, 0, outs[5], outs[6]);
+        if (ret == H_SUCCESS) {
+                rc = hcp_galpas_ctor(&cq->galpas, 0, outs[5], outs[6]);
+                if (rc) {
+                        ehca_gen_err("Could not establish HW access. rc=%d paddr=%#lx",
+                                     rc, outs[5]);
+
+                        ehca_plpar_hcall_norets(H_FREE_RESOURCE,
+                                                adapter_handle.handle,     /* r4 */
+                                                cq->ipz_cq_handle.handle,  /* r5 */
+                                                0, 0, 0, 0, 0);
+                        ret = H_NO_MEM;
+                }
+        }
 
         if (ret == H_NOT_ENOUGH_RESOURCES)
                 ehca_gen_err("Not enough resources. ret=%lli", ret);
@@ -295,6 +307,7 @@ u64 hipz_h_alloc_resource_cq(const struct ipz_adapter_handle adapter_handle,
 u64 hipz_h_alloc_resource_qp(const struct ipz_adapter_handle adapter_handle,
                              struct ehca_alloc_qp_parms *parms, int is_user)
 {
+        int rc;
         u64 ret;
         u64 allocate_controls, max_r10_reg, r11, r12;
         unsigned long outs[PLPAR_HCALL9_BUFSIZE];
@@ -358,8 +371,19 @@ u64 hipz_h_alloc_resource_qp(const struct ipz_adapter_handle adapter_handle,
         parms->rqueue.queue_size =
                 (u32)EHCA_BMASK_GET(H_ALL_RES_QP_RQUEUE_SIZE_PAGES, outs[4]);
 
-        if (ret == H_SUCCESS)
-                hcp_galpas_ctor(&parms->galpas, is_user, outs[6], outs[6]);
+        if (ret == H_SUCCESS) {
+                rc = hcp_galpas_ctor(&parms->galpas, is_user, outs[6], outs[6]);
+                if (rc) {
+                        ehca_gen_err("Could not establish HW access. rc=%d paddr=%#lx",
+                                     rc, outs[6]);
+
+                        ehca_plpar_hcall_norets(H_FREE_RESOURCE,
+                                                adapter_handle.handle,     /* r4 */
+                                                parms->qp_handle.handle,  /* r5 */
+                                                0, 0, 0, 0, 0);
+                        ret = H_NO_MEM;
+                }
+        }
 
         if (ret == H_NOT_ENOUGH_RESOURCES)
                 ehca_gen_err("Not enough resources. ret=%lli", ret);
diff --git a/drivers/infiniband/hw/ehca/hcp_phyp.c b/drivers/infiniband/hw/ehca/hcp_phyp.c
index b3e0e72e8a73..077376ff3d28 100644
--- a/drivers/infiniband/hw/ehca/hcp_phyp.c
+++ b/drivers/infiniband/hw/ehca/hcp_phyp.c
@@ -42,10 +42,9 @@
 #include "ehca_classes.h"
 #include "hipz_hw.h"
 
-int hcall_map_page(u64 physaddr, u64 *mapaddr)
+u64 hcall_map_page(u64 physaddr)
 {
-        *mapaddr = (u64)(ioremap(physaddr, EHCA_PAGESIZE));
-        return 0;
+        return (u64)ioremap(physaddr, EHCA_PAGESIZE);
 }
 
 int hcall_unmap_page(u64 mapaddr)
@@ -58,9 +57,9 @@ int hcp_galpas_ctor(struct h_galpas *galpas, int is_user,
                     u64 paddr_kernel, u64 paddr_user)
 {
         if (!is_user) {
-                int ret = hcall_map_page(paddr_kernel, &galpas->kernel.fw_handle);
-                if (ret)
-                        return ret;
+                galpas->kernel.fw_handle = hcall_map_page(paddr_kernel);
+                if (!galpas->kernel.fw_handle)
+                        return -ENOMEM;
         } else
                 galpas->kernel.fw_handle = 0;
 
diff --git a/drivers/infiniband/hw/ehca/hcp_phyp.h b/drivers/infiniband/hw/ehca/hcp_phyp.h
index 204227d5303a..d1b029910249 100644
--- a/drivers/infiniband/hw/ehca/hcp_phyp.h
+++ b/drivers/infiniband/hw/ehca/hcp_phyp.h
@@ -83,7 +83,7 @@ int hcp_galpas_ctor(struct h_galpas *galpas, int is_user,
 
 int hcp_galpas_dtor(struct h_galpas *galpas);
 
-int hcall_map_page(u64 physaddr, u64 * mapaddr);
+u64 hcall_map_page(u64 physaddr);
 
 int hcall_unmap_page(u64 mapaddr);