Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 497380 - app-admin/hddtemp-0.3_beta15-r7 - wrong output for Samsung SSD 840 PRO Series
Summary: app-admin/hddtemp-0.3_beta15-r7 - wrong output for Samsung SSD 840 PRO Series
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords: NeedPatch
Depends on:
Blocks:
 
Reported: 2014-01-07 11:48 UTC by Evert
Modified: 2018-10-20 16:36 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Evert 2014-01-07 11:48:59 UTC
hddtemp gives the wrong output for Samsung SSD 840 PRO Series (see below):
- Serise should be Series
- extraeneous chars (0x10 0x80)

# hddtemp /dev/sda
/dev/sda: Samsung SSD 840 PRO Serise              ^P<80>: 28°C

# hddtemp /dev/sda |od -Ax -tax1
000000   /   d   e   v   /   s   d   a   :  sp   S   a   m   s   u   n
        2f  64  65  76  2f  73  64  61  3a  20  53  61  6d  73  75  6e
000010   g  sp   S   S   D  sp   8   4   0  sp   P   R   O  sp   S   e
        67  20  53  53  44  20  38  34  30  20  50  52  4f  20  53  65
000020   r   i   s   e  sp  sp  sp  sp  sp  sp  sp  sp  sp  sp  sp  sp
        72  69  73  65  20  20  20  20  20  20  20  20  20  20  20  20
000030  sp  sp dle nul   :  sp   2   8   B   0   C  nl
        20  20  10  80  3a  20  32  38  c2  b0  43  0a

I compared this with the output of udevadm and smartctl and those utilities get the model right:

# udevadm info --name=/dev/sda --query=property |grep ID_MODEL
ID_MODEL=Samsung_SSD_840_PRO_Series
ID_MODEL_ENC=Samsung\x20SSD\x20840\x20PRO\x20Series\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20

# smartctl -a /dev/sda |grep -i samsung
Model Family:     Samsung based SSDs
Device Model:     Samsung SSD 840 PRO Series

Compiling hddtemp with USE=-nls doesn't make any difference.
Note: I already added this drive (190 C) to hddtemp.db (request comes later), otherwise it doesn't show the temperature.

# emerge --info
Portage 2.2.7 (default/linux/amd64/13.0/desktop/gnome/systemd, gcc-4.7.3, glibc-2.17, 3.12.6-3 x86_64)
=================================================================
System uname: Linux-3.12.6-3-x86_64-Intel-R-_Core-TM-_i7-4770K_CPU_@_3.50GHz-with-gentoo-2.2
KiB Mem:    16121728 total,  14565172 free
KiB Swap:   33554428 total,  33554428 free
Timestamp of tree: Mon, 06 Jan 2014 21:45:01 +0000
ld GNU ld (GNU Binutils) 2.23.2
app-shells/bash:          4.2_p45
dev-java/java-config:     2.1.12-r1
dev-lang/python:          2.7.5-r3, 3.3.2-r2
dev-util/cmake:           2.8.11.2
dev-util/pkgconfig:       0.28
sys-apps/baselayout:      2.2
sys-apps/openrc:          0.12.4
sys-apps/sandbox:         2.6-r1
sys-devel/autoconf:       2.13, 2.69
sys-devel/automake:       1.11.6, 1.12.6, 1.13.4
sys-devel/binutils:       2.23.2
sys-devel/gcc:            4.7.3-r1
sys-devel/gcc-config:     1.7.3
sys-devel/libtool:        2.4.2
sys-devel/make:           3.82-r4
sys-kernel/linux-headers: 3.9 (virtual/os-headers)
sys-libs/glibc:           2.17
Repositories: gentoo x-portage
ACCEPT_KEYWORDS="amd64"
ACCEPT_LICENSE="*"
CBUILD="x86_64-pc-linux-gnu"
CFLAGS="-march=core2 -O2 -pipe"
CHOST="x86_64-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/share/gnupg/qualified.txt /var/bind"
CONFIG_PROTECT_MASK="/etc/ca-certificates.conf /etc/dconf /etc/env.d /etc/fonts/fonts.conf /etc/gconf /etc/gentoo-release /etc/php/apache2-php5.3/ext-active/ /etc/php/apache2-php5.5/ext-active/ /etc/php/cgi-php5.3/ext-active/ /etc/php/cgi-php5.5/ext-active/ /etc/php/cli-php5.3/ext-active/ /etc/php/cli-php5.5/ext-active/ /etc/revdep-rebuild /etc/sandbox.d /etc/terminfo /etc/texmf/language.dat.d /etc/texmf/language.def.d /etc/texmf/updmap.d /etc/texmf/web2c"
CXXFLAGS="-march=core2 -O2 -pipe"
DISTDIR="/usr/portage/distfiles"
FCFLAGS="-O2 -pipe"
FEATURES="assume-digests binpkg-logs config-protect-if-modified distlocks fixlafiles merge-sync metadata-transfer news parallel-fetch parallel-install preserve-libs protect-owned sandbox sfperms strict unknown-features-warn unmerge-logs unmerge-orphans userfetch userpriv usersandbox usersync"
FFLAGS="-O2 -pipe"
GENTOO_MIRRORS="rsync://mirror.leaseweb.com/gentoo/ rsync://ftp.snt.utwente.nl/gentoo"
LANG="en_US.UTF-8"
LDFLAGS="-Wl,-O1 -Wl,--as-needed"
MAKEOPTS="-j12"
PKGDIR="/usr/portage/packages"
PORTAGE_CONFIGROOT="/"
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --omit-dir-times --compress --force --whole-file --delete --stats --human-readable --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/local/portage"
SYNC="rsync://rsync.uk.gentoo.org/gentoo-portage"
USE="X a52 aac acl acpi alsa amd64 ao apache2 apm berkdb bluetooth branding bzip2 cairo caps cdda cdr cli colord cpudetection cracklib crypt cups cxx dbus dri dts dvb dvd dvdr dvdread eds emboss encode evo exif fam ffmpeg firefox flac fortran g3dvl gdbm gif gimp gmp gnome gnome-keyring gnome-online-accounts gpm gstreamer gtk iconv icu imlib introspection ipv6 java jpeg latex lcms ldap libass libnotify libsamplerate libsecret lm_sensors mad mikmod mmx mng mod modules mozilla mp3 mp4 mpeg mplayer mudflap multilib nautilus ncurses nls nptl nsplugin ogg opengl openmp pam pango pcre pdf png policykit ppds pulseaudio qt3support qt4 quicktime readline samba sdl session sndfile sni socialweb soundtouch spell sse sse2 ssl ssse3 startup-notification svg systemd tcpd theora threads tiff truetype udev udisks unicode upower usb v4l v4l2 vcd vhosts vim-syntax vorbis wmf wxwidgets x264 xcb xinerama xml xulrunner xv xvid xvmc zlib" ABI_X86="64" ALSA_CARDS="intel8x0" APACHE2_MODULES="actions alias auth_basic auth_digest authn_anon authn_dbd authn_dbm authn_default authn_file authz_dbm authz_default authz_groupfile authz_host authz_owner authz_user cache cgi dav dav_fs dav_lock dbd deflate dir disk_cache env expires ext_filter file_cache filter headers ident imagemap include info log_config logio mem_cache mime mime_magic negotiation proxy proxy_ajp proxy_balancer proxy_connect proxy_http rewrite setenvif so speling status unique_id userdir usertrack vhost_alias" CALLIGRA_FEATURES="kexi words flow plan sheets stage tables krita karbon braindump author" CAMERAS="casio_qv" COLLECTD_PLUGINS="df interface irq load memory rrdtool swap syslog" ELIBC="glibc" GPSD_PROTOCOLS="ashtech aivdm earthmate evermore fv18 garmin garmintxt gpsclock itrax mtk3301 nmea ntrip navcom oceanserver oldstyle oncore rtcm104v2 rtcm104v3 sirf superstar2 timing tsip tripmate tnt ublox ubx" INPUT_DEVICES="keyboard mouse evdev" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LIBREOFFICE_EXTENSIONS="pdfimport" LINGUAS="en en_US en_GB nl de" OFFICE_IMPLEMENTATION="libreoffice" PHP_TARGETS="php5-5" PYTHON_SINGLE_TARGET="python2_7" PYTHON_TARGETS="python2_7 python3_3" RUBY_TARGETS="ruby19 ruby18" SANE_BACKENDS="pixma" USERLAND="GNU" VIDEO_CARDS="vesa intel nouveau fbdev v4l" XTABLES_ADDONS="quota2 psd pknock lscan length2 ipv4options ipset ipp2p iface geoip fuzzy condition tee tarpit sysrq steal rawnat logmark ipmark dhcpmac delude chaos account"
Unset:  CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LC_ALL, PORTAGE_BUNZIP2_COMMAND, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS, USE_PYTHON
Comment 1 Amadeusz Żołnowski (RETIRED) gentoo-dev 2014-01-12 14:56:28 UTC
Hm, I have this drive, too.

$ sudo hddtemp /dev/sda
/dev/sda: Samsung SSD 840 PRO Serise              : 39°C

It looks I can reproduce that bug, but does this string really matter?
Comment 2 Evert 2014-01-12 15:26:26 UTC
(In reply to Amadeusz Żołnowski from comment #1)
> It looks I can reproduce that bug, but does this string really matter?

Not to me since I use hddtemp to show temperares in gkrellm.
Anyway, it seems strange to me a string can be garbled like that. Maybe it has to do something with the extraeneous chars. Looks like the string is 40 bytes fixed (udevadm output) which hddtemp might not do...
Anyway, I already tried to contact the author, but hddtemp@guzu.net doesn't seem to exist...
Comment 3 Larry the Git Cow gentoo-dev 2018-10-20 16:36:28 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=dfea1064475d1d149628e811be2cf4173f04bf53

commit dfea1064475d1d149628e811be2cf4173f04bf53
Author:     Pacho Ramos <pacho@gentoo.org>
AuthorDate: 2018-10-20 16:35:13 +0000
Commit:     Pacho Ramos <pacho@gentoo.org>
CommitDate: 2018-10-20 16:35:42 +0000

    app-admin/hddtemp: Bump to follow latest Debian patchset...
    
    also multiple updates to the DB
    
    Thanks-to: Frank Krömmelbein
    Thanks-to: Thomas Albers
    Thanks-to: Paolo Pedroni
    Thanks-to: Evert
    Thanks-to: Chicago
    Closes: https://bugs.gentoo.org/627272
    Closes: https://bugs.gentoo.org/661256
    Closes: https://bugs.gentoo.org/664150
    Closes: https://bugs.gentoo.org/668176
    Closes: https://bugs.gentoo.org/497380
    Closes: https://bugs.gentoo.org/656138
    Signed-off-by: Pacho Ramos <pacho@gentoo.org>
    Package-Manager: Portage-2.3.51, Repoman-2.3.11

 app-admin/hddtemp/Manifest                      |   1 +
 app-admin/hddtemp/files/hddgentoo.db            |   6 +-
 app-admin/hddtemp/hddtemp-0.3_beta15-r27.ebuild | 114 ++++++++++++++++++++++++
 3 files changed, 119 insertions(+), 2 deletions(-)