Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 8 Jun 2000 19:51:16 +1000 (EST)
From:      Bruce Evans <bde@zeta.org.au>
To:        John-Mark Gurney <jmg@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/lib/libc/sys kqueue.2
Message-ID:  <Pine.BSF.4.21.0006081937330.468-100000@besplex.bde.org>
In-Reply-To: <200006072209.PAA95251@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 7 Jun 2000, John-Mark Gurney wrote:

> jmg         2000/06/07 15:09:16 PDT
> 
>   Modified files:
>     lib/libc/sys         kqueue.2 
>   Log:
>   add a BUGS section on how we can only watch VNODE's on a UFS file system
>   right now...
>   
>   I talked w/ phk last night and "fixing" this in a generic way is going
>   to require a lot of complex thought on stacking let alone the NFS problems..

It can be duplicated in other fs's, just like mounds of other code is
duplicated.  Duplication is especially easy in ext2fs where the surrounding
code was duplicated (except where the duplicates have rotted).

>   add missing sys/time.h for struct timespec def...

This is bogus.  <sys/event.h> forward-declares struct timespec so that
<sys/time.h> is not a prerequisite for <sys/event.h>.  <sys/time.h> only
needs to be included when the complete type for struct timespec is needed.
Even if kevent() is called, the complete type isn't needed unless the
timeout arg is non-NULL and is constructed in the same file.  Anyway,
POSIX.1-1996 says that struct timespec is declared in <time.h>.

Bruce



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0006081937330.468-100000>