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

  • From: ingo_weinhold@xxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 14 Jul 2010 01:42:31 +0200 (CEST)

Author: bonefish
Date: 2010-07-14 01:42:31 +0200 (Wed, 14 Jul 2010)
New Revision: 37502
Changeset: http://dev.haiku-os.org/changeset/37502

Modified:
   haiku/trunk/src/system/kernel/vm/vm_page.cpp
Log:
page_writer(): Added TODO about potential deadlock in case of loop file
systems/devices.


Modified: haiku/trunk/src/system/kernel/vm/vm_page.cpp
===================================================================
--- haiku/trunk/src/system/kernel/vm/vm_page.cpp        2010-07-13 21:47:49 UTC 
(rev 37501)
+++ haiku/trunk/src/system/kernel/vm/vm_page.cpp        2010-07-13 23:42:31 UTC 
(rev 37502)
@@ -2016,6 +2016,11 @@
                        }
 
                        run.AddPage(page);
+                               // TODO: We're possibly adding pages of 
different caches and
+                               // thus maybe of different underlying file 
systems here. This
+                               // is a potential problem for loop file 
systems/devices, since
+                               // we could mark a page busy that would need to 
be accessed
+                               // when writing back another page, thus causing 
a deadlock.
 
                        DEBUG_PAGE_ACCESS_END(page);
 


Other related posts:

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