Changeset 6912 in vbox for trunk

Timestamp:

Feb 11, 2008 10:04:41 PM (17 years ago)

Author:

vboxsync

Message:

Removed the normal (PGMVIRTHANDLERTYPE_NORMAL) kind of virtual access handlers. This type have never been used and is just complicating the code. It can easily be re-added later if found to be useful.

Location:

trunk

Files:

• include/VBox/pgm.h (modified) (1 diff)
• src/VBox/VMM/PGMHandler.cpp (modified) (2 diffs)
• src/VBox/VMM/PGMMap.cpp (modified) (1 diff)
• src/VBox/VMM/VMMAll/PGMAllGst.h (modified) (1 diff)
• src/VBox/VMM/VMMAll/PGMAllHandler.cpp (modified) (3 diffs)

trunk/include/VBox/pgm.h

@@ -178 +178 @@
 typedef enum PGMVIRTHANDLERTYPE
 {
-    /** Natural traps only. */
-    PGMVIRTHANDLERTYPE_NORMAL = 1,
     /** Write access handled. */
-    PGMVIRTHANDLERTYPE_WRITE,
+    PGMVIRTHANDLERTYPE_WRITE = 1,
     /** All access handled. */
     PGMVIRTHANDLERTYPE_ALL,

trunk/src/VBox/VMM/PGMHandler.cpp

@@ -282 +282 @@
     switch (enmType)
     {
-        case PGMVIRTHANDLERTYPE_NORMAL:
         case PGMVIRTHANDLERTYPE_ALL:
         case PGMVIRTHANDLERTYPE_WRITE:
@@ -569 +568 @@
     switch (pCur->enmType)
     {
-        case PGMVIRTHANDLERTYPE_NORMAL: pszType = "Natural"; break;
         case PGMVIRTHANDLERTYPE_WRITE:  pszType = "Write  "; break;
         case PGMVIRTHANDLERTYPE_ALL:    pszType = "All    "; break;

trunk/src/VBox/VMM/PGMMap.cpp

@@ -1090 +1090 @@
     switch (pCur->enmType)
     {
-        case PGMVIRTHANDLERTYPE_NORMAL:
-            RTLogPrintf("NORMAL   %RGv-%RGv size %RGv %s\n", pCur->GCPtr, pCur->GCPtrLast, pCur->cb, pCur->pszDesc);
-            break;
         case PGMVIRTHANDLERTYPE_WRITE:
             RTLogPrintf("WRITE    %RGv-%RGv size %RGv %s\n", pCur->GCPtr, pCur->GCPtrLast, pCur->cb, pCur->pszDesc);

trunk/src/VBox/VMM/VMMAll/PGMAllGst.h

@@ -659 +659 @@
     switch (pCur->enmType)
     {
-        case PGMVIRTHANDLERTYPE_NORMAL:     fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER; break;
         case PGMVIRTHANDLERTYPE_WRITE:      fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER | MM_RAM_FLAGS_VIRTUAL_WRITE; break;
         case PGMVIRTHANDLERTYPE_ALL:        fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER | MM_RAM_FLAGS_VIRTUAL_ALL; break;

trunk/src/VBox/VMM/VMMAll/PGMAllHandler.cpp

@@ -1103 +1103 @@
     switch (pCur->enmType)
     {
-        case PGMVIRTHANDLERTYPE_NORMAL: fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER; break;
         case PGMVIRTHANDLERTYPE_WRITE:  fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER | MM_RAM_FLAGS_VIRTUAL_WRITE; break;
         case PGMVIRTHANDLERTYPE_ALL:    fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER | MM_RAM_FLAGS_VIRTUAL_ALL; break;
@@ -1231 +1230 @@
             switch (pCur->enmType)
             {
-                case PGMVIRTHANDLERTYPE_NORMAL:     pState->fFlagsFound |= MM_RAM_FLAGS_VIRTUAL_HANDLER; break;
                 case PGMVIRTHANDLERTYPE_WRITE:      pState->fFlagsFound |= MM_RAM_FLAGS_VIRTUAL_HANDLER | MM_RAM_FLAGS_VIRTUAL_WRITE; break;
                 case PGMVIRTHANDLERTYPE_ALL:        pState->fFlagsFound |= MM_RAM_FLAGS_VIRTUAL_HANDLER | MM_RAM_FLAGS_VIRTUAL_ALL; break;
@@ -1274 +1272 @@
     switch (pVirt->enmType)
     {
-        case PGMVIRTHANDLERTYPE_NORMAL:     fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER; break;
         case PGMVIRTHANDLERTYPE_WRITE:      fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER | MM_RAM_FLAGS_VIRTUAL_WRITE; break;
         case PGMVIRTHANDLERTYPE_ALL:        fFlags = MM_RAM_FLAGS_VIRTUAL_HANDLER | MM_RAM_FLAGS_VIRTUAL_ALL; break;