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

  • From: ingo_weinhold@xxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 16 Feb 2010 00:01:31 +0100 (CET)

Author: bonefish
Date: 2010-02-16 00:01:31 +0100 (Tue, 16 Feb 2010)
New Revision: 35487
Changeset: http://dev.haiku-os.org/changeset/35487/haiku

Modified:
   haiku/trunk/src/system/kernel/vm/vm_page.cpp
Log:
* page_writer(): Missing continue in the temporary skipping case.
* vm_page_write_modified_page_range(): Need to DEBUG_PAGE_ACCESS_END() a bit
  later.


Modified: haiku/trunk/src/system/kernel/vm/vm_page.cpp
===================================================================
--- haiku/trunk/src/system/kernel/vm/vm_page.cpp        2010-02-15 22:50:28 UTC 
(rev 35486)
+++ haiku/trunk/src/system/kernel/vm/vm_page.cpp        2010-02-15 23:01:31 UTC 
(rev 35487)
@@ -1792,6 +1792,7 @@
                                        set_page_state(page, PAGE_STATE_ACTIVE);
 
                                DEBUG_PAGE_ACCESS_END(page);
+                               continue;
                        }
 
                        // We need our own reference to the store, as it might 
currently be
@@ -2504,17 +2505,19 @@
 
                        wrapper->SetTo(page);
 
-                       DEBUG_PAGE_ACCESS_END(page);
-
                        if (transferEmpty || transfer.AddPage(page)) {
                                if (transferEmpty) {
                                        transfer.SetTo(NULL, page, maxPages);
                                        transferEmpty = false;
                                }
 
+                               DEBUG_PAGE_ACCESS_END(page);
+
                                wrappers[usedWrappers++] = wrapper;
                                continue;
                        }
+
+                       DEBUG_PAGE_ACCESS_END(page);
                }
 
                if (transferEmpty)


Other related posts:

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