PERFORCE change 128216 for review

John Birrell jb at FreeBSD.org
Sat Oct 27 17:28:58 PDT 2007


http://perforce.freebsd.org/chv.cgi?CH=128216

Change 128216 by jb at jb_freebsd1 on 2007/10/28 00:28:14

	The comment "Solaris defines _LITTLE_ENDIAN or _BIG_ENDIAN, but
	never both and decides which architecture it is based on which
	thing is defined." is true, but undefining _LITTLE_ENDIAN or
	_BIG_ENDIAN breaks lots of FreeBSD header files which all
	assume that both _LITTLE_ENDIAN and _BIG_ENDIAN are defined
	and that BYTE_ORDER is set to the one that matches the host.
	
	This was a hack to get ZFS working on FreeBSD, but it prevents
	DTrace code from using FreeBSD headers which include _BIG_ENDIAN
	and _LITTLE_ENDIAN.
	
	ZFS (and other Solaris) code will _have _to be changed to do what
	FreeBSD does: e.g. if BYTE_ORDER == _LITTLE_ENDIAN rather than
	ifdef _LITTLE_ENDIAN.
	
	Without this change, I can't share the OpenSolaris compatibility
	headers that pjd@ has created.

Affected files ...

.. //depot/projects/dtrace/src/sys/compat/opensolaris/machine/endian.h#2 delete

Differences ...


More information about the p4-projects mailing list