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

Commit a0470879 authored by Lajos Molnar's avatar Lajos Molnar Committed by The Android Automerger
Browse files

stagefright: allow for minUndequeuedBufs to be one less 

Bug: 13533477
Change-Id: I909324e3e24f65d8051e0d5474267620efcfc729
parent b635b0e6

media/libstagefright/ACodec.cpp

+4 −3
Original line number Diff line number Diff line
@@ -640,8 +640,8 @@ status_t ACodec::configureOutputBuffersFromNativeWindow( 


    // FIXME: assume that surface is controlled by app (native window

    // returns the number for the case when surface is not controlled by app)

    (*minUndequeuedBuffers)++;



    // FIXME2: This means that minUndeqeueudBufs can be 1 larger than reported

    // For now, try to allocate 1 more buffer, but don't fail if unsuccessful



    // Use conservative allocation while also trying to reduce starvation

    //
@@ -649,7 +649,8 @@ status_t ACodec::configureOutputBuffersFromNativeWindow( 
    //    minimum needed for the consumer to be able to work

    // 2. try to allocate two (2) additional buffers to reduce starvation from

    //    the consumer

    for (OMX_U32 extraBuffers = 2; /* condition inside loop */; extraBuffers--) {

    //    plus an extra buffer to account for incorrect minUndequeuedBufs

    for (OMX_U32 extraBuffers = 2 + 1; /* condition inside loop */; extraBuffers--) {

        OMX_U32 newBufferCount =

            def.nBufferCountMin + *minUndequeuedBuffers + extraBuffers;

        def.nBufferCountActual = newBufferCount;

media/libstagefright/OMXCodec.cpp

+6 −4
Original line number Diff line number Diff line
@@ -1780,7 +1780,8 @@ status_t OMXCodec::allocateOutputBuffersFromNativeWindow() { 
    }

    // FIXME: assume that surface is controlled by app (native window

    // returns the number for the case when surface is not controlled by app)

    minUndequeuedBufs++;

    // FIXME2: This means that minUndeqeueudBufs can be 1 larger than reported

    // For now, try to allocate 1 more buffer, but don't fail if unsuccessful



    // Use conservative allocation while also trying to reduce starvation

    //
@@ -1788,10 +1789,11 @@ status_t OMXCodec::allocateOutputBuffersFromNativeWindow() { 
    //    minimum needed for the consumer to be able to work

    // 2. try to allocate two (2) additional buffers to reduce starvation from

    //    the consumer

    CODEC_LOGI("OMX-buffers: min=%u actual=%u undeq=%d",

    //    plus an extra buffer to account for incorrect minUndequeuedBufs

    CODEC_LOGI("OMX-buffers: min=%u actual=%u undeq=%d+1",

            def.nBufferCountMin, def.nBufferCountActual, minUndequeuedBufs);



    for (OMX_U32 extraBuffers = 2; /* condition inside loop */; extraBuffers--) {

    for (OMX_U32 extraBuffers = 2 + 1; /* condition inside loop */; extraBuffers--) {

        OMX_U32 newBufferCount =

            def.nBufferCountMin + minUndequeuedBufs + extraBuffers;

        def.nBufferCountActual = newBufferCount;
@@ -1810,7 +1812,7 @@ status_t OMXCodec::allocateOutputBuffersFromNativeWindow() { 
            return err;

        }

    }

    CODEC_LOGI("OMX-buffers: min=%u actual=%u undeq=%d",

    CODEC_LOGI("OMX-buffers: min=%u actual=%u undeq=%d+1",

            def.nBufferCountMin, def.nBufferCountActual, minUndequeuedBufs);



    err = native_window_set_buffer_count(