PERFORCE change 132153 for review

M. Warner Losh imp at bsdimp.com
Tue Jan 1 11:08:25 PST 2008


In message: <200712310531.lBV5VvH3092205 at repoman.freebsd.org>
            John Birrell <jb at freebsd.org> writes:
: http://perforce.freebsd.org/chv.cgi?CH=132153
: 
: Change 132153 by jb at 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


More information about the p4-projects mailing list