On Thu, Dec 3, 2015 at 5:21 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Thu, Dec 3, 2015 at 4:55 AM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > >> + if (s->pix_fmt == AV_PIX_FMT_YUV420P) { >> > +#if CONFIG_VP9_DXVA2_HWACCEL >> + *fmtp++ = AV_PIX_FMT_DXVA2_VLD; >> +#endif >> +#if CONFIG_VP9_D3D11VA_HWACCEL >> + *fmtp++ = AV_PIX_FMT_D3D11VA_VLD; >> +#endif >> + } > > > So ... this suggests that hwaccels will ever only support 420, 8bpp. That > may currently be true, but is that likely to hold forever? What happens if > we just do this regardless of pixfmt and let the hwaccel figure out if the > hw supports 420 or something else? >
At the very least the DXVA2 hwaccel spec is limited to profile 0, so maybe it should even check the profile on top of this. In general, there is some disagreement if other chroma formats should use other hwaccel pixfmts, as the calling code can otherwise get confused, as it may need special handling to handle other chroma formats. However, since no hwaccel supports anything else yet (except hevc, which can do 420 10-bit, but I didn't push that to ffmpeg yet), it was not a real problem yet. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel