hiyohiyo / CrystalDiskInfo

CrystalDiskInfo
https://github.com/hiyohiyo/CrystalDiskInfo
MIT License
1.59k stars 177 forks source link

WDC SA530 Total Host Read & Total Host Writes display Error #233

Open black-raison-detre opened 11 months ago

black-raison-detre commented 11 months ago

smart attribute is F1 0xA9 169GB F2 0x21 33GB

but displayed vlaue is Total Host Reads 0GB Total Host Writes 2GB

Total NAND Writes is correct though

CrystalDiskInfo_20230912103252.txt

black-raison-detre commented 11 months ago

Well, it seems to there is special processing for the SA530 drive, without it now the display is correct. So maybe multiple revision of SA530 drive, using different controller or firmware?? Mine one is using Marvell 88SS1074

diff --git a/AtaSmart.cpp b/AtaSmart.cpp
index 3b87e40..3772aa0 100644
--- a/AtaSmart.cpp
+++ b/AtaSmart.cpp
@@ -5738,14 +5738,14 @@ BOOL CAtaSmart::IsSsdWdc(ATA_SMART_INFO& asi)
        {
                flagSmartType = TRUE;

-               if (asi.Model.Find(L"SA530") >= 0)
-               {
-                       asi.HostReadsWritesUnit = HOST_READS_WRITES_16MB;
-               }
-               else
-               {
-                       asi.HostReadsWritesUnit = HOST_READS_WRITES_GB;
-               }
+               //if (asi.Model.Find(L"SA530") >= 0)
+               //{
+               //      asi.HostReadsWritesUnit = HOST_READS_WRITES_16MB;
+               //}
+               //else
+               //{
+               asi.HostReadsWritesUnit = HOST_READS_WRITES_GB;
+               //}
        }

        return flagSmartType;