> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of
> Chen, Wenbin
> Sent: Wednesday, November 10, 2021 4:03 AM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH 1/4] libavutil/hwcontext_d3d11va:
> Add nb_surfaces to AVD3D11VAFramesContext
> 
> > Adding nb_surfaces in AVD3D11VAFramesContext in the end of the
> structure
> > to support flexible size of this arrays and align to
> > AVDXVA2FramesContext and AVVAAPIFramesContext.

There is no flexibility in pool size for D3D11 frames contexts. The
surface count is always identical to initial_pool_size. There's no 
point in exposing it. nb_surfaces could even be removed here.

Also, this change doesn't align with AVVAAPIFramesContext.
In fact not even AVDXVA2FramesContext aligns with AVVAAPIFramesContext.
Both have an nb_surfaces field, but they have different semantics.

The corresponding field to AVVAAPIFramesContext->nb_surfaces is 
DXVA2FramesContext->nb_surfaces_used, not AVDXVA2FramesContext->nb_surfaces.

Kind regards,
softworkz





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

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

Reply via email to