Exclude 980 Pro from samsung NVMe protocol (#754)

LibreHardwareMonitor · Jun 15, 2022 · 7ab8fbb · 7ab8fbb
1 parent 4b0bf58
commit 7ab8fbb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/LibreHardwareMonitorLib/Hardware/Storage/NVMeSamsung.cs b/LibreHardwareMonitorLib/Hardware/Storage/NVMeSamsung.cs
@@ -80,7 +80,7 @@ public bool IdentifyController(SafeHandle hDevice, out Kernel32.NVME_IDENTIFY_CO
                 Marshal.StructureToPtr(buffers, buffer, false);
 
                 validTransfer = Kernel32.DeviceIoControl(hDevice, Kernel32.IOCTL.IOCTL_SCSI_PASS_THROUGH, buffer, length, buffer, length, out _, IntPtr.Zero);
-                if (validTransfer)
+                if (validTransfer && buffers.DataBuf.Any(x => x != 0))
                 {
                     IntPtr offset = Marshal.OffsetOf<Kernel32.SCSI_PASS_THROUGH_WITH_BUFFERS>(nameof(Kernel32.SCSI_PASS_THROUGH_WITH_BUFFERS.DataBuf));
                     IntPtr newPtr = IntPtr.Add(buffer, offset.ToInt32());
@@ -240,7 +240,7 @@ public static SafeHandle IdentifyDevice(StorageInfo storageInfo)
                 {
                     var result = Marshal.PtrToStructure<Kernel32.SCSI_PASS_THROUGH_WITH_BUFFERS>(buffer);
 
-                    if (result.DataBuf.Sum(x => (long)x) == 0)
+                    if (result.DataBuf.All(x => x == 0))
                     {
                         handle.Close();
                         handle = null;

diff --git a/LibreHardwareMonitorLib/Hardware/Storage/NVMeSmart.cs b/LibreHardwareMonitorLib/Hardware/Storage/NVMeSmart.cs
@@ -27,7 +27,11 @@ internal NVMeSmart(StorageInfo storageInfo)
                 _handle = NVMeSamsung.IdentifyDevice(storageInfo);
                 if (_handle != null)
                 {
-                    NVMeDrive = new NVMeSamsung();
+                    NVMeDrive = new NVMeSamsung();
+                    if (!NVMeDrive.IdentifyController(_handle, out _))
+                    {
+                        NVMeDrive = null;
+                    }
                 }
             }