Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Brad Smith <brad-at-comstyle.com@ffmpeg.org>
To: "FFmpeg development discussions and patches"
	<ffmpeg-devel@ffmpeg.org>,
	"Rémi Denis-Courmont" <remi@remlab.net>
Subject: Re: [FFmpeg-devel] aarch64: Implement support for elf_aux_info(3) on FreeBSD and OpenBSD
Date: Sun, 28 Jul 2024 00:37:51 -0400
Message-ID: <dfff33e2-6f98-44dc-a216-c867756383b0@comstyle.com> (raw)
In-Reply-To: <1221B3A6-6931-4E63-A785-D31CE596C448@remlab.net>

On 2024-07-26 7:56 a.m., Rémi Denis-Courmont wrote:
>
> Le 26 juillet 2024 13:58:34 GMT+03:00, Brad Smith <brad-at-comstyle.com@ffmpeg.org> a écrit :
>> aarch64: Implement support for elf_aux_info(3) on FreeBSD and OpenBSD
>>
>> FreeBSD 12.0+, OpenBSD -current and what will be OpenBSD 7.6 support
>> elf_aux_info(3).
>>
>> Signed-off-by: Brad Smith <brad@comstyle.com>
>> ---
>> configure               |  2 ++
>> libavutil/aarch64/cpu.c | 23 ++++++++++++++++++++++-
>> 2 files changed, 24 insertions(+), 1 deletion(-)
>>
>> diff --git a/configure b/configure
>> index f6f5c29fea..e80b549582 100755
>> --- a/configure
>> +++ b/configure
>> @@ -2366,6 +2366,7 @@ SYSTEM_FUNCS="
>>      clock_gettime
>>      closesocket
>>      CommandLineToArgvW
>> +    elf_aux_info
>>      fcntl
>>      getaddrinfo
>>      getauxval
>> @@ -6565,6 +6566,7 @@ check_func_headers mach/mach_time.h mach_absolute_time
>> check_func_headers stdlib.h getenv
>> check_func_headers sys/stat.h lstat
>> check_func_headers sys/auxv.h getauxval
>> +check_func_headers sys/auxv.h elf_aux_info
>> check_func_headers sys/sysctl.h sysctlbyname
>>
>> check_func_headers windows.h GetModuleHandle
>> diff --git a/libavutil/aarch64/cpu.c b/libavutil/aarch64/cpu.c
>> index cfa9306663..05272b4db4 100644
>> --- a/libavutil/aarch64/cpu.c
>> +++ b/libavutil/aarch64/cpu.c
>> @@ -42,6 +42,27 @@ static int detect_flags(void)
>>      return flags;
>> }
>>
>> +#elif (defined(__FreeBSD__) || defined(__OpenBSD__)) && HAVE_ELF_AUX_INFO
>> +#include <stdint.h>
>> +#include <sys/auxv.h>
>> +
>> +static int detect_flags(void)
>> +{
>> +    int flags = 0;
>> +
>> +    unsigned long hwcap = 0;
>> +    elf_aux_info(AT_HWCAP, &hwcap, sizeof hwcap);
>> +    unsigned long hwcap2 = 0;
>> +    elf_aux_info(AT_HWCAP2, &hwcap2, sizeof hwcap2);
>> +
>> +    if (hwcap & HWCAP_ASIMDDP)
>> +        flags |= AV_CPU_FLAG_DOTPROD;
>> +    if (hwcap2 & HWCAP2_I8MM)
>> +        flags |= AV_CPU_FLAG_I8MM;
>> +
>> +    return flags;
>> +}
>> +
> Can't getauxval() be implemented with elf_aux_info(), or vice versa? It seems that otherwise the code should be identical to that from Linux.

QEMU has qemu_getauxval() for example as a wrapper. I will be using this 
elsewhere for arm, ppc and riscv.

I could split this up, but I am not sure where to place such a function.

>> #elif defined(__APPLE__) && HAVE_SYSCTLBYNAME
>> #include <sys/sysctl.h>
>>
>> @@ -65,7 +86,7 @@ static int detect_flags(void)
>>      return flags;
>> }
>>
>> -#elif defined(__OpenBSD__)
>> +#elif defined(__OpenBSD__) && !HAVE_ELF_AUX_INFO
>> #include <machine/armreg.h>
>> #include <machine/cpu.h>
>> #include <sys/types.h>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

  reply	other threads:[~2024-07-28  4:38 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-26 10:58 Brad Smith
2024-07-26 11:56 ` Rémi Denis-Courmont
2024-07-28  4:37   ` Brad Smith [this message]
2024-07-28  7:29     ` Rémi Denis-Courmont
2024-07-28 11:41       ` Martin Storsjö

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=dfff33e2-6f98-44dc-a216-c867756383b0@comstyle.com \
    --to=brad-at-comstyle.com@ffmpeg.org \
    --cc=ffmpeg-devel@ffmpeg.org \
    --cc=remi@remlab.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git