On Tue, 11 Jul 2006, John Partridge wrote:
> James Lentini wrote: > > This is included on IA64 and PPC systems. Since we have not done testing on > > IA64 or PPC, I'm certain that this was a contribution for a IA64 or PPC DAPL > > user. For that reason, I'm not certain why the asm/system.h was included. It > > is certainly possible that it is necessary on PPC but not IA64. > > > > James, > > Thanks your patch has fixed the build errors. I've checked in the patch below to solve this problem: Index: dapl/udapl/linux/dapl_osd.h =================================================================== --- dapl/udapl/linux/dapl_osd.h (revision 8253) +++ dapl/udapl/linux/dapl_osd.h (working copy) @@ -80,6 +80,8 @@ #if defined(__ia64__) || defined(__PPC64__) #include <asm/atomic.h> +#endif +#if defined(__PPC64__) #include <asm/system.h> #endif Index: Makefile.am =================================================================== --- Makefile.am (revision 8253) +++ Makefile.am (working copy) @@ -1,10 +1,11 @@ # $Id: $ +OSFLAGS = -DOS_RELEASE=$(shell expr `uname -r | cut -f1 -d.` \* 65536 + `uname -r | cut -f2 -d.`) # Check for RedHat, needed for ia64 udapl atomic operations (IA64_FETCHADD syntax) if OS_RHEL -OSFLAGS=-DREDHAT_EL4 +OSFLAGS += -DREDHAT_EL4 else -OSFLAGS= +OSFLAGS += endif if DEBUG _______________________________________________ openib-general mailing list [email protected] http://openib.org/mailman/listinfo/openib-general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
