Last two characters of ID_REVISION (firmware version) are inverted

[Fabián Rodríguez]

Binary package hint: udev

When looking for the firmware version of my hard disk (/dev/sda), I found out the last two characters reported by udevadm are inverted:

$ udevadm info --name=/dev/sda --query=property | grep ID_REVISION
ID_REVISION=2CV102DH

I believe the above information is also used by palimpsest, see:
http://launchpadlibrarian.net/57881569/40%20GB%20Solid-State%20Disk%20%28ATA%20INTEL%20SSDSA2M040G2GC%29%20%5B-dev-sda%5D%20%E2%80%94%20Disk%20Utility_002.png

This is very inconvenient when trying to determine if such a disk needs a firmware upgrade on a headless system.

dmesg, however, has the right information:
magicfab@winston:~$ dmesg | grep ATA-7
[ 2.902938] ata3.00: ATA-7: INTEL SSDSA2M040G2GC, 2CV102HD, max UDMA/133

I checked this on several systems and got the same inconsistent information. I haven't checked dmegs output from a live CD to see if it would include the fixed HD information, though.

I've filed a separate bug about lshw reporting only part of such information, Bug #663349.

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: udev 162-2.1
ProcVersionSignature: Ubuntu 2.6.35-22.35-generic 2.6.35.4
Uname: Linux 2.6.35-22-generic x86_64
NonfreeKernelModules: wl
Architecture: amd64
Date: Mon Nov 8 16:26:05 2010
EcryptfsInUse: Yes
InstallationMedia: Ubuntu 10.10 "Maverick Meerkat" - Release amd64 (20101007)
MachineType: Hewlett-Packard HP Pavilion tx1000 Notebook PC
ProcCmdLine: BOOT_IMAGE=/boot/vmlinuz-2.6.35-22-generic root=UUID=41b6b187-be76-4447-b18b-d39cc744b184 ro quiet splash elevator=noop usbhid.quirks=0xeef:0x1:0x40
ProcEnviron:
 LANG=en_US.utf8
 SHELL=/bin/bash
SourcePackage: udev
dmi.bios.date: 04/18/2008
dmi.bios.vendor: Hewlett-Packard
dmi.bios.version: F.20
dmi.board.name: 30BF
dmi.board.vendor: Quanta
dmi.board.version: 69.2E
dmi.chassis.type: 10
dmi.chassis.vendor: Quanta
dmi.chassis.version: N/A
dmi.modalias: dmi:bvnHewlett-Packard:bvrF.20:bd04/18/2008:svnHewlett-Packard:pnHPPaviliontx1000NotebookPC:pvrRev1:rvnQuanta:rn30BF:rvr69.2E:cvnQuanta:ct10:cvrN/A:
dmi.product.name: HP Pavilion tx1000 Notebook PC
dmi.product.version: Rev 1
dmi.sys.vendor: Hewlett-Packard

[Mathieu Trudel-Lapierre (cyphermox)]

Confirming, I see the same issue on very different hardware.

[Shawn Wang (shawn111)]

The fwrev size should be 8 not 6.
However, the upstream already fixed it.

disk_identify_fixup_string(identify, 23, 8); /* fwrev */

http://cgit.freedesktop.org/systemd/systemd/tree/src/udev/ata_id/ata_id.c

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "fwrev.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

[Patrik Lundquist (patrik-lundquist)]

Fixed in Trusty. Don't know about Saucy.

$ sudo hdparm -I /dev/sda | grep Firmware
 Firmware Revision: DXT05L0Q

$ udevadm info --name=/dev/sda --query=property | grep ID_REVISION
ID_REVISION=DXT05L0Q

$ dmesg | grep ATA-9
[ 2.248598] ata1.00: ATA-9: SAMSUNG MZ7TD128HAFV-000L1, DXT05L0Q, max UDMA/133