commit eb8acbf4af7f1ad7ae94337dc3e0d6a38eab4178
author Howard Chen <howardsoc@google.com> Wed May 10 16:32:11 2017 +0800
committer Howard Chen <howardsoc@google.com> Fri May 12 14:31:20 2017 +0800
tree ca246f132ccc7ba439b80363e00e0f166e3a7250
parent 903726f55bf57210bf6523dc909074f46c8c4e5a

Make the binder_lock category optional

The binder_lock event type has been deprecated after binder
driver uses fine grained locking. This patch makes "binder_lock"
an optional category for atrace to support system w/ w/o FGL.

Bug: 37559846
Test: pixel phone with "atrace --async_start binder_lock"
Change-Id: I691efd355e903a26629f896a56a664502e5807f7

cmds/atrace/atrace.cpp

diff --git a/cmds/atrace/atrace.cpp b/cmds/atrace/atrace.cpp
index f8e0ad5..0e68143 100644
--- a/cmds/atrace/atrace.cpp
+++ b/cmds/atrace/atrace.cpp
@@ -183,9 +183,9 @@
         { REQ,      "events/binder/binder_transaction_received/enable" },
     } },
     { "binder_lock", "Binder global lock trace", 0, {
-        { REQ,      "events/binder/binder_lock/enable" },
-        { REQ,      "events/binder/binder_locked/enable" },
-        { REQ,      "events/binder/binder_unlock/enable" },
+        { OPT,      "events/binder/binder_lock/enable" },
+        { OPT,      "events/binder/binder_locked/enable" },
+        { OPT,      "events/binder/binder_unlock/enable" },
     } },
     { "pagecache",  "Page cache", 0, {
         { REQ,      "events/filemap/enable" },
@@ -375,7 +375,7 @@
                     ok = true;
                 }
             } else {
-                ok |= fileIsWritable(path);
+                ok = true;
             }
         }
     }