lilo-installer fails on HP (Compaq) SmartArray controllers (/dev/cciss)

Bug #103291 reported by Alexander "Leo" Bergolth
4
Affects Status Importance Assigned to Milestone
lilo-installer (Ubuntu)
Fix Released
Undecided
Unassigned
Dapper
Fix Released
Medium
Colin Watson

Bug Description

Binary package hint: lilo-installer

During installation of 7.04 beta, the lilo-installer fails when using /dev/cciss/c0d0 (Compaq SmartArray) as boot/root device. (This bug was already present in previous Ubuntu releases.)

The reason are some checks in lilo-installer.postinst that assume that the device has to look like /dev/[hs]d[a-z].
However, SmartArray controllers use device-names like /dev/cciss/c0d0 and partition names like /dev/cciss/c0d0p1.

The attached patch fixes this issue.

--leo

P.S.: Note that the patch only fixes installation in the MBR, installing Lilo in a partition would need further fixes.

Revision history for this message
Alexander "Leo" Bergolth (bergolth) wrote :
Revision history for this message
Stephan Rügamer (sruegamer) wrote :

Suddenly I catched this bug, too.
Installing Feisty didn't let me install lilo on a cciss device.

Grub works as expected.

Revision history for this message
Colin Watson (cjwatson) wrote :

lilo-installer (1.22ubuntu2) gutsy; urgency=low

  * Handle /dev/cciss/* devices (LP: #103291).

 -- Colin Watson <email address hidden> Wed, 12 Sep 2007 15:06:45 +0100

Changed in lilo-installer:
status: New → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Thanks, Colin. Do you think this fix is appropriate for Dapper, too?

Revision history for this message
Colin Watson (cjwatson) wrote :
Revision history for this message
Colin Watson (cjwatson) wrote :

Yes, dapper has roughly the same code modulo indenting. If the fix is confirmed to work on gutsy then I think we should include it.

Note that I used a different patch from that provided by the bug reporter; there were a few cases missing there and in general I took a slightly different approach, though the spirit is the same.

Revision history for this message
Martin Pitt (pitti) wrote :

Thanks, Colin! Let's wait for Stephan's test. I'll accept it once we got a successful report.

Changed in lilo-installer:
importance: Undecided → Medium
status: New → In Progress
Martin Pitt (pitti)
Changed in lilo-installer:
assignee: nobody → kamion
Revision history for this message
Martin Pitt (pitti) wrote :

Stephan, this should be testable in gutsy. Do you have some time to give this a try? Thank you!

Revision history for this message
Stephan Rügamer (sruegamer) wrote :

HI Martin, Colin,

thx for the fix, it works for me here...tested it on a dl360g4p...

\sh

Revision history for this message
Martin Pitt (pitti) wrote :

Accepted into dapper-proposed.

Changed in lilo-installer:
status: In Progress → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

This was tested by Stephan Hermann and by the IS team in the data center. verified.

Revision history for this message
Martin Pitt (pitti) wrote :

Copied to dapper-updates.

Changed in lilo-installer:
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.