(snapshot) removing mbFinalize and usage of finalize

Now there is no need for finalize mechanism at all.
All that was required was resetting the rollback id

diff --git a/main.cc b/main.cc
index 66ad9f1d0a6b9f34b9b416d37d7890957984565d..b38f7ef4816d003f45d876133dacf0dc0d816bab 100644 (file)
--- a/main.cc
+++ b/main.cc
@@ -78,7 +78,6 @@ void real_main() {
        delete model;
 
        DEBUG("Exiting\n");
-       finalize();
 }
 
 int main_numargs;

diff --git a/snapshot-interface.h b/snapshot-interface.h
index 48fa2e3e0915247f6baece67fa46c62d5f21e7cb..5f54edb87f00f3299c20b1dc7beb9b5f4149107d 100644 (file)
--- a/snapshot-interface.h
+++ b/snapshot-interface.h
@@ -13,7 +13,6 @@ typedef void (*VoidFuncPtr)();
 void initSnapShotLibrary(unsigned int numbackingpages,
                unsigned int numsnapshots, unsigned int nummemoryregions,
                unsigned int numheappages, VoidFuncPtr entryPoint);
-void finalize();
 
 struct stackEntry {
   struct stackEntry *next;

diff --git a/snapshot.cc b/snapshot.cc
index 9a69a4b310141cd8fafd8fde5a2b45691b9c9b02..eeeb58c995762ea9c01bf40f2515ef5f956b7d50 100644 (file)
--- a/snapshot.cc
+++ b/snapshot.cc
@@ -120,7 +120,6 @@ void createSharedLibrary(){
        sTheRecord->mStackSize = STACK_SIZE_DEFAULT;
        sTheRecord->mIDToRollback = -1;
        sTheRecord->currSnapShotID = 0;
-       sTheRecord->mbFinalize = false;
 #endif
 }
 
@@ -197,7 +196,7 @@ void initSnapShotLibrary(unsigned int numbackingpages,
        pid_t forkedID = 0;
        snapshotid = sTheRecord->currSnapShotID;
        bool swapContext = false;
-       while( !sTheRecord->mbFinalize ){
+       while( true ){
                sTheRecord->currSnapShotID=snapshotid+1;
                forkedID = fork();
                if( 0 == forkedID ){
@@ -311,14 +310,7 @@ void rollBack( snapshot_id theID ){
                SSDEBUG("Invoked rollback");
                exit(EXIT_SUCCESS);
        }
+       sTheRecord->mIDToRollback = -1;
 #endif
 }
 
-/** The finalize method shuts down the snapshotting system.  */
-//Subramanian -- remove this function from the external interface and
-//have us call it internally
-void finalize(){
-#if !USE_MPROTECT_SNAPSHOT
-       sTheRecord->mbFinalize = true;
-#endif
-}

diff --git a/snapshot.h b/snapshot.h
index cbbe0070fb92328f6690ba604cebbb19f8466fce..629f63a0697c466cfbef348f7aefafd8e08e170e 100644 (file)
--- a/snapshot.h
+++ b/snapshot.h
@@ -11,7 +11,7 @@
 
 /* If USE_MPROTECT_SNAPSHOT=1, then snapshot by using mmap() and mprotect()
    If USE_MPROTECT_SNAPSHOT=0, then snapshot by using fork() */
-#define USE_MPROTECT_SNAPSHOT 1
+#define USE_MPROTECT_SNAPSHOT 0
 
 /* Size of signal stack */
 #define SIGSTACKSIZE 32768

diff --git a/snapshotimp.h b/snapshotimp.h
index 7ab009ec65c1439cc18f17bccadb31317ad2e0b0..e9f6d8bd50a4ab1493a8d0264c275fdfeade3b8b 100644 (file)
--- a/snapshotimp.h
+++ b/snapshotimp.h
@@ -63,7 +63,6 @@ struct Snapshot {
        snapshot_id mIDToRollback;
        ucontext_t mContextToRollback;
        snapshot_id currSnapShotID;
-       volatile bool mbFinalize;
 };
 extern struct Snapshot * sTheRecord;
 #endif