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

Commit 2ceebb30 authored by Lajos Molnar's avatar Lajos Molnar Committed by Android Git Automerger
Browse files

am bd1f8f0a: am fb28c384: am 1faa4170: stagefright: allow for minUndequeuedBufs to be one less 

* commit 'bd1f8f0a':
  stagefright: allow for minUndequeuedBufs to be one less
parents 46d1b4ab bd1f8f0a

media/libstagefright/ACodec.cpp

+4 −3
Original line number Diff line number Diff line
@@ -644,8 +644,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

    //
@@ -653,7 +653,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
@@ -1806,7 +1806,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

    //
@@ -1814,10 +1815,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;
@@ -1836,7 +1838,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(