VirtualBox

Changeset 7642 in vbox


Ignore:
Timestamp:
Mar 31, 2008 10:06:59 AM (17 years ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
29149
Message:

Additional cleanup

Location:
trunk/src/VBox/VMM
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/VMM/PGMInternal.h

    r7635 r7642  
    816816 * @param   pPage       Pointer to the physical guest page tracking structure.
    817817 */
    818 #define PGM_PAGE_HAVE_ANY_PHYSICAL_HANDLERS(pPage)      ( (pPage)->u2HandlerPhysStateX != PGM_PAGE_HNDL_PHYS_STATE_NONE )
     818#define PGM_PAGE_HAS_ANY_PHYSICAL_HANDLERS(pPage)      ( (pPage)->u2HandlerPhysStateX != PGM_PAGE_HNDL_PHYS_STATE_NONE )
    819819
    820820/**
    … …  
    823823 * @param   pPage       Pointer to the physical guest page tracking structure.
    824824 */
    825 #define PGM_PAGE_HAVE_ACTIVE_PHYSICAL_HANDLERS(pPage)   ( (pPage)->u2HandlerPhysStateX >= PGM_PAGE_HNDL_PHYS_STATE_WRITE )
     825#define PGM_PAGE_HAS_ACTIVE_PHYSICAL_HANDLERS(pPage)   ( (pPage)->u2HandlerPhysStateX >= PGM_PAGE_HNDL_PHYS_STATE_WRITE )
    826826
    827827
    … …  
    860860 * @param   pPage       Pointer to the physical guest page tracking structure.
    861861 */
    862 #define PGM_PAGE_HAVE_ANY_VIRTUAL_HANDLERS(pPage)    ( (pPage)->u2HandlerVirtStateX != PGM_PAGE_HNDL_VIRT_STATE_NONE )
    863 
    864 /**
    865  * Same as PGM_PAGE_HAVE_ANY_VIRTUAL_HANDLERS - can't disable pages in
     862#define PGM_PAGE_HAS_ANY_VIRTUAL_HANDLERS(pPage)    ( (pPage)->u2HandlerVirtStateX != PGM_PAGE_HNDL_VIRT_STATE_NONE )
     863
     864/**
     865 * Same as PGM_PAGE_HAS_ANY_VIRTUAL_HANDLERS - can't disable pages in
    866866 * virtual handlers.
    867867 * @returns true/false
    868868 * @param   pPage       Pointer to the physical guest page tracking structure.
    869869 */
    870 #define PGM_PAGE_HAVE_ACTIVE_VIRTUAL_HANDLERS(pPage) PGM_PAGE_HAVE_ANY_VIRTUAL_HANDLERS(pPage)
     870#define PGM_PAGE_HAS_ACTIVE_VIRTUAL_HANDLERS(pPage) PGM_PAGE_HAS_ANY_VIRTUAL_HANDLERS(pPage)
    871871
    872872
    … …  
    877877 * @param   pPage       Pointer to the physical guest page tracking structure.
    878878 */
    879 #define PGM_PAGE_HAVE_ANY_HANDLERS(pPage) \
     879#define PGM_PAGE_HAS_ANY_HANDLERS(pPage) \
    880880    (   (pPage)->u2HandlerPhysStateX != PGM_PAGE_HNDL_PHYS_STATE_NONE \
    881881     || (pPage)->u2HandlerVirtStateX != PGM_PAGE_HNDL_VIRT_STATE_NONE )
    … …  
    886886 * @param   pPage       Pointer to the physical guest page tracking structure.
    887887 */
    888 #define PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPage) \
     888#define PGM_PAGE_HAS_ACTIVE_HANDLERS(pPage) \
    889889    (   (pPage)->u2HandlerPhysStateX >= PGM_PAGE_HNDL_PHYS_STATE_WRITE \
    890890     || (pPage)->u2HandlerVirtStateX >= PGM_PAGE_HNDL_VIRT_STATE_WRITE )
    … …  
    895895 * @param   pPage       Pointer to the physical guest page tracking structure.
    896896 */
    897 #define PGM_PAGE_HAVE_ACTIVE_ALL_HANDLERS(pPage) \
     897#define PGM_PAGE_HAS_ACTIVE_ALL_HANDLERS(pPage) \
    898898    (   (pPage)->u2HandlerPhysStateX == PGM_PAGE_HNDL_PHYS_STATE_ALL \
    899899     || (pPage)->u2HandlerVirtStateX == PGM_PAGE_HNDL_VIRT_STATE_ALL )

  • trunk/src/VBox/VMM/VMMAll/PGMAllBth.h

    r7629 r7642  
    277277            if (VBOX_SUCCESS(rc))
    278278            {
    279                 if (PGM_PAGE_HAVE_ANY_HANDLERS(pPage))
    280                 {
    281                     if (PGM_PAGE_HAVE_ANY_PHYSICAL_HANDLERS(pPage))
     279                if (PGM_PAGE_HAS_ANY_HANDLERS(pPage))
     280                {
     281                    if (PGM_PAGE_HAS_ANY_PHYSICAL_HANDLERS(pPage))
    282282                    {
    283283                        /*
    … …  
    436436                    STAM_COUNTER_INC(&pVM->pgm.s.StatHandlersUnhandled);
    437437
    438                     if (    !PGM_PAGE_HAVE_ACTIVE_ALL_HANDLERS(pPage)
     438                    if (    !PGM_PAGE_HAS_ACTIVE_ALL_HANDLERS(pPage)
    439439                        &&  !(uErr & X86_TRAP_PF_P))
    440440                    {
    … …  
    458458                    LogFlow(("PGM: PGMInterpretInstruction -> rc=%d HCPhys=%RHp%s%s\n",
    459459                             rc, pPage->HCPhys,
    460                              PGM_PAGE_HAVE_ANY_PHYSICAL_HANDLERS(pPage) ? " phys" : "",
    461                              PGM_PAGE_HAVE_ANY_VIRTUAL_HANDLERS(pPage)  ? " virt" : ""));
     460                             PGM_PAGE_HAS_ANY_PHYSICAL_HANDLERS(pPage) ? " phys" : "",
     461                             PGM_PAGE_HAS_ANY_VIRTUAL_HANDLERS(pPage)  ? " virt" : ""));
    462462                    STAM_PROFILE_STOP(&pVM->pgm.s.StatHandlers, b);
    463463                    STAM_STATS({ pVM->pgm.s.CTXSUFF(pStatTrap0eAttribution) = &pVM->pgm.s.StatTrap0eHndUnhandled; });
    … …  
    786786#  else /* PAE */
    787787    unsigned        iPDSrc;
    788     PX86PDPAE       pPDSrc = pgmGstGetPaePDPtr(&pVM->pgm.s, GCPtrPage, &iPDSrc);
    789 
    790     GSTPDE          PdeSrc;;
    791     PdeSrc.u      = pgmGstGetPaePDE(&pVM->pgm.s, GCPtrPage);
     788    PX86PDPAE       pPDSrc      = pgmGstGetPaePDPtr(&pVM->pgm.s, GCPtrPage, &iPDSrc);
     789    GSTPDE          PdeSrc      = pPDSrc->a[iPDSrc];
    792790#  endif
    793791
    … …  
    864862             */
    865863            PPGMPOOLPAGE    pShwPage = pgmPoolGetPageByHCPhys(pVM, PdeDst.u & SHW_PDE_PG_MASK);
    866             RTGCPHYS        GCPhys = PdeSrc.u & GST_PDE_PG_MASK;
     864            RTGCPHYS        GCPhys   = PdeSrc.u & GST_PDE_PG_MASK;
    867865#  if PGM_SHW_TYPE != PGM_TYPE_32BIT
    868866            GCPhys |= (iPDDst & 1) * (PAGE_SIZE/2);
    … …  
    909907            /* Before freeing the page, check if anything really changed. */
    910908            PPGMPOOLPAGE    pShwPage = pgmPoolGetPageByHCPhys(pVM, PdeDst.u & SHW_PDE_PG_MASK);
    911             RTGCPHYS        GCPhys = PdeSrc.u & GST_PDE4M_PG_MASK;
     909            RTGCPHYS        GCPhys   = PdeSrc.u & GST_PDE4M_PG_MASK;
    912910#  if PGM_SHW_TYPE != PGM_TYPE_32BIT
    913911            GCPhys |= GCPtrPage & (1 << X86_PD_PAE_SHIFT);
    … …  
    11151113            const RTHCPHYS HCPhys = pPage->HCPhys; /** @todo FLAGS */
    11161114            SHWPTE PteDst;
    1117             if (PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPage))
     1115            if (PGM_PAGE_HAS_ACTIVE_HANDLERS(pPage))
    11181116            {
    11191117                /** @todo r=bird: Are we actually handling dirty and access bits for pages with access handlers correctly? No. */
    1120                 if (!PGM_PAGE_HAVE_ACTIVE_ALL_HANDLERS(pPage))
     1118                if (!PGM_PAGE_HAS_ACTIVE_ALL_HANDLERS(pPage))
    11211119                    PteDst.u = (PteSrc.u & ~(X86_PTE_PAE_PG_MASK | X86_PTE_AVL_MASK | X86_PTE_PAT | X86_PTE_PCD | X86_PTE_PWT | X86_PTE_RW))
    11221120                             | (HCPhys & X86_PTE_PAE_PG_MASK);
    … …  
    13501348                                    ||  !CSAMDoesPageNeedScanning(pVM, (RTGCPTR)GCPtrCurPage)
    13511349                                    ||  (   (pPage = pgmPhysGetPage(&pVM->pgm.s, PteSrc.u & GST_PTE_PG_MASK))
    1352                                          && PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPage))
     1350                                         && PGM_PAGE_HAS_ACTIVE_HANDLERS(pPage))
    13531351                                   )
    13541352#endif /* else: CSAM not active */
    … …  
    14051403                    PteDst.u = (PdeSrc.u & ~(X86_PTE_PAE_PG_MASK | X86_PTE_AVL_MASK | X86_PTE_PAT | X86_PTE_PCD | X86_PTE_PWT))
    14061404                             | (HCPhys & X86_PTE_PAE_PG_MASK);
    1407                     if (PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPage))
    1408                     {
    1409                         if (!PGM_PAGE_HAVE_ACTIVE_ALL_HANDLERS(pPage))
     1405                    if (PGM_PAGE_HAS_ACTIVE_HANDLERS(pPage))
     1406                    {
     1407                        if (!PGM_PAGE_HAS_ACTIVE_ALL_HANDLERS(pPage))
    14101408                            PteDst.n.u1Write = 0;
    14111409                        else
    … …  
    17671765                        PPGMPAGE pPage = pgmPhysGetPage(&pVM->pgm.s, pPteSrc->u & GST_PTE_PG_MASK);
    17681766                        if (pPage)
    1769                             AssertMsg(!PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPage),
     1767                            AssertMsg(!PGM_PAGE_HAS_ACTIVE_HANDLERS(pPage),
    17701768                                      ("Unexpected dirty bit tracking on monitored page %VGv (phys %VGp)!!!!!!\n", GCPtrPage, pPteSrc->u & X86_PTE_PAE_PG_MASK));
    17711769#  endif
    … …  
    20252023                            ||  !CSAMDoesPageNeedScanning(pVM, (RTGCPTR)((iPDSrc << GST_PD_SHIFT) | (iPTSrc << PAGE_SHIFT)))
    20262024                            ||  (   (pPage = pgmPhysGetPage(&pVM->pgm.s, PteSrc.u & GST_PTE_PG_MASK))
    2027                                  &&  PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPage))
     2025                                 &&  PGM_PAGE_HAS_ACTIVE_HANDLERS(pPage))
    20282026                           )
    20292027#endif
    … …  
    21242122                        }
    21252123
    2126                         if (PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPage))
     2124                        if (PGM_PAGE_HAS_ACTIVE_HANDLERS(pPage))
    21272125                        {
    2128                             if (!PGM_PAGE_HAVE_ACTIVE_ALL_HANDLERS(pPage))
     2126                            if (!PGM_PAGE_HAS_ACTIVE_ALL_HANDLERS(pPage))
    21292127                            {
    21302128                                PteDst.u = PGM_PAGE_GET_HCPHYS(pPage) | PteDstBase.u;
    … …  
    31543152
    31553153                    /* flags */
    3156                     if (PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPhysPage))
    3157                     {
    3158                         if (!PGM_PAGE_HAVE_ACTIVE_ALL_HANDLERS(pPhysPage))
     3154                    if (PGM_PAGE_HAS_ACTIVE_HANDLERS(pPhysPage))
     3155                    {
     3156                        if (!PGM_PAGE_HAS_ACTIVE_ALL_HANDLERS(pPhysPage))
    31593157                        {
    31603158                            if (PteDst.n.u1Write)
    … …  
    33833381
    33843382                    /* flags */
    3385                     if (PGM_PAGE_HAVE_ACTIVE_HANDLERS(pPhysPage))
    3386                     {
    3387                         if (!PGM_PAGE_HAVE_ACTIVE_ALL_HANDLERS(pPhysPage))
     3383                    if (PGM_PAGE_HAS_ACTIVE_HANDLERS(pPhysPage))
     3384                    {
     3385                        if (!PGM_PAGE_HAS_ACTIVE_ALL_HANDLERS(pPhysPage))
    33883386                        {
    33893387                            if (PGM_PAGE_GET_HNDL_PHYS_STATE(pPhysPage) != PGM_PAGE_HNDL_PHYS_STATE_DISABLED)

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

    r6927 r7642  
    13701370        {
    13711371            PGMPAGE const *pPage = &pRam->aPages[iPage];
    1372             if (PGM_PAGE_HAVE_ANY_HANDLERS(pPage))
     1372            if (PGM_PAGE_HAS_ANY_HANDLERS(pPage))
    13731373            {
    13741374                State.GCPhys = pRam->GCPhys + (iPage << PAGE_SHIFT);
    … …  
    13781378                 *                  active on the page, then compare.
    13791379                 */
    1380                 if (PGM_PAGE_HAVE_ANY_PHYSICAL_HANDLERS(pPage))
     1380                if (PGM_PAGE_HAS_ANY_PHYSICAL_HANDLERS(pPage))
    13811381                {
    13821382                    /* the first */
    … …  
    14381438                 * Virtual handlers.
    14391439                 */
    1440                 if (PGM_PAGE_HAVE_ACTIVE_VIRTUAL_HANDLERS(pPage))
     1440                if (PGM_PAGE_HAS_ACTIVE_VIRTUAL_HANDLERS(pPage))
    14411441                {
    14421442                    State.uVirtState = PGM_PAGE_GET_HNDL_VIRT_STATE(pPage);

  • trunk/src/VBox/VMM/VMMAll/PGMAllPhys.cpp

    r7635 r7642  
    14331433                 * Virtual handlers
    14341434                 */
    1435                 if (    PGM_PAGE_HAVE_ACTIVE_VIRTUAL_HANDLERS(pPage)
     1435                if (    PGM_PAGE_HAS_ACTIVE_VIRTUAL_HANDLERS(pPage)
    14361436                    && !(pPage->HCPhys & MM_RAM_FLAGS_MMIO)) /// @todo PAGE FLAGS
    14371437                {
    1438                     if (PGM_PAGE_HAVE_ACTIVE_PHYSICAL_HANDLERS(pPage))
     1438                    if (PGM_PAGE_HAS_ACTIVE_PHYSICAL_HANDLERS(pPage))
    14391439                    {
    14401440                        /*
Note: See TracChangeset for help on using the changeset viewer.

© 2025 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette