Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Wolfgang Denk
Dear Paul Mundt, In message 20090102095023.ga28...@linux-sh.org you wrote: Your main reasons against inclusion of perl seem to be that there is no realistic expectation for target systems that will be self-hosting will have perl included, or the inherent complexity in maintaining a coherent

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Rob Landley
On Friday 02 January 2009 10:04:08 Matthieu CASTET wrote: Rob Landley a écrit : On Friday 02 January 2009 03:26:37 Arkadiusz Miskiewicz wrote: On Friday 02 of January 2009, Rob Landley wrote: Heh, I believe all three scripts run under dash and busybox ash. (The timeconst.sh one

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Rob Landley
On Friday 02 January 2009 12:01:34 Sam Ravnborg wrote: But the serie rased anohter topic: shall we ban use of perl for generating a kernel. I dunno about ban, but every time somebody adds perl to the hot path of the kernel build it breaks my build system, and I write a removal patch anyway.

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread H. Peter Anvin
Sam Ravnborg wrote: With respect to your three patches the plan is to: - add the updated timeconst patch to kbuild-next If you add this, you take the responsibility for the breakages that will occur. The reason his patch is simpler is because he removes the arbitrary-precision arithmetic, and

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Leon Woestenberg
Hello all, On Fri, Jan 2, 2009 at 9:07 AM, Rob Landley r...@landley.net wrote: Before 2.6.25 (specifically git bdc807871d58285737d50dc6163d0feb72cb0dc2 ) building a Linux kernel never required perl to be installed on the build system. (Various development and debugging scripts were written in

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Robert Hancock
Rob Landley wrote: For the record, the reason I can't just pregenerate all these suckers on a system that's got an arbitrary precision calculator (ala dc) and then just ship the resulting header files (more or less the what the first version of that first patch did) is that some architectures

PATCH [0/3]: Simplify the kernel build by removing perl v2

2009-01-03 Thread Rob Landley
Here's an updated set of patches to remove use of perl from the kernel build's hot path (roughly defined as make allnoconfig; make; make headers_install). This update incorporates feedback from Sam Ravnborg, Ted Tso, Joe Perches, Ingo Oeser, and others. It also fixes an integer overflow error

PATCH [1/3]: Replace kernel/timeconst.pl with kernel/timeconst.sh (v2)

2009-01-03 Thread Rob Landley
From: Rob Landley r...@landley.net Replace kernel/timeconst.pl with kernel/timeconst.sh. The new shell script is much simpler, about 1/4 the size, and runs on Red Hat 9 from 2003. It requires a shell which can do 64 bit math, such as bash, busybox ash, or dash running on a 64 bit host. Changes

PATCH [2/3]: Remove perl from make headers_install.

2009-01-03 Thread Rob Landley
From: Rob Landley r...@landley.net Remove perl from make headers_install by replacing a perl script (doing a simple regex search and replace) with a smaller and faster shell script implementation. The new shell script is a single for loop calling sed and piping its output through unifdef to

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Rob Landley
On Friday 02 January 2009 10:04:08 Matthieu CASTET wrote: Rob Landley a écrit : On Friday 02 January 2009 03:26:37 Arkadiusz Miskiewicz wrote: On Friday 02 of January 2009, Rob Landley wrote: Heh, I believe all three scripts run under dash and busybox ash. (The timeconst.sh one

Re: [PATCH 1/3]: Replace kernel/timeconst.pl with kernel/timeconst.sh

2009-01-03 Thread Rob Landley
On Friday 02 January 2009 13:33:02 H. Peter Anvin wrote: Rob Landley wrote: You mean The new shell script is much simpler, about 1/4 the size, runs on Red Hat 9 from 2003, and isn't perl? :) And introduces unclear environment dependencies depending on how external utilities are

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Rob Landley
On Friday 02 January 2009 13:27:45 H. Peter Anvin wrote: Sam Ravnborg wrote: Hi Wookey. Given the simplicitly of these patches I can't see any reason not to put them in Please do NOT do the mistake and think this the same thing. Rob's patch simplyfy the timecost stuff - and will

Re: [PATCH 1/3]: Replace kernel/timeconst.pl with kernel/timeconst.sh

2009-01-03 Thread H. Peter Anvin
Rob Landley wrote: I consider this a step up from code with an implicit dependency on a CPAN library. There is no CPAN library in use. Math::BigInt is a standard part of Perl, and the canned values is there only to support extremely old versions of Perl, or weird system configurations, as

Re: [PATCH 1/3]: Replace kernel/timeconst.pl with kernel/timeconst.sh

2009-01-03 Thread Rob Landley
On Saturday 03 January 2009 06:28:22 Ingo Oeser wrote: +for i in MSEC 1000 USEC 100 +do + NAME=$(echo $i | awk '{print $1}') cut -d' ' -f1 does the same + PERIOD=$(echo $i | awk '{print $2}') cut -d' ' -f2 does the same From a standards perspective

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread David Brownell
On Saturday 03 January 2009, Robert Hancock wrote: Rob Landley wrote: ... some architectures (arm omap and and arm at91) allow you to enter arbitrary HZ values in kconfig. (Their help text says that in many cases values that aren't powers of two won't work, but nothing enforces

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Rob Landley
On Saturday 03 January 2009 17:03:11 H. Peter Anvin wrote: Leon Woestenberg wrote: I agree with Rob that the amount of required dependencies should be kept to a minimum. If we only use 0.5% of a certain language (or: dependent package), then rather implement that 0.5% in the existing

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread H. Peter Anvin
Rob Landley wrote: The new patches have *more* environmental dependencies than that ever did. Could you please be a little more specific? In this case, you're assuming that every version of every shell this is going to get involved with is going to do math correctly with the requisite

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Rob Landley
On Friday 02 January 2009 08:04:09 Theodore Tso wrote: Sounds like though modulo dealing with 64-bit arithmetic, your patches are mostly dash/POSIX.2 comformant, so you're probably mostly good on that front once you address the 32/64-bit issues. I'd also suggest explicitly add a reminder to

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Jamie Lokier
Rob Landley wrote: This doesn't _need_ bignum support. It maxes out around 72 bits and the _result_ can't use more than about $SHIFT bits because you're dividing by the amount you shifted, so just chop off the bottom 32 bits, do a normal 64 bit division on the top (it has to fit), and then

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread H. Peter Anvin
Jamie Lokier wrote: Related query: Does the Perl script being replaced use 64-bit arithmetic? Because many Perl installations only do 32-bit arithmetic. If the Perl version works in 32-bit arithmetic, why does the shell version not do the same thing? The Perl version uses

Re: PATCH [1/3]: Replace kernel/timeconst.pl with kernel/timeconst.sh (v2)

2009-01-03 Thread David Vrabel
Rob Landley wrote: From: Rob Landley r...@landley.net Replace kernel/timeconst.pl with kernel/timeconst.sh. The new shell script is much simpler, about 1/4 the size, and runs on Red Hat 9 from 2003. It requires a shell which can do 64 bit math, such as bash, busybox ash, or dash running on a

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Markus Heidelberg
Rob Landley, 04.01.2009: On Saturday 03 January 2009 18:37:12 Leon Woestenberg wrote: My argument on thin dependencies indeed mostly holds for run-time dependencies (to reduce size) but also for build dependency (to reduce complexity)*. I usually just point to the gnucash 1.6 release as

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Rob Landley
On Saturday 03 January 2009 21:38:13 Markus Heidelberg wrote: Rob Landley, 04.01.2009: Now that you mention this the second time, I have to ask where you have this information from. Since I use Gentoo, I was always able to compile OpenOffice (version 1, 2 and now 3) myself. The gentoo panel

Re: [PATCH 1/3]: Replace kernel/timeconst.pl with kernel/timeconst.sh

2009-01-03 Thread Valdis . Kletnieks
On Sat, 03 Jan 2009 19:36:04 CST, Rob Landley said: On Saturday 03 January 2009 06:28:22 Ingo Oeser wrote: +for i in MSEC 1000 USEC 100 +do + NAME=$(echo $i | awk '{print $1}') cut -d' ' -f1 does the same + PERIOD=$(echo $i | awk '{print $2}') cut -d' ' -f2 does the

Re: PATCH [0/3]: Simplify the kernel build by removing perl.

2009-01-03 Thread Rob Landley
On Saturday 03 January 2009 20:14:44 H. Peter Anvin wrote: Rob Landley wrote: The new patches have *more* environmental dependencies than that ever did. Could you please be a little more specific? In this case, you're assuming that every version of every shell this is going to get

Re: [PATCH 1/3]: Replace kernel/timeconst.pl with kernel/timeconst.sh

2009-01-03 Thread Rob Landley
On Saturday 03 January 2009 23:07:55 valdis.kletni...@vt.edu wrote: On Sat, 03 Jan 2009 19:36:04 CST, Rob Landley said: On Saturday 03 January 2009 06:28:22 Ingo Oeser wrote: +for i in MSEC 1000 USEC 100 +do + NAME=$(echo $i | awk '{print $1}') cut -d' ' -f1 does