hddtemp is broken on sata drives for 2.6.20 kernel

Bug #109498 reported by Alex Murray
6
Affects Status Importance Assigned to Milestone
hddtemp (Debian)
Fix Released
Unknown
hddtemp (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: hddtemp

See bug report in sensors-applet for symptoms
Requires this patch from Debian (see existing debian bug report for actual error):

--- hddtemp-0.3-beta15.orig/src/satacmds.c
+++ hddtemp-0.3-beta15/src/satacmds.c
@@ -80,8 +80,8 @@

   ret = scsi_SG_IO(device, cdb, sizeof(cdb), buffer, cmd[3] * 512, sense, sizeof(sense), dxfer_direction);

- /* Verify SATA magics */
- if (sense[0] != 0x72 || sense[7] != 0x0e || sense[9] != 0x0e || sense[10] != 0x00)
+ /* Verify SATA magic */
+ if (sense[0] != 0x72)
     return 1;
   else
     return ret;

Perhaps we should just sync to latest version?

Revision history for this message
Alex Murray (alexmurray) wrote :

Debian bug report concerning same issue:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=404310

Note the patch provided in the debian bug report is too strict, so we should just use the one from the debian package, as listed above

This bug is causing bug #94558 reported in sensors-applet.

Revision history for this message
William Grant (wgrant) wrote :

Fixed in unstable's 0.3-beta15-34. Gutsy will get this when the autosync is turned on, so no action is necessary.

Changed in hddtemp:
importance: Undecided → Medium
status: Unconfirmed → Confirmed
Changed in hddtemp:
status: Unknown → Fix Released
Revision history for this message
William Grant (wgrant) wrote :

Fixed in Gutsy.

Changed in hddtemp:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.