[FFmpeg-devel] [PATCH] arm: only enable setend on ARMv6

Michael Niedermayer michaelni at gmx.at
Fri Jun 5 00:47:57 CEST 2015


On Thu, Jun 04, 2015 at 11:17:45PM +0200, Andreas Cadhalpun wrote:
> Without this check it causes SIGILL crashes on ARMv5.
> 
> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
> ---
>  libavutil/arm/cpu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavutil/arm/cpu.c b/libavutil/arm/cpu.c
> index f1683e8..ac42a05 100644
> --- a/libavutil/arm/cpu.c
> +++ b/libavutil/arm/cpu.c
> @@ -128,7 +128,7 @@ int ff_get_cpu_flags_arm(void)
>         trickle down. */
>      if (flags & (AV_CPU_FLAG_VFPV3 | AV_CPU_FLAG_NEON))
>          flags |= AV_CPU_FLAG_ARMV6T2;
> -    else
> +    else if (flags & AV_CPU_FLAG_ARMV6)
>      /* Some functions use the 'setend' instruction which is deprecated on ARMv8
>       * and serializing on some ARMv7 cores. This ensures such functions
>       * are only enabled on ARMv6. */

shouldnt this be after
"- if (flags & AV_CPU_FLAG_ARMV6T2)
        flags |= AV_CPU_FLAG_ARMV6;
"

or (AV_CPU_FLAG_ARMV6T2 | AV_CPU_FLAG_ARMV6)

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150605/52a46899/attachment.asc>


More information about the ffmpeg-devel mailing list