From: Keith Owens <kaos@ocs.com.au>
To: "Maciej W. Rozycki" <macro@ds2.pg.gda.pl>
Cc: Ralf Baechle <ralf@uni-koblenz.de>,
Harald Koerfgen <hkoerfg@web.de>,
linux-mips@fnet.fr, linux-mips@oss.sgi.com
Subject: Re: [patch] modutils 2.4.6: Make __dbe_table available to modules
Date: Tue, 14 Aug 2001 01:14:02 +1000 [thread overview]
Message-ID: <16145.997715642@ocs3.ocs-net> (raw)
In-Reply-To: Your message of "Mon, 13 Aug 2001 16:49:22 +0200." <Pine.GSO.3.96.1010813164151.18279K-100000@delta.ds2.pg.gda.pl>
On Mon, 13 Aug 2001 16:49:22 +0200 (MET DST),
"Maciej W. Rozycki" <macro@ds2.pg.gda.pl> wrote:
>On Tue, 14 Aug 2001, Keith Owens wrote:
>> The only other change you have to make is to init_modules(). For mips
>> you create pointers to the kernel dbe tables and fill in archdata start
>> and end in kernel_module. Since init_module is called before kmalloc
>> is ready, make the kernel dbe table a static variable.
>
> __dbe_table is initialized exactly like __ex_table, i.e. it's a separate
>ELF section with pointers to the start and the end computed in a linker
>script. Thus it is fine. The table has actually been present in the
>kernel for quite some time already.
You still need this:
struct archdata {
const struct exception_table_entry *dbe_table_start;
const struct exception_table_entry *dbe_table_end;
};
In init_module:
#ifdef __mips__
{
extern const struct exception_table_entry __start___dbe_table[];
extern const struct exception_table_entry __stop___dbe_table[];
static struct archdata archdata_mips =
{ __start___dbe_table, __end___dbe_table };
kernel_module.archdata_start = (char *)&archdata_mips;
kernel_module.archdata_end = kernel_module.archdata_start +
sizeof(archdata_mips);
}
#endif
I really need to add an arch specific init_module routine as well,
instead of conditional code in init_module, but that means changing all
architectures. Not today.
next prev parent reply other threads:[~2001-08-13 15:14 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-08-13 13:43 [patch] modutils 2.4.6: Make __dbe_table available to modules Maciej W. Rozycki
2001-08-13 14:19 ` Keith Owens
2001-08-13 14:49 ` Maciej W. Rozycki
2001-08-13 15:14 ` Keith Owens [this message]
2001-08-20 14:04 ` [patch] modutils 2.4.6: __dbe_table iteration #2 Maciej W. Rozycki
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=16145.997715642@ocs3.ocs-net \
--to=kaos@ocs.com.au \
--cc=hkoerfg@web.de \
--cc=linux-mips@fnet.fr \
--cc=linux-mips@oss.sgi.com \
--cc=macro@ds2.pg.gda.pl \
--cc=ralf@uni-koblenz.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