Re: [Boost-bugs] [Boost C++ Libraries] #6308: Add sp_counted_base_aix.hpp using AIX atomic operations

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #6308: Add sp_counted_base_aix.hpp using AIX atomic operations
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2011-12-23 17:28:15

#6308: Add sp_counted_base_aix.hpp using AIX atomic operations
-----------------------------------------+----------------------------------
  Reporter:  aaron.riekenberg@…          |       Owner:  pdimov      
      Type:  Feature Requests            |      Status:  reopened    
 Milestone:  To Be Determined            |   Component:  smart_ptr   
   Version:  Boost 1.48.0                |    Severity:  Optimization
Resolution:                              |    Keywords:              
-----------------------------------------+----------------------------------

Comment (by aaron.riekenberg@…):

 I experimented with lwsync.  It was slightly faster than sync in my tests
 (around 300 nanoseconds).  lwsync does not enforce ordering of stores
 followed by loads
 (http://www.ibm.com/developerworks/systems/articles/powerpc.html).  So I
 think I'll stick with sync for this reason, and to be consistent with
 sp_counted_base_gcc_ppc.hpp.

 I am attaching one last (hopefully!) patch which does the following:

 1. Uses builtin functions sync() and isync() instead of inline asm.

 2. Puts the atomic_* functions inside the boost::detail namespace - I
 accidentally put them in the boost namespace previously.

 Thanks again for all your help on this.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/6308#comment:9>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.