| Index: src/platform-linux.cc
|
| diff --git a/src/platform-linux.cc b/src/platform-linux.cc
|
| index 7efb25de94f790aa877b0f702b98f2b178e73e3c..46c74b017ff0181bd8b2b6ccedd5f51def079cd0 100644
|
| --- a/src/platform-linux.cc
|
| +++ b/src/platform-linux.cc
|
| @@ -134,7 +134,9 @@ static bool CPUInfoContainsString(const char * search_string) {
|
| }
|
|
|
| bool OS::ArmCpuHasFeature(CpuFeature feature) {
|
| - const char* search_string = NULL;
|
| + const int max_items = 2;
|
| + const char* search_strings[max_items] = { NULL, NULL };
|
| + int search_items = 0;
|
| // Simple detection of VFP at runtime for Linux.
|
| // It is based on /proc/cpuinfo, which reveals hardware configuration
|
| // to user-space applications. According to ARM (mid 2009), no similar
|
| @@ -142,26 +144,25 @@ bool OS::ArmCpuHasFeature(CpuFeature feature) {
|
| // so it's up to individual OSes to provide such.
|
| switch (feature) {
|
| case VFP3:
|
| - search_string = "vfpv3";
|
| + search_strings[0] = "vfpv3";
|
| + // Some old kernels will report vfp for A8, not vfpv3, so we check for
|
| + // A8 explicitely. The cpuinfo file report the CPU Part which for Cortex
|
| + // A8 is 0xc08.
|
| + search_strings[1] = "0xc08";
|
| + search_items = 2;
|
| + ASSERT(search_items <= max_items);
|
| break;
|
| case ARMv7:
|
| - search_string = "ARMv7";
|
| + search_strings[0] = "ARMv7" ;
|
| + search_items = 1;
|
| + ASSERT(search_items <= max_items);
|
| break;
|
| default:
|
| UNREACHABLE();
|
| }
|
|
|
| - if (CPUInfoContainsString(search_string)) {
|
| - return true;
|
| - }
|
| -
|
| - if (feature == VFP3) {
|
| - // Some old kernels will report vfp not vfpv3. Here we make a last attempt
|
| - // to detect vfpv3 by checking for vfp *and* neon, since neon is only
|
| - // available on architectures with vfpv3.
|
| - // Checking neon on its own is not enough as it is possible to have neon
|
| - // without vfp.
|
| - if (CPUInfoContainsString("vfp") && CPUInfoContainsString("neon")) {
|
| + for (int i = 0; i < search_items; ++i) {
|
| + if (CPUInfoContainsString(search_strings[i])) {
|
| return true;
|
| }
|
| }
|
|
|