[openib-general] Re: [PATCH][DAPL] fix kdapltest Makefile

[Tom Duffy]

On Tue, 2005-05-03 at 15:07 -0400, James Lentini wrote:
> Tom,
> 
> Do you also want to remove the first EXTRA_CLAGS value:
> 
>        -Idrivers/dat             \
> 
> or did you leave that there on purpose?
> 
> You shouldn't need the
> 
>        -I$(src)/../../../dat-provider           \
> 
> line. DAT consumers only need to be aware of the registry (the code in 
> the dat directory).

OK, fair enough, although they should be harmless.  Updated patch below.

Signed-off-by: Tom Duffy <tduffy at sun.com>

Index: Makefile
===================================================================
--- Makefile	(revision 2248)
+++ Makefile	(working copy)
@@ -37,7 +37,7 @@
 ifeq ($(origin KERNELRELEASE), undefined)
 
 KDAPLTEST_ROOT  = `pwd` # directory that contains this makefile 
-KERNEL_ROOT     = /lib/modules/`uname -r`/build # root of kernel sources
+KERNEL_ROOT     ?= /lib/modules/`uname -r`/build # root of kernel sources
 
 # Tools
 ECHO            = /bin/echo
@@ -45,7 +45,7 @@ RM              = /bin/rm
 
 all :
 	@$(ECHO) "--- Compiling kdapltest module ---"
-	$(MAKE) -C $(KERNEL_ROOT) M=$(KDAPLTEST_ROOT) V=1 modules
+	$(MAKE) -C $(KERNEL_ROOT) M=$(KDAPLTEST_ROOT) modules
 
 clean :
 	$(RM) -rf \
@@ -67,14 +67,17 @@ ifeq (${IS_i686},i686)
 # Override -msoft-float in arch/i386/Makefile
 EXTRA_CFLAGS += -mhard-float
 endif
+ifeq (${IS_i686},x86_64)
+EXTRA_CFLAGS += -msse
+endif
 
 EXTRA_CFLAGS += \
-    -Idrivers/dat		\
     -I$(src)/../include        	\
     -I$(src)/../mdep/linux 	\
     -I$(src)/../kdapl          	\
     -I$(src)/../common 		\
-    -I$(src)/../test 
+    -I$(src)/../test 		\
+    -I$(src)/../../../dat 
 
 EXTRA_CFLAGS += \
     -D__KDAPL__ 	\