Opened 12 months ago

Closed 8 weeks ago

#785 closed enhancement (fixed)

New WDC Red Pro drives: WDC WD6002FFWX

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

Description (last modified by chrfranke)

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 12 months ago.
Output from "smartctl -q noserial -x" as requested

Download all attachments as: .zip

Change History (6)

Changed 12 months ago by Kai Harrekilde-Petersen

Attachment: smartctl-WDC-WD6002FFWX.txt added

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

comment:1 Changed 12 months ago by chrfranke

Description: modified (diff)

comment:2 Changed 12 months ago by chrfranke

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

comment:3 in reply to:  2 Changed 12 months ago by Kai Harrekilde-Petersen

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 Changed 8 weeks ago by chrfranke

Owner: set to chrfranke
Status: newaccepted

comment:5 Changed 8 weeks ago by chrfranke

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