[haiku-commits] r36155 - haiku/trunk/src/system/kernel/vm

  • From: ingo_weinhold@xxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 11 Apr 2010 16:49:06 +0200 (CEST)

Author: bonefish
Date: 2010-04-11 16:49:06 +0200 (Sun, 11 Apr 2010)
New Revision: 36155
Changeset: http://dev.haiku-os.org/changeset/36155/haiku

Modified:
   haiku/trunk/src/system/kernel/vm/vm_page.cpp
Log:
full_scan_inactive_pages(): Don't call DEBUG_PAGE_ACCESS_START() before
checking whether the page is busy.


Modified: haiku/trunk/src/system/kernel/vm/vm_page.cpp
===================================================================
--- haiku/trunk/src/system/kernel/vm/vm_page.cpp        2010-04-11 14:26:19 UTC 
(rev 36154)
+++ haiku/trunk/src/system/kernel/vm/vm_page.cpp        2010-04-11 14:49:06 UTC 
(rev 36155)
@@ -2166,16 +2166,15 @@
                        continue;
                }
 
-               DEBUG_PAGE_ACCESS_START(page);
-
                if (page->busy) {
                        // page is busy -- requeue at the end
                        vm_page_requeue(page, true);
                        cache->ReleaseRefAndUnlock();
-                       DEBUG_PAGE_ACCESS_END(page);
                        continue;
                }
 
+               DEBUG_PAGE_ACCESS_START(page);
+
                // Get the page active/modified flags and update the page's 
usage count.
                // We completely unmap inactive temporary pages. This saves us 
to
                // iterate through the inactive list as well, since we'll be 
notified


Other related posts:

  • » [haiku-commits] r36155 - haiku/trunk/src/system/kernel/vm - ingo_weinhold