[FFmpeg-devel] [PATCH v2] lavc/qsv: adding DX11 support

[Max Dmitrichenko]

On Wed, Mar 11, 2020 at 10:58 PM Soft Works <softworkz at hotmail.com> wrote:

>
>
> > -----Original Message-----
> > From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> > Max Dmitrichenko
> > Sent: Wednesday, March 11, 2020 12:49 PM
> > To: FFmpeg development discussions and patches <ffmpeg-
> > devel at ffmpeg.org>
> > Subject: Re: [FFmpeg-devel] [PATCH v2] lavc/qsv: adding DX11 support
> >
> > On Tue, Mar 10, 2020 at 10:36 PM Soft Works <softworkz at hotmail.com>
> > wrote:
> >
> > >
> > > > -----Original Message-----
> > > > From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> > > > Artem Galin
> > > > Sent: Tuesday, March 10, 2020 5:10 PM
> > > > To: FFmpeg development discussions and patches <ffmpeg-
> > > > devel at ffmpeg.org>
> > > > Subject: Re: [FFmpeg-devel] [PATCH v2] lavc/qsv: adding DX11 support
> > > >
> > > > Any comments on updated patch by link below:
> > > >
> > https://patchwork.ffmpeg.org/project/ffmpeg/patch/20200306130954.893
> > > > 8-
> > > > 1-artem.galin at gmail.com/
> > >
> > > I don't see any comments there.
> > >
> > >
> > I can help commenting below:
> > > Allow selection of a specific DXGI adapter
> > QSV can make use only one Intel's adapter for currently available HW, if
> > changed in the future - this comment should be addressed.
>
> 1. It wil change this year because Intel is about to release discrete gpu
> boards
>

we have plans to address it
when appropriate


> 2. Users can have other GPU boars installed so Intel's GPU is not always
> the first
> DXGI/D3D11 adapter
>
>
see d3d11va_device_find_adapter_by_vendor_id() related changes


> > > Change filters to support mfxHandlePair
> > Filters should come in next patches,
> > This is initial implementation which will establish base for later
> filters
> > change(s).
>
> I don't think it is a good idea to submit a path that will break filtering
> functionality.
>
>
it is not possible to break DX11 filtering functionality
when DX11 support doesnt exist at the first place.


> The submitted patch is far away from being complete.  I know it
> because I've already done a full implementation of D3D11
> support for QuickSync (but not yet submitted).
>
>
DX11 support has certain and strong benefits.

we have this patch for DX11 support already
no clear understanding if and when any other implementation(s) to expect.

softworkz
>

regards
Max
(max.dmitrichenko at intel.com)