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

Commit e9ebcdbb authored by Eric Laurent's avatar Eric Laurent
Browse files

Revert "audio policy: refactor audio record APIs" 

This reverts commit 0f4b3c54.

Bug: 72628781
Test: Capture several videos from camera app
Change-Id: I6bcc87c618ac9bfe5c911915cbb37de616bc727a
parent 67651f97

media/libaudioclient/AudioSystem.cpp

+13 −8
Original line number Diff line number Diff line
@@ -904,7 +904,6 @@ status_t AudioSystem::getInputForAttr(const audio_attributes_t *attr, 
                                audio_session_t session,

                                pid_t pid,

                                uid_t uid,

                                const String16& opPackageName,

                                const audio_config_base_t *config,

                                audio_input_flags_t flags,

                                audio_port_handle_t *selectedDeviceId,
@@ -913,29 +912,35 @@ status_t AudioSystem::getInputForAttr(const audio_attributes_t *attr, 
    const sp<IAudioPolicyService>& aps = AudioSystem::get_audio_policy_service();

    if (aps == 0) return NO_INIT;

    return aps->getInputForAttr(

            attr, input, session, pid, uid, opPackageName,

            attr, input, session, pid, uid,

            config, flags, selectedDeviceId, portId);

}



status_t AudioSystem::startInput(audio_port_handle_t portId, bool *silenced)

status_t AudioSystem::startInput(audio_io_handle_t input,

                                 audio_session_t session,

                                 audio_devices_t device,

                                 uid_t uid,

                                 bool *silenced)

{

    const sp<IAudioPolicyService>& aps = AudioSystem::get_audio_policy_service();

    if (aps == 0) return PERMISSION_DENIED;

    return aps->startInput(portId, silenced);

    return aps->startInput(input, session, device, uid, silenced);

}



status_t AudioSystem::stopInput(audio_port_handle_t portId)

status_t AudioSystem::stopInput(audio_io_handle_t input,

                                audio_session_t session)

{

    const sp<IAudioPolicyService>& aps = AudioSystem::get_audio_policy_service();

    if (aps == 0) return PERMISSION_DENIED;

    return aps->stopInput(portId);

    return aps->stopInput(input, session);

}



void AudioSystem::releaseInput(audio_port_handle_t portId)

void AudioSystem::releaseInput(audio_io_handle_t input,

                               audio_session_t session)

{

    const sp<IAudioPolicyService>& aps = AudioSystem::get_audio_policy_service();

    if (aps == 0) return;

    aps->releaseInput(portId);

    aps->releaseInput(input, session);

}



status_t AudioSystem::initStreamVolume(audio_stream_type_t stream,

media/libaudioclient/IAudioPolicyService.cpp

+28 −16
Original line number Diff line number Diff line
@@ -285,7 +285,6 @@ public: 
                                     audio_session_t session,

                                     pid_t pid,

                                     uid_t uid,

                                     const String16& opPackageName,

                                     const audio_config_base_t *config,

                                     audio_input_flags_t flags,

                                     audio_port_handle_t *selectedDeviceId,
@@ -314,7 +313,6 @@ public: 
        data.writeInt32(session);

        data.writeInt32(pid);

        data.writeInt32(uid);

        data.writeString16(opPackageName);

        data.write(config, sizeof(audio_config_base_t));

        data.writeInt32(flags);

        data.writeInt32(*selectedDeviceId);
@@ -333,12 +331,18 @@ public: 
        return NO_ERROR;

    }



    virtual status_t startInput(audio_port_handle_t portId,

    virtual status_t startInput(audio_io_handle_t input,

                                audio_session_t session,

                                audio_devices_t device,

                                uid_t uid,

                                bool *silenced)

    {

        Parcel data, reply;

        data.writeInterfaceToken(IAudioPolicyService::getInterfaceDescriptor());

        data.writeInt32(portId);

        data.writeInt32(input);

        data.writeInt32(session);

        data.writeInt32(device);

        data.writeInt32(uid);

        data.writeInt32(*silenced ? 1 : 0);

        remote()->transact(START_INPUT, data, &reply);

        status_t status = static_cast <status_t> (reply.readInt32());
@@ -346,20 +350,24 @@ public: 
        return status;

    }



    virtual status_t stopInput(audio_port_handle_t portId)

    virtual status_t stopInput(audio_io_handle_t input,

                               audio_session_t session)

    {

        Parcel data, reply;

        data.writeInterfaceToken(IAudioPolicyService::getInterfaceDescriptor());

        data.writeInt32(portId);

        data.writeInt32(input);

        data.writeInt32(session);

        remote()->transact(STOP_INPUT, data, &reply);

        return static_cast <status_t> (reply.readInt32());

    }



    virtual void releaseInput(audio_port_handle_t portId)

    virtual void releaseInput(audio_io_handle_t input,

                              audio_session_t session)

    {

        Parcel data, reply;

        data.writeInterfaceToken(IAudioPolicyService::getInterfaceDescriptor());

        data.writeInt32(portId);

        data.writeInt32(input);

        data.writeInt32(session);

        remote()->transact(RELEASE_INPUT, data, &reply);

    }
@@ -1026,7 +1034,6 @@ status_t BnAudioPolicyService::onTransact( 
            audio_session_t session = (audio_session_t)data.readInt32();

            pid_t pid = (pid_t)data.readInt32();

            uid_t uid = (uid_t)data.readInt32();

            const String16 opPackageName = data.readString16();

            audio_config_base_t config;

            memset(&config, 0, sizeof(audio_config_base_t));

            data.read(&config, sizeof(audio_config_base_t));
@@ -1034,7 +1041,7 @@ status_t BnAudioPolicyService::onTransact( 
            audio_port_handle_t selectedDeviceId = (audio_port_handle_t) data.readInt32();

            audio_port_handle_t portId = (audio_port_handle_t)data.readInt32();

            status_t status = getInputForAttr(&attr, &input, session, pid, uid,

                                              opPackageName, &config,

                                              &config,

                                              flags, &selectedDeviceId, &portId);

            reply->writeInt32(status);

            if (status == NO_ERROR) {
@@ -1047,9 +1054,12 @@ status_t BnAudioPolicyService::onTransact( 


        case START_INPUT: {

            CHECK_INTERFACE(IAudioPolicyService, data, reply);

            audio_port_handle_t portId = static_cast <audio_port_handle_t>(data.readInt32());

            audio_io_handle_t input = static_cast <audio_io_handle_t>(data.readInt32());

            audio_session_t session = static_cast <audio_session_t>(data.readInt32());

            audio_devices_t device = static_cast <audio_devices_t>(data.readInt32());

            uid_t uid = static_cast <uid_t>(data.readInt32());

            bool silenced = data.readInt32() == 1;

            status_t status = startInput(portId, &silenced);

            status_t status = startInput(input, session, device, uid, &silenced);

            reply->writeInt32(static_cast <uint32_t>(status));

            reply->writeInt32(silenced ? 1 : 0);

            return NO_ERROR;
@@ -1057,15 +1067,17 @@ status_t BnAudioPolicyService::onTransact( 


        case STOP_INPUT: {

            CHECK_INTERFACE(IAudioPolicyService, data, reply);

            audio_port_handle_t portId = static_cast <audio_port_handle_t>(data.readInt32());

            reply->writeInt32(static_cast <uint32_t>(stopInput(portId)));

            audio_io_handle_t input = static_cast <audio_io_handle_t>(data.readInt32());

            audio_session_t session = static_cast <audio_session_t>(data.readInt32());

            reply->writeInt32(static_cast <uint32_t>(stopInput(input, session)));

            return NO_ERROR;

        } break;



        case RELEASE_INPUT: {

            CHECK_INTERFACE(IAudioPolicyService, data, reply);

            audio_port_handle_t portId = static_cast <audio_port_handle_t>(data.readInt32());

            releaseInput(portId);

            audio_io_handle_t input = static_cast <audio_io_handle_t>(data.readInt32());

            audio_session_t session = static_cast <audio_session_t>(data.readInt32());

            releaseInput(input, session);

            return NO_ERROR;

        } break;

media/libaudioclient/include/media/AudioSystem.h

+8 −4
Original line number Diff line number Diff line
@@ -239,16 +239,20 @@ public: 
                                    audio_session_t session,

                                    pid_t pid,

                                    uid_t uid,

                                    const String16& opPackageName,

                                    const audio_config_base_t *config,

                                    audio_input_flags_t flags,

                                    audio_port_handle_t *selectedDeviceId,

                                    audio_port_handle_t *portId);



    static status_t startInput(audio_port_handle_t portId,

    static status_t startInput(audio_io_handle_t input,

                               audio_session_t session,

                               audio_devices_t device,

                               uid_t uid,

                               bool *silenced);

    static status_t stopInput(audio_port_handle_t portId);

    static void releaseInput(audio_port_handle_t portId);

    static status_t stopInput(audio_io_handle_t input,

                              audio_session_t session);

    static void releaseInput(audio_io_handle_t input,

                             audio_session_t session);

    static status_t initStreamVolume(audio_stream_type_t stream,

                                      int indexMin,

                                      int indexMax);

media/libaudioclient/include/media/IAudioPolicyService.h

+8 −4
Original line number Diff line number Diff line
@@ -80,15 +80,19 @@ public: 
                              audio_session_t session,

                              pid_t pid,

                              uid_t uid,

                              const String16& opPackageName,

                              const audio_config_base_t *config,

                              audio_input_flags_t flags,

                              audio_port_handle_t *selectedDeviceId,

                              audio_port_handle_t *portId) = 0;

    virtual status_t startInput(audio_port_handle_t portId,

    virtual status_t startInput(audio_io_handle_t input,

                                audio_session_t session,

                                audio_devices_t device,

                                uid_t uid,

                                bool *silenced) = 0;

    virtual status_t stopInput(audio_port_handle_t portId) = 0;

    virtual void releaseInput(audio_port_handle_t portId) = 0;

    virtual status_t stopInput(audio_io_handle_t input,

                               audio_session_t session) = 0;

    virtual void releaseInput(audio_io_handle_t input,

                              audio_session_t session) = 0;

    virtual status_t initStreamVolume(audio_stream_type_t stream,

                                      int indexMin,

                                      int indexMax) = 0;

services/audioflinger/AudioFlinger.cpp

+3 −5
Original line number Diff line number Diff line
@@ -321,7 +321,6 @@ status_t AudioFlinger::openMmapStream(MmapStreamInterface::stream_direction_t di 
                                              actualSessionId,

                                              client.clientPid,

                                              client.clientUid,

                                              client.packageName,

                                              config,

                                              AUDIO_INPUT_FLAG_MMAP_NOIRQ, deviceId, &portId);

    }
@@ -341,7 +340,7 @@ status_t AudioFlinger::openMmapStream(MmapStreamInterface::stream_direction_t di 
        if (direction == MmapStreamInterface::DIRECTION_OUTPUT) {

            AudioSystem::releaseOutput(io, streamType, actualSessionId);

        } else {

            AudioSystem::releaseInput(portId);

            AudioSystem::releaseInput(io, actualSessionId);

        }

        ret = NO_INIT;

    }
@@ -1664,7 +1663,7 @@ sp<media::IAudioRecord> AudioFlinger::createRecord(const CreateRecordInput& inpu 
    // release previously opened input if retrying.

    if (output.inputId != AUDIO_IO_HANDLE_NONE) {

        recordTrack.clear();

        AudioSystem::releaseInput(portId);

        AudioSystem::releaseInput(output.inputId, sessionId);

        output.inputId = AUDIO_IO_HANDLE_NONE;

    }

    lStatus = AudioSystem::getInputForAttr(&input.attr, &output.inputId,
@@ -1672,7 +1671,6 @@ sp<media::IAudioRecord> AudioFlinger::createRecord(const CreateRecordInput& inpu 
                                    // FIXME compare to AudioTrack

                                      clientPid,

                                      clientUid,

                                      input.opPackageName,

                                      &input.config,

                                      output.flags, &output.selectedDeviceId, &portId);
@@ -1741,7 +1739,7 @@ Exit: 
        }

        recordTrack.clear();

        if (output.inputId != AUDIO_IO_HANDLE_NONE) {

            AudioSystem::releaseInput(portId);

            AudioSystem::releaseInput(output.inputId, sessionId);

        }

    }