[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <17789.54777.283849.950002@cargo.ozlabs.ibm.com>
Date: Tue, 12 Dec 2006 09:04:41 +1100
From: Paul Mackerras <paulus@...ba.org>
To: Linus Torvalds <torvalds@...l.org>
Cc: Olaf Hering <olaf@...fle.de>,
Herbert Poetzl <herbert@...hfloor.at>,
Andy Whitcroft <apw@...dowen.org>, Andi Kleen <ak@...e.de>,
Andrew Morton <akpm@...l.org>, linux-kernel@...r.kernel.org,
Steve Fox <drfickle@...ibm.com>, linuxppc-dev@...abs.org
Subject: Re: 2.6.19-git13: uts banner changes break SLES9 (at least)
Linus Torvalds writes:
> On Mon, 11 Dec 2006, Olaf Hering wrote:
> >
> > arch/powerpc/boot/wrapper:156: version=`${CROSS}strings "$kernel" | grep '^Linux version [-0-9.]' | \
>
> This is also obviously broken (and really sad), but actually ends up being
> better than what get_kernel_version apparently does, by at least adding
> the requirement that the string "Linux version" be slightly more correct.
>
> However, it's also TOTALLY BROKEN.
It's the minimum effort for the barely acceptable outcome. :)
The wrapper script, although it currently lives in arch/powerpc/boot,
is designed and intended to be standalone, so that people can use it
outside the kernel tree, and possibly even without having the kernel
source easily to hand. Therefore I didn't want to use any kernel
header files.
Apparently the only reason "mkimage" wants to know the kernel version
is to put it as a comment in the image, which can be displayed to the
user when booting with u-boot (the bootloader used on some embedded
platforms). So it's not critical if the grep fails, and it's slightly
more useful to do the grep than it would be to not even try to provide
any version string to mkimage.
If there is a reliable way to get the version string, great, I'll use
that.
Paul.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists