public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: LGW <large@lilymarleen.de>
To: "Randy.Dunlap" <rddunlap@osdl.org>
Cc: alan@lxorguk.ukuu.org.uk, linux-kernel@vger.kernel.org
Subject: Re: porting driver to 2.6, partly solved
Date: Wed, 27 Aug 2003 19:25:01 +0200	[thread overview]
Message-ID: <3F4CE96D.2060807@lilymarleen.de> (raw)
In-Reply-To: <20030827092918.0981fa71.rddunlap@osdl.org>

Randy.Dunlap wrote:

>On Wed, 27 Aug 2003 18:15:51 +0200 LGW <large@lilymarleen.de> wrote:
>
>| Alan Cox wrote:
>| 
>| >On Mer, 2003-08-27 at 16:34, LGW wrote:
>| >  
>| >
>| >>The driver is mostly a wrapper around a generic driver released by the 
>| >>manufacturer, and that's written in C++. But it worked like this for the 
>| >>2.4.x kernel series, so I think it has something todo with the new 
>| >>module loader code. Possibly ld misses something when linking the object 
>| >>specific stuff like constructors?
>| >>    
>| >>
>| >
>| >The new module loader is kernel side, it may well not know some of the
>| >C++ specific relocation types. 
>| >
>| To you think it's possible to remove those relocations completely, so 
>| that the whole C++ stuff is linked "without" any more open relocations?
>
>Hopefully Rusty will see this and make some comments on it.
>
>You could try using objdump to look for the item(s) that have this
>relocation type (0).  That might help to see what is causing it.
>
>Or you could modify the module loader to ignore relocation type 0...
>and see what happens.
>  
>
Hm objdump -r snd-echoaudio.ko gives about six records (of course many 
others):
R_386_NONE    *ABS*

I'm not really sure what they refer to. Or, I don't even know. I think 
that the driver uses some system functions and they are linked like 
that, or so? I had a look and R_386_NONE means "no relocation" - so it 
should be safe to ignore them?

Modifying the kernel module loader like

switch (reloc_type) {
...
  case R_386_NONE:
    break;
...
}

did the trick, but this feels like a really really really UGLY trick.

Of course the best thing would be to rewrite the whole stuff, as it is 
more than hacked together, but I don't have time to do so, know. At 
least the soundcard device appears and plays sound now. I'll see what 
time brings.

Thanks for your support,
  Lars



  reply	other threads:[~2003-08-27 17:28 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-08-27 13:38 porting driver to 2.6, still unknown relocs... :( LGW
2003-08-27 15:13 ` Randy.Dunlap
2003-08-27 15:34   ` LGW
2003-08-27 15:50     ` Randy.Dunlap
2003-08-27 15:59     ` Alan Cox
2003-08-27 16:15       ` LGW
2003-08-27 16:29         ` Randy.Dunlap
2003-08-27 17:25           ` LGW [this message]
2003-08-27 17:07       ` Stephen Hemminger
2003-08-27 17:23         ` Richard B. Johnson
2003-08-27 17:33           ` LGW

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=3F4CE96D.2060807@lilymarleen.de \
    --to=large@lilymarleen.de \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rddunlap@osdl.org \
    /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