This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: PATCH: Add --size-check=[error|warning]

From: Pekka Enberg <penberg at kernel dot org>
To: Ingo Molnar <mingo at elte dot hu>, Jan Beulich <JBeulich at novell dot com>, "H.J. Lu" <hjl dot tools at gmail dot com>, binutils at sourceware dot org, linux-kernel at vger dot kernel dot org, "H. Peter Anvin" <hpa at zytor dot com>, Andrew Morton <akpm at linux-foundation dot org>, Linus Torvalds <torvalds at linux-foundation dot org>, Thomas Gleixner <tglx at linutronix dot de>
Cc: Alan Modra <amodra at gmail dot com>
Date: Mon, 14 Mar 2011 12:50:44 +0200
Subject: Re: PATCH: Add --size-check=[error|warning]
References: <20110311165802.GA3508@intel.com> <4D7A64670200007800035F4C@vpn.id2.novell.com> <AANLkTikG8wa1Em0bEUddbYpYs2TzFFTDb95qWFJ3xSbv@mail.gmail.com> <4D7DE39302000078000362E6@vpn.id2.novell.com> <20110314095534.GB18058@elte.hu> <20110314104131.GG6275@bubble.grove.modra.org>

Hi Alan,

On Mon, Mar 14, 2011 at 12:41 PM, Alan Modra <amodra@gmail.com> wrote:
> On Mon, Mar 14, 2011 at 10:55:34AM +0100, Ingo Molnar wrote:
>> The thing is, it is absolutely, breath-takingy incompetent for
>
> kernel developers to write such poor asm! ?And not notice the error
> for 4 years! ?Oh, and the binutils developers to write such a poor
> assembler in the first place. ?;-)
>
> Seriously, you are complaining because something is fixed??
>
>> The correct solution is to turn it into a warning as me and others have suggested.
>
> I disagree. ?The whole world is not the linux kernel. ?I think HJ is
> bending over backwards to even offer a switch that turns the error
> into a warning.

So what do you suggest that testers who want to, say, build old Linux
kernel versions with new binutils do?

                        Pekka

Follow-Ups:
- Re: PATCH: Add --size-check=[error|warning]
  - From: Alan Cox

References:
- PATCH: Add --size-check=[error|warning]
  - From: H.J. Lu
- Re: PATCH: Add --size-check=[error|warning]
  - From: Jan Beulich
- Re: PATCH: Add --size-check=[error|warning]
  - From: H.J. Lu
- Re: PATCH: Add --size-check=[error|warning]
  - From: Jan Beulich
- Re: PATCH: Add --size-check=[error|warning]
  - From: Ingo Molnar
- Re: PATCH: Add --size-check=[error|warning]
  - From: Alan Modra

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]