Opened 4 months ago

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

Download all attachments as: .zip

Change History (6)

Changed 4 months ago by Brian Ober

smartctl output

Changed 4 months ago by Brian Ober

630 Series Specification

Changed 4 months ago by Brian Ober

Next Gen Specification

Changed 4 months ago by Brian Ober

Attachment: drivedb.h added

Working Drivedb

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

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