x11/nvidia-driver build failure in head/i386 @r253985 with clang

2013-08-07 Thread David Wolfskill
Builds/runs OK (so far) in stable/9/i386 @r254053 with clang.

Whine is:
...
clang -O2 -pipe -fno-strict-aliasing -DNV_VERSION_STRING=\319.32\ 
-D__KERNEL__ -DNVRM -Wno-unused-function -Wuninitialized -O -UDEBUG -U_DEBUG 
-DNDEBUG -Werror -D_KERNEL -DKLD_MODULE -nostdinc  -I. -I. -I@ -I@/contrib/altq 
-fno-common   -mno-aes -mno-avx -mno-mmx -mno-sse -msoft-float -ffreestanding 
-fstack-protector -std=iso9899:1999 -Qunused-arguments -fstack-protector -Wall 
-Wredundant-decls -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes 
-Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign 
-fformat-extensions  -Wmissing-include-dirs -fdiagnostics-show-option  
-Wno-error-tautological-compare -Wno-error-empty-body  
-Wno-error-parentheses-equality  -c nvidia_sysctl.c
--- nvidia_subr.o ---
nvidia_subr.c:948:33: error: incompatible pointer types passing 'vm_map_t' (aka 
'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
address = kmem_alloc_contig(kernel_map, size, flags, 0,
^~
@/vm/vm_extern.h:54:44: note: passing argument to parameter here
vm_offset_t kmem_alloc_contig(struct vmem *, vm_size_t size, int flags,
   ^
nvidia_subr.c:997:19: error: incompatible pointer types passing 'vm_map_t' (aka 
'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
kmem_free(kernel_map,
  ^~
@/vm/vm_extern.h:58:29: note: passing argument to parameter here
void kmem_free(struct vmem *, vm_offset_t, vm_size_t);
^
nvidia_subr.c:1024:15: error: incompatible pointer types passing 'vm_map_t' 
(aka 'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
kmem_free(kernel_map, at-pte_array[0].virtual_address,
  ^~
@/vm/vm_extern.h:58:29: note: passing argument to parameter here
void kmem_free(struct vmem *, vm_offset_t, vm_size_t);
^
nvidia_subr.c:1088:37: error: incompatible pointer types passing 'vm_map_t' 
(aka 'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
address = kmem_alloc_contig(kernel_map, PAGE_SIZE, flags, 0,
^~
@/vm/vm_extern.h:54:44: note: passing argument to parameter here
vm_offset_t kmem_alloc_contig(struct vmem *, vm_size_t size, int flags,
   ^
nvidia_subr.c:1142:19: error: incompatible pointer types passing 'vm_map_t' 
(aka 'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
kmem_free(kernel_map,
  ^~
@/vm/vm_extern.h:58:29: note: passing argument to parameter here
void kmem_free(struct vmem *, vm_offset_t, vm_size_t);
^
nvidia_subr.c:1172:19: error: incompatible pointer types passing 'vm_map_t' 
(aka 'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
kmem_free(kernel_map,
  ^~
@/vm/vm_extern.h:58:29: note: passing argument to parameter here
void kmem_free(struct vmem *, vm_offset_t, vm_size_t);
^
6 errors generated.
*** [nvidia_subr.o] Error code 1

make[3]: stopped in 
/common/ports/x11/nvidia-driver/work/NVIDIA-FreeBSD-x86-319.32/src
1 error



I am presently updating head to r254052 and that process will attempt
to rebuild x11/nvidia-driver.  If that's successful, I will follow
up.

In the mean time, I will try to figure out what's wrong:
nvidia-driver-304.51 built/ran OK in head/i386 @r253985 (with clang).

Peace,
david
-- 
David H. Wolfskill  da...@catwhisker.org
Taliban: Evil men with guns afraid of truth from a 14-year old girl.

See http://www.catwhisker.org/~david/publickey.gpg for my public key.


pgpWMLn6bcJLv.pgp
Description: PGP signature


Re: x11/nvidia-driver build failure in head/i386 @r253985 with clang

2013-08-07 Thread Alexey Dokuchaev
On Wed, Aug 07, 2013 at 06:02:41AM -0700, David Wolfskill wrote:
 [...]
 nvidia_subr.c:997:19: error: incompatible pointer types passing 'vm_map_t'
 (aka 'struct vm_map *') to parameter of type 'struct vmem *' [-Werror,
 -Wincompatible-pointer-types]
 kmem_free(kernel_map,
   ^~
 @/vm/vm_extern.h:58:29: note: passing argument to parameter here
 void kmem_free(struct vmem *, vm_offset_t, vm_size_t);

I've tested the new driver on my Julyish -CURRENT and it was all fine...
I suspect the problem might be with Jeff's r254025 (CC'ed).  I will see
what I can do about it, thanks for your report!

 In the mean time, I will try to figure out what's wrong:
 nvidia-driver-304.51 built/ran OK in head/i386 @r253985 (with clang).

Please share your findings once you have them. :-)

./danfe
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: x11/nvidia-driver build failure in head/i386 @r253985 with clang

2013-08-07 Thread David Wolfskill
On Wed, Aug 07, 2013 at 01:20:34PM +, Alexey Dokuchaev wrote:
 On Wed, Aug 07, 2013 at 06:02:41AM -0700, David Wolfskill wrote:
  [...]
  nvidia_subr.c:997:19: error: incompatible pointer types passing 'vm_map_t'
  (aka 'struct vm_map *') to parameter of type 'struct vmem *' [-Werror,
  -Wincompatible-pointer-types]
  kmem_free(kernel_map,
^~
  @/vm/vm_extern.h:58:29: note: passing argument to parameter here
  void kmem_free(struct vmem *, vm_offset_t, vm_size_t);
 
 I've tested the new driver on my Julyish -CURRENT and it was all fine...
 I suspect the problem might be with Jeff's r254025 (CC'ed).  I will see
 what I can do about it, thanks for your report!

Glad to heko!

  In the mean time, I will try to figure out what's wrong:
  nvidia-driver-304.51 built/ran OK in head/i386 @r253985 (with clang).
 
 Please share your findings once you have them. :-)

Build still fails in head/i386 @r254052, similarly:

...
clang -O2 -pipe -fno-strict-aliasing -DNV_VERSION_STRING=\319.32\ 
-D__KERNEL__ -DNVRM -Wno-unused-function -Wuninitialized -O -UDEBUG -U_DEBUG 
-DNDEBUG -Werror -D_KERNEL -DKLD_MODULE -nostdinc  -I. -I. -I@ -I@/contrib/altq 
-fno-common   -mno-aes -mno-avx -mno-mmx -mno-sse -msoft-float -ffreestanding 
-fstack-protector -std=iso9899:1999 -Qunused-arguments -fstack-protector -Wall 
-Wredundant-decls -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes 
-Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign 
-fformat-extensions  -Wmissing-include-dirs -fdiagnostics-show-option  
-Wno-error-tautological-compare -Wno-error-empty-body  
-Wno-error-parentheses-equality  -c nvidia_subr.c
nvidia_subr.c:948:33: error: incompatible pointer types passing 'vm_map_t' (aka 
'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
address = kmem_alloc_contig(kernel_map, size, flags, 0,
^~
@/vm/vm_extern.h:54:44: note: passing argument to parameter here
vm_offset_t kmem_alloc_contig(struct vmem *, vm_size_t size, int flags,
   ^
nvidia_subr.c:997:19: error: incompatible pointer types passing 'vm_map_t' (aka 
'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
kmem_free(kernel_map,
  ^~
@/vm/vm_extern.h:58:29: note: passing argument to parameter here
void kmem_free(struct vmem *, vm_offset_t, vm_size_t);
^


Not sure how much further I'll be able to get for a while; I'm
out-of-town and Internet access is a bit flaky.  Sorry...

Peace,
david
-- 
David H. Wolfskill  da...@catwhisker.org
Taliban: Evil men with guns afraid of truth from a 14-year old girl.

See http://www.catwhisker.org/~david/publickey.gpg for my public key.


pgpLVjhSrTObE.pgp
Description: PGP signature


Re: x11/nvidia-driver build failure in head/i386 @r253985 with clang

2013-08-07 Thread Jeff Roberson

On Wed, 7 Aug 2013, David Wolfskill wrote:


On Wed, Aug 07, 2013 at 01:20:34PM +, Alexey Dokuchaev wrote:

On Wed, Aug 07, 2013 at 06:02:41AM -0700, David Wolfskill wrote:

[...]
nvidia_subr.c:997:19: error: incompatible pointer types passing 'vm_map_t'
(aka 'struct vm_map *') to parameter of type 'struct vmem *' [-Werror,
-Wincompatible-pointer-types]
kmem_free(kernel_map,
  ^~
@/vm/vm_extern.h:58:29: note: passing argument to parameter here
void kmem_free(struct vmem *, vm_offset_t, vm_size_t);


I've tested the new driver on my Julyish -CURRENT and it was all fine...
I suspect the problem might be with Jeff's r254025 (CC'ed).  I will see
what I can do about it, thanks for your report!


Glad to heko!


In the mean time, I will try to figure out what's wrong:
nvidia-driver-304.51 built/ran OK in head/i386 @r253985 (with clang).


Please share your findings once you have them. :-)


Build still fails in head/i386 @r254052, similarly:

...
clang -O2 -pipe -fno-strict-aliasing -DNV_VERSION_STRING=\319.32\ 
-D__KERNEL__ -DNVRM -Wno-unused-function -Wuninitialized -O -UDEBUG -U_DEBUG -DNDEBUG 
-Werror -D_KERNEL -DKLD_MODULE -nostdinc  -I. -I. -I@ -I@/contrib/altq -fno-common   
-mno-aes -mno-avx -mno-mmx -mno-sse -msoft-float -ffreestanding -fstack-protector 
-std=iso9899:1999 -Qunused-arguments -fstack-protector -Wall -Wredundant-decls 
-Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline 
-Wcast-qual  -Wundef -Wno-pointer-sign -fformat-extensions  -Wmissing-include-dirs 
-fdiagnostics-show-option  -Wno-error-tautological-compare -Wno-error-empty-body  
-Wno-error-parentheses-equality  -c nvidia_subr.c
nvidia_subr.c:948:33: error: incompatible pointer types passing 'vm_map_t' (aka 
'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
   address = kmem_alloc_contig(kernel_map, size, flags, 0,
   ^~
@/vm/vm_extern.h:54:44: note: passing argument to parameter here
vm_offset_t kmem_alloc_contig(struct vmem *, vm_size_t size, int flags,
  ^
nvidia_subr.c:997:19: error: incompatible pointer types passing 'vm_map_t' (aka 
'struct vm_map *') to parameter of type 'struct vmem *' 
[-Werror,-Wincompatible-pointer-types]
   kmem_free(kernel_map,
 ^~
@/vm/vm_extern.h:58:29: note: passing argument to parameter here
void kmem_free(struct vmem *, vm_offset_t, vm_size_t);
   ^


Not sure how much further I'll be able to get for a while; I'm
out-of-town and Internet access is a bit flaky.  Sorry...



This is my fault.  The first argument to the kmem_ functions should now be 
kernel_arena or kmem_arena.  I don't know how to modify ports but I can 
produce a patch later today to resolve this unless someone beats me to it. 
It will be a simple substitution based on version.


Jeff



Peace,
david
--
David H. Wolfskill  da...@catwhisker.org
Taliban: Evil men with guns afraid of truth from a 14-year old girl.

See http://www.catwhisker.org/~david/publickey.gpg for my public key.


___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: x11/nvidia-driver build failure in head/i386 @r253985 with clang

2013-08-07 Thread David Wolfskill
On Wed, Aug 07, 2013 at 08:53:02AM -1000, Jeff Roberson wrote:
 ...

  Not sure how much further I'll be able to get for a while; I'm
  out-of-town and Internet access is a bit flaky.  Sorry...
 
 This is my fault.  The first argument to the kmem_ functions should now be 
 kernel_arena or kmem_arena.  I don't know how to modify ports but I can 
 produce a patch later today to resolve this unless someone beats me to it. 
 It will be a simple substitution based on version.
 ...

I'll be happy to test.

Peace,
david
-- 
David H. Wolfskill  da...@catwhisker.org
Taliban: Evil men with guns afraid of truth from a 14-year old girl.

See http://www.catwhisker.org/~david/publickey.gpg for my public key.


pgp9fE7OMxyg4.pgp
Description: PGP signature


Re: x11/nvidia-driver build failure in head/i386 @r253985 with clang

2013-08-07 Thread Alexey Dokuchaev
On Wed, Aug 07, 2013 at 08:53:02AM -1000, Jeff Roberson wrote:
 This is my fault.  The first argument to the kmem_ functions should
 now be kernel_arena or kmem_arena.  I don't know how to modify ports
 but I can produce a patch later today to resolve this unless someone
 beats me to it. It will be a simple substitution based on version.

Yep, it is a simple substitution, our users also figured it out.  No
need for a patch, I will adopt the one in PR ports/181118.  Luckily,
__FreeBSD_version was bumped just couple of days ago, otherwise I'd
had to harass Jeff about not bumping it along with API breakage. ;-)

./danfe
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org


Re: x11/nvidia-driver build failure in head/i386 @r253985 with clang

2013-08-07 Thread Alexey Dokuchaev
On Wed, Aug 07, 2013 at 02:03:13PM -0700, David Wolfskill wrote:
 On Wed, Aug 07, 2013 at 08:53:02AM -1000, Jeff Roberson wrote:
  ...
   Not sure how much further I'll be able to get for a while; I'm
   out-of-town and Internet access is a bit flaky.  Sorry...
  
  This is my fault.  The first argument to the kmem_ functions should now be 
  kernel_arena or kmem_arena.  I don't know how to modify ports but I can 
  produce a patch later today to resolve this unless someone beats me to it. 
  It will be a simple substitution based on version.
  ...
 
 I'll be happy to test.

Should be fixed now as of r324376.

./danfe
___
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to freebsd-ports-unsubscr...@freebsd.org