From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Thu, 8 May 2008 06:27:14 +0000 (+0000)
Subject: DEVEL15-windows-buf-invalidate-on-vnovnode-20080507
X-Git-Tag: BP-openafs-devel-1_5_3x~3
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=caaf868614943ddccbce18bcf770adc175c89904;p=packages%2Fo%2Fopenafs.git

DEVEL15-windows-buf-invalidate-on-vnovnode-20080507

LICENSE MIT

Once we know that an attempt to stablize a buffer has failed due to
VNOVNODE, stop trying to Stablize future buffers and just invalidate
all of them belonging to the same FID.


(cherry picked from commit ef619db804d471fda3bf3f77d55e8cd4e4699b29)
---

diff --git a/src/WINNT/afsd/cm_buf.c b/src/WINNT/afsd/cm_buf.c
index 4229aa939..584ede068 100644
--- a/src/WINNT/afsd/cm_buf.c
+++ b/src/WINNT/afsd/cm_buf.c
@@ -1602,10 +1602,16 @@ long buf_FlushCleanPages(cm_scache_t *scp, cm_user_t *userp, cm_req_t *reqp)
             buf_WaitIO(scp, bp);
             lock_ReleaseMutex(&bp->mx);
 
-            code = (*cm_buf_opsp->Stabilizep)(scp, userp, reqp);
-            if (code && code != CM_ERROR_BADFD) 
-                goto skip;
-
+            /* 
+             * if the error for the previous buffer was BADFD
+             * then all buffers for the FID are bad.  Do not
+             * attempt to stabalize.
+             */
+            if (code != CM_ERROR_BADFD) {
+                code = (*cm_buf_opsp->Stabilizep)(scp, userp, reqp);
+                if (code && code != CM_ERROR_BADFD) 
+                    goto skip;
+            }
             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