diff options
author | mukesh agrawal <quiche@google.com> | 2016-04-11 17:08:19 -0700 |
---|---|---|
committer | mukesh agrawal <quiche@google.com> | 2016-04-12 15:39:22 -0700 |
commit | f8c50b1e8239d843dd01e6dcf956629dad0771c1 (patch) | |
tree | b6baff81561a155fcaaef9550d5c8b944494496e /service | |
parent | e31290e1c2eebf70c5ab22091a9520c7a001161e (diff) |
WifiLogger: make firmware dump conditional on verbose
We want to make firmware memory dump conditional on
verbose logging for it's own sake. But this also
sets the stage for the driver dump feature to be
conditional on verbose. (Driver dump is still in
the works.)
BUG=21661292
BUG=27554392
TEST=compile (more tests to come)
Change-Id: I22a753d785d1023a94f5c2e753714bc918b41fdf
Diffstat (limited to 'service')
-rw-r--r-- | service/java/com/android/server/wifi/WifiLogger.java | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/service/java/com/android/server/wifi/WifiLogger.java b/service/java/com/android/server/wifi/WifiLogger.java index 552049f74..df677b823 100644 --- a/service/java/com/android/server/wifi/WifiLogger.java +++ b/service/java/com/android/server/wifi/WifiLogger.java @@ -180,13 +180,13 @@ class WifiLogger extends BaseWifiLogger { @Override public synchronized void captureBugReportData(int reason) { - BugReport report = captureBugreport(reason, true); + BugReport report = captureBugreport(reason, isVerboseLoggingEnabled()); mLastBugReports.addLast(report); } @Override public synchronized void captureAlertData(int errorCode, byte[] alertData) { - BugReport report = captureBugreport(errorCode, /* captureFWDump = */ true); + BugReport report = captureBugreport(errorCode, isVerboseLoggingEnabled()); report.alertData = alertData; mLastAlerts.addLast(report); } @@ -347,6 +347,10 @@ class WifiLogger extends BaseWifiLogger { } } + private boolean isVerboseLoggingEnabled() { + return mLogLevel > VERBOSE_NORMAL_LOG; + } + private boolean fetchRingBuffers() { if (mRingBuffers != null) return true; |