[FFmpeg-devel] [PATCH] libavutil/ppc/cpu.c: check that AT_HWCAP2 is defined

Sean McGovern gseanmcg at gmail.com
Wed Oct 18 20:18:54 EEST 2023


On Sat, Oct 14, 2023, 23:27 Sean McGovern <gseanmcg at gmail.com> wrote:

> It was not introduced until glibc 2.18.
> ---
> This should fix the ppc32 FATE node.
> ---
>  libavutil/ppc/cpu.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/libavutil/ppc/cpu.c b/libavutil/ppc/cpu.c
> index 96b491c716..bc8bb5f47c 100644
> --- a/libavutil/ppc/cpu.c
> +++ b/libavutil/ppc/cpu.c
> @@ -95,12 +95,15 @@ int ff_get_cpu_flags_ppc(void)
>  #endif
>                  if (ret & AV_CPU_FLAG_VSX)
>                      av_assert0(ret & AV_CPU_FLAG_ALTIVEC);
> -            } else if (buf[i] == AT_HWCAP2) {
> +            }
> +#ifdef AT_HWCAP2 /* not introduced until glibc 2.18 */
> +            else if (buf[i] == AT_HWCAP2) {
>  #ifdef PPC_FEATURE2_ARCH_2_07
>                  if (buf[i + 1] & PPC_FEATURE2_ARCH_2_07)
>                      ret |= AV_CPU_FLAG_POWER8;
>  #endif
>              }
> +#endif /* AT_HWCAP2 */
>          }
>      }
>
> --
> 2.39.2
>

Ping review.

Alternatively, can the ppc32 FATE nodes be upgraded to a distribution
release with glibc 2.18 or higher?
Can I assist with that somehow?

Thanks,
Sean McGovern


More information about the ffmpeg-devel mailing list