Opened 4 years ago

Closed 3 months ago

#1317 closed patch (fixed)

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

Reported by: Artem S. Tashkinov Owned by: Gabriele Pohl
Priority: minor Milestone: Release 7.5
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 4 years ago.
Attributes in the OEM app
drivedb.h.patch (1.3 KB ) - added by Artem S. Tashkinov 4 years ago.
smartmontools patch for SanDisk SDSSDA(120|240|480)G - OK to merge
smartctlx.txt (9.2 KB ) - added by Ivan 3 years ago.
smartctlx-new.txt (9.2 KB ) - added by Ivan 3 years ago.
drivedb.h-final.patch (1.8 KB ) - added by Ivan 3 years ago.

Download all attachments as: .zip

Change History (13)

by Artem S. Tashkinov, 4 years ago

Attachment: smart_ssd.png added

Attributes in the OEM app

by Artem S. Tashkinov, 4 years ago

Attachment: drivedb.h.patch added

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

comment:1 by Artem S. Tashkinov, 4 years ago

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

comment:2 by Christian Franke, 4 years ago

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.

by Ivan, 3 years ago

Attachment: smartctlx.txt added

by Ivan, 3 years ago

Attachment: smartctlx-new.txt added

in reply to:  2 comment:3 by Ivan, 3 years ago

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 3 years ago by Ivan (previous) (diff)

by Ivan, 3 years ago

Attachment: drivedb.h-final.patch added

comment:4 by Artem S. Tashkinov, 23 months ago

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

What's the holdout exactly?

comment:5 by Artem S. Tashkinov, 23 months ago

Ping.

comment:6 by Christian Franke, 23 months ago

Milestone: undecidedunscheduled

comment:7 by Gabriele Pohl, 3 months ago

Owner: set to Gabriele Pohl
Status: newaccepted

added in r5560

comment:8 by Christian Franke, 3 months ago

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