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

Commit b41690ce authored by Mikhail Naganov's avatar Mikhail Naganov Committed by Automerger Merge Worker
Browse files

Merge "audio: Update FMQ error reporting handler" into main am: 9a24a944 am: 611c082c 

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3272910



Change-Id: I8a834be41c3a3d13d54c186db1829e6e80051ac7
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 5ceff283 611c082c

audio/aidl/default/Stream.cpp

+21 −9
Original line number Diff line number Diff line
@@ -47,6 +47,17 @@ using aidl::android::media::audio::common::MicrophoneInfo; 


namespace aidl::android::hardware::audio::core {



namespace {



template <typename MQTypeError>

auto fmqErrorHandler(const char* mqName) {

    return [m = std::string(mqName)](MQTypeError fmqError, std::string&& errorMessage) {

        CHECK_EQ(fmqError, MQTypeError::NONE) << m << ": " << errorMessage;

    };

}



}  // namespace



void StreamContext::fillDescriptor(StreamDescriptor* desc) {

    if (mCommandMQ) {

        desc->command = mCommandMQ->dupeDesc();
@@ -332,11 +343,7 @@ StreamInWorkerLogic::Status StreamInWorkerLogic::cycle() { 
bool StreamInWorkerLogic::read(size_t clientSize, StreamDescriptor::Reply* reply) {

    ATRACE_CALL();

    StreamContext::DataMQ* const dataMQ = mContext->getDataMQ();

    StreamContext::DataMQ::Error fmqError = StreamContext::DataMQ::Error::NONE;

    std::string fmqErrorMsg;

    const size_t byteCount = std::min(

            {clientSize, dataMQ->availableToWrite(&fmqError, &fmqErrorMsg), mDataBufferSize});

    CHECK(fmqError == StreamContext::DataMQ::Error::NONE) << fmqErrorMsg;

    const size_t byteCount = std::min({clientSize, dataMQ->availableToWrite(), mDataBufferSize});

    const bool isConnected = mIsConnected;

    const size_t frameSize = mContext->getFrameSize();

    size_t actualFrameCount = 0;
@@ -612,10 +619,7 @@ StreamOutWorkerLogic::Status StreamOutWorkerLogic::cycle() { 
bool StreamOutWorkerLogic::write(size_t clientSize, StreamDescriptor::Reply* reply) {

    ATRACE_CALL();

    StreamContext::DataMQ* const dataMQ = mContext->getDataMQ();

    StreamContext::DataMQ::Error fmqError = StreamContext::DataMQ::Error::NONE;

    std::string fmqErrorMsg;

    const size_t readByteCount = dataMQ->availableToRead(&fmqError, &fmqErrorMsg);

    CHECK(fmqError == StreamContext::DataMQ::Error::NONE) << fmqErrorMsg;

    const size_t readByteCount = dataMQ->availableToRead();

    const size_t frameSize = mContext->getFrameSize();

    bool fatal = false;

    int32_t latency = mContext->getNominalLatencyMs();
@@ -719,6 +723,14 @@ ndk::ScopedAStatus StreamCommonImpl::initInstance( 
            LOG(WARNING) << __func__ << ": invalid worker tid: " << workerTid;

        }

    }

    getContext().getCommandMQ()->setErrorHandler(

            fmqErrorHandler<StreamContext::CommandMQ::Error>("CommandMQ"));

    getContext().getReplyMQ()->setErrorHandler(

            fmqErrorHandler<StreamContext::ReplyMQ::Error>("ReplyMQ"));

    if (getContext().getDataMQ() != nullptr) {

        getContext().getDataMQ()->setErrorHandler(

                fmqErrorHandler<StreamContext::DataMQ::Error>("DataMQ"));

    }

    return ndk::ScopedAStatus::ok();

}