IB/hfi1: Fix pio map initialization
authorJubin John <jubin.john@intel.com>
Tue, 24 May 2016 19:51:06 +0000 (12:51 -0700)
committerDoug Ledford <dledford@redhat.com>
Thu, 26 May 2016 16:21:10 +0000 (12:21 -0400)
The pio map initialization function is off by 1 causing the last
kernel send context that is allocated to not get mapped into the
pio map which leads to the last kernel send context not being used
by any of the qps.

The send context reserved for VL15 is taken care of by setting the
scontext variable that is used as the index into the kernel send
context array to 1 and does not need to be accounted for in the
kernel send context counting loop as it is currently done.

Fix the kernel send context counting loop to account for all the
allocated send contexts and map all of them to the different VLs.

Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Reviewed-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Reviewed-by: Jianxin Xiong <jianxin.xiong@intel.com>
Signed-off-by: Jubin John <jubin.john@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/hfi1/pio.c

index c67b9ad..d5edb1a 100644 (file)
@@ -1835,8 +1835,7 @@ int pio_map_init(struct hfi1_devdata *dd, u8 port, u8 num_vls, u8 *vl_scontexts)
        struct pio_vl_map *oldmap, *newmap;
 
        if (!vl_scontexts) {
-               /* send context 0 reserved for VL15 */
-               for (i = 1; i < dd->num_send_contexts; i++)
+               for (i = 0; i < dd->num_send_contexts; i++)
                        if (dd->send_contexts[i].type == SC_KERNEL)
                                num_kernel_send_contexts++;
                /* truncate divide */