Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 655d5710 authored by Steven Moreland's avatar Steven Moreland Committed by Gerrit Code Review
Browse files

Revert "libbinder: Flush excess refs after single async transaction" 

This reverts commit 6aae5369.

Reason for revert: b/268238590
Bug: 268238590
Test: binderRpcTestNoKernel --gtest_filter="*OnewayCallQueueingWithFds/unix_domain_socket_bootstrap_raw_clientV4026531840_serverV4026531840_no_kernel*" --gtest_repeat=10000 --gtest_break_on_failure

(this change causes tests to flake, reverting for now)

Change-Id: I2dd4e347228791085f3f2534f948839ee27c16b6
parent 6aae5369

libs/binder/RpcState.cpp

+2 −2
Original line number Diff line number Diff line
@@ -1036,8 +1036,8 @@ processTransactInternalTailCall: 
                return DEAD_OBJECT;

            }



            if (it->second.asyncTodo.size() != 0 &&

                it->second.asyncTodo.top().asyncNumber == it->second.asyncNumber) {

            if (it->second.asyncTodo.size() == 0) return OK;

            if (it->second.asyncTodo.top().asyncNumber == it->second.asyncNumber) {

                LOG_RPC_DETAIL("Found next async transaction %" PRIu64 " on %" PRIu64,

                               it->second.asyncNumber, addr);