void buf_HoldLocked(cm_buf_t *bp)
{
- osi_assert(bp->magic == CM_BUF_MAGIC);
- bp->refCount++;
+ osi_assertx(bp->magic == CM_BUF_MAGIC,"incorrect cm_buf_t magic");
+ InterlockedIncrement(&bp->refCount);
}
/* hold a reference to an already held buffer */
void buf_Hold(cm_buf_t *bp)
{
- lock_ObtainWrite(&buf_globalLock);
+ lock_ObtainRead(&buf_globalLock);
buf_HoldLocked(bp);
- lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&buf_globalLock);
}
/* code to drop reference count while holding buf_globalLock */
-void buf_ReleaseLocked(cm_buf_t *bp)
+void buf_ReleaseLocked(cm_buf_t *bp, afs_uint32 writeLocked)
{
+ afs_int32 refCount;
+
+ if (writeLocked)
+ lock_AssertWrite(&buf_globalLock);
+ else
+ lock_AssertRead(&buf_globalLock);
+
/* ensure that we're in the LRU queue if our ref count is 0 */
- osi_assert(bp->magic == CM_BUF_MAGIC);
+ osi_assertx(bp->magic == CM_BUF_MAGIC,"incorrect cm_buf_t magic");
+
+ refCount = InterlockedDecrement(&bp->refCount);
#ifdef DEBUG
- if (bp->refCount == 0)
+ if (refCount < 0)
osi_panic("buf refcount 0",__FILE__,__LINE__);;
#else
- osi_assert(bp->refCount > 0);
+ osi_assertx(refCount >= 0, "cm_buf_t refCount == 0");
#endif
- if (--bp->refCount == 0) {
- if (!(bp->flags & CM_BUF_INLRU)) {
+ if (refCount == 0) {
+ /*
+ * If we are read locked there could be a race condition
+ * with buf_Find() so we must obtain a write lock and
+ * double check that the refCount is actually zero
+ * before we remove the buffer from the LRU queue.
+ */
+ if (!writeLocked)
+ lock_ConvertRToW(&buf_globalLock);
+
+ if (bp->refCount == 0 &&
+ !(bp->flags & CM_BUF_INLRU)) {
osi_QAdd((osi_queue_t **) &cm_data.buf_freeListp, &bp->q);
/* watch for transition from empty to one element */
cm_data.buf_freeListEndp = cm_data.buf_freeListp;
bp->flags |= CM_BUF_INLRU;
}
+
+ if (!writeLocked)
+ lock_ConvertWToR(&buf_globalLock);
}
}
/* release a buffer. Buffer must be referenced, but unlocked. */
void buf_Release(cm_buf_t *bp)
{
- lock_ObtainWrite(&buf_globalLock);
- buf_ReleaseLocked(bp);
- lock_ReleaseWrite(&buf_globalLock);
+ afs_int32 refCount;
+
+ /* ensure that we're in the LRU queue if our ref count is 0 */
+ osi_assertx(bp->magic == CM_BUF_MAGIC,"incorrect cm_buf_t magic");
+
+ refCount = InterlockedDecrement(&bp->refCount);
+#ifdef DEBUG
+ if (refCount < 0)
+ osi_panic("buf refcount 0",__FILE__,__LINE__);;
+#else
+ osi_assertx(refCount >= 0, "cm_buf_t refCount == 0");
+#endif
+ if (refCount == 0) {
+ lock_ObtainWrite(&buf_globalLock);
+ if (bp->refCount == 0 &&
+ !(bp->flags & CM_BUF_INLRU)) {
+ osi_QAdd((osi_queue_t **) &cm_data.buf_freeListp, &bp->q);
+
+ /* watch for transition from empty to one element */
+ if (!cm_data.buf_freeListEndp)
+ cm_data.buf_freeListEndp = cm_data.buf_freeListp;
+ bp->flags |= CM_BUF_INLRU;
+ }
+ lock_ReleaseWrite(&buf_globalLock);
+ }
}
/* incremental sync daemon. Writes all dirty buffers every 5000 ms */
bp->dirtyp = NULL;
if (cm_data.buf_dirtyListp == NULL)
cm_data.buf_dirtyListEndp = NULL;
- buf_ReleaseLocked(bp);
+ buf_ReleaseLocked(bp, TRUE);
lock_ReleaseWrite(&buf_globalLock);
} else {
/* advance the pointer so we don't loop forever */
cm_data.buf_nOrigBuffers = cm_data.buf_nbuffers;
/* lower hash size to a prime number */
- cm_data.buf_hashSize = osi_PrimeLessThan(CM_BUF_HASHSIZE);
+ cm_data.buf_hashSize = osi_PrimeLessThan((afs_uint32)(cm_data.buf_nbuffers/7 + 1));
/* create hash table */
- memset((void *)cm_data.buf_hashTablepp, 0, cm_data.buf_hashSize * sizeof(cm_buf_t *));
+ memset((void *)cm_data.buf_scacheHashTablepp, 0, cm_data.buf_hashSize * sizeof(cm_buf_t *));
/* another hash table */
memset((void *)cm_data.buf_fileHashTablepp, 0, cm_data.buf_hashSize * sizeof(cm_buf_t *));
cm_data.buf_allp = NULL;
for (i=0; i<cm_data.buf_nbuffers; i++) {
- osi_assert(bp >= cm_data.bufHeaderBaseAddress && bp < (cm_buf_t *)cm_data.bufDataBaseAddress);
- osi_assert(data >= cm_data.bufDataBaseAddress && data < cm_data.bufEndOfData);
+ osi_assertx(bp >= cm_data.bufHeaderBaseAddress && bp < (cm_buf_t *)cm_data.bufDataBaseAddress,
+ "invalid cm_buf_t address");
+ osi_assertx(data >= cm_data.bufDataBaseAddress && data < cm_data.bufEndOfData,
+ "invalid cm_buf_t data address");
/* allocate and zero some storage */
memset(bp, 0, sizeof(cm_buf_t));
int release = 0;
if (scp)
- osi_assert(scp->magic == CM_SCACHE_MAGIC);
- osi_assert(bp->magic == CM_BUF_MAGIC);
+ osi_assertx(scp->magic == CM_SCACHE_MAGIC, "invalid cm_scache_t magic");
+ osi_assertx(bp->magic == CM_BUF_MAGIC, "invalid cm_buf_t magic");
while (1) {
/* if no IO is happening, we're done */
*/
cm_buf_t *buf_FindLocked(struct cm_scache *scp, osi_hyper_t *offsetp)
{
- long i;
+ afs_uint32 i;
cm_buf_t *bp;
i = BUF_HASH(&scp->fid, offsetp);
- for(bp = cm_data.buf_hashTablepp[i]; bp; bp=bp->hashp) {
+ for(bp = cm_data.buf_scacheHashTablepp[i]; bp; bp=bp->hashp) {
if (cm_FidCmp(&scp->fid, &bp->fid) == 0
&& offsetp->LowPart == bp->offset.LowPart
&& offsetp->HighPart == bp->offset.HighPart) {
{
cm_buf_t *bp;
- lock_ObtainWrite(&buf_globalLock);
+ lock_ObtainRead(&buf_globalLock);
bp = buf_FindLocked(scp, offsetp);
- lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&buf_globalLock);
return bp;
}
long code = 0;
long isdirty = 0;
cm_scache_t * scp = NULL;
+ osi_hyper_t offset;
- osi_assert(bp->magic == CM_BUF_MAGIC);
+ osi_assertx(bp->magic == CM_BUF_MAGIC, "invalid cm_buf_t magic");
while ((bp->flags & CM_BUF_DIRTY) == CM_BUF_DIRTY) {
isdirty = 1;
scp = cm_FindSCache(&bp->fid);
if (scp) {
osi_Log2(buf_logp, "buf_CleanAsyncLocked starts I/O on scp 0x%p buf 0x%p", scp, bp);
- code = (*cm_buf_opsp->Writep)(scp, &bp->offset,
- cm_data.buf_blockSize, 0, bp->userp,
- reqp);
+
+ offset = bp->offset;
+ LargeIntegerAdd(offset, ConvertLongToLargeInteger(bp->dirty_offset));
+ code = (*cm_buf_opsp->Writep)(scp, &offset, bp->dirty_length, 0, bp->userp, reqp);
osi_Log3(buf_logp, "buf_CleanAsyncLocked I/O on scp 0x%p buf 0x%p, done=%d", scp, bp, code);
cm_ReleaseSCache(scp);
* because we aren't going to be able to write this data to the file
* server.
*/
- if (code == CM_ERROR_NOSUCHFILE){
+ if (code == CM_ERROR_NOSUCHFILE || code == CM_ERROR_BADFD){
bp->flags &= ~CM_BUF_DIRTY;
bp->flags |= CM_BUF_ERROR;
- bp->error = CM_ERROR_NOSUCHFILE;
+ bp->dirty_offset = 0;
+ bp->dirty_length = 0;
+ bp->error = code;
bp->dataVersion = -1; /* bad */
bp->dirtyCounter++;
}
*/
void buf_Recycle(cm_buf_t *bp)
{
- int i;
+ afs_uint32 i;
cm_buf_t **lbpp;
cm_buf_t *tbp;
cm_buf_t *prevBp, *nextBp;
- osi_assert(bp->magic == CM_BUF_MAGIC);
+ osi_assertx(bp->magic == CM_BUF_MAGIC, "invalid cm_buf_t magic");
/* if we get here, we know that the buffer still has a 0 ref count,
* and that it is clean and has no currently pending I/O. This is
* have any lock conflicts, so we can grab the buffer lock out of
* order in the locking hierarchy.
*/
- osi_Log2( buf_logp, "buf_Recycle recycles 0x%p, off 0x%x",
- bp, bp->offset.LowPart);
+ osi_Log3( buf_logp, "buf_Recycle recycles 0x%p, off 0x%x:%08x",
+ bp, bp->offset.HighPart, bp->offset.LowPart);
- osi_assert(bp->refCount == 0);
- osi_assert(!(bp->flags & (CM_BUF_READING | CM_BUF_WRITING | CM_BUF_DIRTY)));
+ osi_assertx(bp->refCount == 0, "cm_buf_t refcount != 0");
+ osi_assertx(!(bp->flags & (CM_BUF_READING | CM_BUF_WRITING | CM_BUF_DIRTY)),
+ "incorrect cm_buf_t flags");
lock_AssertWrite(&buf_globalLock);
if (bp->flags & CM_BUF_INHASH) {
/* Remove from hash */
i = BUF_HASH(&bp->fid, &bp->offset);
- lbpp = &(cm_data.buf_hashTablepp[i]);
+ lbpp = &(cm_data.buf_scacheHashTablepp[i]);
for(tbp = *lbpp; tbp; lbpp = &tbp->hashp, tbp = *lbpp) {
- if (tbp == bp) break;
+ if (tbp == bp)
+ break;
}
/* we better find it */
osi_assertx(tbp != NULL, "buf_Recycle: hash table screwup");
*lbpp = bp->hashp; /* hash out */
+ bp->hashp = NULL;
/* Remove from file hash */
i = BUF_FILEHASH(&bp->fid);
prevBp = bp->fileHashBackp;
+ bp->fileHashBackp = NULL;
nextBp = bp->fileHashp;
+ bp->fileHashp = NULL;
if (prevBp)
prevBp->fileHashp = nextBp;
else
*/
long buf_GetNewLocked(struct cm_scache *scp, osi_hyper_t *offsetp, cm_buf_t **bufpp)
{
- cm_buf_t *bp; /* buffer we're dealing with */
+ cm_buf_t *bp; /* buffer we're dealing with */
cm_buf_t *nextBp; /* next buffer in file hash chain */
- long i; /* temp */
+ afs_uint32 i; /* temp */
cm_req_t req;
cm_InitReq(&req); /* just in case */
while(1) {
retry:
+ lock_ObtainRead(&scp->bufCreateLock);
lock_ObtainWrite(&buf_globalLock);
/* check to see if we lost the race */
if (scp) {
*/
bp->refCount--;
lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&scp->bufCreateLock);
return CM_BUF_EXISTS;
}
}
/* does this fix the problem below? it's a simple solution. */
if (!cm_data.buf_freeListEndp)
- {
+ {
lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&scp->bufCreateLock);
+ osi_Log0(afsd_logp, "buf_GetNewLocked: Free Buffer List is empty - sleeping 200ms");
Sleep(200);
goto retry;
- }
+ }
/* for debugging, assert free list isn't empty, although we
* really should try waiting for a running tranasction to finish
*/
buf_HoldLocked(bp);
lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&scp->bufCreateLock);
/* grab required lock and clean; this only
* starts the I/O. By the time we're back,
#endif
bp->offset = *offsetp;
i = BUF_HASH(&scp->fid, offsetp);
- bp->hashp = cm_data.buf_hashTablepp[i];
- cm_data.buf_hashTablepp[i] = bp;
+ bp->hashp = cm_data.buf_scacheHashTablepp[i];
+ cm_data.buf_scacheHashTablepp[i] = bp;
i = BUF_FILEHASH(&scp->fid);
nextBp = cm_data.buf_fileHashTablepp[i];
bp->fileHashp = nextBp;
cm_data.buf_fileHashTablepp[i] = bp;
}
- /* prepare to return it. Start by giving it a good
- * refcount */
- bp->refCount = 1;
-
- /* and since it has a non-zero ref count, we should move
- * it from the lru queue. It better be still there,
- * since we've held the global (big) lock since we found
- * it there.
+ /* we should move it from the lru queue. It better still be there,
+ * since we've held the global (big) lock since we found it there.
*/
osi_assertx(bp->flags & CM_BUF_INLRU,
"buf_GetNewLocked: LRU screwup");
+
if (cm_data.buf_freeListEndp == bp) {
/* we're the last guy in this queue, so maintain it */
cm_data.buf_freeListEndp = (cm_buf_t *) osi_QPrev(&bp->q);
osi_QRemove((osi_queue_t **) &cm_data.buf_freeListp, &bp->q);
bp->flags &= ~CM_BUF_INLRU;
- /* finally, grab the mutex so that people don't use it
+ /* grab the mutex so that people don't use it
* before the caller fills it with data. Again, no one
* should have been able to get to this dude to lock it.
*/
- osi_assertx(lock_TryMutex(&bp->mx),
- "buf_GetNewLocked: TryMutex failed");
+ if (!lock_TryMutex(&bp->mx)) {
+ osi_Log2(afsd_logp, "buf_GetNewLocked bp 0x%p cannot be mutex locked. refCount %d should be 0",
+ bp, bp->refCount);
+ osi_panic("buf_GetNewLocked: TryMutex failed",__FILE__,__LINE__);
+ }
+ /* prepare to return it. Give it a refcount */
+ bp->refCount = 1;
+
lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&scp->bufCreateLock);
*bufpp = bp;
#ifdef TESTING
return 0;
} /* for all buffers in lru queue */
lock_ReleaseWrite(&buf_globalLock);
- Sleep(100); /* give some time for a buffer to be freed */
+ lock_ReleaseRead(&scp->bufCreateLock);
+ osi_Log0(afsd_logp, "buf_GetNewLocked: Free Buffer List has no buffers with a zero refcount - sleeping 100ms");
+ Sleep(100); /* give some time for a buffer to be freed */
} /* while loop over everything */
/* not reached */
} /* the proc */
*/
lock_ReleaseMutex(&bp->mx);
*bufpp = bp;
- osi_Log3(buf_logp, "buf_GetNew returning bp 0x%p for scp 0x%p, offset 0x%x",
- bp, scp, offsetp->LowPart);
+ osi_Log4(buf_logp, "buf_GetNew returning bp 0x%p for scp 0x%p, offset 0x%x:%08x",
+ bp, scp, offsetp->HighPart, offsetp->LowPart);
return 0;
}
*/
if (created) {
/* load the page; freshly created pages should be idle */
- osi_assert(!(bp->flags & (CM_BUF_READING | CM_BUF_WRITING)));
-
- /* setup offset, event */
- bp->over.Offset = bp->offset.LowPart;
- bp->over.OffsetHigh = bp->offset.HighPart;
+ osi_assertx(!(bp->flags & (CM_BUF_READING | CM_BUF_WRITING)), "incorrect cm_buf_t flags");
/* start the I/O; may drop lock */
bp->flags |= CM_BUF_READING;
}
lock_ReleaseWrite(&buf_globalLock);
- osi_Log3(buf_logp, "buf_Get returning bp 0x%p for scp 0x%p, offset 0x%x",
- bp, scp, offsetp->LowPart);
+ osi_Log4(buf_logp, "buf_Get returning bp 0x%p for scp 0x%p, offset 0x%x:%08x",
+ bp, scp, offsetp->HighPart, offsetp->LowPart);
#ifdef TESTING
buf_ValidateBufQueues();
#endif /* TESTING */
long buf_CleanAsync(cm_buf_t *bp, cm_req_t *reqp)
{
long code;
- osi_assert(bp->magic == CM_BUF_MAGIC);
+ osi_assertx(bp->magic == CM_BUF_MAGIC, "invalid cm_buf_t magic");
lock_ObtainMutex(&bp->mx);
code = buf_CleanAsyncLocked(bp, reqp);
/* wait for a buffer's cleaning to finish */
void buf_CleanWait(cm_scache_t * scp, cm_buf_t *bp)
{
- osi_assert(bp->magic == CM_BUF_MAGIC);
+ osi_assertx(bp->magic == CM_BUF_MAGIC, "invalid cm_buf_t magic");
lock_ObtainMutex(&bp->mx);
if (bp->flags & CM_BUF_WRITING) {
*
* The buffer must be locked before calling this routine.
*/
-void buf_SetDirty(cm_buf_t *bp)
+void buf_SetDirty(cm_buf_t *bp, afs_uint32 offset, afs_uint32 length)
{
- osi_assert(bp->magic == CM_BUF_MAGIC);
- osi_assert(bp->refCount > 0);
-
+ osi_assertx(bp->magic == CM_BUF_MAGIC, "invalid cm_buf_t magic");
+ osi_assertx(bp->refCount > 0, "cm_buf_t refcount 0");
+
if (bp->flags & CM_BUF_DIRTY) {
+
osi_Log1(buf_logp, "buf_SetDirty 0x%p already dirty", bp);
+
+ if (bp->dirty_offset <= offset) {
+ if (bp->dirty_offset + bp->dirty_length >= offset + length) {
+ /* dirty_length remains the same */
+ } else {
+ bp->dirty_length = offset + length - bp->dirty_offset;
+ }
+ } else /* bp->dirty_offset > offset */ {
+ if (bp->dirty_offset + bp->dirty_length >= offset + length) {
+ bp->dirty_length = bp->dirty_offset + bp->dirty_length - offset;
+ } else {
+ bp->dirty_length = length;
+ }
+ bp->dirty_offset = offset;
+ }
} else {
osi_Log1(buf_logp, "buf_SetDirty 0x%p", bp);
- }
- /* set dirty bit */
- bp->flags |= CM_BUF_DIRTY;
- /* and turn off EOF flag, since it has associated data now */
- bp->flags &= ~CM_BUF_EOF;
+ /* set dirty bit */
+ bp->flags |= CM_BUF_DIRTY;
- /* and add to the dirty list.
- * we obtain a hold on the buffer for as long as it remains
- * in the list. buffers are only removed from the list by
- * the buf_IncrSyncer function regardless of when else the
- * dirty flag might be cleared.
- *
- * This should never happen but just in case there is a bug
- * elsewhere, never add to the dirty list if the buffer is
- * already there.
- */
- lock_ObtainWrite(&buf_globalLock);
- if (bp->dirtyp == NULL && cm_data.buf_dirtyListEndp != bp) {
- buf_HoldLocked(bp);
- if (!cm_data.buf_dirtyListp) {
- cm_data.buf_dirtyListp = cm_data.buf_dirtyListEndp = bp;
- } else {
- cm_data.buf_dirtyListEndp->dirtyp = bp;
- cm_data.buf_dirtyListEndp = bp;
- }
- bp->dirtyp = NULL;
+ /* and turn off EOF flag, since it has associated data now */
+ bp->flags &= ~CM_BUF_EOF;
+
+ bp->dirty_offset = offset;
+ bp->dirty_length = length;
+
+ /* and add to the dirty list.
+ * we obtain a hold on the buffer for as long as it remains
+ * in the list. buffers are only removed from the list by
+ * the buf_IncrSyncer function regardless of when else the
+ * dirty flag might be cleared.
+ *
+ * This should never happen but just in case there is a bug
+ * elsewhere, never add to the dirty list if the buffer is
+ * already there.
+ */
+ lock_ObtainWrite(&buf_globalLock);
+ if (bp->dirtyp == NULL && cm_data.buf_dirtyListEndp != bp) {
+ buf_HoldLocked(bp);
+ if (!cm_data.buf_dirtyListp) {
+ cm_data.buf_dirtyListp = cm_data.buf_dirtyListEndp = bp;
+ } else {
+ cm_data.buf_dirtyListEndp->dirtyp = bp;
+ cm_data.buf_dirtyListEndp = bp;
+ }
+ bp->dirtyp = NULL;
+ }
+ lock_ReleaseWrite(&buf_globalLock);
}
- lock_ReleaseWrite(&buf_globalLock);
}
/* clean all buffers, reset log pointers and invalidate all buffers.
*/
long buf_CleanAndReset(void)
{
- long i;
+ afs_uint32 i;
cm_buf_t *bp;
cm_req_t req;
- lock_ObtainWrite(&buf_globalLock);
+ lock_ObtainRead(&buf_globalLock);
for(i=0; i<cm_data.buf_hashSize; i++) {
- for(bp = cm_data.buf_hashTablepp[i]; bp; bp = bp->hashp) {
+ for(bp = cm_data.buf_scacheHashTablepp[i]; bp; bp = bp->hashp) {
if ((bp->flags & CM_BUF_DIRTY) == CM_BUF_DIRTY) {
buf_HoldLocked(bp);
- lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&buf_globalLock);
/* now no locks are held; clean buffer and go on */
cm_InitReq(&req);
buf_CleanWait(NULL, bp);
/* relock and release buffer */
- lock_ObtainWrite(&buf_globalLock);
- buf_ReleaseLocked(bp);
+ lock_ObtainRead(&buf_globalLock);
+ buf_ReleaseLocked(bp, FALSE);
} /* dirty */
} /* over one bucket */
} /* for loop over all hash buckets */
/* release locks */
- lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&buf_globalLock);
#ifdef TESTING
buf_ValidateBufQueues();
osi_hyper_t bufEnd;
long code;
long bufferPos;
- long i;
+ afs_uint32 i;
/* assert that cm_bufCreateLock is held in write mode */
lock_AssertWrite(&scp->bufCreateLock);
i = BUF_FILEHASH(&scp->fid);
- lock_ObtainWrite(&buf_globalLock);
+ lock_ObtainRead(&buf_globalLock);
bufp = cm_data.buf_fileHashTablepp[i];
if (bufp == NULL) {
- lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&buf_globalLock);
return 0;
}
buf_HoldLocked(bufp);
- lock_ReleaseWrite(&buf_globalLock);
- for(; bufp; bufp = nbufp) {
+ lock_ReleaseRead(&buf_globalLock);
+ while (bufp) {
lock_ObtainMutex(&bufp->mx);
bufEnd.HighPart = 0;
| CM_SCACHESYNC_BUFLOCKED);
- lock_ObtainWrite(&buf_globalLock);
/* if we succeeded in our locking, and this applies to the right
* file, and the truncate request overlaps the buffer either
* totally or partially, then do something.
if (LargeIntegerLessThanOrEqualTo(*sizep, bufp->offset)) {
/* truncating the entire page */
bufp->flags &= ~CM_BUF_DIRTY;
+ bufp->dirty_offset = 0;
+ bufp->dirty_length = 0;
bufp->dataVersion = -1; /* known bad */
bufp->dirtyCounter++;
}
* visible again.
*/
bufferPos = sizep->LowPart & (cm_data.buf_blockSize - 1);
- osi_assert(bufferPos != 0);
+ osi_assertx(bufferPos != 0, "non-zero bufferPos");
memset(bufp->datap + bufferPos, 0,
cm_data.buf_blockSize - bufferPos);
}
if (!code) {
nbufp = bufp->fileHashp;
if (nbufp)
- buf_HoldLocked(nbufp);
+ buf_Hold(nbufp);
} else {
/* This forces the loop to end and the error code
* to be returned. */
nbufp = NULL;
}
- buf_ReleaseLocked(bufp);
- lock_ReleaseWrite(&buf_globalLock);
+ buf_Release(bufp);
+ bufp = nbufp;
}
#ifdef TESTING
cm_buf_t *bp; /* buffer we're hacking on */
cm_buf_t *nbp;
int didRelease;
- long i;
+ afs_uint32 i;
i = BUF_FILEHASH(&scp->fid);
code = 0;
- lock_ObtainWrite(&buf_globalLock);
+ lock_ObtainRead(&buf_globalLock);
bp = cm_data.buf_fileHashTablepp[i];
if (bp)
buf_HoldLocked(bp);
- lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&buf_globalLock);
for (; bp; bp = nbp) {
didRelease = 0; /* haven't released this buffer yet */
lock_ReleaseMutex(&bp->mx);
code = (*cm_buf_opsp->Stabilizep)(scp, userp, reqp);
- if (code)
+ if (code && code != CM_ERROR_BADFD)
goto skip;
- lock_ObtainWrite(&buf_globalLock);
+ if (code == CM_ERROR_BADFD) {
+ /* if the scp's FID is bad its because we received VNOVNODE
+ * when attempting to FetchStatus before the write. This
+ * page therefore contains data that can no longer be stored.
+ */
+ lock_ObtainMutex(&bp->mx);
+ bp->flags &= ~CM_BUF_DIRTY;
+ bp->flags |= CM_BUF_ERROR;
+ bp->error = CM_ERROR_BADFD;
+ bp->dirty_offset = 0;
+ bp->dirty_length = 0;
+ bp->dataVersion = -1; /* known bad */
+ bp->dirtyCounter++;
+ lock_ReleaseMutex(&bp->mx);
+ }
+
/* actually, we only know that buffer is clean if ref
* count is 1, since we don't have buffer itself locked.
*/
if (!(bp->flags & CM_BUF_DIRTY)) {
+ lock_ObtainWrite(&buf_globalLock);
if (bp->refCount == 1) { /* bp is held above */
nbp = bp->fileHashp;
if (nbp)
buf_HoldLocked(nbp);
- buf_ReleaseLocked(bp);
+ buf_ReleaseLocked(bp, TRUE);
didRelease = 1;
buf_Recycle(bp);
}
+ lock_ReleaseWrite(&buf_globalLock);
}
- lock_ReleaseWrite(&buf_globalLock);
- (*cm_buf_opsp->Unstabilizep)(scp, userp);
+ if (code == 0)
+ (*cm_buf_opsp->Unstabilizep)(scp, userp);
}
skip:
if (!didRelease) {
- lock_ObtainWrite(&buf_globalLock);
+ lock_ObtainRead(&buf_globalLock);
nbp = bp->fileHashp;
if (nbp)
buf_HoldLocked(nbp);
- buf_ReleaseLocked(bp);
- lock_ReleaseWrite(&buf_globalLock);
+ buf_ReleaseLocked(bp, FALSE);
+ lock_ReleaseRead(&buf_globalLock);
}
} /* for loop over a bunch of buffers */
return code;
}
+/* Must be called with scp->mx held */
+long buf_ForceDataVersion(cm_scache_t * scp, afs_uint64 fromVersion, afs_uint64 toVersion)
+{
+ cm_buf_t * bp;
+ afs_uint32 i;
+ int found = 0;
+
+ lock_AssertMutex(&scp->mx);
+
+ i = BUF_FILEHASH(&scp->fid);
+
+ lock_ObtainRead(&buf_globalLock);
+
+ for (bp = cm_data.buf_fileHashTablepp[i]; bp; bp = bp->fileHashp) {
+ if (cm_FidCmp(&bp->fid, &scp->fid) == 0) {
+ if (bp->dataVersion == fromVersion) {
+ bp->dataVersion = toVersion;
+ found = 1;
+ }
+ }
+ }
+ lock_ReleaseRead(&buf_globalLock);
+
+ if (found)
+ return 0;
+ else
+ return ENOENT;
+}
+
long buf_CleanVnode(struct cm_scache *scp, cm_user_t *userp, cm_req_t *reqp)
{
long code = 0;
long wasDirty = 0;
cm_buf_t *bp; /* buffer we're hacking on */
cm_buf_t *nbp; /* next one */
- long i;
+ afs_uint32 i;
i = BUF_FILEHASH(&scp->fid);
- lock_ObtainWrite(&buf_globalLock);
+ lock_ObtainRead(&buf_globalLock);
bp = cm_data.buf_fileHashTablepp[i];
if (bp)
buf_HoldLocked(bp);
- lock_ReleaseWrite(&buf_globalLock);
+ lock_ReleaseRead(&buf_globalLock);
for (; bp; bp = nbp) {
/* clean buffer synchronously */
if (cm_FidCmp(&bp->fid, &scp->fid) == 0) {
lock_ReleaseMutex(&bp->mx);
}
- lock_ObtainWrite(&buf_globalLock);
+ lock_ObtainRead(&buf_globalLock);
nbp = bp->fileHashp;
if (nbp)
buf_HoldLocked(nbp);
- buf_ReleaseLocked(bp);
- lock_ReleaseWrite(&buf_globalLock);
+ buf_ReleaseLocked(bp, FALSE);
+ lock_ReleaseRead(&buf_globalLock);
} /* for loop over a bunch of buffers */
#ifdef TESTING
}
#endif /* TESTING */
-/* dump the contents of the buf_hashTablepp. */
+/* dump the contents of the buf_scacheHashTablepp. */
int cm_DumpBufHashTable(FILE *outputFile, char *cookie, int lock)
{
int zilch;
cm_buf_t *bp;
char output[1024];
- int i;
+ afs_uint32 i;
- if (cm_data.buf_hashTablepp == NULL)
+ if (cm_data.buf_scacheHashTablepp == NULL)
return -1;
if (lock)
for (i = 0; i < cm_data.buf_hashSize; i++)
{
- for (bp = cm_data.buf_hashTablepp[i]; bp; bp=bp->hashp)
+ for (bp = cm_data.buf_scacheHashTablepp[i]; bp; bp=bp->hashp)
{
- if (bp->refCount)
- {
- StringCbPrintfA(output, sizeof(output),
- "%s bp=0x%08X, hash=%d, fid (cell=%d, volume=%d, "
- "vnode=%d, unique=%d), size=%d refCount=%d\r\n",
- cookie, (void *)bp, i, bp->fid.cell, bp->fid.volume,
- bp->fid.vnode, bp->fid.unique, bp->size, bp->refCount);
- WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
- }
+ StringCbPrintfA(output, sizeof(output),
+ "%s bp=0x%08X, hash=%d, fid (cell=%d, volume=%d, "
+ "vnode=%d, unique=%d), offset=%x:%08x, dv=%I64d, "
+ "flags=0x%x, cmFlags=0x%x, refCount=%d\r\n",
+ cookie, (void *)bp, i, bp->fid.cell, bp->fid.volume,
+ bp->fid.vnode, bp->fid.unique, bp->offset.HighPart,
+ bp->offset.LowPart, bp->dataVersion, bp->flags,
+ bp->cmFlags, bp->refCount);
+ WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
}
}
StringCbPrintfA(output, sizeof(output), "%s - Done dumping buf_HashTable.\r\n", cookie);
WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
+ StringCbPrintfA(output, sizeof(output), "%s - dumping buf_freeListEndp\r\n", cookie);
+ WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
+ for(bp = cm_data.buf_freeListEndp; bp; bp=(cm_buf_t *) osi_QPrev(&bp->q)) {
+ StringCbPrintfA(output, sizeof(output),
+ "%s bp=0x%08X, fid (cell=%d, volume=%d, "
+ "vnode=%d, unique=%d), offset=%x:%08x, dv=%I64d, "
+ "flags=0x%x, cmFlags=0x%x, refCount=%d\r\n",
+ cookie, (void *)bp, bp->fid.cell, bp->fid.volume,
+ bp->fid.vnode, bp->fid.unique, bp->offset.HighPart,
+ bp->offset.LowPart, bp->dataVersion, bp->flags,
+ bp->cmFlags, bp->refCount);
+ WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
+ }
+ StringCbPrintfA(output, sizeof(output), "%s - Done dumping buf_FreeListEndp.\r\n", cookie);
+ WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
+
+ StringCbPrintfA(output, sizeof(output), "%s - dumping buf_dirtyListEndp\r\n", cookie);
+ WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
+ for(bp = cm_data.buf_dirtyListEndp; bp; bp=(cm_buf_t *) osi_QPrev(&bp->q)) {
+ StringCbPrintfA(output, sizeof(output),
+ "%s bp=0x%08X, fid (cell=%d, volume=%d, "
+ "vnode=%d, unique=%d), offset=%x:%08x, dv=%I64d, "
+ "flags=0x%x, cmFlags=0x%x, refCount=%d\r\n",
+ cookie, (void *)bp, bp->fid.cell, bp->fid.volume,
+ bp->fid.vnode, bp->fid.unique, bp->offset.HighPart,
+ bp->offset.LowPart, bp->dataVersion, bp->flags,
+ bp->cmFlags, bp->refCount);
+ WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
+ }
+ StringCbPrintfA(output, sizeof(output), "%s - Done dumping buf_dirtyListEndp.\r\n", cookie);
+ WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
+
if (lock)
lock_ReleaseRead(&buf_globalLock);
return 0;
{
cm_buf_t *bp;
afs_uint32 bcount = 0;
+ afs_uint32 i;
- for (bp = cm_data.buf_allp; bp; bp=bp->allp, bcount++) {
+ i = BUF_FILEHASH(fidp);
+
+ for (bp = cm_data.buf_fileHashTablepp[i]; bp; bp=bp->allp, bcount++) {
if (!cm_FidCmp(fidp, &bp->fid) && (bp->flags & CM_BUF_DIRTY))
return 1;
}
for (bp = cm_data.buf_allp; bp; bp=bp->allp, bcount++) {
if (!cm_FidCmp(fidp, &bp->fid) && (bp->flags & CM_BUF_DIRTY)) {
- buf_Hold(bp);
+ buf_Hold(bp);
lock_ObtainMutex(&bp->mx);
bp->cmFlags &= ~CM_BUF_CMSTORING;
bp->flags &= ~CM_BUF_DIRTY;
+ bp->dirty_offset = 0;
+ bp->dirty_length = 0;
bp->flags |= CM_BUF_ERROR;
bp->error = VNOVNODE;
bp->dataVersion = -1; /* bad */
git://git.openafs.org / openafs.git / blobdiff