Opened 2 years ago

Last modified 5 weeks ago

#1317 new patch

Proper attributes for SanDisk SDSSDA(120|240|480)G

Reported by: Artem S. Tashkinov Owned by:
Priority: minor Milestone: unscheduled
Component: drivedb Version: 7.1
Keywords: ssd Cc:

Description

No idea how to create a proper patch, so:

"-v 5,raw48,Reassigned_Block_Count "
"-v 166,raw48,Min_PE_Cycles "
"-v 167,raw48,Max_Bad_Blocks_Per_Die "
"-v 168,raw48,Max_PE_Cycles "
"-v 169,raw48,Total_Bad_Blocks "
"-v 170,raw48,Grown_Bad_Blocks "
"-v 173,raw48,Average_PE_Cycles "
"-v 230,raw48,Media_Wearout_Indicator "
"-v 233,raw48,NAND_GiB_Written "

Source: Sandisk SSD Utility

Attachments (5)

smart_ssd.png (30.8 KB) - added by Artem S. Tashkinov 2 years ago.
Attributes in the OEM app
drivedb.h.patch (1.3 KB) - added by Artem S. Tashkinov 2 years ago.
smartmontools patch for SanDisk? SDSSDA(120|240|480)G - OK to merge
smartctlx.txt (9.2 KB) - added by Ivan 11 months ago.
smartctlx-new.txt (9.2 KB) - added by Ivan 11 months ago.
drivedb.h-final.patch (1.8 KB) - added by Ivan 11 months ago.

Download all attachments as: .zip

Change History (11)

Changed 2 years ago by Artem S. Tashkinov

Attachment: smart_ssd.png added

Attributes in the OEM app

Changed 2 years ago by Artem S. Tashkinov

Attachment: drivedb.h.patch added

smartmontools patch for SanDisk? SDSSDA(120|240|480)G - OK to merge

comment:1 Changed 2 years ago by Artem S. Tashkinov

Summary: Proper attributes for SanDisk SDSSDA240G (SandForce Driven SSDs)Proper attributes for SanDisk SDSSDA(120|240|480)G
Type: enhancementpatch

comment:2 Changed 2 years ago by Christian Franke

Keywords: ssd added
Milestone: Release 7.2undecided

Thanks for this patch.

Please provide a sample smartctl -x output for this device.

Please add //"-v N,... " comments for the attributes of this device which are handled by the DEFAULT settings. See other SSD entries for examples.

Here is a quote from the smart_list.xml file from some 2018 version of the SanDisk tool (see tickets #464 and #799):

<!-- SanDisk SDSSDA* is being used by multiple hardware, so we define those drive by fw version -->

This comment is followed by 16 firmware patterns. Please check which one matches this device and add a firmware regex to the entry.

Changed 11 months ago by Ivan

Attachment: smartctlx.txt added

Changed 11 months ago by Ivan

Attachment: smartctlx-new.txt added

comment:3 in reply to:  2 Changed 11 months ago by Ivan

Hello!

I have just faced with the same disk in our server. I've adopted the patch for the drivedb.h version
7.2/5224 2021-06-06 15:55:46. I also attached smartctl -x output before and after applying the patch. Speaking about filtering by firmware version, I've digged into smart_list.xml and found no matches with the disk and its firmware version. The first suggestion was

  <ref fname="aphrodite_generic.xml" matchType="prefix" length="8" fwprefix="U2" version="1.1" />

but contents of the aphrodite_generic.xml surely describes another drive model. Even amount of metrics differs.

So I gave up using smart_list.xml and looked for a file with exactly 20 metrics. Happily there is only one and it's plus_generic.xml. From the name of the file I've done a suggestion that there is no need in firmware filtering and it should be general block for SanDisk SSD Plus like it's done in Artem's and my patch.

Last edited 11 months ago by Ivan (previous) (diff)

Changed 11 months ago by Ivan

Attachment: drivedb.h-final.patch added

comment:4 Changed 6 weeks ago by Artem S. Tashkinov

This was filed two years ago (!) along with the patch.

What's the holdout exactly?

comment:5 Changed 5 weeks ago by Artem S. Tashkinov

Ping.

comment:6 Changed 5 weeks ago by Christian Franke

Milestone: undecidedunscheduled
Note: See TracTickets for help on using tickets.