mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@kernel.org>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: "Toralf Förster" <toralf.foerster@gmx.de>,
	ambx1@neo.rr.com, linux-kernel@vger.kernel.org,
	"Bjorn Helgaas" <bjorn.helgaas@hp.com>
Subject: Re: build issue #538 for v2.6.27-rc1 :undefined reference to `pnp_get_resource'
Date: Wed, 30 Jul 2008 14:22:11 +0300	[thread overview]
Message-ID: <20080730112211.GA19966@cs181140183.pp.htv.fi> (raw)
In-Reply-To: <20080730030344.25cba78a.akpm@linux-foundation.org>

On Wed, Jul 30, 2008 at 03:03:44AM -0700, Andrew Morton wrote:
> On Tue, 29 Jul 2008 18:11:46 +0200 Toralf F__rster <toralf.foerster@gmx.de> wrote:
> 
> > Hello,
> > 
> > the build (.config attached) failed, make ends with :
> > ...
> >   AS      arch/x86/lib/thunk_32.o
> >   CC      arch/x86/lib/usercopy_32.o
> >   AR      arch/x86/lib/lib.a
> >   LD      vmlinux.o
> >   MODPOST vmlinux.o
> > WARNING: modpost: Found 13 section mismatch(es).
> > To see full details build your kernel with:
> > 'make CONFIG_DEBUG_SECTION_MISMATCH=y'
> >   GEN     .version
> >   CHK     include/linux/compile.h
> >   UPD     include/linux/compile.h
> >   CC      init/version.o
> >   LD      init/built-in.o
> >   LD      .tmp_vmlinux1
> > drivers/built-in.o: In function `pnp_port_start':
> > /home/tfoerste/devel/linux-2.6/include/linux/pnp.h:51: undefined reference to `pnp_get_resource'
> > make: *** [.tmp_vmlinux1] Error 1
> > 
> 
> y:/usr/src/25> grep PNP .config 
> y:/usr/src/25> grep pnp_port_start $(find . -name '*.o')
> Binary file ./drivers/built-in.o matches
> Binary file ./drivers/telephony/built-in.o matches
> Binary file ./drivers/telephony/ixj.o matches
> Binary file ./vmlinux.o matches
> 
> I assume that drivers/telephony/ixj.c needs to depend on PNP.
>...

No, it does not need this dependency.

CONFIG_OPTIMIZE_INLINING broke it, and Bjorn has already done a fix 
(it's silly to get an out-of-line pnp_port_start() function only due to 
it having two callers in one source file, but that is what gcc 4.2 does 
as "optimized inlining"...).

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed


      reply	other threads:[~2008-07-30 11:23 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-29 16:11 Toralf Förster
2008-07-30 10:03 ` Andrew Morton
2008-07-30 11:22   ` Adrian Bunk [this message]

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=20080730112211.GA19966@cs181140183.pp.htv.fi \
    --to=bunk@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=ambx1@neo.rr.com \
    --cc=bjorn.helgaas@hp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=toralf.foerster@gmx.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

all inboxes | Powered by JetHome®