metrics/base: log whether drives have seek penalties.

base/sys_info_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/sys_info.h"
 
 #include <windows.h>
+#include <winioctl.h>
 
+#include "base/files/file.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/stringprintf.h"
 #include "base/threading/thread_restrictions.h"
 #include "base/win/windows_version.h"
 
 namespace {
 
 int64 AmountOfMemory(DWORDLONG MEMORYSTATUSEX::* memory_field) {
   MEMORYSTATUSEX memory_info;
   memory_info.dwLength = sizeof(memory_info);
   if (!GlobalMemoryStatusEx(&memory_info)) {
     NOTREACHED();
     return 0;
   }
 
   int64 rv = static_cast<int64>(memory_info.*memory_field);
-  if (rv < 0)
-    rv = kint64max;
-  return rv;
+  return rv < 0 ? kint64max : rv;
 }
 
 }  // namespace
 
 namespace base {
 
 // static
 int SysInfo::NumberOfProcessors() {
   return win::OSInfo::GetInstance()->processors();
 }
 
 // static
 int64 SysInfo::AmountOfPhysicalMemory() {
   return AmountOfMemory(&MEMORYSTATUSEX::ullTotalPhys);
 }
 
 // static
 int64 SysInfo::AmountOfAvailablePhysicalMemory() {
   return AmountOfMemory(&MEMORYSTATUSEX::ullAvailPhys);
 }
 
 // static
 int64 SysInfo::AmountOfVirtualMemory() {
   return 0;
 }
 
 // static
 int64 SysInfo::AmountOfFreeDiskSpace(const FilePath& path) {
-  base::ThreadRestrictions::AssertIOAllowed();
+  ThreadRestrictions::AssertIOAllowed();
 
   ULARGE_INTEGER available, total, free;
-  if (!GetDiskFreeSpaceExW(path.value().c_str(), &available, &total, &free)) {
+  if (!GetDiskFreeSpaceExW(path.value().c_str(), &available, &total, &free))
     return -1;
-  }
+
   int64 rv = static_cast<int64>(available.QuadPart);
-  if (rv < 0)
-    rv = kint64max;
-  return rv;
+  return rv < 0 ? kint64max : rv;
+}
+
+bool SysInfo::HasSeekPenalty(const FilePath& path, bool* has_seek_penalty) {
+  ThreadRestrictions::AssertIOAllowed();
+
+  DCHECK(has_seek_penalty);
+
+  // TODO(dbeam): Vista, XP support.
+  if (!path.IsAbsolute() || win::GetVersion() < win::VERSION_WIN7)

[rvargas (doing something else), 2015/03/13 21:03:09]

I have not heard why allowing callers to use relative paths is a good thing :)

[Dan Beam, 2015/03/13 21:15:51]

Done. (changed to DCHECK)

+    return false;
+
+  std::vector<FilePath::StringType> components;
+  native_path.GetComponents(&components);
+  File drive(FilePath(L"\\\\.\\" + components[0]), File::FLAG_OPEN);

[rvargas (doing something else), 2015/03/13 21:03:09]

Still requires if (!drive.IsValid())...

[Dan Beam, 2015/03/13 21:15:51]

Done.

+
+  STORAGE_PROPERTY_QUERY query;
+  query.QueryType = PropertyStandardQuery;
+  query.PropertyId = StorageDeviceSeekPenaltyProperty;
+
+  DEVICE_SEEK_PENALTY_DESCRIPTOR result;
+
+  success = DeviceIoControl(drive.GetPlatformFile(),
+                            IOCTL_STORAGE_QUERY_PROPERTY,
+                            &query, sizeof(query),
+                            &result, sizeof(result),
+                            &bytes_returned,

[rvargas (doing something else), 2015/03/13 21:03:09]

not defined?

[Dan Beam, 2015/03/13 21:15:51]

whoops, Done.

+                            NULL);
+  if (success == FALSE || bytes_returned < sizeof(result))
+    return false;
+
+  *has_seek_penalty = result.IncursSeekPenalty == TRUE;

[rvargas (doing something else), 2015/03/13 21:03:09]

Comparing against TRUE is not a good idea (in general) because TRUE == 1 and
some functions can return another value even though it is supposed to be a bool.

[Dan Beam, 2015/03/13 21:15:51]

Done.

+  return true;
 }
 
 // static
 std::string SysInfo::OperatingSystemName() {
   return "Windows NT";
 }
 
 // static
 std::string SysInfo::OperatingSystemVersion() {
   win::OSInfo* os_info = win::OSInfo::GetInstance();
@@ skipping 43 matching lines @@
 void SysInfo::OperatingSystemVersionNumbers(int32* major_version,
                                             int32* minor_version,
                                             int32* bugfix_version) {
   win::OSInfo* os_info = win::OSInfo::GetInstance();
   *major_version = os_info->version_number().major;
   *minor_version = os_info->version_number().minor;
   *bugfix_version = 0;
 }
 
 }  // namespace base