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.8.0_pre1^2~3742
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=586139140f5b28a99ae80faea099166f7dfe45b7;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: I247ad7090a9484ae76b2e56f6f53ef8ee5af1abc
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>
---

diff --git a/src/WINNT/afsd/cm_buf.c b/src/WINNT/afsd/cm_buf.c
index e871f2e27..e62a20b15 100644
--- a/src/WINNT/afsd/cm_buf.c
+++ b/src/WINNT/afsd/cm_buf.c
@@ -658,6 +658,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
@@ -1068,6 +1070,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