8000 decavcodec: Increase decoder thread count from (cpu_count/2 + 1) to (… by dashsantosh-mcw · Pull Request #6908 · HandBrake/HandBrake · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

decavcodec: Increase decoder thread count from (cpu_count/2 + 1) to (… #6908

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

decavcodec: Increase decoder thread count from (cpu_count/2 + 1) to (… #6908

wants to merge 1 commit into from

Conversation

dashsantosh-mcw
Copy link
Contributor

Increase decoder thread count from (cpu_count/2 + 1) to (cpu_count + 1) for snapdragon devices

On profiling different modules of Handbrake in Snapdragon Xelite devices, idle times in encoder threads were seen, due to waiting on decoder threads. Increasing the decoder threads resulted in lesser idle times in encoder threads as well as higher performance in terms of FPS overall

Results:

VERY FAST 1080P PRESET

Decoder Encoder FPS ( Decoder Threads = CPU_COUNT/2 + 1) FPS (DecoderThreads = CPU_COUNT + 1)
h264 x264_10bit 98.12 104.89
x265_10bit 74.57 78.02
svt_av1_10bit 68.52 70.84
mf_h264 119.09 151.61
mf_h265 119.26 151.82
hevc x264_10bit 82.47 96.62
x265_10bit 66.91 73.68
svt_av1_10bit 62.37 66.06
mf_h264 91.94 123.25
mf_h265 92.43 122.90

VERY FAST 720P PRESET

Decoder Encoder FPS (Decoder Threads = CPU_COUNT/2 + 1) FPS (Decoder Threads = CPU_COUNT + 1)
h264 x264_10bit 110.99 129.35
x265_10bit 96.72 107.45
svt_av1_10bit 85.53 93.36
mf_h264 121.15 155.73
mf_h265 120.55 155.56
hevc x264_10bit 89.28 111.86
x265_10bit 82.02 97.03
svt_av1_10bit 73.93 85.99
mf_h264 94.34 123.34
mf_h265 94.29 123.86

Tested on:

  • Windows 10+ (via MinGW)
  • macOS 10.13+
  • Ubuntu Linux

@dashsantosh-mcw
Copy link
Contributor Author

Ping @galad87 @sr55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dashsantosh-mcw @Harshitha-Suresh1410
0