Re: [PATCH v2 5/5] blkdebug: protect rules and suspended_reqs with a lock

[Paolo Bonzini]

On 07/05/21 17:29, Eric Blake wrote:
+    qemu_mutex_lock(&s->lock);
      QLIST_FOREACH(r, &s->suspended_reqs, next) {
          if (!strcmp(r->tag, tag)) {
+            qemu_mutex_unlock(&s->lock);
              return true;
          }
      }
+    qemu_mutex_unlock(&s->lock);
      return false;
Would code like this be easier to write by using QEMU_LOCK_GUARD from
lockable.h?

Yes, this one would.  In other cases (rule_check) it's not so clear cut. 
 It depends whether you prefer to have the simplest code, or rather to 
have homogeneous use of either guards or lock/unlock.

Paolo