From owner-freebsd-ports@FreeBSD.ORG Thu Feb 9 16:23:02 2006 Return-Path: X-Original-To: ports@FreeBSD.org Delivered-To: freebsd-ports@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C9DD116A422; Thu, 9 Feb 2006 16:23:02 +0000 (GMT) (envelope-from marcus@FreeBSD.org) Received: from av-tac-rtp.cisco.com (bantam.cisco.com [64.102.19.199]) by mx1.FreeBSD.org (Postfix) with ESMTP id EBA4743D68; Thu, 9 Feb 2006 16:23:00 +0000 (GMT) (envelope-from marcus@FreeBSD.org) X-TACSUNS: Virus Scanned Received: from rooster.cisco.com (localhost [127.0.0.1]) by av-tac-rtp.cisco.com (8.11.7p1+Sun/8.11.7) with ESMTP id k19GN0l28674; Thu, 9 Feb 2006 11:23:00 -0500 (EST) Received: from [64.102.192.190] (dhcp-64-102-192-190.cisco.com [64.102.192.190]) by rooster.cisco.com (8.11.7p1+Sun/8.11.7) with ESMTP id k19GMxm10008; Thu, 9 Feb 2006 11:22:59 -0500 (EST) Message-ID: <43EB6C7D.3020601@FreeBSD.org> Date: Thu, 09 Feb 2006 11:23:25 -0500 From: Joe Marcus Clarke Organization: FreeBSD, Inc. User-Agent: Thunderbird 1.5 (Macintosh/20051201) MIME-Version: 1.0 To: Jean-Yves Lefort References: <43E9FFAE.5020500@FreeBSD.org> <1139468008.54060.5.camel@shumai.marcuscom.com> <20060209131009.47dd1398.jylefort@FreeBSD.org> In-Reply-To: <20060209131009.47dd1398.jylefort@FreeBSD.org> X-Enigmail-Version: 0.93.2.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: ports@FreeBSD.org, ale@FreeBSD.org Subject: Re: gamin 0.1.7 X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 09 Feb 2006 16:23:03 -0000 -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Jean-Yves Lefort wrote: > On Thu, 09 Feb 2006 01:53:28 -0500 > Joe Marcus Clarke wrote: > >> On Wed, 2006-02-08 at 15:26 +0100, Alex Dupre wrote: >>> In the last days I worked on porting the latest release of gamin (0.1.7) >>> to FreeBSD (and cygwin, too, but this is for another mailing list ;-)). >>> If you don't want to read more, the result was successfully and the new >>> port may be downloaded for testing from >>> http://www.alexdupre.com/gamin.tar.gz. >>> If you are interested in porting work and changed internals go ahead. >>> The 0.1.7 release works (with about the same bugs of 0.1.5) >>> out-of-the-box only on Linux with kernel backends (dnotify/inotify). >>> Polling is broken, kqueue doesn't compile. I fixed all the bugs I found >>> and tried to make polling and kqueue work flawless. >>> Internally I replaced the read of the linux /etc/mtab file with a call >>> to getmntinfo() to know which file system could be monitored by the >>> kernel and which should be polled. In the latter case the monitoring job >>> is carried out by the polling backend, in the former the job is >>> completed by the kqueue backend in the same way is done actually in the >>> 0.1.5 release. >>> I did many tests on a 6-STABLE with thousands of monitored dirs/files on >>> ufs and smbfs file systems to catch any possible bug, but it seems it's >>> working great. >>> Unless you will find anything wrong with the port, I'm going to commit >>> it in the end of the week. >> I committed this with a few changes. Someone using NFS along with gamin >> might want to validate whether kqueue on NFS is working well. The same >> goes for smbfs, msdosfs, and ext2fs. > > Please address the following issues, or revert: > > - we now have two different pollers; one is used when > gam_kqueue_monitor_enable_kqueue() returns FALSE (for instance when > the fd limit is exhausted, or when kevent() fails); one is used for > "nfs" and "smbfs" filesystems > - the two pollers behave differently, compare: stat() vs lstat(), > gam_poll_generic_node_changed() vs gam_kqueue_differs(), > scalability, ... > - using filesystem names to choose between kqueue and polling is a > bad idea, for obvious reasons; one should use fstatfs() and enable > kqueue if the MNT_LOCAL flag is set > - testing no longer works: > make > cd $WRKDIR/tests > export GAMIN_DEBUG_SERVER=../server/gam_server > ./testgam - > connect test > -> it connects to the already running gam_server (the installed one) > - the patch which removed a stale socket has been dropped > I did a full make regression-test as well as some practical Nautilus tests before committing. I didn't notice the socket patch change. I'll try and address some of these other issues tonight. Joe - -- Joe Marcus Clarke FreeBSD GNOME Team :: gnome@FreeBSD.org FreeNode / #freebsd-gnome http://www.FreeBSD.org/gnome -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (Darwin) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFD62x9b2iPiv4Uz4cRArsfAKCEHC2h8Cgr4PzHOc6GvtBDyNz4owCgpLh3 4F5Q6JGqUL88AZvN+KgaOGU= =okh+ -----END PGP SIGNATURE-----