Commit d97793af authored by Rohit Maheshwari's avatar Rohit Maheshwari Committed by David S. Miller

chcr: Fix CPU hard lockup

Soft lock should be taken in place of hard lock.
Signed-off-by: default avatarRohit Maheshwari <rohitm@chelsio.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent f35d1297
...@@ -120,12 +120,10 @@ static int chcr_ktls_save_keys(struct chcr_ktls_info *tx_info, ...@@ -120,12 +120,10 @@ static int chcr_ktls_save_keys(struct chcr_ktls_info *tx_info,
static int chcr_ktls_update_connection_state(struct chcr_ktls_info *tx_info, static int chcr_ktls_update_connection_state(struct chcr_ktls_info *tx_info,
int new_state) int new_state)
{ {
unsigned long flags;
/* This function can be called from both rx (interrupt context) and tx /* This function can be called from both rx (interrupt context) and tx
* queue contexts. * queue contexts.
*/ */
spin_lock_irqsave(&tx_info->lock, flags); spin_lock_bh(&tx_info->lock);
switch (tx_info->connection_state) { switch (tx_info->connection_state) {
case KTLS_CONN_CLOSED: case KTLS_CONN_CLOSED:
tx_info->connection_state = new_state; tx_info->connection_state = new_state;
...@@ -169,7 +167,7 @@ static int chcr_ktls_update_connection_state(struct chcr_ktls_info *tx_info, ...@@ -169,7 +167,7 @@ static int chcr_ktls_update_connection_state(struct chcr_ktls_info *tx_info,
pr_err("unknown KTLS connection state\n"); pr_err("unknown KTLS connection state\n");
break; break;
} }
spin_unlock_irqrestore(&tx_info->lock, flags); spin_unlock_bh(&tx_info->lock);
return tx_info->connection_state; return tx_info->connection_state;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment