VirtualBox

Changeset 74791 in vbox for trunk/src/VBox/VMM/VMMR3


Ignore:
Timestamp:
Oct 12, 2018 10:44:17 AM (7 years ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
125735
Message:

vm.h,VMM,REM: s/VM_FF_IS_PENDING/VM_FF_IS_ANY_SET/g to emphasize the plurality of the flags argument and encourage using VM_FF_IS_SET. bugref:9180

Location:
trunk/src/VBox/VMM/VMMR3
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/VMM/VMMR3/DBGF.cpp

    r74790 r74791  
    834834        {
    835835            int rc;
    836             if (    !VM_FF_IS_PENDING(pVM, VM_FF_EMT_RENDEZVOUS | VM_FF_REQUEST)
     836            if (    !VM_FF_IS_ANY_SET(pVM, VM_FF_EMT_RENDEZVOUS | VM_FF_REQUEST)
    837837                &&  !VMCPU_FF_IS_SET(pVCpu, VMCPU_FF_REQUEST))
    838838            {

  • trunk/src/VBox/VMM/VMMR3/EM.cpp

    r74790 r74791  
    11871187             * important FFs while we were busy switching the state. So, check again.
    11881188             */
    1189             if (    VM_FF_IS_PENDING(pVM, VM_FF_REQUEST | VM_FF_PDM_QUEUES | VM_FF_DBGF | VM_FF_CHECK_VM_STATE | VM_FF_RESET)
     1189            if (    VM_FF_IS_ANY_SET(pVM, VM_FF_REQUEST | VM_FF_PDM_QUEUES | VM_FF_DBGF | VM_FF_CHECK_VM_STATE | VM_FF_RESET)
    11901190                ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_TIMER | VMCPU_FF_REQUEST))
    11911191            {
    … …  
    12221222         * else.  Sync back the state and leave the lock to be on the safe side.
    12231223         */
    1224         if (    VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
     1224        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
    12251225            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_POST_MASK))
    12261226        {
    … …  
    12741274#endif
    12751275        AssertCompile(VMCPU_FF_ALL_REM_MASK & VMCPU_FF_TIMER);
    1276         if (    VM_FF_IS_PENDING(pVM, VM_FF_ALL_REM_MASK)
     1276        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_ALL_REM_MASK)
    12771277            ||  VMCPU_FF_IS_ANY_SET(pVCpu,
    12781278                                     VMCPU_FF_ALL_REM_MASK
    … …  
    13991399         * Check for pending actions.
    14001400         */
    1401         if (   VM_FF_IS_PENDING(pVM, VM_FF_ALL_REM_MASK)
     1401        if (   VM_FF_IS_ANY_SET(pVM, VM_FF_ALL_REM_MASK)
    14021402            || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_ALL_REM_MASK & ~VMCPU_FF_UNHALT))
    14031403            return VINF_SUCCESS;
    … …  
    18631863     * Post execution chunk first.
    18641864     */
    1865     if (    VM_FF_IS_PENDING(pVM, VM_FF_NORMAL_PRIORITY_POST_MASK)
     1865    if (    VM_FF_IS_ANY_SET(pVM, VM_FF_NORMAL_PRIORITY_POST_MASK)
    18661866        ||  (VMCPU_FF_NORMAL_PRIORITY_POST_MASK && VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_NORMAL_PRIORITY_POST_MASK)) )
    18671867    {
    … …  
    20942094     * (Executed in ascending priority order.)
    20952095     */
    2096     if (    VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_PRE_MASK)
     2096    if (    VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_PRE_MASK)
    20972097        ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_PRE_MASK))
    20982098    {
    … …  
    24162416                && rc != VINF_EM_TERMINATE
    24172417                && rc != VINF_EM_OFF
    2418                 && (   VM_FF_IS_PENDING(pVM, VM_FF_ALL_REM_MASK)
     2418                && (   VM_FF_IS_ANY_SET(pVM, VM_FF_ALL_REM_MASK)
    24192419                    || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_ALL_REM_MASK & ~VMCPU_FF_UNHALT)))
    24202420            {

  • trunk/src/VBox/VMM/VMMR3/EMHM.cpp

    r74790 r74791  
    9595         * Service necessary FFs before going into HM.
    9696         */
    97         if (   VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
     97        if (   VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
    9898            || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_PRE_RAW_MASK))
    9999        {
    … …  
    119119         */
    120120        VMCPU_FF_CLEAR(pVCpu, VMCPU_FF_RESUME_GUEST_MASK);
    121         if (   VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
     121        if (   VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
    122122            || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_POST_MASK))
    123123        {
    … …  
    404404        Assert(!VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_SELM_SYNC_TSS | VMCPU_FF_SELM_SYNC_GDT | VMCPU_FF_SELM_SYNC_LDT));
    405405#endif
    406         if (    VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
     406        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
    407407            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_PRE_RAW_MASK))
    408408        {
    … …  
    465465         */
    466466        VMCPU_FF_CLEAR(pVCpu, VMCPU_FF_RESUME_GUEST_MASK);
    467         if (    VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
     467        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
    468468            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_POST_MASK))
    469469            rc = VBOXSTRICTRC_TODO(emR3HighPriorityPostForcedActions(pVM, pVCpu, rc));
    … …  
    485485        TMTimerPollVoid(pVM, pVCpu);
    486486#endif
    487         if (    VM_FF_IS_PENDING(pVM, VM_FF_ALL_MASK)
     487        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_ALL_MASK)
    488488            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_ALL_MASK))
    489489        {

  • trunk/src/VBox/VMM/VMMR3/EMR3Nem.cpp

    r74790 r74791  
    9595         * Service necessary FFs before going into HM.
    9696         */
    97         if (   VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
     97        if (   VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
    9898            || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_PRE_RAW_MASK))
    9999        {
    … …  
    119119         */
    120120        VMCPU_FF_CLEAR(pVCpu, VMCPU_FF_RESUME_GUEST_MASK);
    121         if (   VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
     121        if (   VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
    122122            || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_POST_MASK))
    123123        {
    … …  
    379379         * Process high priority pre-execution raw-mode FFs.
    380380         */
    381         if (   VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
     381        if (   VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
    382382            || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_PRE_RAW_MASK))
    383383        {
    … …  
    448448         */
    449449        VMCPU_FF_CLEAR(pVCpu, VMCPU_FF_RESUME_GUEST_MASK);
    450         if (   VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
     450        if (   VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
    451451            || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_POST_MASK))
    452452            rcStrict = emR3HighPriorityPostForcedActions(pVM, pVCpu, rcStrict);
    … …  
    468468        TMTimerPollVoid(pVM, pVCpu);
    469469#endif
    470         if (   VM_FF_IS_PENDING(pVM, VM_FF_ALL_MASK)
     470        if (   VM_FF_IS_ANY_SET(pVM, VM_FF_ALL_MASK)
    471471            || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_ALL_MASK))
    472472        {

  • trunk/src/VBox/VMM/VMMR3/EMRaw.cpp

    r74790 r74791  
    172172         * Check vital forced actions, but ignore pending interrupts and timers.
    173173         */
    174         if (    VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
     174        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
    175175            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_PRE_RAW_MASK))
    176176        {
    … …  
    13171317         * Process high priority pre-execution raw-mode FFs.
    13181318         */
    1319         if (    VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
     1319        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
    13201320            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_PRE_RAW_MASK))
    13211321        {
    … …  
    13481348            CSAMR3CheckCodeEx(pVM, &pVCpu->cpum.GstCtx, pVCpu->cpum.GstCtx.eip);
    13491349            STAM_PROFILE_ADV_RESUME(&pVCpu->em.s.StatRAWEntry, b);
    1350             if (    VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
     1350            if (    VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_PRE_RAW_MASK)
    13511351                ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_PRE_RAW_MASK))
    13521352            {
    … …  
    14171417        rc = CPUMRawLeave(pVCpu, rc);
    14181418        VMCPU_FF_CLEAR(pVCpu, VMCPU_FF_RESUME_GUEST_MASK);
    1419         if (    VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
     1419        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_POST_MASK)
    14201420            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_HIGH_PRIORITY_POST_MASK))
    14211421            rc = VBOXSTRICTRC_TODO(emR3HighPriorityPostForcedActions(pVM, pVCpu, rc));
    … …  
    14841484#endif
    14851485        STAM_PROFILE_ADV_STOP(&pVCpu->em.s.StatRAWTail, d);
    1486         if (    VM_FF_IS_PENDING(pVM, ~VM_FF_HIGH_PRIORITY_PRE_RAW_MASK | VM_FF_PGM_NO_MEMORY)
     1486        if (    VM_FF_IS_ANY_SET(pVM, ~VM_FF_HIGH_PRIORITY_PRE_RAW_MASK | VM_FF_PGM_NO_MEMORY)
    14871487            ||  VMCPU_FF_IS_ANY_SET(pVCpu, ~VMCPU_FF_HIGH_PRIORITY_PRE_RAW_MASK))
    14881488        {

  • trunk/src/VBox/VMM/VMMR3/NEMR3Native-win.cpp

    r74789 r74791  
    17051705                    if (rcStrict == VINF_NEM_FLUSH_TLB)
    17061706                    {
    1707                         if (   !VM_FF_IS_PENDING(pVM, VM_FF_HIGH_PRIORITY_POST_MASK | VM_FF_HP_R0_PRE_HM_MASK)
     1707                        if (   !VM_FF_IS_ANY_SET(pVM, VM_FF_HIGH_PRIORITY_POST_MASK | VM_FF_HP_R0_PRE_HM_MASK)
    17081708                            && !VMCPU_FF_IS_ANY_SET(pVCpu,   (VMCPU_FF_HIGH_PRIORITY_POST_MASK | VMCPU_FF_HP_R0_PRE_HM_MASK)
    17091709                                                           & ~VMCPU_FF_RESUME_GUEST_MASK))

  • trunk/src/VBox/VMM/VMMR3/VMEmt.cpp

    r74790 r74791  
    372372        uint64_t const cNsElapsedTimers = RTTimeNanoTS() - u64StartTimers;
    373373        STAM_REL_PROFILE_ADD_PERIOD(&pUVCpu->vm.s.StatHaltTimers, cNsElapsedTimers);
    374         if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_HALTED_MASK)
     374        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_HALTED_MASK)
    375375            ||  VMCPU_FF_IS_ANY_SET(pVCpu, fMask))
    376376            break;
    377377        uint64_t u64NanoTS;
    378378        TMTimerPollGIP(pVM, pVCpu, &u64NanoTS);
    379         if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_HALTED_MASK)
     379        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_HALTED_MASK)
    380380            ||  VMCPU_FF_IS_ANY_SET(pVCpu, fMask))
    381381            break;
    … …  
    571571        uint64_t const cNsElapsedTimers = RTTimeNanoTS() - u64StartTimers;
    572572        STAM_REL_PROFILE_ADD_PERIOD(&pUVCpu->vm.s.StatHaltTimers, cNsElapsedTimers);
    573         if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_HALTED_MASK)
     573        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_HALTED_MASK)
    574574            ||  VMCPU_FF_IS_ANY_SET(pVCpu, fMask))
    575575            break;
    … …  
    580580        uint64_t u64NanoTS;
    581581        TMTimerPollGIP(pVM, pVCpu, &u64NanoTS);
    582         if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_HALTED_MASK)
     582        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_HALTED_MASK)
    583583            ||  VMCPU_FF_IS_ANY_SET(pVCpu, fMask))
    584584            break;
    … …  
    720720        uint64_t const cNsElapsedTimers = RTTimeNanoTS() - u64StartTimers;
    721721        STAM_REL_PROFILE_ADD_PERIOD(&pUVCpu->vm.s.StatHaltTimers, cNsElapsedTimers);
    722         if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_HALTED_MASK)
     722        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_HALTED_MASK)
    723723            ||  VMCPU_FF_IS_ANY_SET(pVCpu, fMask))
    724724            break;
    … …  
    730730        uint64_t u64Delta;
    731731        uint64_t u64GipTime = TMTimerPollGIP(pVM, pVCpu, &u64Delta);
    732         if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_HALTED_MASK)
     732        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_HALTED_MASK)
    733733            ||  VMCPU_FF_IS_ANY_SET(pVCpu, fMask))
    734734            break;
    … …  
    740740        {
    741741            VMMR3YieldStop(pVM);
    742             if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_HALTED_MASK)
     742            if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_HALTED_MASK)
    743743                ||  VMCPU_FF_IS_ANY_SET(pVCpu, fMask))
    744744                break;
    … …  
    808808         * Check Relevant FFs.
    809809         */
    810         if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_SUSPENDED_MASK)
     810        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_SUSPENDED_MASK)
    811811            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_EXTERNAL_SUSPENDED_MASK))
    812812            break;
    … …  
    898898
    899899        if (    pUVCpu->pVM
    900             &&  (   VM_FF_IS_PENDING(pUVCpu->pVM, VM_FF_EXTERNAL_SUSPENDED_MASK)
     900            &&  (   VM_FF_IS_ANY_SET(pUVCpu->pVM, VM_FF_EXTERNAL_SUSPENDED_MASK)
    901901                 || VMCPU_FF_IS_ANY_SET(VMMGetCpu(pUVCpu->pVM), VMCPU_FF_EXTERNAL_SUSPENDED_MASK)
    902902                )
    … …  
    960960         * Check Relevant FFs.
    961961         */
    962         if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_SUSPENDED_MASK)
     962        if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_SUSPENDED_MASK)
    963963            ||  VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_EXTERNAL_SUSPENDED_MASK))
    964964            break;
    … …  
    11091109        ? VMCPU_FF_EXTERNAL_HALTED_MASK
    11101110        : VMCPU_FF_EXTERNAL_HALTED_MASK & ~(VMCPU_FF_UPDATE_APIC | VMCPU_FF_INTERRUPT_APIC | VMCPU_FF_INTERRUPT_PIC);
    1111     if (    VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_HALTED_MASK)
     1111    if (    VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_HALTED_MASK)
    11121112        ||  VMCPU_FF_IS_ANY_SET(pVCpu, fMask))
    11131113    {
    … …  
    11901190
    11911191    if (    pVM
    1192         &&  (   VM_FF_IS_PENDING(pVM, VM_FF_EXTERNAL_SUSPENDED_MASK)
     1192        &&  (   VM_FF_IS_ANY_SET(pVM, VM_FF_EXTERNAL_SUSPENDED_MASK)
    11931193             || VMCPU_FF_IS_ANY_SET(pVCpu, VMCPU_FF_EXTERNAL_SUSPENDED_MASK)
    11941194            )
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