dtraceall.ko with old nfsclient

[Andrew Boyer]

On Jul 12, 2012, at 3:39 PM, Andriy Gapon wrote:

> on 12/07/2012 22:36 Fabian Keil said the following:
>> Andriy Gapon <avg at FreeBSD.org> wrote:
>> 
>>> on 12/07/2012 21:17 Fabian Keil said the following:
>>>> Benjamin Kaduk <kaduk at MIT.EDU> wrote:
>>>> 
>>>>> On Wed, 11 Jul 2012, Fabian Keil wrote:
>>>>> 
>>>>>> I'm using the following modification of Sean's patch:
>>>> 
>>>> This way it seems to work as expected:
>>>> 
>>>> diff --git a/sys/modules/dtrace/dtraceall/Makefile 
>>>> b/sys/modules/dtrace/dtraceall/Makefile index 456efd1..628583b 100644
>>>> --- a/sys/modules/dtrace/dtraceall/Makefile +++ 
>>>> b/sys/modules/dtrace/dtraceall/Makefile @@ -1,7 +1,7 @@ # $FreeBSD: 
>>>> src/sys/modules/dtrace/dtraceall/Makefile,v 1.3 2011/04/09 09:07:31 uqs
>>>> Exp $
>>>> 
>>>> KMOD=          dtraceall -SRCS=          dtraceall.c opt_compat.h
>>>> +SRCS= dtraceall.c opt_compat.h opt_nfs.h
>>>> 
>>>> CFLAGS+=       -I${.CURDIR}/../../..
>>>> 
>>> 
>>> If you do cd sys/modules/dtrace/dtraceall && make [obj depend] all, does
>>> it compile OK with the above change?
>> 
>> Depends on your expectations I guess. As neither NFS-related option gets
>> defined, no dependency on either NFS module is registered. The compiler has
>> no complaints, though.
> 
> Interesting.  Could you repeat after sufficient cleaning up?
> I am not sure where from opt_nfs.h file could come.
> 

Maybe related: check out sys/modules/ipfw/Makefile.  It makes its own option headers for INET and INET6.

-A

--------------------------------------------------
Andrew Boyer	aboyer at averesystems.com