[v3,0/4] media: mediatek: vcodec: Fix decoder under flow and plt test fails randomly

Message

Yunfei Dong April 24, 2023, 6:01 a.m. UTC

1: Getting decoder under flow error randomly when do stress test with youtube;
2: Video display black when do plt test with one night.

patch 1 avoid unneeded error logging.
patch 2 using decoder status instead of core work count.
patch 3 move core context from device to each instance.
patch 4 using empty lat buffer as the last one to flush decoder.
---
changed with v2:
- fix build error for patch 4
changed with v1:
- re-write commit messgage for patch 1
- squash v1 patch 2 to v1 patch 3 for patch 2
- squash v1 patch 4 to v1 patch 6 for patch 4
---
Yunfei Dong (4):
  media: mediatek: vcodec: Avoid unneeded error logging
  media: mediatek: vcodec: using decoder status instead of core work
    count
  media: mediatek: vcodec: move core context from device to each
    instance
  media: mediatek: vcodec: using empty lat buffer as the last one

 .../mediatek/vcodec/mtk_vcodec_dec_drv.c      |   1 -
 .../vcodec/mtk_vcodec_dec_stateless.c         |   2 +-
 .../platform/mediatek/vcodec/mtk_vcodec_drv.h |   2 -
 .../vcodec/vdec/vdec_h264_req_multi_if.c      |   6 +-
 .../vcodec/vdec/vdec_vp9_req_lat_if.c         |   4 +-
 .../platform/mediatek/vcodec/vdec_msg_queue.c | 105 ++++++++----------
 .../platform/mediatek/vcodec/vdec_msg_queue.h |  28 ++++-
 7 files changed, 74 insertions(+), 74 deletions(-)

Comments

Yunfei Dong May 17, 2023, 3:34 a.m. UTC | #1

Dear Hans,

Sorry to disturb you.
Could you please help to review & merge this patch series when you are
free?

Best Regards,
Yunfei Dong

On Mon, 2023-04-24 at 14:01 +0800, Yunfei Dong wrote:
> 1: Getting decoder under flow error randomly when do stress test with
> youtube;
> 2: Video display black when do plt test with one night.
> 
> patch 1 avoid unneeded error logging.
> patch 2 using decoder status instead of core work count.
> patch 3 move core context from device to each instance.
> patch 4 using empty lat buffer as the last one to flush decoder.
> ---
> changed with v2:
> - fix build error for patch 4
> changed with v1:
> - re-write commit messgage for patch 1
> - squash v1 patch 2 to v1 patch 3 for patch 2
> - squash v1 patch 4 to v1 patch 6 for patch 4
> ---
> Yunfei Dong (4):
>   media: mediatek: vcodec: Avoid unneeded error logging
>   media: mediatek: vcodec: using decoder status instead of core work
>     count
>   media: mediatek: vcodec: move core context from device to each
>     instance
>   media: mediatek: vcodec: using empty lat buffer as the last one
> 
>  .../mediatek/vcodec/mtk_vcodec_dec_drv.c      |   1 -
>  .../vcodec/mtk_vcodec_dec_stateless.c         |   2 +-
>  .../platform/mediatek/vcodec/mtk_vcodec_drv.h |   2 -
>  .../vcodec/vdec/vdec_h264_req_multi_if.c      |   6 +-
>  .../vcodec/vdec/vdec_vp9_req_lat_if.c         |   4 +-
>  .../platform/mediatek/vcodec/vdec_msg_queue.c | 105 ++++++++------
> ----
>  .../platform/mediatek/vcodec/vdec_msg_queue.h |  28 ++++-
>  7 files changed, 74 insertions(+), 74 deletions(-)
>