On Tue, 2022-07-19 at 07:18 +0000, Xiang, Haihao wrote:
> On Mon, 2022-07-18 at 15:02 +0200, Anton Khirnov wrote:
> > Quoting Xiang, Haihao (2022-07-12 08:27:32)
> > > +static int qsv_va_update_config(void *ctx, mfxHDL handle, mfxConfig cfg)
> > > +{
> > > +#if CONFIG_VAAPI
> > > +#if VA_CHECK_VERSION(1, 5, 0)
> > > +#define LOCAL_VADISPLAYPCIID VADisplayPCIID
> > > +#else
> > > +#define LOCAL_VADISPLAYPCIID 21
> > > +#endif
> > > + mfxStatus sts;
> > > + VADisplay dpy = handle;
> > > + VAStatus vas;
> > > + VADisplayAttribute attr = {
> > > + .type = LOCAL_VADISPLAYPCIID
> > > + };
> > > + mfxVariant impl_value;
> > > +
> > > + vas = vaGetDisplayAttributes(dpy, &attr, 1);
> > > + if (vas == VA_STATUS_SUCCESS && attr.flags !=
> > > VA_DISPLAY_ATTRIB_NOT_SUPPORTED) {
> > > + impl_value.Type = MFX_VARIANT_TYPE_U16;
> > > + impl_value.Data.U16 = (attr.value & 0xFFFF);
> > > + sts = MFXSetConfigFilterProperty(cfg,
> > > + (const mfxU8
> > > *)"mfxExtendedDeviceId.DeviceID", impl_value);
> > > + if (sts != MFX_ERR_NONE) {
> > > + av_log(ctx, AV_LOG_ERROR, "Error adding a MFX configuration"
> > > + "DeviceID property: %d.\n", sts);
> > > + goto fail;
> > > + }
> > > + } else
> > > + av_log(ctx, AV_LOG_WARNING, "Cannot get device id from the
> > > driver,
> > > the default "
> > > + "MFX implementation will be loaded for this device. Please
> > > consider to "
> > > + "upgrade the driver to support VAAPI 1.5.0. \n");
> >
> > I would still prefer to fail here. The user requested a specific device,
> > disregarding that request is evil.
>
> Thanks for the comment. There is only one available device for most users, so
> the default one and the given one from user should be the same, otherwise it
> won't work. I don't want to make them in trouble if they don't have a driver
> to
> support the new interface. However I agree with you it is a little evil to
> ignore the request. I'll update the patch to return error here.
>
Fixed in v11,
Thanks
Haihao
_______________________________________________
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".
