mythtv doesnt play high bitrate(9/10bit) content smoothly

[slackerlinux85@…]

when playing 10bit content mythtv pauses every few seconds due to no useable video buffers left and then dumps all the frames till the video is done playing back(mythffplay plays the content fine).

9/10bit content also cannot be played with vdpau so a check is needed so it will fall back to software decoder.

[slackerlinux85@…]

small test video

[slackerlinux85@…]

log from mythavtest with -v general, playback

[beirdo]

More samples at ​http://ps-auxw.de/10bit-h264-sample/

[slackerlinux85@…]

done some more debugging it seems the buffers for 9/10bit content is using FF_BUFFER_TYPE_INTERNAL and removing the if (!decode.contains(*it)) check in VideoBuffers::DoneDisplayingFrame? makes the video play smoothly so i think the problem might be with how mythtv handles FF_BUFFER_TYPE_INTERNAL buffers

i still have no idea on how to properly fix it but i hope this information will help

[moores.martin@…]

Just to add my sample here, which appears to have the same issue. Was pointed to this and ticket #10942 by Shawn King on the users list:

​http://www.gossamer-threads.com/lists/mythtv/users/531135

It is commented in #10942 that these two tickets are most likely duplicates, so just to mention that here also

My sample is here, hopefully it might help with testing:

​http://dl.dropbox.com/u/419095/2012-10-20-mythvideo/2012-09-09_16-32-30.AVI

[slackerlinux85@…]

finally got around to finding a fix for this

kinda disappointed that this has been opened for 1.5 years and kinda disappointed it took me this long to find a fix that im happy with releasing

the main issue is that for non-directrendering codecs ffmpeg handles the buffers but the buffers are still added to the decode queue so they are never released(till the buffers full and all frames get dropped) what this does is add a direct rendering setting to videoframe and if set it skips adding the frames to the decode list

i dont know much about the buffering code but this code fixes 9/10bit aswell as the camera samples i have. this is the best way i have found to do it but it feels more like a workaround then a fix and i really dont know enough about mythtv to know if its correct(and theres probably a much better way to fix this)

there still needs to be a software fallback check for 9/10bit content with vdpau. i had a go at that months ago(only about 5 mins) but i ended up with black screen + audio(better then failed to init avsync) unfortunally i the dev macine im using atm is an intel chip so i wont be doing a patch for that(atleast for now)

[slackerlinux85@…]

should fix 9/10bit content and MJPEG content in mythtv

[Jean-Yves Avenard <jyavenard@…>]

In 8f0437c7a66626f29fff18b04c60035cd8fb3b91/mythtv:

Fix playback of 9/10bit h264 videos

The fashion this fix is implemented is a hack more than anything else.
It disable the following H264 profile to be decoded via the GPU:
FF_PROFILE_H264_HIGH_10
FF_PROFILE_H264_HIGH_10_INTRA
FF_PROFILE_H264_HIGH_422
FF_PROFILE_H264_HIGH_422_INTRA
FF_PROFILE_H264_HIGH_444_PREDICTIVE
FF_PROFILE_H264_HIGH_444_INTRA
FF_PROFILE_H264_CAVLC_444

regardless of the GPU acceleration method (VDPAU, VAAPI, VDA, DXVA or CrystalHD). Which at this present time is acceptable enough as no hardware handle those profiles.

A proper implementation, per decoder, would require significant changes and code cleanup to the mess that is currently avformatdecoder.cpp

Fixes #10706

[Jean-Yves Avenard <jyavenard@…>]

In 329b25db98845b2c1ef9e0f393e889864bb1ecc9/mythtv:

Fix playback of 9/10bit h264 videos

The fashion this fix is implemented is a hack more than anything else.
It disable the following H264 profile to be decoded via the GPU:
FF_PROFILE_H264_HIGH_10
FF_PROFILE_H264_HIGH_10_INTRA
FF_PROFILE_H264_HIGH_422
FF_PROFILE_H264_HIGH_422_INTRA
FF_PROFILE_H264_HIGH_444_PREDICTIVE
FF_PROFILE_H264_HIGH_444_INTRA
FF_PROFILE_H264_CAVLC_444

regardless of the GPU acceleration method (VDPAU, VAAPI, VDA, DXVA or CrystalHD). Which at this present time is acceptable enough as no hardware handle those profiles.

A proper implementation, per decoder, would require significant changes and code cleanup to the mess that is currently avformatdecoder.cpp

Fixes #10706
(cherry picked from commit 8f0437c7a66626f29fff18b04c60035cd8fb3b91)