Commit eefa1d89 authored by Mike Marciniszyn's avatar Mike Marciniszyn Committed by Doug Ledford

IB/rdmavt: Correct sparse annotation

The __must_hold() is sufficent to correct the sparse
context imbalance inside a function.

Per Documentation/sparse.txt:
__must_hold - The specified lock is held on function entry and exit.

Fixes: Commit c0a67f6b ("IB/rdmavt: Annotate rvt_reset_qp()")
Reviewed-by: default avatarDennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: default avatarMike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: default avatarDennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
parent 584d9577
...@@ -500,12 +500,9 @@ static void rvt_remove_qp(struct rvt_dev_info *rdi, struct rvt_qp *qp) ...@@ -500,12 +500,9 @@ static void rvt_remove_qp(struct rvt_dev_info *rdi, struct rvt_qp *qp)
*/ */
static void rvt_reset_qp(struct rvt_dev_info *rdi, struct rvt_qp *qp, static void rvt_reset_qp(struct rvt_dev_info *rdi, struct rvt_qp *qp,
enum ib_qp_type type) enum ib_qp_type type)
__releases(&qp->s_lock) __must_hold(&qp->r_lock)
__releases(&qp->s_hlock) __must_hold(&qp->s_hlock)
__releases(&qp->r_lock) __must_hold(&qp->s_lock)
__acquires(&qp->r_lock)
__acquires(&qp->s_hlock)
__acquires(&qp->s_lock)
{ {
if (qp->state != IB_QPS_RESET) { if (qp->state != IB_QPS_RESET) {
qp->state = IB_QPS_RESET; qp->state = IB_QPS_RESET;
......
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