IB/hfi1: Handle initial value of 0 for CCTI setting
authorDennis Dalessandro <dennis.dalessandro@intel.com>
Mon, 6 Nov 2017 14:39:29 +0000 (06:39 -0800)
committerDoug Ledford <dledford@redhat.com>
Mon, 13 Nov 2017 20:53:57 +0000 (15:53 -0500)
When the driver is loaded it sets the default CCTI value to be 0. When the FM
starts and CCA is disabled the driver sets the max value to 65535 due the driver
subtracting 1 from 0 and the fact that the CCTI value is a u16.

Special case the subtraction to find the index for a 0 value.

Reviewed-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/hfi1/mad.c

index 1277e1c..cf8dba3 100644 (file)
@@ -3701,7 +3701,11 @@ static void apply_cc_state(struct hfi1_pportdata *ppd)
 
        *new_cc_state = *old_cc_state;
 
-       new_cc_state->cct.ccti_limit = ppd->total_cct_entry - 1;
+       if (ppd->total_cct_entry)
+               new_cc_state->cct.ccti_limit = ppd->total_cct_entry - 1;
+       else
+               new_cc_state->cct.ccti_limit = 0;
+
        memcpy(new_cc_state->cct.entries, ppd->ccti_entries,
               ppd->total_cct_entry * sizeof(struct ib_cc_table_entry));