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