From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Tue, 9 Apr 2013 14:48:12 +0000 (-0400)
Subject: Windows: Comment CommonWrite VDL Change
X-Git-Tag: upstream/1.8.0_pre1^2~1221
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=6580f37357e9c9c85404d81fb0fca8eb3e28b3c3;p=packages%2Fo%2Fopenafs.git

Windows: Comment CommonWrite VDL Change

Add a comment reminding the reader that CcSetFileSizes only needs
to be called on a ValidDataLength change if the VDL value has decreased.
A write operation cannot result in a decrease therefore CcSetFileSizes
does not need to be called from within AFSCommonWrite().

Change-Id: Iaf867ec876a6265dc2c8a7ba2319fdf67503a467
Reviewed-on: http://gerrit.openafs.org/9757
Reviewed-by: Rod Widdowson <rdw@steadingsoftware.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Peter Scott <pscott@kerneldrivers.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp b/src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp
index 744ef0aa0..2d5f44b70 100644
--- a/src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp
+++ b/src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp
@@ -709,7 +709,8 @@ try_exit:
                 }
 
                 //
-                // If this extended the Vdl, then update it accordingly
+                // If this extended the VDL, then update it accordingly.
+                // Increasing the VDL does not require a call to CcSetFileSizes.
                 //
 
                 if( liStartingByte.QuadPart + ulByteCount > pFcb->Header.ValidDataLength.QuadPart)