Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 06 Oct 2011 01:35:14 +0000
From:      Nali Toja <nalitoja@gmail.com>
To:        Ali Mashtizadeh <mashtizadeh@gmail.com>
Cc:        fidaj@ukr.net, Alexey Dokuchaev <danfe@freebsd.org>, gobledb@gmail.com, yanegomi@gmail.com, Olivier Smedts <olivier@gid0.org>, freebsd-current@freebsd.org
Subject:   Re: x11/nvidia-driver / Compilation has failed
Message-ID:  <86vcs2q4vh.fsf@gmail.com>
In-Reply-To: <CAFL8=tjnhGjg%2BUh4TCx65nLtqjHzhUzg4deqdF5ei-Gf6u6GOg@mail.gmail.com> (Ali Mashtizadeh's message of "Wed, 5 Oct 2011 13:52:24 -0700")
References:  <4e4ab684.0c4c970a.68e8.6846SMTPIN_ADDED@mx.google.com> <9A6F7C9D-32D8-4387-BB03-B11CB0A91BFA@gmail.com> <CAGH67wS=KgH19CaQMFKaDxN%2BFB4n-05fwyZo_mTdt-o5vSsK-A@mail.gmail.com> <20110829.214640.1568838937885804931.ken@tydfam.jp> <CABzXLYORe5HpJQq%2BUCtZ3OfSQpBAuWTeLUX0yXvAVzk5QXp6jw@mail.gmail.com> <20110831124126.GB82908@FreeBSD.org> <CAFL8=tjnhGjg%2BUh4TCx65nLtqjHzhUzg4deqdF5ei-Gf6u6GOg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
--=-=-=
Content-Type: text/plain

Ali Mashtizadeh <mashtizadeh@gmail.com> writes:

> 2011/8/31 Alexey Dokuchaev <danfe@freebsd.org>:
>> On Mon, Aug 29, 2011 at 02:59:48PM +0200, Olivier Smedts wrote:
>>> 2011/8/29 ken <ken@tydfam.jp>:
>>> > Could I test your patch for nvidia-driver, too?
>>> > I cannot find your patch in this mail.
>>>
>>> I took the patch in :
>>> http://lists.freebsd.org/pipermail/freebsd-current/2011-August/026515.html
>>>
>>> And it worked for me.
>>
>> Should be fixed in the port itself now (also updated to 280.13).
>
> Is there any reason I should still be hitting this bug when building
> on 9-STABLE? With Linux compatibility disabled I can build the driver,
> but the kernel refuses to load it saying it's incompatible with my
> kernel version.

Only if you're using 285.05.09 with the port. And it'd affect both
/stable/9 and /head users.

  // from src/nv-freebsd.h:
  #if __FreeBSD_version >= 900041
  #include <sys/capability.h>
  #else
  #define fget(td, fd, cap, fp) fget(td, fd, fp)
  #endif


--=-=-=
Content-Type: message/rfc822
Content-Disposition: inline

X-From-Line: nobody Tue Oct  4 09:44:48 2011
From: Nali Toja <nalitoja@gmail.com>
To: danfe@FreeBSD.org
Subject: [patch] x11/nvidia-driver: add 285.05.09 support
Date: Tue, 04 Oct 2011 05:44:48 +0000
Message-ID: <86pqidmhtb.fsf@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain

Can you commit below tiny change? It should make testing the new version a
bit easier for people who are impatient to wait for the next port update.

That version also includes tunable support similar to ports/156386.

  $ make
  [...]
  nvidia_linux.c:40:51: error: too many arguments to function call, expected 4, have 5
      if ((error = fget(td, args->fd, CAP_IOCTL, 0, &fp)) != 0)
                   ~~~~                             ^~~
  @/sys/file.h:186:1: note: 'fget' declared here
  int fget(struct thread *td, int fd, cap_rights_t rights, struct file **fpp);
  ^
  1 error generated.

Index: x11/nvidia-driver/Makefile
===================================================================
RCS file: /a/.csup/ports/x11/nvidia-driver/Makefile,v
retrieving revision 1.109
diff -u -p -r1.109 Makefile
--- x11/nvidia-driver/Makefile	31 Aug 2011 12:30:24 -0000	1.109
+++ x11/nvidia-driver/Makefile	4 Oct 2011 05:33:52 -0000
@@ -112,7 +113,7 @@ post-patch: .SILENT
 	${REINPLACE_CMD} -e 's/status != status/status != RM_OK/' \
 		${WRKSRC}/src/nvidia_os.c
 # Fix the build after fget(9) KPI was changed in r224778
-.if ${OSVERSION} > 900040 && ${NVVERSION} >= 1952200
+.if ${OSVERSION} > 900040 && ${NVVERSION} >= 1952200 && ${NVVERSION} < 2850509
 	${REINPLACE_CMD} -e '/fget/s/&fp/0, &/' ${WRKSRC}/src/nvidia_linux.c
 .endif
 .if defined(WITH_FREEBSD_AGP)

--=-=-=--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?86vcs2q4vh.fsf>