[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#426737: lilo-installer fails on HP SmartArray controllers (/dev/ida)



On 06/18/2007 08:10 AM, Robert Millan wrote:
> On Wed, May 30, 2007 at 06:36:11PM +0200, Alexander 'Leo' Bergolth wrote:
>> Package: lilo-installer
>> Version: 1.20
>>
>> During installation of Debian Etch, lilo-installer fails when using
>> /dev/ida/c0d0 (Compaq/HP SmartArray) as boot/root device.
>>
>> 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/ida/c0d0
>> (/dev/cciss/c0d0 on Ubuntu) and partition names like /dev/ida/c0d0p1 (or
>> /dev/cciss/c0d0p1).
>>
>> The attached patch fixes this issue.
> 
> Thank you.  I checked this in svn.  However...

Thanks.

>> P.S.: Note that the patch only fixes installation in the MBR, installing
>> lilo in a partition would need further fixes.
>> [...]
>>  # If installing to a partition (not MBR), offer to make it active
>>  # Do not try to make MD device active
>> -if (echo "${bootdev}" | grep -q '[0-9]$') && [ "${raid_boot}" = "no" ]; then
>> +if (echo "${bootdev}" | grep -v '/c[0-9]d[0-9]$' | grep -q '[0-9]$') \
>> +  && [ "${raid_boot}" = "no" ]; then
[...]
> This "grep -v '/c[0-9]d[0-9]$'" does indeed seem to break cciss / ida
> partitions.  Why did you add it?

This if-expression should return true, if $bootdev is a _partition_, not
a _disk_.

However, with SmartArray, _disks_ are named like /dev/cciss/c0d0, which
would also be matched by "grep -q '[0-9]$'". That's why I'm filtering
out this special case.
SmartArray _partitions_ look like /dev/cciss/c0d0p1, and are not
affected by my "grep -v".

Cheers,
--leo
-- 
e-mail   ::: Alexander.Bergolth (at) wu-wien.ac.at
fax      ::: +43-1-31336-906050
location ::: Computer Center | Vienna University of Economics | Austria




Reply to: