Opened 8 years ago

Closed 7 years ago

#785 closed enhancement (fixed)

New WDC Red Pro drives: WDC WD6002FFWX

Reported by: Kai Harrekilde-Petersen Owned by: Christian Franke
Priority: minor Milestone: Release 6.6
Component: drivedb Version: 6.5
Keywords: Cc:

Description (last modified by Christian Franke)

I found that my new 6TB WDC Red Pro drive wasn't recognized by drivedb.h so I created a patch so all Red Pro's can be detected.
I extracted model names from https://www.wdc.com/content/dam/wdc/website/downloadable_assets/eng/spec_data_sheet/2879-800022.pdf

--- drivedb.h.old  2016-10-23 17:17:15.000000000 +0200
+++ drivedb.h   2017-01-01 14:06:13.042894951 +0100
@@ -3406,7 +3406,11 @@
     "-v 22,raw48,Helium_Level" // WD80EFZX
   },
   { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81
-    "WDC WD[234]001FFSX-.*",
+    "WDC WD([234]001|2002)FFSX-.*",
+    "", "", ""
+  },
+  { "Western Digital Red Pro", // tested with WDC WD6002FFWX-68TZ4N0/83.H0A83
+    "WDC WD([568]001|[46]002)FFWX-.*",
     "", "", ""
   },
   { "Western Digital Purple", // tested with WDC WD40PURX-64GVNY0/80.00A80

Attachments (1)

smartctl-WDC-WD6002FFWX.txt (10.7 KB ) - added by Kai Harrekilde-Petersen 8 years ago.
Output from "smartctl -q noserial -x" as requested

Download all attachments as: .zip

Change History (6)

by Kai Harrekilde-Petersen, 8 years ago

Attachment: smartctl-WDC-WD6002FFWX.txt added

Output from "smartctl -q noserial -x" as requested

comment:1 by Christian Franke, 8 years ago

Description: modified (diff)

comment:2 by Christian Franke, 8 years ago

Milestone: Release 6.6
Type: patchenhancement

Thanks for this patch. IMO an extra entry is not needed. I would suggest this simplified version:

--- drivedb.h	(revision 4367)
+++ drivedb.h	(working copy)
@@ -3405,8 +3405,9 @@
     "", "",
     "-v 22,raw48,Helium_Level" // WD80EFZX
   },
-  { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81
-    "WDC WD[234]001FFSX-.*",
+  { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81,
+      // WDC WD6002FFWX-68TZ4N0/83.H0A83
+    "WDC WD([2-68]00[12])FF[SW]X-.*",
     "", "", ""
   },
   { "Western Digital Purple", // tested with WDC WD40PURX-64GVNY0/80.00A80

in reply to:  2 comment:3 by Kai Harrekilde-Petersen, 8 years ago

I'm fine with either - I just wanted to avoid matching too much ;-)

Replying to chrfranke:

Thanks for this patch. IMO an extra entry is not needed. I would suggest this simplified version:

--- drivedb.h	(revision 4367)
+++ drivedb.h	(working copy)
@@ -3405,8 +3405,9 @@
     "", "",
     "-v 22,raw48,Helium_Level" // WD80EFZX
   },
-  { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81
-    "WDC WD[234]001FFSX-.*",
+  { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81,
+      // WDC WD6002FFWX-68TZ4N0/83.H0A83
+    "WDC WD([2-68]00[12])FF[SW]X-.*",
     "", "", ""
   },
   { "Western Digital Purple", // tested with WDC WD40PURX-64GVNY0/80.00A80

comment:4 by Christian Franke, 7 years ago

Owner: set to Christian Franke
Status: newaccepted

comment:5 by Christian Franke, 7 years ago

Resolution: fixed
Status: acceptedclosed
Note: See TracTickets for help on using tickets.