diff options
author | Alex Estrin <alex.estrin@intel.com> | 2017-07-24 16:46:36 +0200 |
---|---|---|
committer | Doug Ledford <dledford@redhat.com> | 2017-07-31 21:18:37 +0200 |
commit | 5e2d6764a729bf8d43894b0368b0ae11a19485c0 (patch) | |
tree | 320cfa734fabe64797f78669e7f63bfe1c3aaa6e /drivers/infiniband/hw/hfi1/intr.c | |
parent | IB/hfi1: Move saving PCI values to a separate function (diff) | |
download | linux-5e2d6764a729bf8d43894b0368b0ae11a19485c0.tar.xz linux-5e2d6764a729bf8d43894b0368b0ae11a19485c0.zip |
IB/hfi1: Verify port data VLs credits on transition to Armed
There is a window where the FM can read the buffer control table
and decide not to program buffers. When a port goes down, the code
clears the table and if it is not programmed, posted SDMA descriptors
will never complete due to no buffer credits.
Reviewed-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Alex Estrin <alex.estrin@intel.com>
Signed-off-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband/hw/hfi1/intr.c')
-rw-r--r-- | drivers/infiniband/hw/hfi1/intr.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/infiniband/hw/hfi1/intr.c b/drivers/infiniband/hw/hfi1/intr.c index 9469be9940e0..96845dfed5c5 100644 --- a/drivers/infiniband/hw/hfi1/intr.c +++ b/drivers/infiniband/hw/hfi1/intr.c @@ -164,6 +164,7 @@ void handle_linkup_change(struct hfi1_devdata *dd, u32 linkup) ppd->linkup = 0; /* clear HW details of the previous connection */ + ppd->actual_vls_operational = 0; reset_link_credits(dd); /* freeze after a link down to guarantee a clean egress */ |