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 <[EMAIL PROTECTED]>

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__        \

_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to