[openib-general] [Bug 146] OFED-1.0 DAPL fails to build on SLES10 on IA64 with IA64_FETCHADD error

[James Lentini]

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