On Sat, 28 Jun 2014, Katerina Barone-Adesi wrote:

Is there a reason not to use getauxval (with AT_HWCAP)? Glibc
implements it, and googling suggests bionic might too.

No reason other than the fact that we don't have code for doing that yet, while we do have code for parsing /proc/cpuinfo, which is quite tested and proven.

Also getauxval isn't officially exposed in bionic (it's not visible in the public headers) until the next major release. In practice it seems to have appeared already in 4.3 though. Therefore the function would have to be fetched dynamically with dlsym.

Thus; we need to keep the code for parsing /proc/cpuinfo in any case (for supporting older versions of android), and extending that code, as in this patch, is quite simple. Adding optional support for loading getauxval dynamically if available, and using that in favor of /proc/cpuinfo, is of course possible as well (patch welcome). The android cpufeatures library actually does that, in addition to parsing the "CPU architecture" field.

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to