Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 31 Dec 2007 05:31:57 GMT
From:      John Birrell <jb@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 132153 for review
Message-ID:  <200712310531.lBV5VvH3092205@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
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

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
 
 	if (dmp->dm_info.objfs_info_primary)
 		dmp->dm_flags |= DT_DM_PRIMARY;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200712310531.lBV5VvH3092205>