Date: Mon, 09 Sep 2013 16:49:23 -0600 From: Ian Lepore <ian@FreeBSD.org> To: "O. Hartmann" <ohartman@zedat.fu-berlin.de> Cc: Ivan Klymenko <fidaj@ukr.net>, FreeBSD CURRENT <freebsd-current@FreeBSD.org>, FreeBSD Ports <freebsd-ports@FreeBSD.org> Subject: Re: CURRENT r255426: x11/nvidia-driver: nvidia_subr.c:835:46: error: too few arguments to function call, expected 10, have 9 (VM_PROT_READ | VM_PROT_WRITE), 0); Message-ID: <1378766963.1111.579.camel@revolution.hippie.lan> In-Reply-To: <20130910004240.70bc281f@thor.walstatt.dyndns.org> References: <20130909223748.61dd8859@thor.walstatt.dyndns.org> <20130910000237.6b5604da@nonamehost.local> <20130909231713.34ebfbdd@thor.walstatt.dyndns.org> <20130910002405.4aa701d0@nonamehost.local> <20130910004240.70bc281f@thor.walstatt.dyndns.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 2013-09-10 at 00:42 +0200, O. Hartmann wrote: > On Tue, 10 Sep 2013 00:24:05 +0300 > Ivan Klymenko <fidaj@ukr.net> wrote: >=20 > > =F7 Mon, 9 Sep 2013 23:17:13 +0200 > > "O. Hartmann" <ohartman@zedat.fu-berlin.de> =D0=C9=DB=C5=D4: > >=20 > > > On Tue, 10 Sep 2013 00:02:37 +0300 > > > Ivan Klymenko <fidaj@ukr.net> wrote: > > >=20 > > > > =F7 Mon, 9 Sep 2013 22:37:48 +0200 > > > > "O. Hartmann" <ohartman@zedat.fu-berlin.de> =D0=C9=DB=C5=D4: > > > > > /usr/ports/emulators/virtualbox-ose-kmod/work/VirtualBox-4.2.1= 8/out/freebsd.amd64/release/bin/src/vboxdrv/r0drv/freebsd/alloc-r0drv-fre= ebsd.c:83:76: > > > > > error: too few arguments to function call, expected 10, have 9 > > > > > cbAllocated, TRUE, VM_PROT_ALL, VM_PROT_ALL, 0); ^ > > > > > @/vm/vm_map.h:368:1: note: 'vm_map_find' declared here int > > > > > vm_map_find(vm_map_t, vm_object_t, vm_ooffset_t, vm_offset_t *, > > > > > vm_size_t, > > > >=20 > > > > Try the following changes: > > > > int rc =3D vm_map_find(kernel_map, pVmObject, 0, &Addr, > > > > --- cbAllocated, TRUE, VM_PROT_ALL, > > > > VM_PROT_ALL, 0); +++ cbAllocated, 0, > > > > VMFS_OPTIMAL_SPACE, VM_PROT_ALL, VM_PROT_ALL, 0); > > > >=20 > > > > for Nvidia driver need to make similar changes... > > > The man page of vm_map_find() says the last parameter is of int, > > > named int cow. On the hurry, I didn't find any explanation of that > > > parameter. Setting it to "0" (zero) also in the nvidia_subr.c makes > > > the driver compile again. > > >=20 > > > But simply filling in a int zero is a bit strange without knowing > > > what to do, isn't it? > > >=20 > > > Thanks anyway, > > >=20 > > > Oliver > >=20 > > This commit you to find an appropriate example ... > > http://docs.freebsd.org/cgi/getmsg.cgi?fetch=3D291456+0+current/svn-s= rc-head > > And in this case, the driver Nvidia really need to specify 0 in place > > of the sixth argument ... > > but I'm not sure at 100% :) >=20 > I have a patch attached for the x11/nvidia-driver Makefile. It doesn't > work and I loose hairs due to not knowing why. >=20 > When issuing the following command sequence on the console: >=20 > cd /usr/ports/x11/nvidia-driver > make clean extract > sed -i -e '/(VM_PROT_READ | VM_PROT_WRITE), 0);$/s/0);$/0, 0);/g' \ > work/NVIDIA-FreeBSD-x86_64-325.15/src/nvidia_subr.c >=20 >=20 > and check then line 835 (the corrupt one) in file=20 > work/NVIDIA-FreeBSD-x86_64-325.15/src/nvidia_subr.c, >=20 > I see this BEFORE: >=20 > status =3D vm_map_find(kernel_map, at->object, (i * PAGE_SIZE), > &virtual_address, size, VMFS_ANY_SPACE, > (VM_PROT_READ | VM_PROT_WRITE), > =3D=3D=3D(835)>> (VM_PROT_READ | VM_PROT_WRITE), 0); >=20 >=20 > and I see this AFTER the sed'ed replacement: >=20 > status =3D vm_map_find(kernel_map, at->object, (i * PAGE_SIZE), > &virtual_address, size, VMFS_ANY_SPACE, > (VM_PROT_READ | VM_PROT_WRITE), > =3D=3D=3D(835)>> (VM_PROT_READ | VM_PROT_WRITE), 0, 0); >=20 > BUT: Using the patch (see attached, please apply to > x11/nvidia-driver/Makefile) gives me an error in x11/nvidia-driver: >=20 > make clean patch >=20 > =3D=3D=3D> Cleaning for nvidia-driver-325.15 > =3D=3D=3D> License NVIDIA accepted by the user > =3D=3D=3D> Found saved configuration for nvidia-driver-325.08_1 > =3D=3D=3D> nvidia-driver-325.15 depends on file: /usr/local/sbin/pkg = - found > =3D=3D=3D> Fetching all distfiles required by nvidia-driver-325.15 for > building > =3D=3D=3D> Extracting for nvidia-driver-325.15 > =3D> SHA256 Checksum OK for NVIDIA-FreeBSD-x86_64-325.15.tar.gz. > =3D=3D=3D> Patching for nvidia-driver-325.15 > sed: 1: "/(VM_PROT_READ | VM_PRO ...": invalid command code 0 > *** Error code 1 >=20 > The Shell/make should be able to substitute within a single-quotet > command to sed, but the error message reports differently. >=20 > Do not be confused about the driver revision I use. I tried the > official one (319.XX) as well and it is not working the very same way > and the patch won't either. >=20 > Oliver I don't know about the sed error, but I think the change you're trying to make is wrong. The new '0' parameter to vm_map_find() isn't added at the end, it goes after the size parameter. -- Ian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1378766963.1111.579.camel>