Date: Tue, 01 Jan 2008 12:06:01 -0700 (MST) From: "M. Warner Losh" <imp@bsdimp.com> To: jb@FreeBSD.org Cc: perforce@FreeBSD.org Subject: Re: PERFORCE change 132153 for review Message-ID: <20080101.120601.106977236.imp@bsdimp.com> In-Reply-To: <200712310531.lBV5VvH3092205@repoman.freebsd.org> References: <200712310531.lBV5VvH3092205@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <200712310531.lBV5VvH3092205@repoman.freebsd.org>
John Birrell <jb@freebsd.org> writes:
: http://perforce.freebsd.org/chv.cgi?CH=132153
:
: Change 132153 by jb@jb_freebsd1 on 2007/12/31 05:31:02
:
: amd64 doesn't seem to have a program header in the kernel modules
: like i386 does. I don't know enough about amd64 to know if that's a
: problem or not. I have a feeling I'm not far off learning that. :-P
I believe that amd64 is the only one that doesn't...
Warner
: Affected files ...
:
: .. //depot/projects/dtrace/src/contrib/opensolaris/lib/libdtrace/common/dt_module.c#14 edit
:
: Differences ...
:
: ==== //depot/projects/dtrace/src/contrib/opensolaris/lib/libdtrace/common/dt_module.c#14 (text) ====
:
: @@ -927,6 +927,7 @@
: #if defined(sun)
: dmp->dm_modid = (int)OBJFS_MODID(st.st_ino);
: #else
: +#if defined(__i386__)
: /*
: * Find the first load section and figure out the relocation
: * offset for the symbols. The kernel module will not need
: @@ -939,6 +940,7 @@
: }
: }
: #endif
: +#endif
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080101.120601.106977236.imp>