From 954b42791160bd60c244d4d129c94b7aa39f74d4 Mon Sep 17 00:00:00 2001 From: Ocean Chen Date: Tue, 14 Sep 2021 09:13:16 +0000 Subject: [PATCH] Pixelstats: add more UFS error count paths The UFS resume flow also increase the reset count. It makes the reset count cannot represent the real error count. So we use other count instead of reset count Bug: 199459367 Test: run storage-qa pixel/022 Change-Id: I7eabf238bb0963af9430b47c9ffa92bb3ad3b6fc --- pixelstats/service.cpp | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/pixelstats/service.cpp b/pixelstats/service.cpp index 2bc19b85..4fcf913b 100644 --- a/pixelstats/service.cpp +++ b/pixelstats/service.cpp @@ -26,6 +26,7 @@ using android::hardware::google::pixel::SysfsCollector; using android::hardware::google::pixel::UeventListener; #define UFSHC_PATH(filename) "/dev/sys/block/bootdevice/" #filename +#define UFS_ERR_PATH(err_type) UFSHC_PATH(err_stats/) #err_type const struct SysfsCollector::SysfsPaths sysfs_paths = { .SlowioReadCntPath = UFSHC_PATH(slowio_read_cnt), .SlowioWriteCntPath = UFSHC_PATH(slowio_write_cnt), @@ -34,10 +35,18 @@ const struct SysfsCollector::SysfsPaths sysfs_paths = { .UFSLifetimeA = UFSHC_PATH(health_descriptor/life_time_estimation_a), .UFSLifetimeB = UFSHC_PATH(health_descriptor/life_time_estimation_b), .UFSLifetimeC = UFSHC_PATH(health_descriptor/life_time_estimation_c), - .UFSHostResetPath = UFSHC_PATH(err_stats/dev_reset_count), .F2fsStatsPath = "/sys/fs/f2fs/", .ImpedancePath = "/sys/devices/platform/audiometrics/speaker_impedance", .CodecPath = "/sys/devices/platform/audiometrics/codec_state", + .UFSErrStatsPath = { + UFS_ERR_PATH(pa_err_count), + UFS_ERR_PATH(dl_err_count), + UFS_ERR_PATH(nl_err_count), + UFS_ERR_PATH(tl_err_count), + UFS_ERR_PATH(dme_err_count), + UFS_ERR_PATH(fatal_err_count), + UFS_ERR_PATH(auto_hibern8_err_count) + }, }; const char *const kAudioUevent = "/devices/virtual/amcs/amcs";