From: Eric Valette <eric.valette@free.fr>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: linux-kernel@vger.kernel.org, Ingo Molnar <mingo@elte.hu>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: Typo in linux-2.6.38/arch/x86/kernel/entry_32.S
Date: Wed, 16 Mar 2011 21:53:29 +0100 [thread overview]
Message-ID: <4D812349.1050501@free.fr> (raw)
In-Reply-To: <1300308282.16880.57.camel@gandalf.stny.rr.com>
On 16/03/2011 21:44, Steven Rostedt wrote:
> On Wed, 2011-03-16 at 21:32 +0100, Eric Valette wrote:
>
>
>>> The old binutils never complained.
>>
>> Anyway this show once again that the world like symetry and you better
>> remember it ;-) (lock/unlock, malloc free, ...)
>
> Right, but it should be a warning not an error. If it worked before, why
> not work now and just warn about it. This "BUG" existed forever in the
> kernel, and its in more than one place. Having the compiler crash on the
> first instance means you need to fix it and run the compiler again to
> find the next instance and so on.
Yes like any other name change in a .h file with incompatible function
signature...
The exact error message is a complaint about the .size directive not
being a "constant"...
> binutils made a fatal error in making this a bug and not an warning.
>
>>
>> Probably a stable candidate for 2.6.38.1
>
> Perhaps, but what about all older kernels. Now this version of binutils
> does not work for bisecting.
I guess its binutils git of the 2011/03/02
As far as I'm concerned I only found one occurence on one machine
although I compiled tree 2.6.38 kernels for different machines.
because its under the CONFIG_KVM_GUEST flag
--eric
next prev parent reply other threads:[~2011-03-16 20:53 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-16 13:15 Typo in linux-2.6.38/arch/x86/kernel/entry_32.S Eric Valette
2011-03-16 19:52 ` Steven Rostedt
2011-03-16 20:09 ` Eric Vlette
2011-03-16 20:22 ` Steven Rostedt
2011-03-16 20:32 ` Eric Valette
2011-03-16 20:44 ` Steven Rostedt
2011-03-16 20:53 ` Eric Valette [this message]
2011-03-17 8:31 ` Ingo Molnar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4D812349.1050501@free.fr \
--to=eric.valette@free.fr \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=rostedt@goodmis.org \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox