[FFmpeg-devel] [PATCH] avcodec/d3d12va_decode: don't change the resource state if the referenced frame is the same as the current frame

[FFmpeg-devel] [PATCH] avcodec/d3d12va_decode: don't change the resource state if the referenced frame is the same as the current frame

[Wu Jianhua]

[-- Attachment #1: Type: text/plain, Size: 1009 bytes --]

avcodec/d3d12va_decode: don't change the resource state if the referenced frame is the same as the current frame

 This commit removes the follow warning and error:

    D3D12 WARNING: ID3D12CommandList::ResourceBarrier: Called on the same subresource(s) of
    Resource(0x000002236E0E00D0:'Unnamed ID3D12Resource Object') in separate Barrier Descs
    which is inefficient and likely unintentional. Desc[0] and Desc[1] on (subresource :
    4294967295). [RESOURCE_MANIPULATION WARNING #1008: RESOURCE_BARRIER_DUPLICATE_SUBRESOURCE_TRANSITIONS]

    D3D12 ERROR: ID3D12CommandList::ResourceBarrier: Before state (0x0: D3D12_RESOURCE_STATE_[COMMON|PRESENT])
    of resource (0x000002236E0E00D0:'Unnamed ID3D12Resource Object') (subresource: 0) specified
    by transition barrier does not match with the state (0x20000: D3D12_RESOURCE_STATE_VIDEO_DECODE_WRITE)
    specified in the previous call to ResourceBarrier [RESOURCE_MANIPULATION ERROR #527:
    RESOURCE_BARRIER_BEFORE_AFTER_MISMATCH]

Patch attached

[-- Attachment #2: 0001-avcodec-d3d12va_decode-don-t-change-the-resource-sta.patch --]
[-- Type: application/octet-stream, Size: 3824 bytes --]

[-- Attachment #3: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avcodec/d3d12va_decode: don't change the resource state if the referenced frame is the same as the current frame

[Wu, Tong1]

>From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Wu
>Jianhua
>Sent: Tuesday, December 26, 2023 9:21 PM
>To: FFmpeg development discussions and patches <ffmpeg-
>devel@ffmpeg.org>
>Subject: [FFmpeg-devel] [PATCH] avcodec/d3d12va_decode: don't change the
>resource state if the referenced frame is the same as the current frame
>
>avcodec/d3d12va_decode: don't change the resource state if the referenced
>frame is the same as the current frame
>
> This commit removes the follow warning and error:
>
>    D3D12 WARNING: ID3D12CommandList::ResourceBarrier: Called on the
>same subresource(s) of
>    Resource(0x000002236E0E00D0:'Unnamed ID3D12Resource Object') in
>separate Barrier Descs
>    which is inefficient and likely unintentional. Desc[0] and Desc[1] on
>(subresource :
>    4294967295). [RESOURCE_MANIPULATION WARNING #1008:
>RESOURCE_BARRIER_DUPLICATE_SUBRESOURCE_TRANSITIONS]
>
>    D3D12 ERROR: ID3D12CommandList::ResourceBarrier: Before state (0x0:
>D3D12_RESOURCE_STATE_[COMMON|PRESENT])
>    of resource (0x000002236E0E00D0:'Unnamed ID3D12Resource Object')
>(subresource: 0) specified
>    by transition barrier does not match with the state (0x20000:
>D3D12_RESOURCE_STATE_VIDEO_DECODE_WRITE)
>    specified in the previous call to ResourceBarrier [RESOURCE_MANIPULATION
>ERROR #527:
>    RESOURCE_BARRIER_BEFORE_AFTER_MISMATCH]
>
>Patch attached

Could you please split the function declaration(header) into 2 lines since it's a little bit long?

Thx,
Tong

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] 回复: [PATCH] avcodec/d3d12va_decode: don't change the resource state if the referenced frame is the same as the current frame

[Wu Jianhua]

> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Wu, Tong1 <tong1.wu-at-intel.com@ffmpeg.org>
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH] avcodec/d3d12va_decode: don't change the resource state if the referenced frame is > the same as the current frame
>
>>From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Wu
>>Jianhua
>>Sent: Tuesday, December 26, 2023 9:21 PM
>>To: FFmpeg development discussions and patches <ffmpeg-
>>devel@ffmpeg.org>
>>Subject: [FFmpeg-devel] [PATCH] avcodec/d3d12va_decode: don't change the
>>resource state if the referenced frame is the same as the current frame
>>
>>avcodec/d3d12va_decode: don't change the resource state if the referenced
>>frame is the same as the current frame
>>
>> This commit removes the follow warning and error:
>>
>>    D3D12 WARNING: ID3D12CommandList::ResourceBarrier: Called on the
>>same subresource(s) of
>>    Resource(0x000002236E0E00D0:'Unnamed ID3D12Resource Object') in
>>separate Barrier Descs
>>    which is inefficient and likely unintentional. Desc[0] and Desc[1] on
>>(subresource :
>>    4294967295). [RESOURCE_MANIPULATION WARNING #1008:
>>RESOURCE_BARRIER_DUPLICATE_SUBRESOURCE_TRANSITIONS]
>>
>>    D3D12 ERROR: ID3D12CommandList::ResourceBarrier: Before state (0x0:
>>D3D12_RESOURCE_STATE_[COMMON|PRESENT])
>>    of resource (0x000002236E0E00D0:'Unnamed ID3D12Resource Object')
>>(subresource: 0) specified
>>    by transition barrier does not match with the state (0x20000:
>>D3D12_RESOURCE_STATE_VIDEO_DECODE_WRITE)
>>    specified in the previous call to ResourceBarrier [RESOURCE_MANIPULATION
>>ERROR #527:
>>    RESOURCE_BARRIER_BEFORE_AFTER_MISMATCH]
>>
>>Patch attached
>
> Could you please split the function declaration(header) into 2 lines since it's a little bit long?
>
> Thx,
> Tong

Sure. Will do in the v2.

Thanks,
Jianhua
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".