All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yoshinori Sato <ysato@users.sourceforge.jp>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: vapier.adi@gmail.com, linux-kernel@vger.kernel.org,
	Jeremy Fitzhardinge <jeremy@goop.org>
Subject: Re: [PATCH] Missing symbol prefix on vmlinux.lds.h
Date: Wed, 30 Jul 2008 23:20:27 -0400	[thread overview]
Message-ID: <87sktqyc1w.wl%ysato@users.sourceforge.jp> (raw)
In-Reply-To: <20080730190659.fcf01dff.akpm@linux-foundation.org>

At Wed, 30 Jul 2008 19:06:59 -0700,
Andrew Morton wrote:
> 
> On Wed, 30 Jul 2008 20:26:24 -0400 Yoshinori Sato <ysato@users.sourceforge.jp> wrote:
> 
> > Sorry.
> > 
> > I got error ARCH=h8300.
> > init/main.c:781: undefined reference to `___early_initcall_end'
> > 
> > Same problem have
> > __start___bug_table
> > __stop___bug_table
> > __tracedata_start
> > __tracedata_end
> > __per_cpu_start
> > __per_cpu_end
> > 
> > If define symbol in vmlinux.lds, Use VMLINUX_SYMBOL macro.
> > VMLINUX_SYMBOL is add prefix charactor.
> > 
> > Signed-off-by: Yoshinori Sato <ysato@users.sourceforge.jp>
> > 
> > ---
> >  include/asm-generic/vmlinux.lds.h |   14 +++++++-------
> >  1 files changed, 7 insertions(+), 7 deletions(-)
> > 
> > diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
> > index 6d88a92..04cae96 100644
> > --- a/include/asm-generic/vmlinux.lds.h
> > +++ b/include/asm-generic/vmlinux.lds.h
> > @@ -333,9 +333,9 @@
> >  #define BUG_TABLE							\
> >  	. = ALIGN(8);							\
> >  	__bug_table : AT(ADDR(__bug_table) - LOAD_OFFSET) {		\
> > -		__start___bug_table = .;				\
> > +		VMLINUX_SYMBOL(__start___bug_table) = .;		\
> >  		*(__bug_table)						\
> > -		__stop___bug_table = .;					\
> > +		VMLINUX_SYMBOL(__stop___bug_table) = .;			\
> >  	}
> >  #else
> >  #define BUG_TABLE
> 
> Strange.  The code you're fixing there was added by
> 
>   commit 7664c5a1da4711bb6383117f51b94c8dc8f3f1cd
>   Author: Jeremy Fitzhardinge <jeremy@goop.org>
>   Date:   Fri Dec 8 02:36:19 2006 -0800
> 
>      [PATCH] Generic BUG implementation
>     
> 
> over a year ago.  I doubt if h8300 has been broken for that long, so
> something else must have triggered this failure.
> 
> Do you know what it was?

I don't know.
Check it.

-- 
Yoshinori Sato
<ysato@users.sourceforge.jp>

  reply	other threads:[~2008-07-31  3:20 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-30 15:58 [PATCH] blackfin / h8300 build error fix Yoshinori Sato
2008-07-30 16:11 ` Mike Frysinger
2008-07-30 19:22   ` Yoshinori Sato
2008-07-30 20:01     ` Andrew Morton
2008-07-31  0:26       ` [PATCH] Missing symbol prefix on vmlinux.lds.h Yoshinori Sato
2008-07-31  2:06         ` Andrew Morton
2008-07-31  3:20           ` Yoshinori Sato [this message]
2008-07-31  3:26       ` [PATCH] blackfin / h8300 build error fix Mike Frysinger
2008-07-31  3:37         ` Andrew Morton
2008-07-31  3:51           ` Mike Frysinger

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=87sktqyc1w.wl%ysato@users.sourceforge.jp \
    --to=ysato@users.sourceforge.jp \
    --cc=akpm@linux-foundation.org \
    --cc=jeremy@goop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=vapier.adi@gmail.com \
    /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.