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

Commit aef7c3d8 authored by Gopalakrishnan Nallasamy's avatar Gopalakrishnan Nallasamy Committed by Automerger Merge Worker
Browse files

C2AllocatorIon:protect mMappings using mutex am: 416da6e8 am: 67002488 am:... 

C2AllocatorIon:protect mMappings using mutex am: 416da6e8 am: 67002488 am: d6545589 am: 1785465c

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/16622627

Change-Id: I9d4e4e6221f198dfa29de0c4f3f1d123f7408ddd
parents 20a508fb 1785465c

media/codec2/vndk/C2AllocatorIon.cpp

+22 −15
Original line number Diff line number Diff line
@@ -207,6 +207,7 @@ public: 


        c2_status_t err = mapInternal(mapSize, mapOffset, alignmentBytes, prot, flags, &(map.addr), addr);

        if (map.addr) {

            std::lock_guard<std::mutex> guard(mMutexMappings);

            mMappings.push_back(map);

        }

        return err;
@@ -217,6 +218,8 @@ public: 
            ALOGD("tried to unmap unmapped buffer");

            return C2_NOT_FOUND;

        }

        { // Scope for the lock_guard of mMutexMappings.

            std::lock_guard<std::mutex> guard(mMutexMappings);

            for (auto it = mMappings.begin(); it != mMappings.end(); ++it) {

                if (addr != (uint8_t *)it->addr + it->alignmentBytes ||

                        size + it->alignmentBytes != it->size) {
@@ -231,9 +234,11 @@ public: 
                    *fence = C2Fence(); // not using fences

                }

                (void)mMappings.erase(it);

            ALOGV("successfully unmapped: addr=%p size=%zu fd=%d", addr, size, mHandle.bufferFd());

                ALOGV("successfully unmapped: addr=%p size=%zu fd=%d", addr, size,

                          mHandle.bufferFd());

                return C2_OK;

            }

        }

        ALOGD("unmap failed to find specified map");

        return C2_BAD_VALUE;

    }
@@ -241,6 +246,7 @@ public: 
    virtual ~Impl() {

        if (!mMappings.empty()) {

            ALOGD("Dangling mappings!");

            std::lock_guard<std::mutex> guard(mMutexMappings);

            for (const Mapping &map : mMappings) {

                (void)munmap(map.addr, map.size);

            }
@@ -320,6 +326,7 @@ protected: 
        size_t size;

    };

    std::list<Mapping> mMappings;

    std::mutex mMutexMappings;

};



class C2AllocationIon::ImplV2 : public C2AllocationIon::Impl {