public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Luca Olivetti <luca@ventoso.org>
To: Daniel Walker <dwalker@fifo99.com>
Cc: Ingo Molnar <mingo@elte.hu>, Greg KH <gregkh@suse.de>,
	Mauro Carvalho Chehab <mchehab@infradead.org>,
	linux-kernel@vger.kernel.org, Hans Verkuil <hverkuil@xs4all.nl>,
	Janne Grunau <janne-dvb@grunau.be>
Subject: Re: [crash] af9005_usb_module_init(): BUG: unable to handle kernel paging request at ff100000
Date: Wed, 04 Feb 2009 16:49:39 +0100	[thread overview]
Message-ID: <4989B913.1020702@ventoso.org> (raw)
In-Reply-To: <1233760607.15119.85.camel@desktop>

En/na Daniel Walker ha escrit:

>> Doesn't symbol_request return a NULL in such a case?
>> At the time I didn't try the above configuration 
>> (CONFIG_DVB_USB_AF9005_REMOTE=n), but removed the compiled 
>> dvb-usb-af9005-remote.ko, and all was well (i.e. the symbol_request 
>> returned NULL and the remote handling was disabled).
> 
> I'm not sure what symbol request returns when CONFIG_MODULES=n .. Since
> it basically drops symbol_request() into a macro , 

Take the following with a grain of salt, since I'm not well versed in C 
(and I'm not interested) but it seems that the macro, if 
CONFIG_MODULES=n, transforms it into a weak symbol reference

> and you have externs
> for all these functions 

No, I havent: the source file isn't compiled, so those symbols are 
unavailable to the linker.
The only reference I could find (well, I haven't searched that much) on 
the expected behaviour of the linker when a weak attribute is not found 
is this:

http://software.intel.com/en-us/articles/software-convention-models-using-elf-visibility-attributes/

"# STB_WEAK. A weak symbol behaves as does a global symbol, with a few 
differences. If there are both a weak and a global definition of a name, 
the global definition takes precedence and any weak definitions are 
ignored. The Linker will not extract archive members to resolve 
undefined weak symbols. It is not an error to have unresolved weak 
references; unresolved weak symbols have the value zero. "

So it should work without your patch.

Bye

-- 
Luca


  reply	other threads:[~2009-02-04 15:49 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-03 17:28 [crash] af9005_usb_module_init(): BUG: unable to handle kernel paging request at ff100000 Ingo Molnar
2009-02-03 17:45 ` Arjan van de Ven
2009-02-03 18:22 ` Daniel Walker
2009-02-03 19:30   ` Ingo Molnar
2009-02-03 20:41     ` Luca Olivetti
2009-02-03 21:18       ` Luca Olivetti
2009-02-03 21:32       ` Ingo Molnar
2009-02-04  1:14       ` Daniel Walker
2009-02-04 14:52         ` Luca Olivetti
2009-02-04 15:16           ` Daniel Walker
2009-02-04 15:49             ` Luca Olivetti [this message]
2009-02-04 16:12               ` Daniel Walker
2009-02-04 18:12                 ` Luca Olivetti
2009-02-04 18:30 ` Daniel Walker
2009-02-04 18:41   ` Luca Olivetti
2009-02-04 19:27     ` Daniel Walker

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=4989B913.1020702@ventoso.org \
    --to=luca@ventoso.org \
    --cc=dwalker@fifo99.com \
    --cc=gregkh@suse.de \
    --cc=hverkuil@xs4all.nl \
    --cc=janne-dvb@grunau.be \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=mingo@elte.hu \
    /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