DEVEL15-windows-scache-recycle-20060603
authorJeffrey Altman <jaltman@secure-endpoints.com>
Sat, 3 Jun 2006 19:01:19 +0000 (19:01 +0000)
committerJeffrey Altman <jaltman@secure-endpoints.com>
Sat, 3 Jun 2006 19:01:19 +0000 (19:01 +0000)
commit10125417e2edb7e378611e302e44b34fb9a806a4
tree7ced001982f93ee6ea06404dc99d501c731b537f
parent1eb78d4f8d05ec55a7908832b16e78af554a5f73
DEVEL15-windows-scache-recycle-20060603

A deadlock was detected when performing "fs flushall" if the file
server reports VNOVNODE.   The scp->createBufferLock is already
held by the current thread and there is no mechanism to propagate
the knowledge.  Therefore, an alternate mechanism for clearing the
cache must be developed.  A new function cm_RecycleSCache(scp,flags)
has been extracted from cm_GetNewSCache().  This function performs
the task of recycling an cm_scache_t object.  When called from
cm_GetNewSCache() with no flags the expectation is that there are
no associated buffers that are queued to be read or written.  When
called from cm_Analyze() with the CM_SCACHE_RECYCLEFLAG_DESTROY_BUFFERS
flag, any queued buffers will be de-queued and marked as if the operations
were performed so the data can be discarded.

This patch also addresses the following issues:

* CM_ERROR codes were logged as "unknown" by cm_Analyze.

* In response to VNOVNODE, the parent is only discarded if the current
  cm_scache_t is not a directory.

* In the Ioctl FlushFile and FlushVolume functions, there were no
  checks to protect against flushing the Freelance SCache entries.

* In FlushFile, the wrong cm_scache_t object was being released.

* In cm_GetNewSCache, do not allow Freelance SCache entries to be
  recycled.  Choose a new entry if cm_RecycleSCache fails.

(cherry picked from commit f4f09d1c03cf738de6f73fc9d56502babd6743b4)
src/WINNT/afsd/cm_conn.c
src/WINNT/afsd/cm_ioctl.c
src/WINNT/afsd/cm_scache.c
src/WINNT/afsd/cm_scache.h
src/WINNT/afsd/cm_utils.c
src/WINNT/afsd/cm_vnodeops.c