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

Commit 4b3ebec6 authored by Chih-Yu Huang's avatar Chih-Yu Huang Committed by Chih-yu Huang
Browse files

C2AllocatorGralloc: Convert C2Rect to Rect correctly 

The field of C2Rect is (left, top, width, height), and the field of
Rect is (left, top, right, bottom). This CL fixes the conversion from
C2Rect to Rect.

Bug: 170697563
Test: android.media.cts.MediaCodecBlockModelTest#testDecodeShortAudio

Change-Id: I74c7197587fbdaa0916046d1a9fad24d00e4f7a3
(cherry picked from commit 3d0e82ce1b1d1c594fee6796f49509bef5be9ca2)
parent bea48ae9

media/codec2/vndk/C2AllocatorGralloc.cpp

+11 −18
Original line number Original line Diff line number Diff line
@@ -25,6 +25,7 @@ 
#include <hardware/gralloc.h>

#include <hardware/gralloc.h>

#include <ui/GraphicBufferAllocator.h>

#include <ui/GraphicBufferAllocator.h>

#include <ui/GraphicBufferMapper.h>

#include <ui/GraphicBufferMapper.h>

#include <ui/Rect.h>





#include <C2AllocatorGralloc.h>

#include <C2AllocatorGralloc.h>

#include <C2Buffer.h>

#include <C2Buffer.h>
@@ -253,7 +254,7 @@ public: 
    virtual ~C2AllocationGralloc() override;

    virtual ~C2AllocationGralloc() override;





    virtual c2_status_t map(

    virtual c2_status_t map(

            C2Rect rect, C2MemoryUsage usage, C2Fence *fence,

            C2Rect c2Rect, C2MemoryUsage usage, C2Fence *fence,

            C2PlanarLayout *layout /* nonnull */, uint8_t **addr /* nonnull */) override;

            C2PlanarLayout *layout /* nonnull */, uint8_t **addr /* nonnull */) override;

    virtual c2_status_t unmap(

    virtual c2_status_t unmap(

            uint8_t **addr /* nonnull */, C2Rect rect, C2Fence *fence /* nullable */) override;

            uint8_t **addr /* nonnull */, C2Rect rect, C2Fence *fence /* nullable */) override;
@@ -336,8 +337,12 @@ C2AllocationGralloc::~C2AllocationGralloc() { 
}

}





c2_status_t C2AllocationGralloc::map(

c2_status_t C2AllocationGralloc::map(

        C2Rect rect, C2MemoryUsage usage, C2Fence *fence,

        C2Rect c2Rect, C2MemoryUsage usage, C2Fence *fence,

        C2PlanarLayout *layout /* nonnull */, uint8_t **addr /* nonnull */) {

        C2PlanarLayout *layout /* nonnull */, uint8_t **addr /* nonnull */) {

    const Rect rect{(int32_t)c2Rect.left, (int32_t)c2Rect.top,

                    (int32_t)(c2Rect.left + c2Rect.width) /* right */,

                    (int32_t)(c2Rect.top + c2Rect.height) /* bottom */};



    uint64_t grallocUsage = static_cast<C2AndroidMemoryUsage>(usage).asGrallocUsage();

    uint64_t grallocUsage = static_cast<C2AndroidMemoryUsage>(usage).asGrallocUsage();

    ALOGV("mapping buffer with usage %#llx => %#llx",

    ALOGV("mapping buffer with usage %#llx => %#llx",

          (long long)usage.expected, (long long)grallocUsage);

          (long long)usage.expected, (long long)grallocUsage);
@@ -386,10 +391,7 @@ c2_status_t C2AllocationGralloc::map( 
            void *pointer = nullptr;

            void *pointer = nullptr;

            // TODO: fence

            // TODO: fence

            status_t err = GraphicBufferMapper::get().lock(

            status_t err = GraphicBufferMapper::get().lock(

                                const_cast<native_handle_t *>(mBuffer), grallocUsage,

                    const_cast<native_handle_t *>(mBuffer), grallocUsage, rect, &pointer);

                                { (int32_t)rect.left, (int32_t)rect.top,

                                  (int32_t)rect.width, (int32_t)rect.height },

                                &pointer);

            if (err) {

            if (err) {

                ALOGE("failed transaction: lock(RGBA_1010102)");

                ALOGE("failed transaction: lock(RGBA_1010102)");

                return C2_CORRUPTED;

                return C2_CORRUPTED;
@@ -464,10 +466,7 @@ c2_status_t C2AllocationGralloc::map( 
            void *pointer = nullptr;

            void *pointer = nullptr;

            // TODO: fence

            // TODO: fence

            status_t err = GraphicBufferMapper::get().lock(

            status_t err = GraphicBufferMapper::get().lock(

                                const_cast<native_handle_t*>(mBuffer), grallocUsage,

                    const_cast<native_handle_t*>(mBuffer), grallocUsage, rect, &pointer);

                                { (int32_t)rect.left, (int32_t)rect.top,

                                  (int32_t)rect.width, (int32_t)rect.height },

                                &pointer);

            if (err) {

            if (err) {

                ALOGE("failed transaction: lock(RGBA_8888)");

                ALOGE("failed transaction: lock(RGBA_8888)");

                return C2_CORRUPTED;

                return C2_CORRUPTED;
@@ -524,10 +523,7 @@ c2_status_t C2AllocationGralloc::map( 
            void *pointer = nullptr;

            void *pointer = nullptr;

            // TODO: fence

            // TODO: fence

            status_t err = GraphicBufferMapper::get().lock(

            status_t err = GraphicBufferMapper::get().lock(

                                const_cast<native_handle_t*>(mBuffer), grallocUsage,

                    const_cast<native_handle_t*>(mBuffer), grallocUsage, rect, &pointer);

                                { (int32_t)rect.left, (int32_t)rect.top,

                                  (int32_t)rect.width, (int32_t)rect.height },

                                &pointer);

            if (err) {

            if (err) {

                ALOGE("failed transaction: lock(BLOB)");

                ALOGE("failed transaction: lock(BLOB)");

                return C2_CORRUPTED;

                return C2_CORRUPTED;
@@ -544,10 +540,7 @@ c2_status_t C2AllocationGralloc::map( 
            android_ycbcr ycbcrLayout;

            android_ycbcr ycbcrLayout;





            status_t err = GraphicBufferMapper::get().lockYCbCr(

            status_t err = GraphicBufferMapper::get().lockYCbCr(

                        const_cast<native_handle_t*>(mBuffer), grallocUsage,

                    const_cast<native_handle_t*>(mBuffer), grallocUsage, rect, &ycbcrLayout);

                        { (int32_t)rect.left, (int32_t)rect.top,

                          (int32_t)rect.width, (int32_t)rect.height },

                        &ycbcrLayout);

            if (err) {

            if (err) {

                ALOGE("failed transaction: lockYCbCr");

                ALOGE("failed transaction: lockYCbCr");

                return C2_CORRUPTED;

                return C2_CORRUPTED;