Bug 1301627 - Change the name of commented variables when calling perf_event_open. r=erahm
authorKirti Singla <singlakirti3@gmail.com>
Sun, 23 Oct 2016 10:10:00 -0700
changeset 319144 93cdd187a2535349195fd5d4ec71bee88fa04f54
parent 319080 215f9686117673a2c914ed207bc7da9bb8d741ad
child 319145 a99a23306822790c74956a79955579631a3a16f0
push id30862
push usercbook@mozilla.com
push dateMon, 24 Oct 2016 14:55:55 +0000
treeherdermozilla-central@c845bfd0accb [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerserahm
bugs1301627
milestone52.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1301627 - Change the name of commented variables when calling perf_event_open. r=erahm
tools/power/rapl.cpp
--- a/tools/power/rapl.cpp
+++ b/tools/power/rapl.cpp
@@ -513,18 +513,18 @@ public:
 
     struct perf_event_attr attr;
     memset(&attr, 0, sizeof(attr));
     attr.type = aType;
     attr.size = uint32_t(sizeof(attr));
     attr.config = config;
 
     // Measure all processes/threads. The specified CPU doesn't matter.
-    mFd = perf_event_open(&attr, /* pid = */ -1, /* cpu = */ 0,
-                          /* group_fd = */ -1, /* flags = */ 0);
+    mFd = perf_event_open(&attr, /* aPid = */ -1, /* aCpu = */ 0,
+                          /* aGroupFd = */ -1, /* aFlags = */ 0);
     if (mFd < 0) {
       Abort("perf_event_open() failed\n"
             "- Did you run as root (e.g. with |sudo|) or set\n"
             "  /proc/sys/kernel/perf_event_paranoid to 0, as required?");
     }
 
     mPrevTicks = 0;
   }