Re: CVS commit: src/share/man/man4

2016-01-06 Thread Christos Zoulas
In article , Paul Goyette wrote: > >Yeah, this is workable, even if it is a HACK ! :) > >The attached patch borrows extensively from fd_free() routine in >kern/kern_descrip.c > >Let me know if this looks "good enough" and

Re: CVS commit: src/share/man/man4

2016-01-06 Thread Christos Zoulas
In article , Christos Zoulas wrote: > >As a temporary hack it is probably good enough. Longer term filemon >should be removed/rewritten, and the close ordering problem should >be handled. I changed my mind; it does not help because one can

Re: CVS commit: src/share/man/man4

2016-01-06 Thread Taylor R Campbell
Date: Wed, 6 Jan 2016 12:00:32 +0800 (PHT) From: Paul Goyette On Wed, 6 Jan 2016, Paul Goyette wrote: > Hmmm. I'm looking at the filemon_open() code. It seems to have a "fd" > variable that gets set by fd_allocfile(). The value is later passed to >

Re: CVS commit: src/share/man/man4

2016-01-06 Thread Paul Goyette
On Wed, 6 Jan 2016, Christos Zoulas wrote: As a temporary hack it is probably good enough. Longer term filemon should be removed/rewritten, and the close ordering problem should be handled. I think that a "better" approach here is probably to remove filemon(4)'s SET_FD ioctl (for the