Bug#453749: Fix serial console detection on ia64

On Friday 30 November 2007, dann frazier wrote:
> The attached patch to di-utils adds a slightly modified version of

I compiled this on amd64 and after stripping it is still 4k. As it is only needed during finish install, I wonder if it should be in di-utils and thus in initrds.
Maybe we should create a di-utils-extra udeb that includes utils normally needed only after anna has been run. This could also include in-target for example. Not sure though.

> Bryan's utility. This utility was originally called getserialdev - I've
> renamed it to serial-console-info, since I think it better describes
> the function.

If I run the app on my system, I get an error code 255. It's probably correct that it returns an error, but I wonder if the value 255 is correct?

> The attached patch to finish-install adds support for using
> serial-console-info, if available, to 90console. Processing behaves
> just as before, until it finds that /proc/<di-pid>/fd/0 is pointing at
> /dev/console. It will only then run serial-console-info to see if it
> can determine that /dev/console is actually a serial device.

I think that, because of the "ttyS" assumption" this patch may break hvsi/hvc virtualized console support (bottom half of 90console). This should be checked.

I don't think there's any need to write the output of s-c-i to a temp file. Just assigning it to a variable should work just as well:

+if [ "$rawconsole" == "/dev/console" ] && \
+   [ -x /bin/serial-console-info ] && \
+   sci=$(/bin/serial-console-info); then
+	rawconsole="/dev/$(echo "$sci" | grep ^DEV= | cut -d= -f2)"
+	sci_baud=$(echo "$sci" | grep ^SPEED= | cut -d= -f2)
+fi

Cheers,
FJP

Do you need help?

In addition to our free technical library, Pantek provides professional IT services to companies who utilize Open Source software: Learn more about Pantek's Technical Support offerings Place an order for 24/7/365 Technical Support here Learn more about Pantek, who we are, and what we do Chat with a Live Operator Related lists.debian.org Links debian-announce debian-apache debian-arm-changes debian-beowulf debian-boot debian-cd debian-cd-vendors debian-changes debian-changes-digest debian-isp debian-kde debian-laptop debian-mirrors-announce debian-news debian-security debian-security-announce debian-testing debian-user debian-user-digest Request more information about Pantek

-- 
To UNSUBSCRIBE, email to debian-boot-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org


application/pgp-signature attachment: This is a digitally signed message part.