Changeset 44509 in vbox for trunk/src/VBox/VMM

Timestamp:

Feb 1, 2013 1:11:17 PM (12 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

83546

Message:

Document PIC and APIC callback method locking rules.

File:

• trunk/src/VBox/VMM/VMMAll/PDMAll.cpp (modified) (2 diffs)

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

@@ -318 +318 @@
  *
  * @remarks No-long-jump zone!!!
-*/
+ */
 VMMDECL(int) PDMApicGetTPR(PVMCPU pVCpu, uint8_t *pu8TPR, bool *pfPending)
 {
@@ -324 +324 @@
     if (pVM->pdm.s.Apic.CTX_SUFF(pDevIns))
     {
+        /*
+         * Note! We don't acquire the PDM lock here as we're just reading
+         *       information. Doing so causes massive contention as this
+         *       function is called very often by each and every VCPU.
+         */
         Assert(pVM->pdm.s.Apic.CTX_SUFF(pfnGetTPR));
-        /* We don't acquire the PDM lock here as we're just reading information. Doing so causes massive
-         * contention as this function is called very often by each and every VCPU.
-         */
         *pu8TPR = pVM->pdm.s.Apic.CTX_SUFF(pfnGetTPR)(pVM->pdm.s.Apic.CTX_SUFF(pDevIns), pVCpu->idCpu);
         if (pfPending)