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 e66e8f96ceabd40bdb7027eab0015b7bd3928cf8..728ed457e447e98a8ea81711f51d43d69a16d414 100644 (file)
@@ -1297,6 +1297,13 @@ int hfi1_reset_device(int);
 /* return the driver's idea of the physical OPA port state */
 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