Add cost estimator for SSD (#1480)

ydb-platform · Jan 31, 2024 · a456eb6 · a456eb6
1 parent 458220e
commit a456eb6
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/ydb/core/blobstorage/vdisk/common/blobstorage_cost_tracker.cpp b/ydb/core/blobstorage/vdisk/common/blobstorage_cost_tracker.cpp
@@ -8,6 +8,12 @@ const TDiskOperationCostEstimator TBsCostModelBase::HDDEstimator{
     { 6.089e+06, 8.1 }, // HugeWriteCoefficients
 };
 
+const TDiskOperationCostEstimator TBsCostModelBase::SSDEstimator{
+    { 180000, 3.00 },   // ReadCoefficients
+    { 430, 4.2 },     // WriteCoefficients
+    { 110000, 3.6 },     // HugeWriteCoefficients
+};
+
 const TDiskOperationCostEstimator TBsCostModelBase::NVMEEstimator{
     { 10000, 1.3 },   // ReadCoefficients
     { 3300, 1.5 },     // WriteCoefficients

diff --git a/ydb/core/blobstorage/vdisk/common/blobstorage_cost_tracker.h b/ydb/core/blobstorage/vdisk/common/blobstorage_cost_tracker.h
@@ -63,6 +63,7 @@ class TBsCostModelBase {
     ui64 PDiskWriteBlockSize = 4ull * 1'000'000; // 4MB
 
     static const TDiskOperationCostEstimator HDDEstimator;
+    static const TDiskOperationCostEstimator SSDEstimator;
     static const TDiskOperationCostEstimator NVMEEstimator;
 
 private:
@@ -100,6 +101,9 @@ class TBsCostModelBase {
             case NPDisk::DEVICE_TYPE_ROT: {
                 return HDDEstimator.Write(chunkSize);
             }
+            case NPDisk::DEVICE_TYPE_SSD: {
+                return SSDEstimator.Write(chunkSize);
+            }
             case NPDisk::DEVICE_TYPE_NVME: {
                 return NVMEEstimator.Write(chunkSize);
             }
@@ -116,6 +120,9 @@ class TBsCostModelBase {
             case NPDisk::DEVICE_TYPE_ROT: {
                 return HDDEstimator.HugeWrite(chunkSize);
             }
+            case NPDisk::DEVICE_TYPE_SSD: {
+                return SSDEstimator.HugeWrite(chunkSize);
+            }
             case NPDisk::DEVICE_TYPE_NVME: {
                 return NVMEEstimator.HugeWrite(chunkSize);
             }
@@ -133,6 +140,9 @@ class TBsCostModelBase {
             case NPDisk::DEVICE_TYPE_ROT: {
                 return HDDEstimator.Read(chunkSize);
             }
+            case NPDisk::DEVICE_TYPE_SSD: {
+                return SSDEstimator.Read(chunkSize);
+            }
             case NPDisk::DEVICE_TYPE_NVME: {
                 return NVMEEstimator.Read(chunkSize);
             }