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

Commit 52a19954 authored by John Reck's avatar John Reck Committed by Automerger Merge Worker
Browse files

Merge "Fix USAGE_FRONT_BUFFER failure on Cuttlefish" into udc-dev am: 1889c6a8 am: 384705e5 

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



Change-Id: I877f3c194d015d3cb3e6017f211ba35e2c452b2c
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 2079f021 384705e5

libs/ui/Gralloc5.cpp

+11 −8
Original line number Original line Diff line number Diff line
@@ -343,14 +343,17 @@ status_t Gralloc5Mapper::validateBufferSize(buffer_handle_t bufferHandle, uint32 
            return BAD_VALUE;

            return BAD_VALUE;

        }

        }

    }

    }

    {

    // TODO: This can false-positive fail if the allocator adjusted the USAGE bits internally

        auto value = getStandardMetadata<StandardMetadataType::USAGE>(mMapper, bufferHandle);

    //       Investigate further & re-enable or remove, but for now ignoring usage should be OK

        if (static_cast<BufferUsage>(usage) != value) {

    (void)usage;

            ALOGW("Usage didn't match, expected %" PRIu64 " got %" PRId64, usage,

    // {

                  static_cast<int64_t>(value.value_or(BufferUsage::CPU_READ_NEVER)));

    //     auto value = getStandardMetadata<StandardMetadataType::USAGE>(mMapper, bufferHandle);

            return BAD_VALUE;

    //     if (static_cast<BufferUsage>(usage) != value) {

        }

    //         ALOGW("Usage didn't match, expected %" PRIu64 " got %" PRId64, usage,

    }

    //               static_cast<int64_t>(value.value_or(BufferUsage::CPU_READ_NEVER)));

    //         return BAD_VALUE;

    //     }

    // }

    {

    {

        auto value = getStandardMetadata<StandardMetadataType::STRIDE>(mMapper, bufferHandle);

        auto value = getStandardMetadata<StandardMetadataType::STRIDE>(mMapper, bufferHandle);

        if (stride != value) {

        if (stride != value) {