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

  • From: ingo_weinhold@xxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sat, 9 Jan 2010 21:59:03 +0100 (CET)

Author: bonefish
Date: 2010-01-09 21:59:03 +0100 (Sat, 09 Jan 2010)
New Revision: 34980
Changeset: http://dev.haiku-os.org/changeset/34980/haiku
Ticket: http://dev.haiku-os.org/ticket/5228

Modified:
   haiku/trunk/src/system/kernel/vm/vm_page.cpp
Log:
vm_page_schedule_write_page_range(): The debug code was a bit too possessive.
We can only access the page, if it is not busy. Fixes #5228.


Modified: haiku/trunk/src/system/kernel/vm/vm_page.cpp
===================================================================
--- haiku/trunk/src/system/kernel/vm/vm_page.cpp        2010-01-09 20:27:11 UTC 
(rev 34979)
+++ haiku/trunk/src/system/kernel/vm/vm_page.cpp        2010-01-09 20:59:03 UTC 
(rev 34980)
@@ -1813,14 +1813,12 @@
                if (page->cache_offset >= endPage)
                        break;
 
-               DEBUG_PAGE_ACCESS_START(page);
-
                if (page->state == PAGE_STATE_MODIFIED) {
+                       DEBUG_PAGE_ACCESS_START(page);
                        vm_page_requeue(page, false);
                        modified++;
+                       DEBUG_PAGE_ACCESS_END(page);
                }
-
-               DEBUG_PAGE_ACCESS_END(page);
        }
 
        if (modified > 0)


Other related posts:

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