LGTM. This rename clarifies this object's purpose w.r.t. other objects in 
struct zilog so that you do not have counter-intuitive code such as in the new 
zil_prune_commit_list() where an assertion is made that a mutex named 
\`zl_writer_lock' is held but then just below an object named \`zl_lock' must 
be acquired.

It also accords with the new documentation of the lock's purpose to protect 
"the creation and issuance of lwb zio's."

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/456#issuecomment-326669289
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/T54c57375fea79b4d-Md937016459493e2af7bd0c0f
Powered by Topicbox: https://topicbox.com

Reply via email to