List:Commits« Previous MessageNext Message »
From:Jim Starkey Date:October 19 2007 10:35pm
Subject:bk commit into 6.0 tree (jas:1.2670)
View as plain text  
Below is the list of changes that have just been committed into a local
6.0 repository of jas. When jas does a push these changes will
be propagated to the main repository and, within 24 hours after the
push, to the public repository.
For information on how to access the public repository
see http://dev.mysql.com/doc/mysql/en/installing-source-tree.html

ChangeSet@stripped, 2007-10-19 18:35:17-04:00, jas@stripped +1 -0
  Moved lock operation to avoid deadlock.

  storage/falcon/Section.cpp@stripped, 2007-10-19 18:35:13-04:00, jas@stripped +1 -1
    Moved lock operation to avoid deadlock.

diff -Nrup a/storage/falcon/Section.cpp b/storage/falcon/Section.cpp
--- a/storage/falcon/Section.cpp	2007-10-19 10:47:35 -04:00
+++ b/storage/falcon/Section.cpp	2007-10-19 18:35:13 -04:00
@@ -1313,6 +1313,7 @@ Bdb* Section::getSectionPage(int sequenc
 	Sync sync(&syncObject, "Section::getSectionPage");
 	sync.lock(Shared);
 	int pageNumber = sectionPages.get(sequence);
+	sync.unlock();
 	
 	if (pageNumber)
 		{
@@ -1328,7 +1329,6 @@ Bdb* Section::getSectionPage(int sequenc
 		bdb->release(REL_HISTORY);
 		}
 	
-	sync.unlock();
 	sync.lock(Exclusive);
 	Bdb *bdb = getSectionPage(dbb, root, sequence, lockType, transId);
 	BDB_HISTORY(bdb);
Thread
bk commit into 6.0 tree (jas:1.2670)Jim Starkey20 Oct