From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Mon, 16 May 2011 01:53:56 +0000 (-0400)
Subject: Windows: add lock assertions to cm_buf.c
X-Git-Tag: upstream/1.6.0.pre6^2~20
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=2910dca6489954c01fecbc21807310fb6584c45e;p=packages%2Fo%2Fopenafs.git

Windows: add lock assertions to cm_buf.c

add lock assertions to cm_buf.c in hopes of detecting why
periodic buf_scacheHashTable corruption is being detected.

Change-Id: Ib93ac7d449c7f2cb065219981f16ba319a9273b1
Reviewed-on: http://gerrit.openafs.org/4659
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-on: http://gerrit.openafs.org/4674
---

diff --git a/src/WINNT/afsd/cm_buf.c b/src/WINNT/afsd/cm_buf.c
index 6dc8c6b26..676bed6fb 100644
--- a/src/WINNT/afsd/cm_buf.c
+++ b/src/WINNT/afsd/cm_buf.c
@@ -655,6 +655,8 @@ cm_buf_t *buf_FindLocked(struct cm_scache *scp, osi_hyper_t *offsetp)
     afs_uint32 i;
     cm_buf_t *bp;
 
+    lock_AssertAny(&buf_globalLock);
+
     i = BUF_HASH(&scp->fid, offsetp);
     for(bp = cm_data.buf_scacheHashTablepp[i]; bp; bp=bp->hashp) {
         if (cm_FidCmp(&scp->fid, &bp->fid) == 0
@@ -1065,6 +1067,8 @@ long buf_GetNewLocked(struct cm_scache *scp, osi_hyper_t *offsetp, cm_req_t *req
              * appropriate label, if requested.
              */
             if (scp) {
+                lock_AssertWrite(&buf_globalLock);
+
                 bp->qFlags |= CM_BUF_QINHASH;
                 bp->fid = scp->fid;
 #ifdef DEBUG