PerfProvider: Run a randomly selected perf command

chrome/browser/metrics/perf/perf_provider_chromeos_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/metrics/perf/perf_provider_chromeos.h"
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/test/test_simple_task_runner.h"
 #include "base/thread_task_runner_handle.h"
 #include "chrome/browser/metrics/perf/windowed_incognito_observer.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/login/login_state.h"
 #include "components/metrics/proto/sampled_profile.pb.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace metrics {
 
 namespace {
 
 // Return values for perf.
 const int kPerfSuccess = 0;
 const int kPerfFailure = 1;
 
+const char* kPerfRecordCyclesCmd =

[Alexei Svitkine (slow), 2015/10/14 15:53:58]

Nit:

const char kPerfRecordCyclesCmd[] =

Same for the other ones.

[dhsharp, 2015/10/14 18:07:04]

Sorry, missed it. Done.

+  "perf record -a -e cycles -c 1000003";
+const char* kPerfRecordCallgraphCmd =
+  "perf record -a -e cycles -g -c 4000037";
+const char* kPerfRecordLBRCmd =
+  "perf record -a -e r2c4 -b -c 20011";
+const char* kPerfStatMemoryBandwidthCmd =
+  "perf stat -a -e cycles -e instructions "
+  "-e uncore_imc/data_reads/ -e uncore_imc/data_writes/ "
+  "-e cpu/event=0xD0,umask=0x11,name=MEM_UOPS_RETIRED-STLB_MISS_LOADS/ "
+  "-e cpu/event=0xD0,umask=0x12,name=MEM_UOPS_RETIRED-STLB_MISS_STORES/";
+
 // Converts a protobuf to serialized format as a byte vector.
 std::vector<uint8_t> SerializeMessageToVector(
     const google::protobuf::MessageLite& message) {
   std::vector<uint8_t> result(message.ByteSize());
   message.SerializeToArray(result.data(), result.size());
   return result;
 }
 
 // Returns an example PerfDataProto. The contents don't have to make sense. They
 // just need to constitute a semantically valid protobuf.
@@ skipping 414 matching lines @@
   EXPECT_EQ(SampledProfile::RESUME_FROM_SUSPEND, profile3.trigger_event());
   EXPECT_TRUE(profile3.has_ms_after_login());
   EXPECT_EQ(60000, profile3.suspend_duration_ms());
   EXPECT_EQ(1500, profile3.ms_after_resume());
   ASSERT_TRUE(profile3.has_perf_data());
   EXPECT_FALSE(profile3.has_perf_stat());
   EXPECT_EQ(SerializeMessageToVector(perf_data_proto_),
             SerializeMessageToVector(profile3.perf_data()));
 }
 
+TEST_F(PerfProviderTest, DefaultCommandsBasedOnUarch_IvyBridge) {
+  std::vector<RandomSelector::WeightAndValue> cmds;
+  CPUIdentity cpuid;
+  cpuid.arch = "x86_64";
+  cpuid.vendor = "GenuineIntel";
+  cpuid.family = 0x06;
+  cpuid.model = 0x3a;  // IvyBridge
+  cpuid.model_name = "";
+  cmds = internal::GetDefaultCommandsForCpu(cpuid);
+  ASSERT_GE(cmds.size(), 2UL);
+  EXPECT_EQ(cmds[0].value, kPerfRecordCyclesCmd);
+  EXPECT_EQ(cmds[1].value, kPerfRecordCallgraphCmd);
+  auto found = std::find_if(
+      cmds.begin(), cmds.end(),
+      [](const RandomSelector::WeightAndValue& cmd) -> bool {
+        return cmd.value == kPerfStatMemoryBandwidthCmd;
+      });
+  EXPECT_NE(cmds.end(), found);
+  found = std::find_if(
+      cmds.begin(), cmds.end(),
+      [](const RandomSelector::WeightAndValue& cmd) -> bool {
+        return cmd.value == kPerfRecordLBRCmd;
+      });
+  EXPECT_NE(cmds.end(), found);
+}
+
+TEST_F(PerfProviderTest, DefaultCommandsBasedOnUarch_SandyBridge) {
+  std::vector<RandomSelector::WeightAndValue> cmds;
+  CPUIdentity cpuid;
+  cpuid.arch = "x86_64";
+  cpuid.vendor = "GenuineIntel";
+  cpuid.family = 0x06;
+  cpuid.model = 0x2a;  // SandyBridge
+  cpuid.model_name = "";
+  cmds = internal::GetDefaultCommandsForCpu(cpuid);
+  ASSERT_GE(cmds.size(), 2UL);
+  EXPECT_EQ(cmds[0].value, kPerfRecordCyclesCmd);
+  EXPECT_EQ(cmds[1].value, kPerfRecordCallgraphCmd);
+  auto found = std::find_if(
+      cmds.begin(), cmds.end(),
+      [](const RandomSelector::WeightAndValue& cmd) -> bool {
+        return cmd.value == kPerfStatMemoryBandwidthCmd;
+      });
+  EXPECT_EQ(cmds.end(), found) << "SandyBridge does not support this command";
+  found = std::find_if(
+      cmds.begin(), cmds.end(),
+      [](const RandomSelector::WeightAndValue& cmd) -> bool {
+        return cmd.value == kPerfRecordLBRCmd;
+      });
+  EXPECT_NE(cmds.end(), found);
+}
+
+TEST_F(PerfProviderTest, DefaultCommandsBasedOnArch_Arm) {
+  std::vector<RandomSelector::WeightAndValue> cmds;
+  CPUIdentity cpuid;
+  cpuid.arch = "armv7l";
+  cpuid.vendor = "";
+  cpuid.family = 0;
+  cpuid.model = 0;
+  cpuid.model_name = "";
+  cmds = internal::GetDefaultCommandsForCpu(cpuid);
+  ASSERT_GE(cmds.size(), 2UL);
+  EXPECT_EQ(cmds[0].value, kPerfRecordCyclesCmd);
+  EXPECT_EQ(cmds[1].value, kPerfRecordCallgraphCmd);
+  const auto found = std::find_if(
+      cmds.begin(), cmds.end(),
+      [](const RandomSelector::WeightAndValue& cmd) -> bool {
+        return cmd.value == kPerfRecordLBRCmd;
+      });
+  EXPECT_EQ(cmds.end(), found) << "ARM does not support this command";
+}
+
+TEST_F(PerfProviderTest, DefaultCommandsBasedOnArch_x86_32) {
+  std::vector<RandomSelector::WeightAndValue> cmds;
+  CPUIdentity cpuid;
+  cpuid.arch = "x86";
+  cpuid.vendor = "GenuineIntel";
+  cpuid.family = 0x06;
+  cpuid.model = 0x2f;  // Westmere
+  cpuid.model_name = "";
+  cmds = internal::GetDefaultCommandsForCpu(cpuid);
+  ASSERT_GE(cmds.size(), 2UL);
+  EXPECT_EQ(cmds[0].value, kPerfRecordCyclesCmd);
+  EXPECT_EQ(cmds[1].value, kPerfRecordCallgraphCmd);
+  auto found = std::find_if(
+      cmds.begin(), cmds.end(),
+      [](const RandomSelector::WeightAndValue& cmd) -> bool {
+        return cmd.value == kPerfStatMemoryBandwidthCmd;
+      });
+  EXPECT_EQ(cmds.end(), found) << "x86_32 does not support this command";
+  found = std::find_if(
+      cmds.begin(), cmds.end(),
+      [](const RandomSelector::WeightAndValue& cmd) -> bool {
+        return cmd.value == kPerfRecordLBRCmd;
+      });
+  EXPECT_EQ(cmds.end(), found) << "x86_32 does not support this command";
+}
+
+TEST_F(PerfProviderTest, DefaultCommandsBasedOnArch_Unknown) {
+  std::vector<RandomSelector::WeightAndValue> cmds;

[Alexei Svitkine (slow), 2015/10/14 15:53:58]

Nit: Merge with the assignment on line 580.

Do this for the other tests too.

[dhsharp, 2015/10/14 18:07:04]

Done.

+  CPUIdentity cpuid;
+  cpuid.arch = "nonsense";
+  cpuid.vendor = "";
+  cpuid.family = 0;
+  cpuid.model = 0;
+  cpuid.model_name = "";
+  cmds = internal::GetDefaultCommandsForCpu(cpuid);
+  EXPECT_EQ(1UL, cmds.size());
+  EXPECT_EQ(cmds[0].value, kPerfRecordCyclesCmd);
+}
+
 }  // namespace metrics