Remove heap allocation of xscom mutex.

Since mutexes are simply a uint64 with a static initializer, the xscom mutex
is now a instance variable directly in the cpu object.

Remove unneeded mutex_create function and changed the behavior of
mutex_destroy.

Change-Id: If6e1d1bf0083c32ef9e7502d27678811bdaf7e1e
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/150
Tested-by: Jenkins Server
Reviewed-by: Douglas R. Gilbert <dgilbert@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>

1 files changed, 5 insertions, 10 deletions

diff --git a/src/include/sys/sync.h b/src/include/sys/sync.h
index 68b570aec..b6274a0a6 100644
--- a/src/include/sys/sync.h
+++ b/src/include/sys/sync.h
@@ -51,12 +51,6 @@ void barrier_destroy (barrier_t * i_barrier);
 void barrier_wait (barrier_t * i_barrier);
 
 /**
- * Create a mutex and initialize a mutex
- * @returns a pointer to the mutex
- */
-mutex_t * mutex_create();
-
-/**
  * Initialize a mutex object
  * @param[out] o_mutex the mutex
  * @pre an uninitialized mutex object
@@ -65,11 +59,12 @@ mutex_t * mutex_create();
 void mutex_init(mutex_t * o_mutex);
 
 /**
- * Destroy a mutex
- * @param[in/out] i_mutex The mutex
- * @pre mutex must have been created with mutex_create()
+ * Destroy / Uninitialize a mutex object.
+ * @param[in] i_mutex The mutex
+ * @note This does not free the memory associated with the object if the mutex
+ *       was allocated off the heap.
  */
-void mutex_destroy(mutex_t *& io_mutex);
+void mutex_destroy(mutex_t * i_mutex);
 
 /**
  * Obtain a lock on a mutex