AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

d3d12: Avoid local allocations for D3D12_RESOURCE_BARRIER on hot paths

Browse Source 
Reviewed-by: Giancarlo Devich <gdevich@microsoft.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18328>
This commit is contained in:
Sil Vilerino
2022-09-02 10:42:14 -04:00
parent 5c2ec8efc8
commit d3f0733d1d
3 changed files with 7 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/gallium/drivers/d3d12/d3d12_video_dec.h
+1
View File
@@ -109,6 +109,7 @@ struct d3d12_video_decoder
ComPtr<ID3D12VideoDecodeCommandList1> m_spDecodeCommandList;
std::vector<D3D12_RESOURCE_BARRIER> m_transitionsBeforeCloseCmdList;
std::vector<D3D12_RESOURCE_BARRIER> m_transitionsStorage;
D3D12_VIDEO_DECODER_DESC m_decoderDesc = {};
D3D12_VIDEO_DECODER_HEAP_DESC m_decoderHeapDesc = {};
src/gallium/drivers/d3d12/d3d12_video_dec_h264.cpp
+3 -5
View File
@@ -113,16 +113,14 @@ d3d12_video_decoder_prepare_current_frame_references_h264(struct d3d12_video_dec
// RefFrameList array of the associated picture parameters structure.For more information, see section 6.2. In
// all cases, when Index7Bits does not contain a valid index, the value is 127.
std::vector<D3D12_RESOURCE_BARRIER>
neededStateTransitions; // Returned by update_entries to perform by the method caller
pD3D12Dec->m_spDPBManager->update_entries(
d3d12_video_decoder_get_current_dxva_picparams<DXVA_PicParams_H264>(pD3D12Dec)->RefFrameList,
neededStateTransitions);
pD3D12Dec->m_transitionsStorage);
pD3D12Dec->m_spDecodeCommandList->ResourceBarrier(neededStateTransitions.size(), neededStateTransitions.data());
pD3D12Dec->m_spDecodeCommandList->ResourceBarrier(pD3D12Dec->m_transitionsStorage.size(), pD3D12Dec->m_transitionsStorage.data());
// Schedule reverse (back to common) transitions before command list closes for current frame
for (auto BarrierDesc : neededStateTransitions) {
for (auto BarrierDesc : pD3D12Dec->m_transitionsStorage) {
std::swap(BarrierDesc.Transition.StateBefore, BarrierDesc.Transition.StateAfter);
pD3D12Dec->m_transitionsBeforeCloseCmdList.push_back(BarrierDesc);
}
src/gallium/drivers/d3d12/d3d12_video_dec_hevc.cpp
+3 -5
View File
@@ -105,16 +105,14 @@ d3d12_video_decoder_prepare_current_frame_references_hevc(struct d3d12_video_dec
// When Index7Bits is used in the CurrPic and RefPicList members of the picture parameters structure, the value directly specifies the DXVA index of an uncompressed surface.
// When Index7Bits is 127 (0x7F), this indicates that it does not contain a valid index.
std::vector<D3D12_RESOURCE_BARRIER>
neededStateTransitions; // Returned by update_entries to perform by the method caller
pD3D12Dec->m_spDPBManager->update_entries(
d3d12_video_decoder_get_current_dxva_picparams<DXVA_PicParams_HEVC>(pD3D12Dec)->RefPicList,
neededStateTransitions);
pD3D12Dec->m_transitionsStorage);
pD3D12Dec->m_spDecodeCommandList->ResourceBarrier(neededStateTransitions.size(), neededStateTransitions.data());
pD3D12Dec->m_spDecodeCommandList->ResourceBarrier(pD3D12Dec->m_transitionsStorage.size(), pD3D12Dec->m_transitionsStorage.data());
// Schedule reverse (back to common) transitions before command list closes for current frame
for (auto BarrierDesc : neededStateTransitions) {
for (auto BarrierDesc : pD3D12Dec->m_transitionsStorage) {
std::swap(BarrierDesc.Transition.StateBefore, BarrierDesc.Transition.StateAfter);
pD3D12Dec->m_transitionsBeforeCloseCmdList.push_back(BarrierDesc);
}