]> granicus.if.org Git - zfs/commit
Removing unneeded mutex for reading vq_pending_tree size
authorGregorKopka <mailfrom-github@kopka.net>
Thu, 19 Sep 2013 14:42:17 +0000 (16:42 +0200)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 25 Sep 2013 22:29:45 +0000 (15:29 -0700)
commitf9f3f1ef983e987a2e09a49c3684405561fed634
treed7c90d355ed2cf4e6c5f7f1de5ce59669e7ca75e
parent77831e17385ba822fe70436d862c0e14df5d67b2
Removing unneeded mutex for reading vq_pending_tree size

Locking mutex &vq->vq_lock in vdev_mirror_pending is unneeded:

* no data is modified
* only vq_pending_tree is read
* in case garbage is returned (eg. vq_pending_tree being updated
  while the read is made) the worst case would be that a single
  read could be queued on a mirror side which more busy than thought

The benefit of this change is streamlining of the code path since
it is taken for *every* mirror member on *every* read.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #1739
module/zfs/vdev_mirror.c