RE: [PATCH] media: coda: avoid starvation on well-compressed data

From: Benjamin Bara - SKIDATA
Date: Fri Sep 18 2020 - 04:35:46 EST


Hi all,

there are still cases where the decoder starves.
Also, the failure log at the bottom contradicts with the 2x256 bytes assumption.
When I increase the threshold to 512 bytes, the respective video (and all my other tests) work.

Is it possible that the limitation is 2x512 bytes or is there another "limitation",
beside the "two window" one, which is "accidentally fulfilled" with this approach?

What do you think about the approach in the patch; is it possible to get it mainline?
Any help, feedback, hints or suggestions would be really appreciated!

I will do some additional testing to see if the 2x512 threshold finally catches the problem.
When I'm done, I will provide a second version of the patch.

Many thanks & best regards
Benjamin

*Failure Log:*
[ 108.108711] coda 2040000.vpu: 0: active metas:
[ 108.108716] coda 2040000.vpu: 0: - payload: 4240
[ 108.108721] coda 2040000.vpu: 0: - payload: 900
[ 108.108726] coda 2040000.vpu: 0: - payload: 170
[ 108.108730] coda 2040000.vpu: 0: - payload: 403
[ 108.108734] coda 2040000.vpu: 0: want to queue: payload: 405
[ 109.057738] coda 2040000.vpu: CODA PIC_RUN timeout