All of lore.kernel.org
 help / color / mirror / Atom feed
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



  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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.