Fix a redundant BeforeAdditionalRefReleased call (#3174)

Author: chenBright

src/brpc/versioned_ref_with_id.h

@@ -504,10 +504,9 @@ int VersionedRefWithId<T>::ReleaseAdditionalReference() {
     do {
         AdditionalRefStatus expect = ADDITIONAL_REF_USING;
         if (_additional_ref_status.compare_exchange_strong(
-            expect, ADDITIONAL_REF_RECYCLED,
-            butil::memory_order_relaxed,
-            butil::memory_order_relaxed)) {
-            BeforeAdditionalRefReleasedWrapper();
+                expect, ADDITIONAL_REF_RECYCLED,
+                butil::memory_order_relaxed,
+                butil::memory_order_relaxed)) {
             WRAPPER_CALL(BeforeAdditionalRefReleased, static_cast<T*>(this));
             return Dereference();
         }
@@ -591,7 +590,7 @@ void VersionedRefWithId<T>::Revive(int32_t at_least_nref) {
 
         int32_t nref = NRefOfVRef(vref);
         if (nref < at_least_nref) {
-            // Set the status to REF_RECYLED since no one uses this socket
+            // Set the status to REF_RECYCLED since no one uses this socket
             _additional_ref_status.store(
                 ADDITIONAL_REF_RECYCLED, butil::memory_order_relaxed);
             CHECK_EQ(1, nref);

test/bthread_work_stealing_queue_unittest.cpp

@@ -21,6 +21,7 @@
 #include "butil/macros.h"
 #include "butil/scoped_lock.h"
 #include "bthread/work_stealing_queue.h"
+#include "bthread/processor.h"
 
 namespace {
 typedef size_t value_type;
@@ -39,13 +40,7 @@ void* steal_thread(void* arg) {
         if (q->steal(&val)) {
             stolen->push_back(val);
         } else {
-#if defined(ARCH_CPU_ARM_FAMILY)
-            asm volatile("yield\n": : :"memory");
-#elif defined(ARCH_CPU_LOONGARCH64_FAMILY)
-            asm volatile("nop\n": : :"memory");
-#else
-            asm volatile("pause\n": : :"memory");
-#endif
+            cpu_relax();
         }
     }
     return stolen;