In looking for a response to what they mean/do, the following message might explain things a bit http://lkml.org/lkml/2007/9/30/114 The thread below also illustrates this, I guess it was done to make the code more functional http://www.ussg.iu.edu/hypermail/linux/kernel/0709.3/2581.html In my humble opinion, the Makefile found in kmodule directory after martian-200804047.tar.gz is extracted is the one. However, it needs to be tested across several distributions and has to work for most of them if not all of them. I know that it works in at least two of the many distributions, Slax and Fedora namely. Regards, Antonio On Mon, Jun 9, 2008 at 5:30 PM, Marvin Stodolsky <marvin.stodolsky@xxxxxxxxx> wrote: > RE: > EXTRA_CFLAGS += -DKMARTIAN_STAMP=20061202 > EXTRA_CPPFLAGS += -DKMARTIAN_STAMP=20061202 > > I just need the syxtax for alternate choosing between these tol > depending on SUBLEVEL > EXTRA_CFLAGS and EXTRA_CPPFLAGS specifications during the compile, > the result on compiling for the different SUBLEVELs of the kernels is obvious. > > I'll do the martian-full-20080407 update and send it out for testing > once the Conditional syntax is resolved. > > MarvS > > While not fully understanding what the difference is between > > > On Mon, Jun 9, 2008 at 5:00 PM, Antonio Olivares > <olivares14031@xxxxxxxxx> wrote: >> It is the other way around Jacques. the EXTRA_CPPFLAGS is for kernels >> 2.6.24 and above while the other is for previous kernels pre 2.6.24. >> Take a look at grujakg's message in the thread while I will quote >> >> \begin{QUOTE} >> >> You would also need to keep in mind the patch for Makefile already >> applied to martian-full-20080407. >> >> diff -u -w -r martian-full-20080407/kmodule/Makefile >> martian/kmodule/Makefile >> --- martian-full-20080407/kmodule/Makefile 2008-04-08 >> 02:52:13.000000000 +0200 >> +++ martian/kmodule/Makefile 2006-12-03 00:39:24.000000000 +0100 >> @@ -1,14 +1,8 @@ >> KRELEASE = $(shell uname -r) >> - >> -ifdef KERNEL_DIR >> -KBUILD_DIR = $(KERNEL_DIR) >> -else >> KBUILD_DIR = /lib/modules/$(KRELEASE)/build >> -endif >> - >> MSRC_DIR = $(shell pwd) >> >> -EXTRA_CFLAGS += -DKMARTIAN_STAMP=20061202 >> +EXTRA_CPPFLAGS += -DKMARTIAN_STAMP=20061202 >> CPPFLAGS += -DKMARTIAN_STAMP=20061202 >> >> # kbuild >> >> Think that EXTRA_CFLAGS make a difference. >> \end{QUOTE} >> >> Regards, >> >> Antonio >> >> On 6/9/08, Jacques Goldberg <Jacques.Goldberg@xxxxxxx> wrote: >>> Marv, >>> I will try tomorrow but let me please candidly ask: >>> Is EXTRA_CPPFLAGS used for kernels < 24 and is EXTRA_CFLAGS used with 24 >>> and above? >>> This would mean that not the same compiler is used by the Makefile, or >>> that I missed something. >>> Why couldn't you then just define both? >>> Or do you have a typo and mean to define a different value for >>> DKMARTIAN_STAMP in the same variable EXTRA_CFLAGS or EXTRA_CPPFLAGS? >>> >>> Jacques >>> >>> Marvin Stodolsky wrote: >>>> Folks >>>> >>>> Is any erudite enough in Makefile syntax to help with a needed >>>> Conditional. >>>> >>>> With SUBLEVEL being the third component of the Kernel-Version >>>> which can be read out with >>>> SUBLEVEL := $(shell uname -r | cut -d. -f3 | cut -d. -f1 | cut -d- -f1) >>>> with cogency in the 22 24+ range >>>> >>>> a Makefile conditional is needed to use >>>> EXTRA_CFLAGS += -DKMARTIAN_STAMP=20061202 >>>> for SUBLEVEL < 24 , >>>> and for greater or equal 24 >>>> EXTRA_CPPFLAGS += -DKMARTIAN_STAMP=20061202 >>>> >>>> Otherwise we would need separate martian packages for 2.6.24 and later >>>> kernels >>>> versus 2.6.22 and earlier >>>> >>>> Sadly I'm not adequately knowledgeable in Makefile syntax. >>>> >>>> Incidentally, in the kmodule/Makefile , the role of this section: >>>> >>>> ifdef KERNEL_DIR >>>> KBUILD_DIR = $(KERNEL_DIR) >>>> else >>>> KBUILD_DIR = /lib/modules/$(KRELEASE)/build >>>> endif >>>> >>>> is to enable compiling for other kernels, other than the current boot >>>> kernel. >>>> The syntax is like: >>>> $ make KERNEL_DIR=/usr/src/linux-headers-SomeKernelVersion >>>> that is >>>> KERNEL_DIR is the path to the linux-headers for the desired kernel >>>> version. >>>> This saves rebooting when one wants to test compiling for several >>>> different linux-headers >>>> >>>> IIf the command is only: >>>> $ make >>>> then there is used the default /lib/modules/kernel-version-bootup/build >>>> >>>> Thus I will retain this section in the update. >>>> >>>> MarvS >>>> >>>> On Sat, Jun 7, 2008 at 3:08 PM, Ivica Grujic <grujakg@xxxxxxxxx> wrote: >>>>> Antonio I'm thinking that others should know about your good deed, so >>>>> I've >>>>> posted this on the thread also ;) >>>>> >>>>> You would also need to keep in mind the patch for Makefile already >>>>> applied to martian-full-20080407. >>>>> >>>>> diff -u -w -r martian-full-20080407/kmodule/Makefile >>>>> martian/kmodule/Makefile >>>>> --- martian-full-20080407/kmodule/Makefile 2008-04-08 >>>>> 02:52:13.000000000 +0200 >>>>> +++ martian/kmodule/Makefile 2006-12-03 00:39:24.000000000 +0100 >>>>> @@ -1,14 +1,8 @@ >>>>> KRELEASE = $(shell uname -r) >>>>> - >>>>> -ifdef KERNEL_DIR >>>>> -KBUILD_DIR = $(KERNEL_DIR) >>>>> -else >>>>> KBUILD_DIR = /lib/modules/$(KRELEASE)/build >>>>> -endif >>>>> - >>>>> MSRC_DIR = $(shell pwd) >>>>> >>>>> -EXTRA_CFLAGS += -DKMARTIAN_STAMP=20061202 >>>>> +EXTRA_CPPFLAGS += -DKMARTIAN_STAMP=20061202 >>>>> CPPFLAGS += -DKMARTIAN_STAMP=20061202 >>>>> >>>>> # kbuild >>>>> >>>>> Think that EXTRA_CFLAGS make a difference. >>>>> >>>>> This Makefile is all I had to keep, not to build but to successfully run >>>>> martian. >>>>> >>>>> Thanks again for great work. >>>>> >>>>> Regards >>>>> >>>>> grujakg >>>>> >>>>> Antonio Olivares wrote: >>>>>> If compiling fails, there is a file that asks for # linux/ioctl32.h> >>>>>> >>>>>> comment it out and the compilation will succeed. >>>>>> /* # linux/ioctl32.h> */ >>>>>> >>>>>> I forgot to mention this in the previous emails. >>>>>> >>>>>> Regards, >>>>>> >>>>>> Antonio >>>>>> >>>>>> --- On Sat, 6/7/08, Ivica Grujic <grujakg@xxxxxxxxx> wrote: >>>>>> >>>>>>> From: Ivica Grujic <grujakg@xxxxxxxxx> >>>>>>> Subject: Re: Fw: [Fwd: [Fwd: Re: Another Patch will be needed for >>>>>>> kernel >>>>>>> version 2.6.25]] >>>>>>> To: olivares14031@xxxxxxxxx >>>>>>> Date: Saturday, June 7, 2008, 8:27 AM >>>>>>> Yes, I got it. >>>>>>> >>>>>>> Thanks a lot. >>>>>>> >>>>>>> Regards, >>>>>>> >>>>>>> grujakg >>>>>>> >>>>>>> Antonio Olivares wrote: >>>>>>>> Done, >>>>>>>> >>>>>>>> please let me know if you receive it without problems. >>>>>>> :) >>>>>>>> Regards, >>>>>>>> >>>>>>>> Antonio >>>>>>>> >>> >> >
