Opened 14 months ago

Last modified 14 months 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 14 months ago.
smartctl output
SQFlash SMART ID Definition(630)_20170620.pdf (331.4 KB) - added by Brian Ober 14 months ago.
630 Series Specification
SQFlash SMART ID Definition(830 910 640)_20170620.pdf (340.6 KB) - added by Brian Ober 14 months ago.
Next Gen Specification
drivedb.h (197.4 KB) - added by Brian Ober 14 months ago.
Working Drivedb

Download all attachments as: .zip

Change History (6)

Changed 14 months ago by Brian Ober

smartctl output

Changed 14 months ago by Brian Ober

630 Series Specification

Changed 14 months ago by Brian Ober

Next Gen Specification

Changed 14 months ago by Brian Ober

Attachment: drivedb.h added

Working Drivedb

comment:1 Changed 14 months 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 14 months ago by Christian Franke

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