commit 2c1e98d69fabe66a445c774e2795b740b25525c8
author Rubin Xu <rubinxu@google.com> Fri Mar 22 06:17:06 2019 -0700
committer android-build-merger <android-build-merger@google.com> Fri Mar 22 06:17:06 2019 -0700
tree dfd5815a329774d8ba5f5f2d5e191920f4ea544c
parent 5da6d2a0f93b769c47d10df25b8cb5ac2e9f5fdf
parent b640f4e337531b79b9dc0df1b5a3357ae0b868f9

Merge "Remove excess logging in secdiscard"
am: b640f4e337

Change-Id: Ic4509611efc55b0953ccca02fd7730740982728a

FileDeviceUtils.cpp

diff --git a/FileDeviceUtils.cpp b/FileDeviceUtils.cpp
index ce938a9..c745b54 100644
--- a/FileDeviceUtils.cpp
+++ b/FileDeviceUtils.cpp
@@ -61,7 +61,6 @@
         LOG(ERROR) << "Didn't find a mountpoint to match path " << path;
         return "";
     }
-    LOG(DEBUG) << "For path " << path << " block device is " << result;
     return result;
 }
 

secdiscard.cpp

diff --git a/secdiscard.cpp b/secdiscard.cpp
index 2d9dc35..cb2eca9 100644
--- a/secdiscard.cpp
+++ b/secdiscard.cpp
@@ -94,7 +94,6 @@
         }
         set = 0;
         ioctl(fd, F2FS_IOC_SET_PIN_FILE, &set);
-        LOG(DEBUG) << "Discarded: " << target;
     }
     return 0;
 }
@@ -143,9 +142,8 @@
         range[0] = fiemap->fm_extents[i].fe_physical;
         range[1] = fiemap->fm_extents[i].fe_length;
         if (ioctl(fs_fd.get(), BLKSECDISCARD, range) == -1) {
-            PLOG(ERROR) << "Unable to BLKSECDISCARD " << path;
+            // Use zero overwrite as a fallback for BLKSECDISCARD
             if (!overwrite_with_zeros(fs_fd.get(), range[0], range[1])) return false;
-            LOG(DEBUG) << "Used zero overwrite";
         }
     }
     return true;