IB/hfi1: Use host_link_state to read state when DC is shut down
authorBartlomiej Dudek <bartlomiej.dudek@intel.com>
Fri, 4 Aug 2017 20:52:32 +0000 (13:52 -0700)
committerDoug Ledford <dledford@redhat.com>
Tue, 22 Aug 2017 18:22:36 +0000 (14:22 -0400)
When DC is shut down (by e.g.  disconnecting the cable), the
driver should use host_link_state to get port's current
physical state. This is due to the fact that physical state
is read from DC's CSRs and when DC is shut down and state is
changed, its registers are not impacted.

Reviewed-by: Jakub Byczkowski <jakub.byczkowski@intel.com>
Signed-off-by: Bartlomiej Dudek <bartlomiej.dudek@intel.com>
Signed-off-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/hfi1/hfi.h

index e66e8f9..728ed45 100644 (file)
@@ -1298,6 +1298,13 @@ int hfi1_reset_device(int);
 static inline u32 driver_pstate(struct hfi1_pportdata *ppd)
 {
        /*
+        * When DC is shut down and state is changed, its CSRs are not
+        * impacted, therefore host_link_state should be used to get
+        * current physical state.
+        */
+       if (ppd->dd->dc_shutdown)
+               return driver_physical_state(ppd);
+       /*
         * The driver does some processing from the time the physical
         * link state is at LINKUP to the time the SM can be notified
         * as such. Return IB_PORTPHYSSTATE_TRAINING until the software