Opened 11 years ago

Closed 10 years ago

#291 closed enhancement (fixed)

Temparature printout broken on tempminmax device once drive has been below 0C

Reported by: nadeau Owned by: Christian Franke
Priority: minor Milestone: Release 6.3
Component: all Version: 6.1
Keywords: Cc:


This is the output(below) I get from an SSD that has been in a temperature chamber.

Output ain't pretty anymore, used to be something like 23 (min/max 10/77)

I created a patch on github for that:

I understand that you might not be interested in this patch for various reasons(drivedb.h not backward compatible, too many undocumented drive, etc.) Anyhow, I thought you might be interested.

smartctl 6.1 2013-03-16 r3800 [x86_64-linux-3.10.9-200.fc19.x86_64] (local build)
Copyright (C) 2002-13, Bruce Allen, Christian Franke,

Model Family:     SandForce Driven SSDs
Device Model:     KINGSTON SVP200S3480G
Serial Number:    50026B7226020429
LU WWN Device Id: 5 0026b7 226020429
Firmware Version: 501ABBF0
User Capacity:    480 103 981 056 bytes [480 GB]
Sector Size:      512 bytes logical/physical
Rotation Rate:    Solid State Device
Device is:        In smartctl database [for details use: -P show]
ATA Version is:   ATA8-ACS, ACS-2 T13/2015-D revision 3
SATA Version is:  SATA 3.0, 6.0 Gb/s (current: 3.0 Gb/s)
Local Time is:    Fri Aug 23 13:13:12 2013 EDT
SMART support is: Available - device has SMART capability.
SMART support is: Enabled


SMART Attributes Data Structure revision number: 10
Vendor Specific SMART Attributes with Thresholds:
  1 Raw_Read_Error_Rate     0x000f   120   120   050    Pre-fail  Always       -       0/0
194 Temperature_Celsius     0x0022   242   083   000    Old_age   Always       -       242 (0 239 0 83 0)
242 Lifetime_Reads_GiB      0x0032   000   000   000    Old_age   Always       -       2

Attachments (1)

tempminmax-format.patch (3.8 KB ) - added by Christian Franke 11 years ago.
Patch for -v,tempminmax format. Handles negative temperatures and WDC counter.

Download all attachments as: .zip

Change History (4)

comment:1 by Christian Franke, 11 years ago

Milestone: Release 6.3
Owner: changed from somebody to Christian Franke
Priority: trivialminor
Status: newaccepted

Thanks for the patch.

IMO its complexity and the break of backward compatibility is not needed. I will enhance the current heuristics used by tempminmax format such that signed 8-bit values are handled properly. This would then also work for other drives using this format in the rare case of actual negative temperatures.

by Christian Franke, 11 years ago

Attachment: tempminmax-format.patch added

Patch for -v,tempminmax format. Handles negative temperatures and WDC counter.

comment:2 by Christian Franke, 11 years ago

Please apply above patch and test "smartctl -A ..." with affected devices.

comment:3 by Christian Franke, 10 years ago

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