[FFmpeg-devel] [PATCH 3/3] ffmpeg_dxva2: add a profile check for hevc

Hendrik Leppkes h.leppkes at gmail.com
Sat Jan 30 17:03:39 CET 2016


On Thu, Jan 28, 2016 at 3:52 PM, Michael Niedermayer
<michael at niedermayer.cc> wrote:
> On Thu, Jan 28, 2016 at 12:58:27PM +0100, Hendrik Leppkes wrote:
>> ---
>>  ffmpeg_dxva2.c | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/ffmpeg_dxva2.c b/ffmpeg_dxva2.c
>> index 1b62bb9..35bedec 100644
>> --- a/ffmpeg_dxva2.c
>> +++ b/ffmpeg_dxva2.c
>> @@ -637,6 +637,12 @@ int dxva2_init(AVCodecContext *s)
>>          return AVERROR(EINVAL);
>>      }
>>
>> +    if (s->codec_id == AV_CODEC_ID_HEVC &&
>> +        s->profile != FF_PROFILE_HEVC_MAIN && s->profile != FF_PROFILE_HEVC_MAIN_10) {
>> +        av_log(NULL, loglevel, "Unsupported HEVC profile for DXVA2 HWAccel: %d\n", s->profile);
>                   ^^^^
> this should be some context if possible
>
> otherwise LGTM
>

The entire file doesn't use contexts for some reason, I would rather
keep it like this in this patch and then send a follow up to make all
the log commands in that file use contexts.

- Hendrik


More information about the ffmpeg-devel mailing list