Changeset 8234

Timestamp:

04/21/08 16:29:54 (9 months ago)

Author:

vboxsync

Message:

MMIO: Cleanup up (removed duplicate code)
Disassembler: simplified general purpose register access

Files:

• trunk/include/VBox/dis.h (modified) (1 diff)
• trunk/src/VBox/Disassembler/DisasmCore.cpp (modified) (13 diffs)
• trunk/src/VBox/Disassembler/DisasmReg.cpp (modified) (14 diffs)
• trunk/src/VBox/VMM/IOMInternal.h (modified) (1 diff)
• trunk/src/VBox/VMM/PATM/PATMPatch.cpp (modified) (6 diffs)
• trunk/src/VBox/VMM/VMMAll/EMAll.cpp (modified) (10 diffs)
• trunk/src/VBox/VMM/VMMAll/IOMAll.cpp (modified) (17 diffs)
• trunk/src/VBox/VMM/VMMAll/IOMAllMMIO.cpp (modified) (3 diffs)
• trunk/src/VBox/VMM/VMMAll/PGMAllPool.cpp (modified) (1 diff)

trunk/include/VBox/dis.h

@@ -342 +342 @@
-    union
-    {
-
-
-
-
+
-        /** ST(0) - ST(7) */
-        uint32_t    reg_fp;

trunk/src/VBox/Disassembler/DisasmCore.cpp

@@ -532 +532 @@
-
-        pParam->flags |= USE_BASE | USE_REG_GEN32;
-32
+
-    }
-    return;   /* Already fetched everything in ParseSIB; no size returned */
@@ -1240 +1240 @@
-        {
-            /* Use 32-bit registers. */
-32
+
-            pParam->flags |= USE_REG_GEN32;
-            pParam->size   = 4;
@@ -1247 +1247 @@
-        {
-            /* Use 16-bit registers. */
-16
+
-            pParam->flags |= USE_REG_GEN16;
-            pParam->size   = 2;
@@ -1265 +1265 @@
-    {
-        /* 16-bit AX..DI registers. */
-16
+
-        pParam->flags |= USE_REG_GEN16;
-        pParam->size   = 2;
@@ -1273 +1273 @@
-    {
-        /* 8-bit AL..DL, AH..DH registers. */
-8
+
-        pParam->flags |= USE_REG_GEN8;
-        pParam->size   = 1;
@@ -1299 +1299 @@
-    if (pCpu->addrmode == CPUMODE_32BIT)
-    {
-32
+
-        pParam->flags |= USE_REG_GEN32;
-    }
-    else
-    {
-16
+
-        pParam->flags |= USE_REG_GEN16;
-    }
@@ -1318 +1318 @@
-    if (pCpu->addrmode == CPUMODE_32BIT)
-    {
-32
+
-        pParam->flags |= USE_REG_GEN32;
-    }
-    else
-    {
-16
+
-        pParam->flags |= USE_REG_GEN16;
-    }
@@ -1338 +1338 @@
-    if (pCpu->addrmode == CPUMODE_32BIT)
-    {
-32
+
-        pParam->flags |= USE_REG_GEN32;
-    }
-    else
-    {
-16
+
-        pParam->flags |= USE_REG_GEN16;
-    }
@@ -1357 +1357 @@
-    if (pCpu->addrmode == CPUMODE_32BIT)
-    {
-32
+
-        pParam->flags |= USE_REG_GEN32;
-    }
-    else
-    {
-16
+
-        pParam->flags |= USE_REG_GEN16;
-    }
@@ -1826 +1826 @@
-        disasmAddString(pParam->szParam, szModRMReg8[idx]);
-        pParam->flags |= USE_REG_GEN8;
-8
+
-        break;
-
@@ -1832 +1832 @@
-        disasmAddString(pParam->szParam, szModRMReg16[idx]);
-        pParam->flags |= USE_REG_GEN16;
-16
+
-        break;
-
@@ -1838 +1838 @@
-        disasmAddString(pParam->szParam, szModRMReg32[idx]);
-        pParam->flags |= USE_REG_GEN32;
-32
+
-        break;
-
@@ -1860 +1860 @@
-    disasmAddString(pParam->szParam, szModRMReg1616[idx]);
-    pParam->flags |= USE_REG_GEN16;
-16
+
-    if (idx < 4)
-    {

trunk/src/VBox/Disassembler/DisasmReg.cpp

@@ -207 +207 @@
-        return 8;
-
-
+ /* far pointer */
-        if (pCpu->addrmode == CPUMODE_32BIT)
-
-
-
+
+
+
+
+
+
-
-    default:
@@ -232 +235 @@
-    {
-        /* Guess segment register by parameter type. */
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
-                return USE_REG_SS;
-        }
@@ -486 +487 @@
-            {
-                pParamVal->flags |= PARAM_VAL8;
-8
+
-            }
-            else
@@ -492 +493 @@
-            {
-                pParamVal->flags |= PARAM_VAL16;
-16
+
-            }
-            else
@@ -498 +499 @@
-            {
-                pParamVal->flags |= PARAM_VAL32;
-32
+
-            }
-            else
@@ -504 +505 @@
-            {
-                pParamVal->flags |= PARAM_VAL64;
-64
+
-            }
-            else {
@@ -581 +582 @@
-            pParamVal->flags |= PARAM_VAL8;
-            pParamVal->size   = sizeof(uint8_t);
-8
+
-        }
-        else
@@ -588 +589 @@
-            pParamVal->flags |= PARAM_VAL16;
-            pParamVal->size   = sizeof(uint16_t);
-16
+
-        }
-        else
@@ -595 +596 @@
-            pParamVal->flags |= PARAM_VAL32;
-            pParamVal->size   = sizeof(uint32_t);
-32
+
-        }
-        else
@@ -602 +603 @@
-            pParamVal->flags |= PARAM_VAL64;
-            pParamVal->size   = sizeof(uint64_t);
-64
+
-        }
-        else
@@ -698 +699 @@
-        {
-            uint8_t *pu8Reg;
-8
+
-            {
-                *pcbSize = sizeof(uint8_t);
@@ -709 +710 @@
-        {
-            uint16_t *pu16Reg;
-16
+
-            {
-                *pcbSize = sizeof(uint16_t);
@@ -720 +721 @@
-        {
-            uint32_t *pu32Reg;
-32
+
-            {
-                *pcbSize = sizeof(uint32_t);
@@ -731 +732 @@
-        {
-            uint64_t *pu64Reg;
-64
+
-            {
-                *pcbSize = sizeof(uint64_t);

trunk/src/VBox/VMM/IOMInternal.h

@@ -514 +514 @@
-#endif
-
+/* Disassembly helpers used in IOMAll.cpp & IOMAllMMIO.cpp */
+bool iomGetRegImmData(PDISCPUSTATE pCpu, PCOP_PARAMETER pParam, PCPUMCTXCORE pRegFrame, uint32_t *pu32Data, unsigned *pcbSize);
+bool iomSaveDataToReg(PDISCPUSTATE pCpu, PCOP_PARAMETER pParam, PCPUMCTXCORE pRegFrame, unsigned u32Data);
+
-__END_DECLS
-
+
-#ifdef IN_RING3
-

trunk/src/VBox/VMM/PATM/PATMPatch.cpp

@@ -1166 +1166 @@
-
-        dbgreg = pCpu->param1.base.reg_dbg;
-32
+
-    }
-    else
@@ -1175 +1175 @@
-
-        pPB[0] = 0x8B;      // mov GPR, disp32
-32
+
-        dbgreg = pCpu->param2.base.reg_dbg;
-    }
@@ -1242 +1242 @@
-        pPB[0] = 0x89;      //mov disp32, GPR
-        ctrlreg = pCpu->param1.base.reg_ctrl;
-32
+
-        Assert(pCpu->param1.flags & USE_REG_CR);
-        Assert(pCpu->param2.flags & USE_REG_GEN32);
@@ -1253 +1253 @@
-
-        pPB[0]  = 0x8B;      // mov GPR, disp32
-32
+
-        ctrlreg = pCpu->param2.base.reg_ctrl;
-    }
@@ -1319 +1319 @@
-    if (pCpu->prefix & PREFIX_OPSIZE)
-        pPB[offset++] = 0x66; /* size override -> 16 bits pop */
-32
+
-    PATCHGEN_EPILOG(pPatch, offset);
-
@@ -1362 +1362 @@
-        pPB[offset++] = 0x8B;              // mov       destreg, CPUMCTX.tr/ldtr
-        /* Modify REG part according to destination of original instruction */
-32
+
-        if (pCpu->pCurInstr->opcode == OP_STR)
-        {

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

@@ -354 +354 @@
-
-#if defined(VBOX_STRICT) || defined(LOG_ENABLED)
-# ifdef IN_GC
-/** 
- * Get the mnemonic for the disassembled instruction.
@@ -375 +374 @@
-    }
-}
-# endif
-#endif
-
@@ -462 +460 @@
-                {
-                case 1: //special case for AH etc
-8
-32
-32
+
+
+
-                default: AssertFailedReturn(VERR_EM_INTERPRETER);
-                }
@@ -487 +485 @@
-                {
-                case 1: //special case for AH etc
-8
-32
-32
+
+
+
-                default: AssertFailedReturn(VERR_EM_INTERPRETER);
-                }
@@ -626 +624 @@
-                if (    (pCpu->param1.flags & USE_BASE)
-                    &&  (pCpu->param1.flags & (USE_REG_GEN16|USE_REG_GEN32))
-32
+
-                   )
-                   pParam1 = (RTGCPTR)((RTGCUINTPTR)pParam1 + param1.size);
@@ -1221 +1219 @@
-                switch(param1.size)
-                {
-8
-16
-32
+
+
+
-                default:
-                    return VERR_EM_INTERPRETER;
@@ -1252 +1250 @@
-
-#ifdef LOG_ENABLED
-onst c
+
-
-    if (pCpu->prefix & PREFIX_LOCK)
@@ -1340 +1338 @@
-
-#ifdef LOG_ENABLED
-onst c
+
-
-    if (pCpu->prefix & PREFIX_LOCK)
@@ -1814 +1812 @@
-{
-    if (pCpu->param1.flags == USE_REG_GEN32 && pCpu->param2.flags == USE_REG_CR)
-32
+
-    if (pCpu->param1.flags == USE_REG_CR && pCpu->param2.flags == USE_REG_GEN32)
-32
+
-    AssertMsgFailedReturn(("Unexpected control register move\n"), VERR_EM_INTERPRETER);
-    return VERR_EM_INTERPRETER;
@@ -1878 +1876 @@
-    if(pCpu->param1.flags == USE_REG_GEN32 && pCpu->param2.flags == USE_REG_DBG)
-    {
-32
+
-    }
-    else
-    if(pCpu->param1.flags == USE_REG_DBG && pCpu->param2.flags == USE_REG_GEN32)
-    {
-32
+
-    }
-    else

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

@@ -46 +46 @@
- * @param   pParam              Pointer to parameter of instruction to proccess.
- */
-CG
+
-{
-    if (pParam->flags & (USE_BASE | USE_INDEX | USE_SCALE | USE_DISPLACEMENT8 | USE_DISPLACEMENT16 | USE_DISPLACEMENT32 | USE_IMMEDIATE8 | USE_IMMEDIATE16 | USE_IMMEDIATE32 | USE_IMMEDIATE16_SX8 | USE_IMMEDIATE32_SX8))
@@ -59 +59 @@
-    if (pParam->flags & USE_REG_GEN8)
-        return 1;
+
+    if (pParam->flags & USE_REG_GEN64)
+        return 8;
-
-    if (pParam->flags & USE_REG_SEG)
@@ -77 +80 @@
- * @param   pcbSize             Where to store the size of data (1, 2, 4).
- */
-static bool iomGC
+bool iom
-{
-    if (pParam->flags & (USE_BASE | USE_INDEX | USE_SCALE | USE_DISPLACEMENT8 | USE_DISPLACEMENT16 | USE_DISPLACEMENT32))
@@ -89 +92 @@
-    {
-        *pcbSize  = 4;
-32
+
-        return true;
-    }
@@ -96 +99 @@
-    {
-        *pcbSize  = 2;
-16
+
-        return true;
-    }
@@ -103 +106 @@
-    {
-        *pcbSize  = 1;
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-        return true;
-    }
@@ -110 +129 @@
-    {
-        *pcbSize  = 4;
-
+//
-        return true;
-    }
@@ -135 +154 @@
-    } /* Else - error. */
-
+    AssertFailed();
-    *pcbSize  = 0;
-    *pu32Data = 0;
@@ -151 +171 @@
- * @param   u32Data             8/16/32 bit data to store.
- */
-static bool iomGC
+bool iom
-{
-    if (pParam->flags & (USE_BASE | USE_INDEX | USE_SCALE | USE_DISPLACEMENT8 | USE_DISPLACEMENT16 | USE_DISPLACEMENT32 | USE_IMMEDIATE8 | USE_IMMEDIATE16 | USE_IMMEDIATE32 | USE_IMMEDIATE32_SX8 | USE_IMMEDIATE16_SX8))
@@ -160 +180 @@
-    if (pParam->flags & USE_REG_GEN32)
-    {
-32
+
-        return true;
-    }
@@ -166 +186 @@
-    if (pParam->flags & USE_REG_GEN16)
-    {
-16
+
-        return true;
-    }
@@ -172 +192 @@
-    if (pParam->flags & USE_REG_GEN8)
-    {
-8
+
-        return true;
-    }
@@ -189 +209 @@
- * Internal - statistics only.
- */
-GC
+
-{
-#ifdef VBOX_WITH_STATISTICS
@@ -857 +877 @@
-    uint32_t    uPort = 0;
-    unsigned    cbSize = 0;
-CG
+
-    AssertMsg(fRc, ("Failed to get reg/imm port number!\n")); NOREF(fRc);
-
-CG
+
-    Assert(cbSize > 0);
-    int rc = IOMInterpretCheckPortIOAccess(pVM, pRegFrame, uPort, cbSize);
@@ -875 +895 @@
-             * Store the result in the AL|AX|EAX register.
-             */
-GC
+
-            AssertMsg(fRc, ("Failed to store register value!\n")); NOREF(fRc);
-        }
@@ -916 +936 @@
-    uint32_t    uPort = 0;
-    unsigned    cbSize = 0;
-CG
+
-    AssertMsg(fRc, ("Failed to get reg/imm port number!\n")); NOREF(fRc);
-
@@ -923 +943 @@
-    {
-        uint32_t    u32Data = 0;
-CG
+
-        AssertMsg(fRc, ("Failed to get reg value!\n")); NOREF(fRc);
-

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

@@ -45 +45 @@
-#include <iprt/string.h>
-
-/*******************************************************************************
-*   Internal Functions                                                         *
-*******************************************************************************/
-static bool iomGetRegImmData(PDISCPUSTATE pCpu, PCOP_PARAMETER pParam, PCPUMCTXCORE pRegFrame, uint32_t *pu32Data, unsigned *pcbSize);
-static bool iomSaveDataToReg(PDISCPUSTATE pCpu, PCOP_PARAMETER pParam, PCPUMCTXCORE pRegFrame, uint32_t u32Data);
-
-
@@ -55 +50 @@
-*   Global Variables                                                           *
-*******************************************************************************/
-/**
- * Array for accessing 32-bit general registers in VMMREGFRAME structure
- * by register's index from disasm.
- */
-static unsigned g_aReg32Index[] =
-{
-    RT_OFFSETOF(CPUMCTXCORE, eax),        /* USE_REG_EAX */
-    RT_OFFSETOF(CPUMCTXCORE, ecx),        /* USE_REG_ECX */
-    RT_OFFSETOF(CPUMCTXCORE, edx),        /* USE_REG_EDX */
-    RT_OFFSETOF(CPUMCTXCORE, ebx),        /* USE_REG_EBX */
-    RT_OFFSETOF(CPUMCTXCORE, esp),        /* USE_REG_ESP */
-    RT_OFFSETOF(CPUMCTXCORE, ebp),        /* USE_REG_EBP */
-    RT_OFFSETOF(CPUMCTXCORE, esi),        /* USE_REG_ESI */
-    RT_OFFSETOF(CPUMCTXCORE, edi)         /* USE_REG_EDI */
-};
-
-/**
- * Macro for accessing 32-bit general purpose registers in CPUMCTXCORE structure.
- */
-#define ACCESS_REG32(p, idx) (*((uint32_t *)((char *)(p) + g_aReg32Index[idx])))
-
-/**
- * Array for accessing 16-bit general registers in CPUMCTXCORE structure
- * by register's index from disasm.
- */
-static unsigned g_aReg16Index[] =
-{
-    RT_OFFSETOF(CPUMCTXCORE, eax),        /* USE_REG_AX */
-    RT_OFFSETOF(CPUMCTXCORE, ecx),        /* USE_REG_CX */
-    RT_OFFSETOF(CPUMCTXCORE, edx),        /* USE_REG_DX */
-    RT_OFFSETOF(CPUMCTXCORE, ebx),        /* USE_REG_BX */
-    RT_OFFSETOF(CPUMCTXCORE, esp),        /* USE_REG_SP */
-    RT_OFFSETOF(CPUMCTXCORE, ebp),        /* USE_REG_BP */
-    RT_OFFSETOF(CPUMCTXCORE, esi),        /* USE_REG_SI */
-    RT_OFFSETOF(CPUMCTXCORE, edi)         /* USE_REG_DI */
-};
-
-/**
- * Macro for accessing 16-bit general purpose registers in CPUMCTXCORE structure.
- */
-#define ACCESS_REG16(p, idx) (*((uint16_t *)((char *)(p) + g_aReg16Index[idx])))
-
-/**
- * Array for accessing 8-bit general registers in CPUMCTXCORE structure
- * by register's index from disasm.
- */
-static unsigned g_aReg8Index[] =
-{
-    RT_OFFSETOF(CPUMCTXCORE, eax),        /* USE_REG_AL */
-    RT_OFFSETOF(CPUMCTXCORE, ecx),        /* USE_REG_CL */
-    RT_OFFSETOF(CPUMCTXCORE, edx),        /* USE_REG_DL */
-    RT_OFFSETOF(CPUMCTXCORE, ebx),        /* USE_REG_BL */
-    RT_OFFSETOF(CPUMCTXCORE, eax) + 1,    /* USE_REG_AH */
-    RT_OFFSETOF(CPUMCTXCORE, ecx) + 1,    /* USE_REG_CH */
-    RT_OFFSETOF(CPUMCTXCORE, edx) + 1,    /* USE_REG_DH */
-    RT_OFFSETOF(CPUMCTXCORE, ebx) + 1     /* USE_REG_BH */
-};
-
-/**
- * Macro for accessing 8-bit general purpose registers in CPUMCTXCORE structure.
- */
-#define ACCESS_REG8(p, idx) (*((uint8_t *)((char *)(p) + g_aReg8Index[idx])))
-
-/**
- * Array for accessing segment registers in CPUMCTXCORE structure
- * by register's index from disasm.
- */
-static unsigned g_aRegSegIndex[] =
-{
-    RT_OFFSETOF(CPUMCTXCORE, es),         /* USE_REG_ES */
-    RT_OFFSETOF(CPUMCTXCORE, cs),         /* USE_REG_CS */
-    RT_OFFSETOF(CPUMCTXCORE, ss),         /* USE_REG_SS */
-    RT_OFFSETOF(CPUMCTXCORE, ds),         /* USE_REG_DS */
-    RT_OFFSETOF(CPUMCTXCORE, fs),         /* USE_REG_FS */
-    RT_OFFSETOF(CPUMCTXCORE, gs)          /* USE_REG_GS */
-};
-
-/**
- * Macro for accessing segment registers in CPUMCTXCORE structure.
- */
-#define ACCESS_REGSEG(p, idx) (*((uint16_t *)((char *)(p) + g_aRegSegIndex[idx])))
-
-/**
@@ -211 +125 @@
-    return rc;
-}
-
-
-/**
- * Returns the contents of register or immediate data of instruction's parameter.
- *
- * @returns true on success.
- *
- * @param   pCpu                Pointer to current disassembler context.
- * @param   pParam              Pointer to parameter of instruction to proccess.
- * @param   pRegFrame           Pointer to CPUMCTXCORE guest structure.
- * @param   pu32Data            Where to store retrieved data.
- * @param   pcbSize             Where to store the size of data (1, 2, 4).
- */
-static bool iomGetRegImmData(PDISCPUSTATE pCpu, PCOP_PARAMETER pParam, PCPUMCTXCORE pRegFrame, uint32_t *pu32Data, unsigned *pcbSize)
-{
-    if (pParam->flags & (USE_BASE | USE_INDEX | USE_SCALE | USE_DISPLACEMENT8 | USE_DISPLACEMENT16 | USE_DISPLACEMENT32))
-    {
-        *pcbSize  = 0;
-        *pu32Data = 0;
-        return false;
-    }
-
-    if (pParam->flags & USE_REG_GEN32)
-    {
-        *pcbSize  = 4;
-        *pu32Data = ACCESS_REG32(pRegFrame, pParam->base.reg_gen32);
-        return true;
-    }
-
-    if (pParam->flags & USE_REG_GEN16)
-    {
-        *pcbSize  = 2;
-        *pu32Data = ACCESS_REG16(pRegFrame, pParam->base.reg_gen16);
-        return true;
-    }
-
-    if (pParam->flags & USE_REG_GEN8)
-    {
-        *pcbSize  = 1;
-        *pu32Data = ACCESS_REG8(pRegFrame, pParam->base.reg_gen8);
-        return true;
-    }
-
-    if (pParam->flags & (USE_IMMEDIATE32|USE_IMMEDIATE32_SX8))
-    {
-        *pcbSize  = 4;
-        *pu32Data = (uint32_t)pParam->parval;
-        return true;
-    }
-
-    if (pParam->flags & (USE_IMMEDIATE16|USE_IMMEDIATE16_SX8))
-    {
-        *pcbSize  = 2;
-        *pu32Data = (uint16_t)pParam->parval;
-        return true;
-    }
-
-    if (pParam->flags & USE_IMMEDIATE8)
-    {
-        *pcbSize  = 1;
-        *pu32Data = (uint8_t)pParam->parval;
-        return true;
-    }
-
-    if (pParam->flags & USE_REG_SEG)
-    {
-        *pcbSize  = 2;
-        *pu32Data = ACCESS_REGSEG(pRegFrame, pParam->base.reg_seg);
-        return true;
-    } /* Else - error. */
-
-    *pcbSize  = 0;
-    *pu32Data = 0;
-    return false;
-}
-
-
-/**
- * Saves data to 8/16/32 general purpose or segment register defined by
- * instruction's parameter.
- *
- * @returns true on success.
- * @param   pCpu                Pointer to current disassembler context.
- * @param   pParam              Pointer to parameter of instruction to proccess.
- * @param   pRegFrame           Pointer to CPUMCTXCORE guest structure.
- * @param   u32Data             8/16/32 bit data to store.
- */
-static bool iomSaveDataToReg(PDISCPUSTATE pCpu, PCOP_PARAMETER pParam, PCPUMCTXCORE pRegFrame, unsigned u32Data)
-{
-    if (pParam->flags & (USE_BASE | USE_INDEX | USE_SCALE | USE_DISPLACEMENT8 | USE_DISPLACEMENT16 | USE_DISPLACEMENT32 | USE_IMMEDIATE8 | USE_IMMEDIATE16 | USE_IMMEDIATE32 | USE_IMMEDIATE32_SX8 | USE_IMMEDIATE16_SX8))
-    {
-        return false;
-    }
-
-    if (pParam->flags & USE_REG_GEN32)
-    {
-        ACCESS_REG32(pRegFrame, pParam->base.reg_gen32) = u32Data;
-        return true;
-    }
-
-    if (pParam->flags & USE_REG_GEN16)
-    {
-        ACCESS_REG16(pRegFrame, pParam->base.reg_gen16) = (uint16_t)u32Data;
-        return true;
-    }
-
-    if (pParam->flags & USE_REG_GEN8)
-    {
-        ACCESS_REG8(pRegFrame, pParam->base.reg_gen8) = (uint8_t)u32Data;
-        return true;
-    }
-
-    if (pParam->flags & USE_REG_SEG)
-    {
-        ACCESS_REGSEG(pRegFrame, pParam->base.reg_seg) = (uint16_t)u32Data;
-        return true;
-    }
-
-    /* Else - error. */
-    return false;
-}
-
-
-/*

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

@@ -563 +563 @@
-    }
-    if (    (pCpu->param1.flags & USE_REG_GEN32)
-32
+
-    {
-        Log4(("pgmPoolMonitorIsReused: ESP\n"));