summaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorRaju Rangoju <rajur@chelsio.com>2017-06-19 14:10:48 +0200
committerDavid S. Miller <davem@davemloft.net>2017-06-19 16:51:45 +0200
commit910603818c6c0558fe9b5e056a3bd5195aaae1a5 (patch)
tree4b9c5b4f4861960179a59682d13b2cac56c21b29 /include
parentcxgb4: add new T6 pci device id's (diff)
downloadlinux-910603818c6c0558fe9b5e056a3bd5195aaae1a5.tar.xz
linux-910603818c6c0558fe9b5e056a3bd5195aaae1a5.zip
cxgb4: notify uP to route ctrlq compl to rdma rspq
During the module initialisation there is a possible race (basically race between uld and lld) where neither the uld nor lld notifies the uP about where to route the ctrl queue completions. LLD skips notifying uP as the rdma queues were not created by then (will leave it to ULD to notify the uP). As the ULD comes up, it also skips notifying the uP as the flag FULL_INIT_DONE is not set yet (ULD assumes that the interface is not up yet). Consequently, this race between uld and lld leaves uP unnotified about where to send the ctrl queue completions to, leading to iwarp RI_RES WR failure. Here is the race: CPU 0 CPU1 - allocates nic rx queus - t4_sge_alloc_ctrl_txq() (if rdma rsp queues exists, tell uP to route ctrl queue compl to rdma rspq) - acquires the mutex_lock - allocates rdma response queues - if FULL_INIT_DONE set, tell uP to route ctrl queue compl to rdma rspq - relinquishes mutex_lock - acquires the mutex_lock - enable_rx() - set FULL_INIT_DONE - relinquishes mutex_lock This patch fixes the above issue. Fixes: e7519f9926f1('cxgb4: avoid enabling napi twice to the same queue') Signed-off-by: Raju Rangoju <rajur@chelsio.com> Acked-by: Steve Wise <swise@opengridcomputing.com> CC: Stable <stable@vger.kernel.org> # 4.9+ Signed-off-by: Ganesh Goudar <ganeshgr@chelsio.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions