diff options
author | Martin Storsjö <martin@martin.st> | 2024-02-14 23:00:54 +0200 |
---|---|---|
committer | Martin Storsjö <martin@martin.st> | 2024-03-04 10:13:31 +0200 |
commit | e30369bc1c683aeab6ea74bc37b4ae77b03f79b5 (patch) | |
tree | 95b590ef4d1b86d3444e0890a298a3f735d9d13f /fftools/cmdutils.h | |
parent | 4ed0fea0fa6189a69f549f4c40a6b51c155ef584 (diff) | |
download | ffmpeg-e30369bc1c683aeab6ea74bc37b4ae77b03f79b5.tar.gz |
aarch64: Use regular hwcaps flags instead of HWCAP_CPUID for CPU feature detection on Linux
This makes the code much simpler (especially for adding support
for other instruction set extensions), avoids needing inline
assembly for this feature, and generally is more of the canonical
way to do this.
The CPU feature detection was added in
493fcde50a84cb23854335bcb0e55c6f383d55db, using HWCAP_CPUID.
The argument for using that, was that HWCAP_CPUID was added much
earlier in the kernel (in Linux v4.11), while the HWCAP flags for
individual features always come later. This allows detecting support
for new CPU extensions before the kernel exposes information about
them via hwcap flags.
However in practice, there's probably quite little advantage in this.
E.g. HWCAP2_I8MM was added in Linux v5.10 - long after HWCAP_CPUID,
but there's probably very little practical cases where one would
run a kernel older than that on a CPU that supports those instructions.
Additionally, we provide our own definitions of the flag values to
check (as they are fixed constants anyway), with names not conflicting
with the ones from system headers. This reduces the number of ifdefs
needed, and allows detecting those features even if building with
userland headers that are lacking the definitions of those flags.
Also, slightly older versions of QEMU, e.g. 6.2 in Ubuntu 22.04,
do expose support for these features via HWCAP flags, but the
emulated cpuid registers are missing the bits for exposing e.g. I8MM.
(This issue is fixed in later versions of QEMU though.)
Signed-off-by: Martin Storsjö <martin@martin.st>
Diffstat (limited to 'fftools/cmdutils.h')
0 files changed, 0 insertions, 0 deletions