Re: x11/nvidia-driver-173 build failed on CURRENT

[redirecting to ports_at_, where this belongs]

On 2013-05-06 08:19, sig6247 wrote:
...
> cc -O2 -pipe -fno-strict-aliasing -DNV_VERSION_STRING=\"173.14.35\" -D__KERNEL__ -DNVRM -UDEBUG -U_DEBUG -DNDEBUG -O -Werror -D_KERNEL -DKLD_MODULE -nostdinc  -I/wrkdirs/usr/ports/x11/nvidia-driver-173/work/NVIDIA-FreeBSD-x86-173.14.35/src -I. -I_at_ -I_at_/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_os.c
> nvidia_os.c:28:23: error: function declared with regparm(0) attribute was previously declared without the regparm attribute
> RM_STATUS NV_API_CALL os_alloc_contig_pages(
>                        ^
> /wrkdirs/usr/ports/x11/nvidia-driver-173/work/NVIDIA-FreeBSD-x86-173.14.35/src/nv-freebsd.h:145:11: note: previous declaration is here
> RM_STATUS os_alloc_contig_pages(void **, U032);
>            ^

Please try the attached patch.  I am not sure if there are more driver
versions that include these inconsisent prototypes, but if anybody is
aware of them, we can adjust the ${NVVERSION} check a little.

-Dimitry