Make AMF encoder utilize all VCNs #11752
Open
+394
−62
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
exeldro
reviewed
Jan 20, 2025
VCNs are HW units which can process encoders in parallel boosting performance if several encoders are run simultaneously. Added controller to balance load of VCNs. Made AMF context and D3D11 device and context shared between all encoder instances. Using KeyedMutex to synchronize copying from texture from OBS core to texture allocated and used in encoder caused submitting a lot of HW fences to all HW queues leading to parallel encode queues waiting for other queue's job's done and hence significant performance degradation. To work this around, encoder's texture is created as shared, it's opened and copy is done on a device used in OBS core, using synchronization on CPU.
nikita-pletnev
force-pushed
the
two_vcn_support
branch
from
b8364e7 to
262a981
Compare
|
Addressed comments above, squashed commits and rebased on the latest master |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Make AMF encoder utilize all VCNs (HW encoding units in AMD GPUs) which are present in used GPU, currently only one VCN is used. Reduce amount of D3D devices being created by making one shared device being used by all AMF encode sessions.
Motivation and Context
When use obs-multi-rtmp plugin to run multiple streams at once this change significantly increases performance allowing to run more streams simultaneously.
How Has This Been Tested?
Run OBS on Windows 10, AMD Radeon RX 6800, and run multiple streams with obs-multi-rtmp plugin
Types of changes
Checklist: