Opened 12 days ago

Last modified 3 days ago

#1191 new patch

Add Support for Advantech-SQF-S25M4-128G-S9E

Reported by: Brian Ober Owned by:
Priority: minor Milestone: undecided
Component: drivedb Version:
Keywords: ssd Cc:

Description

Add support for the 630 (current version nearing EOL), 830, 910, 640 (next-gen version) Series Advantech hard disks.

Attached are vendor SMART specifications, the output of smartctrl on a 630-series drive and a drivedb that I have been using in our system.

Attachments (4)

smartctl-Advantech-SQF-S25M4-128G-S9E.txt (6.1 KB) - added by Brian Ober 12 days ago.
smartctl output
SQFlash SMART ID Definition(630)_20170620.pdf (331.4 KB) - added by Brian Ober 12 days ago.
630 Series Specification
SQFlash SMART ID Definition(830 910 640)_20170620.pdf (340.6 KB) - added by Brian Ober 12 days ago.
Next Gen Specification
drivedb.h (197.4 KB) - added by Brian Ober 12 days ago.
Working Drivedb

Download all attachments as: .zip

Change History (6)

Changed 12 days ago by Brian Ober

smartctl output

Changed 12 days ago by Brian Ober

630 Series Specification

Changed 12 days ago by Brian Ober

Next Gen Specification

Changed 12 days ago by Brian Ober

Attachment: drivedb.h added

Working Drivedb

comment:1 Changed 3 days ago by Christian Franke

Milestone: undecided
Type: enhancementpatch

Thanks.

For future submissions, please don't post the full drivedb.h file. Provide a patch (output of svn diff or diff -u) or a local drivedb.h file (see -B option on smartctl man page).

Issues:

  • None of the new entries match the Device model string from the sample output:
    $ smartctl -B ./drivedb.h -P showall 'SQF-S25M4-128G-S9E'
    No presets are defined for this drive.  Its identity strings:
    MODEL:    SQF-S25M4-128G-S9E
    FIRMWARE: (any)
    do not match any of the known regular expressions.
    
  • The second new entry contains a syntax error:
    $ smartctl -B ./drivedb.h -P showall
    ...
                        175 Power_Failure_Protection
              Error: Attribute name too long ------^
    ...
    Found 1 syntax error(s) in database.
    
  • Conclusion: PLEASE TEST an entry before submission.
  • The 128GB in the model family names do not make sense as the entries cover more capacities.
  • The attribute sets are identical except the extra attribute 175. Please merge both entries into one.
  • If possible, please also provide sample outputs for the other series.
  • -v options with are already covered by the DEFAULT entry should be commented-out (see various other SSD entries for examples).
  • Excess long lines should be avoided.

comment:2 Changed 3 days ago by Christian Franke

Keywords: ssd added
Note: See TracTickets for help on using tickets.