Skip to content

Commit

Permalink
Merge pull request eclipse-omr#2769 from amicic/master
Browse files Browse the repository at this point in the history
Remove deferredVMAccessRelease
  • Loading branch information
charliegracie authored Jul 20, 2018
2 parents 65a5628 + 7d533a4 commit 3d7cabd
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 7 deletions.
2 changes: 1 addition & 1 deletion example/glue/EnvironmentDelegate.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ MM_EnvironmentDelegate::releaseExclusiveVMAccess()
* @see assumeExclusiveVMAccess(uintptr_t)
*/
uintptr_t
MM_EnvironmentDelegate::relinquishExclusiveVMAccess(bool *deferredVMAccessRelease)
MM_EnvironmentDelegate::relinquishExclusiveVMAccess()
{
uintptr_t relinquishedExclusiveCount = _env->getOmrVMThread()->exclusiveCount;
_env->getOmrVMThread()->exclusiveCount = 0;
Expand Down
4 changes: 2 additions & 2 deletions example/glue/EnvironmentDelegate.hpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*******************************************************************************
* Copyright (c) 2017, 2017 IBM Corp. and others
* Copyright (c) 2017, 2018 IBM Corp. and others
*
* This program and the accompanying materials are made available under
* the terms of the Eclipse Public License 2.0 which accompanies this
Expand Down Expand Up @@ -213,7 +213,7 @@ class MM_EnvironmentDelegate
* @return the exclusive count of the current thread before relinquishing
* @see assumeExclusiveVMAccess(uintptr_t)
*/
uintptr_t relinquishExclusiveVMAccess(bool *deferredVMAccessRelease);
uintptr_t relinquishExclusiveVMAccess();

/**
* Assume exclusive access from a collaborating thread (i.e. main-to-master or master-to-main).
Expand Down
4 changes: 2 additions & 2 deletions gc/base/EnvironmentBase.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -487,9 +487,9 @@ MM_EnvironmentBase::unwindExclusiveVMAccessForGC()
}

uintptr_t
MM_EnvironmentBase::relinquishExclusiveVMAccess(bool *deferredVMAccessRelease)
MM_EnvironmentBase::relinquishExclusiveVMAccess()
{
return _delegate.relinquishExclusiveVMAccess(deferredVMAccessRelease);
return _delegate.relinquishExclusiveVMAccess();
}

void
Expand Down
3 changes: 1 addition & 2 deletions gc/base/EnvironmentBase.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -441,10 +441,9 @@ class MM_EnvironmentBase : public MM_BaseVirtual

/**
* Give up exclusive access in preparation for transferring it to a collaborating thread (i.e. main-to-master or master-to-main)
* @deferredVMAccessRelease Set to true if caller wants to defer releasing VM access (for the calling thread) at a later point (to be done explicitly by the caller). The method may set it back to false, if unable to obey the request.
* @return the exclusive count of the current thread before relinquishing
*/
uintptr_t relinquishExclusiveVMAccess(bool *deferredVMAccessRelease = NULL);
uintptr_t relinquishExclusiveVMAccess();

/**
* Assume exclusive access from a collaborating thread i.e. main-to-master or master-to-main)
Expand Down

0 comments on commit 3d7cabd

Please sign in to comment.