OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_1-223-gbb13f1a
Gerrit Code Review
gerrit@openafs.org
Fri, 2 Dec 2011 16:07:06 -0800 (PST)
The following commit has been merged in the openafs-devel-1_7_x branch:
commit bb13f1a3fb54eba1982395544dc64de8e2486a5e
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date: Fri Dec 2 11:14:11 2011 -0500
Windows: buf_CleanAsync is not async; rename it
buf_CleanAsync() calls cm_BufWrite() which stores the dirty
buffers synchronously. There is nothing asynchronous about
buf_CleanAsync() so rename it to buf_Clean() and buf_CleanAsyncLocked()
to buf_CleanLocked(). Update the comments to remove the references
to the asynchronous processing which doesn't exist.
That is not to say that the call to buf_Clean() in buf_GetNewLocked()
should not be asynchronous; it should. There is no such functionality
at the moment. One approach would be to modify buf_IncrSyncer to
trigger on an event set by buf_GetNewLocked() instead of the call
to buf_Clean(). Another approach would be registering a background
store event. In any case, that is for another patchset.
Reviewed-on: http://gerrit.openafs.org/6172
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@secure-endpoints.com>
(cherry picked from commit c712403f192bd9932f3ec1dce5fb23461d3d0349)
Change-Id: I4baf6425cd3d7c8b6d2a32ea1fac29431d3a0259
Reviewed-on: http://gerrit.openafs.org/6185
Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@secure-endpoints.com>
src/WINNT/afsd/cm_buf.c | 36 +++++++++++++++++++-----------------
src/WINNT/afsd/cm_buf.h | 4 ++--
2 files changed, 21 insertions(+), 19 deletions(-)
--
OpenAFS Master Repository