Re: [FFmpeg-devel] [PATCH] avcodec/mediacodec_wrapper: fix false positives in swdec blacklist

[Matthieu Bouron]

On Wed, Mar 07, 2018 at 01:35:20AM +0200, Jan Ekström wrote:
> On Wed, Mar 7, 2018 at 12:19 AM, Stefan _  wrote:
> > Hi,
> >
> > attached patch fixes an issue with the previous mediacodec patch.
> >
> 
> LGTM.
> 
> `strstr(name, "OMX.SEC") && strstr(name, ".sw.")`
> 
> ..will most likely have less false positives as OMX.SEC seems to be
> utilized for HW decoders on some Exynos SoCs.

Patch applied.

Thanks,

-- 
Matthieu B.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/mediacodec_wrapper: fix false positives in swdec blacklist

[Jan Ekström]

On Wed, Mar 7, 2018 at 12:19 AM, Stefan _  wrote:
> Hi,
>
> attached patch fixes an issue with the previous mediacodec patch.
>

LGTM.

`strstr(name, "OMX.SEC") && strstr(name, ".sw.")`

..will most likely have less false positives as OMX.SEC seems to be
utilized for HW decoders on some Exynos SoCs.

Jan
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avcodec/mediacodec_wrapper: fix false positives in swdec blacklist

[Stefan _]

Hi,

attached patch fixes an issue with the previous mediacodec patch.

From b6a8721679483900b4f824504fdb1f7944ec268f Mon Sep 17 00:00:00 2001
From: sfan5 
Date: Tue, 6 Mar 2018 23:14:09 +0100
Subject: [PATCH] avcodec/mediacodec_wrapper: fix false positives in swdec
 blacklist

'OMX.SEC.avc.dec' is a valid hardware decoder, while the decoders
we seek to blacklist all match 'OMX.SEC.*.sw.dec'.
---
 libavcodec/mediacodec_wrapper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/mediacodec_wrapper.c b/libavcodec/mediacodec_wrapper.c
index 8381128..11d7f66 100644
--- a/libavcodec/mediacodec_wrapper.c
+++ b/libavcodec/mediacodec_wrapper.c
@@ -473,7 +473,7 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
 if (
 strstr(name, "OMX.google") ||
 strstr(name, "OMX.ffmpeg") ||
-strstr(name, "OMX.SEC") ||
+(strstr(name, "OMX.SEC") && strstr(name, ".sw.")) ||
 !strcmp(name, "OMX.qcom.video.decoder.hevcswvdec")) {
 av_freep();
 goto done_with_type;
-- 
2.16.2

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel