Code optimizations.

   - Reduce DCBZ and ICBI calls in memory copy and init functions.
   - Reduce strlen calls in trace.
   - Set thread to low priority while waiting on in-kernel barrier.

Change-Id: Ic9c23b1e26797ff393e5862819830de60554747e
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/871
Tested-by: Jenkins Server
Reviewed-by: Douglas R. Gilbert <dgilbert@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Brian H. Horton <brianh@linux.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>

2 files changed, 5 insertions, 4 deletions

diff --git a/src/kernel/barrier.C b/src/kernel/barrier.C
index a5dadc63a..c0fea2fa4 100644
--- a/src/kernel/barrier.C
+++ b/src/kernel/barrier.C
@@ -31,7 +31,7 @@ void Barrier::wait()
     {
         size_t l_event = iv_event;
         iv_spinlock.unlock();
-        while(iv_event == l_event);
+        while(iv_event == l_event) { setThreadPriorityLow(); }
     }
     else
     {
@@ -39,4 +39,5 @@ void Barrier::wait()
         ++iv_event;
         iv_spinlock.unlock();
     }
+    setThreadPriorityHigh();
 }
diff --git a/src/kernel/pagemgr.C b/src/kernel/pagemgr.C
index 858215ecf..83e99b9b3 100644
--- a/src/kernel/pagemgr.C
+++ b/src/kernel/pagemgr.C
@@ -83,7 +83,7 @@ PageManager::PageManager() : iv_pagesAvail(0), iv_pagesTotal(0)
     while (cache_line != end_cache_line)
     {
         dcbz(cache_line);
-        cache_line++;
+        cache_line += getCacheLineWords();
     }
 
     // Allocate pages to buckets.
@@ -206,7 +206,7 @@ void PageManager::_coalesce( void )
         while(NULL != (p = pq.remove()))
         {
             // p needs to be the even buddy to prevent merging of wrong block.
-            // To determine this, get the index of the block as if the whole 
+            // To determine this, get the index of the block as if the whole
             // page memory space were blocks of this size.
             uint64_t p_idx = (reinterpret_cast<uint64_t>(p) - firstPageAddr())/
                              ((1 << bucket)*PAGESIZE);
@@ -220,7 +220,7 @@ void PageManager::_coalesce( void )
                 // match.  The address of p also can't be greater than what's
                 // in pq or something is really messed up, therefore if
                 // pq.remove_if() returns something then it's a match.
-                page_t * p_seek = (page_t*)((uint64_t)p + 
+                page_t * p_seek = (page_t*)((uint64_t)p +
                                             (1 << bucket)*PAGESIZE);
                 page_t * p_next = pq.remove_if(p_seek);
                 if(p_next == p_seek)