Index: /trunk/src/VBox/VMM/PGMPhys.cpp
===================================================================
--- /trunk/src/VBox/VMM/PGMPhys.cpp	(revision 22714)
+++ /trunk/src/VBox/VMM/PGMPhys.cpp	(revision 22715)
@@ -386,4 +386,6 @@
             PGMPoolFlushPage(pVM, *pGCPhys);
             Assert(!PGM_PAGE_HAS_ACTIVE_HANDLERS(pPage));
+            /** @todo r=bird: return VERR_PGM_PHYS_PAGE_RESERVED here if it still has
+             *        active handlers, see the PGMR3PhysGCPhys2CCPtrExternal docs. */
         }
     }
@@ -2947,5 +2949,5 @@
         Req.idChunkUnmap = pgmR3PhysChunkFindUnmapCandidate(pVM);
 /** @todo This is wrong. Any thread in the VM process should be able to do this,
- *        there are depenenecies on this.  What currently saves the day is that 
+ *        there are depenenecies on this.  What currently saves the day is that
  *        we don't unmap anything and that all non-zero memory will therefore
  *        be present when non-EMTs tries to access it.  */