Skip to content

Commit bc2f39a

Browse files
Dan Carpenteranguy11
authored andcommitted
iavf: missing unlocks in iavf_watchdog_task()
This code was re-organized and there some unlocks missing now. Fixes: 898ef1c ("iavf: Combine init and watchdog state machines") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
1 parent 884d2b8 commit bc2f39a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/net/ethernet/intel/iavf/iavf_main.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2046,6 +2046,7 @@ static void iavf_watchdog_task(struct work_struct *work)
20462046
}
20472047
adapter->aq_required = 0;
20482048
adapter->current_op = VIRTCHNL_OP_UNKNOWN;
2049+
mutex_unlock(&adapter->crit_lock);
20492050
queue_delayed_work(iavf_wq,
20502051
&adapter->watchdog_task,
20512052
msecs_to_jiffies(10));
@@ -2076,9 +2077,8 @@ static void iavf_watchdog_task(struct work_struct *work)
20762077
iavf_detect_recover_hung(&adapter->vsi);
20772078
break;
20782079
case __IAVF_REMOVE:
2079-
mutex_unlock(&adapter->crit_lock);
2080-
return;
20812080
default:
2081+
mutex_unlock(&adapter->crit_lock);
20822082
return;
20832083
}
20842084

0 commit comments

Comments
 (0)