Firmware Version is invalid in /proc/driver/cciss/cciss0

Bug #284031 reported by Bryan Stillwell
8
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Medium
Unassigned
Intrepid
Fix Released
Low
Unassigned

Bug Description

Binary package hint: linux-image-2.6.27-7-generic

While testing intrepid on an HP DL380G5, I noticed that the Firmware Version was not displayed in /proc/driver/cciss/cciss0 like it has been in all previous releases:

---------------------------------------------------
bryan@ian:~$ cat /proc/driver/cciss/cciss0
cciss0: HP Smart Array P400 Controller
Board ID: 0x3234103c
Firmware Version:
IRQ: 2290
Logical drives: 1
Current Q depth: 0
Current # commands on controller: 0
Max Q depth since init: 8
Max # commands on controller since init: 19
Max SG entries since init: 31
Sequential access devices: 0

cciss/c0d0: 220.12GB RAID 5
---------------------------------------------------

Using hexdump it appears that the firmware version is being displayed as four null characters:

---------------------------------------------------
bryan@ian:~$ hexdump -C /proc/driver/cciss/cciss0
00000000 63 63 69 73 73 30 3a 20 48 50 20 53 6d 61 72 74 |cciss0: HP Smart|
00000010 20 41 72 72 61 79 20 50 34 30 30 20 43 6f 6e 74 | Array P400 Cont|
00000020 72 6f 6c 6c 65 72 0a 42 6f 61 72 64 20 49 44 3a |roller.Board ID:|
00000030 20 30 78 33 32 33 34 31 30 33 63 0a 46 69 72 6d | 0x3234103c.Firm|
00000040 77 61 72 65 20 56 65 72 73 69 6f 6e 3a 20 00 00 |ware Version: ..|
00000050 00 00 0a 49 52 51 3a 20 32 32 39 30 0a 4c 6f 67 |...IRQ: 2290.Log|
00000060 69 63 61 6c 20 64 72 69 76 65 73 3a 20 31 0a 43 |ical drives: 1.C|
00000070 75 72 72 65 6e 74 20 51 20 64 65 70 74 68 3a 20 |urrent Q depth: |
00000080 30 0a 43 75 72 72 65 6e 74 20 23 20 63 6f 6d 6d |0.Current # comm|
00000090 61 6e 64 73 20 6f 6e 20 63 6f 6e 74 72 6f 6c 6c |ands on controll|
000000a0 65 72 3a 20 31 0a 4d 61 78 20 51 20 64 65 70 74 |er: 1.Max Q dept|
000000b0 68 20 73 69 6e 63 65 20 69 6e 69 74 3a 20 38 0a |h since init: 8.|
000000c0 4d 61 78 20 23 20 63 6f 6d 6d 61 6e 64 73 20 6f |Max # commands o|
000000d0 6e 20 63 6f 6e 74 72 6f 6c 6c 65 72 20 73 69 6e |n controller sin|
000000e0 63 65 20 69 6e 69 74 3a 20 31 39 0a 4d 61 78 20 |ce init: 19.Max |
000000f0 53 47 20 65 6e 74 72 69 65 73 20 73 69 6e 63 65 |SG entries since|
00000100 20 69 6e 69 74 3a 20 33 31 0a 53 65 71 75 65 6e | init: 31.Sequen|
00000110 74 69 61 6c 20 61 63 63 65 73 73 20 64 65 76 69 |tial access devi|
00000120 63 65 73 3a 20 30 0a 0a 63 63 69 73 73 2f 63 30 |ces: 0..cciss/c0|
00000130 64 30 3a 09 20 32 32 30 2e 31 32 47 42 09 52 41 |d0:. 220.12GB.RA|
00000140 49 44 20 35 0a |ID 5.|
00000145
---------------------------------------------------

The attached patch fixes this problem.

CVE References

Revision history for this message
Bryan Stillwell (bryan-stillwell) wrote :
Revision history for this message
Leann Ogasawara (leannogasawara) wrote :

Hi Bryan,

Just wanted to let you know the kernel is currently frozen for Intrepid. However, this may qualify for an Intrepid SRU and can definitely be looked at for Jaunty. Thanks.

Changed in linux:
assignee: nobody → ubuntu-kernel-team
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Tim Gardner (timg-tpi) wrote :
Changed in linux:
assignee: nobody → timg-tpi
importance: Undecided → Low
milestone: none → intrepid-updates
status: New → Fix Committed
Revision history for this message
Tim Gardner (timg-tpi) wrote :

SRU Justification

Impact: Controller firmware version displays as blanks

Patch Description: Add code to emit an INQUIRY command which responds with the firmware version.

Patch: http://kernel.ubuntu.com/git?p=ubuntu/ubuntu-intrepid.git;a=commit;h=b659b0b0e14f5e3c049bf5c8d2883587cdbfe344

Test Case: See bug description

Changed in linux:
status: Triaged → Fix Committed
Revision history for this message
Tim Gardner (timg-tpi) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (5.7 KiB)

This bug was fixed in the package linux - 2.6.27-9.18

---------------
linux (2.6.27-9.18) jaunty; urgency=low

  [ Amit Kucheria ]

  * Add the armel architecture

  [ Leann Ogasawara ]

  * Reboot stalls after installing Intrepid from alternate on a Dell
    Optiplex 330
    - LP: #293372

  [ Stefan Bader ]

  * (no-up) buildenv: Update git-ubuntu-log

  [ Tim Gardner ]

  * SAUCE: Firmware Version is invalid in cciss (version 2)
    - LP: #284031
  * Add igb to debian installer udeb
    - LP: #290073

  [ Upstream Kernel Changes ]

  * Revert "HID: Invert HWHEEL mappings for some Logitech mice"
  * md: fix bug in raid10 recovery.
    - LP: #285156
  * SCSI: sd: Fix handling of NO_SENSE check condition
  * S390: Fix sysdev class file creation.
  * sysfs: Fix return values for sysdev_store_{ulong, int}
  * USB: fix crash when URBs are unlinked after the device is gone
  * ALSA: hda - Add reboot notifier
  * kbuild: mkspec - fix build rpm
  * x86: fix /dev/mem mmap breakage when PAT is disabled
  * atl1: fix vlan tag regression
  * libertas: fix buffer overrun
  * libata: initialize port_task when !CONFIG_ATA_SFF
  * syncookies: fix inclusion of tcp options in syn-ack
  * tcpv6: fix option space offsets with md5
  * pkt_sched: sch_generic: Fix oops in sch_teql
  * sparc64: Fix race in arch/sparc64/kernel/trampoline.S
  * math-emu: Fix signalling of underflow and inexact while packing result.
  * firewire: fix setting tag and sy in iso transmission
  * firewire: fix ioctl() return code
  * firewire: Survive more than 256 bus resets
  * firewire: fix struct fw_node memory leak
  * firewire: fw-sbp2: delay first login to avoid retries
  * firewire: fw-sbp2: fix races
  * ACPI: Always report a sync event after a lid state change
  * powerpc: fix i2c on PPC linkstation / kurobox machines
  * powerpc: Reserve in bootmem lmb reserved regions that cross NUMA nodes
  * powerpc/numa: Make memory reserve code more robust
  * powerpc: Don't use a 16G page if beyond mem= limits
  * i2c: The i2c mailing list is moving
  * scx200_i2c: Add missing class parameter
  * ALSA: use correct lock in snd_ctl_dev_disconnect()
  * V4L: pvrusb2: Keep MPEG PTSs from drifting away
  * DVB: s5h1411: bugfix: Setting serial or parallel mode could destroy
    bits
  * DVB: s5h1411: Perform s5h1411 soft reset after tuning
  * DVB: s5h1411: Power down s5h1411 when not in use
  * PCI: fix 64-vbit prefetchable memory resource BARs
  * sched: disable the hrtick for now
  * sched_clock: prevent scd->clock from moving backwards
  * x86: avoid dereferencing beyond stack + THREAD_SIZE
  * rtc-cmos: look for PNP RTC first, then for platform RTC
  * USB: storage: Avoid I/O errors when issuing SCSI ioctls to JMicron
    USB/ATA bridge
  * sata_promise: add ATA engine reset to reset ops
  * sata_nv: fix generic, nf2/3 detection regression
  * ACPI: EC: Rename some variables
  * ACPI: EC: Check for IBF=0 periodically if not in GPE mode
  * libata: Fix LBA48 on pata_it821x RAID volumes.
  * ACPI: Ingore the RESET_REG_SUP bit when using ACPI reset mechanism
  * ACPI: Clear WAK_STS on resume
  * hfsplus: fix Buffer overflow with a corrupted image
  * hfsplus: check read_mapping_page() retur...

Read more...

Changed in linux:
status: Fix Committed → Fix Released
Changed in linux:
status: Fix Committed → Fix Released
Martin Pitt (pitti)
Changed in linux:
status: Fix Released → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted linux into intrepid-proposed, please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Revision history for this message
Bryan Stillwell (bryan-stillwell) wrote :

Appears to be fixed in 2.6.27-10.20 from intrepid-proposed:

bryan@ian:~$ cat /proc/driver/cciss/cciss0
cciss0: HP Smart Array P400 Controller
Board ID: 0x3234103c
Firmware Version: 5.20
IRQ: 2290
Logical drives: 3
Current Q depth: 0
Current # commands on controller: 0
Max Q depth since init: 16
Max # commands on controller since init: 19
Max SG entries since init: 31
Sequential access devices: 0

cciss/c0d0: 146.74GB RAID 0
cciss/c0d1: 146.74GB RAID 0
cciss/c0d2: 73.37GB RAID 0

Revision history for this message
Launchpad Janitor (janitor) wrote : Kernel team bugs

Per a decision made by the Ubuntu Kernel Team, bugs will longer be assigned to the ubuntu-kernel-team in Launchpad as part of the bug triage process. The ubuntu-kernel-team is being unassigned from this bug report. Refer to https://wiki.ubuntu.com/KernelTeamBugPolicies for more information. Thanks.

Tim Gardner (timg-tpi)
Changed in linux (Ubuntu Intrepid):
assignee: Tim Gardner (timg-tpi) → nobody
status: Fix Committed → 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.