From: VMware, Inc <>
Date: Thu, 17 Jun 2010 21:40:26 +0000 (-0700)
Subject: lib/lock: pick up review comments
X-Git-Tag: 2010.06.16-268169~91
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f2a8b8e15d56ef34f3d92722173139dec72bcf7e;p=thirdparty%2Fopen-vm-tools.git

lib/lock: pick up review comments

Signed-off-by: Marcelo Vanzin <mvanzin@vmware.com>
---

diff --git a/open-vm-tools/lib/lock/ul.c b/open-vm-tools/lib/lock/ul.c
index d4a57f360..251c1888e 100644
--- a/open-vm-tools/lib/lock/ul.c
+++ b/open-vm-tools/lib/lock/ul.c
@@ -21,6 +21,7 @@
 #include "util.h"
 #include "userlock.h"
 #include "ulInt.h"
+#include "ulIntShared.h"
 #include "hashTable.h"
 
 
diff --git a/open-vm-tools/lib/lock/ulExcl.c b/open-vm-tools/lib/lock/ulExcl.c
index 82f0ccb9b..4b90e409c 100644
--- a/open-vm-tools/lib/lock/ulExcl.c
+++ b/open-vm-tools/lib/lock/ulExcl.c
@@ -20,6 +20,7 @@
 #include "str.h"
 #include "util.h"
 #include "userlock.h"
+#include "hostinfo.h"
 #include "ulInt.h"
 
 #define MXUSER_EXCL_SIGNATURE 0x58454B4C // 'LKEX' in memory
@@ -30,7 +31,7 @@ struct MXUserExclLock
    MXRecLock               recursiveLock;
 
 #if defined(MXUSER_STATS)
-   uint64                  holdStart;
+   VmTimeType              holdStart;
 
    MXUserAcquisitionStats  acquisitionStats;
    Atomic_Ptr              acquisitionHisto;
@@ -268,8 +269,8 @@ void
 MXUser_AcquireExclLock(MXUserExclLock *lock)  // IN/OUT:
 {
 #if defined(MXUSER_STATS)
-   uint64 begin;
-   uint64 value;
+   VmTimeType begin;
+   VmTimeType value;
    Bool contended;
    MXUserHisto *histo;
 #endif
@@ -330,7 +331,7 @@ void
 MXUser_ReleaseExclLock(MXUserExclLock *lock)  // IN/OUT:
 {
 #if defined(MXUSER_STATS)
-   uint64 value;
+   VmTimeType value;
    MXUserHisto *histo;
 #endif
 
@@ -390,7 +391,7 @@ MXUser_TryAcquireExclLock(MXUserExclLock *lock)  // IN/OUT:
    Bool success;
 
 #if defined(MXUSER_STATS)
-   uint64 begin;
+   VmTimeType begin;
 #endif
 
    ASSERT(lock && (lock->header.lockSignature == MXUSER_EXCL_SIGNATURE));
@@ -407,7 +408,7 @@ MXUser_TryAcquireExclLock(MXUserExclLock *lock)  // IN/OUT:
 
    if (success) {
 #if defined(MXUSER_STATS)
-      uint64 end = Hostinfo_SystemTimerNS();
+      VmTimeType end = Hostinfo_SystemTimerNS();
 #endif
 
       MXUserAcquisitionTracking(&lock->header, FALSE);
diff --git a/open-vm-tools/lib/lock/ulInt.h b/open-vm-tools/lib/lock/ulInt.h
index 37ca5f2cd..29f573d94 100644
--- a/open-vm-tools/lib/lock/ulInt.h
+++ b/open-vm-tools/lib/lock/ulInt.h
@@ -38,8 +38,6 @@ typedef pthread_t MXThreadID;
 
 #include "vm_basic_types.h"
 #include "vthreadBase.h"
-#include "hostinfo.h"
-#include "ulIntShared.h"
 
 #if defined(MXUSER_STATS)
 #include "circList.h"
diff --git a/open-vm-tools/lib/lock/ulRW.c b/open-vm-tools/lib/lock/ulRW.c
index 3f2d32771..0dd0e3494 100644
--- a/open-vm-tools/lib/lock/ulRW.c
+++ b/open-vm-tools/lib/lock/ulRW.c
@@ -25,6 +25,7 @@
 #include "util.h"
 #include "hashTable.h"
 #include "userlock.h"
+#include "hostinfo.h"
 #include "ulInt.h"
 
 #define MXUSER_RW_SIGNATURE 0x57524B4C // 'LKRW' in memory
@@ -232,7 +233,7 @@ typedef enum {
 
 typedef struct {
 #if defined(MXUSER_STATS)
-   uint64       holdStart;
+   VmTimeType   holdStart;
 #endif
 
    HolderState  state;
@@ -601,8 +602,8 @@ MXUserAcquisition(MXUserRWLock *lock,  // IN/OUT:
    HolderContext *myContext;
 
 #if defined(MXUSER_STATS)
-   uint64 begin;
-   uint64 value;
+   VmTimeType begin;
+   VmTimeType value;
    MXUserHisto *histo;
 #endif
 
@@ -783,8 +784,8 @@ MXUser_ReleaseRWLock(MXUserRWLock *lock)  // IN/OUT:
    HolderContext *myContext;
 
 #if defined(MXUSER_STATS)
-   uint64 holdEnd = Hostinfo_SystemTimerNS();
-   uint64 duration;
+   VmTimeType holdEnd = Hostinfo_SystemTimerNS();
+   VmTimeType duration;
    MXUserHisto *histo;
 #endif
 
diff --git a/open-vm-tools/lib/lock/ulRec.c b/open-vm-tools/lib/lock/ulRec.c
index 9158a90f3..ca27248c1 100644
--- a/open-vm-tools/lib/lock/ulRec.c
+++ b/open-vm-tools/lib/lock/ulRec.c
@@ -20,6 +20,7 @@
 #include "str.h"
 #include "util.h"
 #include "userlock.h"
+#include "hostinfo.h"
 #include "ulInt.h"
 
 #define MXUSER_REC_SIGNATURE 0x43524B4C // 'LKRC' in memory
@@ -303,7 +304,7 @@ MXUser_AcquireRecLock(MXUserRecLock *lock)  // IN/OUT:
    } else {
 #if defined(MXUSER_STATS)
       Bool contended;
-      uint64 begin;
+      VmTimeType begin;
 #endif
       /* Rank checking is only done on the first acquisition */
       MXUserAcquisitionTracking(&lock->header, TRUE);
@@ -319,7 +320,7 @@ MXUser_AcquireRecLock(MXUserRecLock *lock)  // IN/OUT:
 #if defined(MXUSER_STATS)
       if (MXRecLockCount(&lock->recursiveLock) == 1) {
          MXUserHisto *histo;
-         uint64 value = Hostinfo_SystemTimerNS() - begin;
+         VmTimeType value = Hostinfo_SystemTimerNS() - begin;
 
          MXUserAcquisitionSample(&lock->acquisitionStats, contended, value);
 
@@ -363,7 +364,7 @@ MXUser_ReleaseRecLock(MXUserRecLock *lock)  // IN/OUT:
    } else {
 #if defined(MXUSER_STATS)
       if (MXRecLockCount(&lock->recursiveLock) == 1) {
-         uint64 value = Hostinfo_SystemTimerNS() - lock->holdStart;
+         VmTimeType value = Hostinfo_SystemTimerNS() - lock->holdStart;
          MXUserHisto *histo = Atomic_ReadPtr(&lock->heldHisto);
 
          MXUserBasicStatsSample(&lock->heldStats, value);
@@ -425,7 +426,7 @@ MXUser_TryAcquireRecLock(MXUserRecLock *lock)  // IN/OUT:
       success = (*MXUserMX_TryLockRec)(lock->vmmLock);
    } else {
 #if defined(MXUSER_STATS)
-      uint64 begin;
+      VmTimeType begin;
 #endif
 
       if (MXUserTryAcquireFail(lock->header.lockName)) {