[FFmpeg-devel] [PATCH] aarch64: Add OpenBSD runtime detection of dotprod and i8mm using sysctl

Martin Storsjö martin at martin.st
Mon Jun 24 11:15:33 EEST 2024


On Sat, 22 Jun 2024, Brad Smith wrote:

> [PATCH] aarch64: Add OpenBSD runtime detection of dotprod and i8mm using sysctl
>
> Signed-off-by: Brad Smith <brad at comstyle.com>
> ---
> libavutil/aarch64/cpu.c | 35 +++++++++++++++++++++++++++++++++++
> 1 file changed, 35 insertions(+)
>
> diff --git a/libavutil/aarch64/cpu.c b/libavutil/aarch64/cpu.c
> index 196bdaf6b0..40fcc8d1ff 100644
> --- a/libavutil/aarch64/cpu.c
> +++ b/libavutil/aarch64/cpu.c
> @@ -65,6 +65,41 @@ static int detect_flags(void)
>     return flags;
> }
>
> +#elif defined(__OpenBSD__)
> +#include <machine/armreg.h>
> +#include <machine/cpu.h>
> +#include <sys/types.h>
> +#include <sys/sysctl.h>
> +
> +static int detect_flags(void)
> +{
> +    int flags = 0;
> +    int mib[2];
> +    uint64_t isar0;
> +    uint64_t isar1;
> +    size_t len;
> +
> +    mib[0] = CTL_MACHDEP;
> +    mib[1] = CPU_ID_AA64ISAR0;
> +    len = sizeof(isar0);
> +    if (sysctl(mib, 2, &isar0, &len, NULL, 0) != -1) {
> +        if (ID_AA64ISAR0_DP(isar0) >= ID_AA64ISAR0_DP_IMPL)
> +            flags |= AV_CPU_FLAG_DOTPROD;
> +    }
> +
> +    mib[0] = CTL_MACHDEP;
> +    mib[1] = CPU_ID_AA64ISAR1;
> +    len = sizeof(isar1);
> +    if (sysctl(mib, 2, &isar1, &len, NULL, 0) != -1) {
> +#ifdef ID_AA64ISAR1_I8MM_IMPL
> +        if (ID_AA64ISAR1_I8MM(isar1) >= ID_AA64ISAR1_I8MM_IMPL)
> +            flags |= AV_CPU_FLAG_I8MM;
> +#endif
> +    }
> +
> +    return flags;
> +}
> +
> #elif defined(_WIN32)
> #include <windows.h>

This LGTM. Although, in 
https://code.videolan.org/videolan/dav1d/-/merge_requests/1673 you wrapped 
most of this in an #ifdef CPU_ID_AA64ISAR0, so would that be useful here 
too?

Feel free to push either with or without that.

// Martin



More information about the ffmpeg-devel mailing list