Re: svn commit: r196460 - head/sys/kern

2009-08-26 Thread Bruce Evans
On Tue, 25 Aug 2009, Jilles Tjoelker wrote: On Tue, Aug 25, 2009 at 04:07:11AM +1000, Bruce Evans wrote: On Sun, 23 Aug 2009, Jilles Tjoelker wrote: I think poll on fifos should instead be fixed by closing the half-connection corresponding to writing from fi_readsock to fi_writesock. I have

Re: svn commit: r196460 - head/sys/kern

2009-08-26 Thread Jilles Tjoelker
On Wed, Aug 26, 2009 at 05:08:57PM +1000, Bruce Evans wrote: On Tue, 25 Aug 2009, Jilles Tjoelker wrote: On Tue, Aug 25, 2009 at 04:07:11AM +1000, Bruce Evans wrote: On Sun, 23 Aug 2009, Jilles Tjoelker wrote: % Index: sys/fs/fifofs/fifo_vnops.c %

Re: svn commit: r196460 - head/sys/kern

2009-08-25 Thread Jilles Tjoelker
On Tue, Aug 25, 2009 at 04:07:11AM +1000, Bruce Evans wrote: On Sun, 23 Aug 2009, Jilles Tjoelker wrote: I think poll on fifos should instead be fixed by closing the half-connection corresponding to writing from fi_readsock to fi_writesock. I have tried this out, see attached patch. With

Re: svn commit: r196460 - head/sys/kern

2009-08-24 Thread Bruce Evans
On Sun, 23 Aug 2009, Jilles Tjoelker wrote: I think poll on fifos should instead be fixed by closing the half-connection corresponding to writing from fi_readsock to fi_writesock. I have tried this out, see attached patch. With the patch, pipepoll only gives expected POLLHUP; got POLLIN |

Re: svn commit: r196460 - head/sys/kern

2009-08-24 Thread Bruce Evans
On Tue, 25 Aug 2009, Bruce Evans wrote: On Sun, 23 Aug 2009, Jilles Tjoelker wrote: I think poll on fifos should instead be fixed by closing the half-connection corresponding to writing from fi_readsock to fi_writesock. I have tried this out, see attached patch. With the patch, pipepoll only

svn commit: r196460 - head/sys/kern

2009-08-23 Thread Konstantin Belousov
Author: kib Date: Sun Aug 23 12:44:15 2009 New Revision: 196460 URL: http://svn.freebsd.org/changeset/base/196460 Log: Fix the conformance of poll(2) for sockets after r195423 by returning POLLHUP instead of POLLIN for several cases. Now, the tools/regression/poll results for FreeBSD are

Re: svn commit: r196460 - head/sys/kern

2009-08-23 Thread Jilles Tjoelker
On Sun, Aug 23, 2009 at 12:44:15PM +, Konstantin Belousov wrote: Author: kib Date: Sun Aug 23 12:44:15 2009 New Revision: 196460 URL: http://svn.freebsd.org/changeset/base/196460 Log: Fix the conformance of poll(2) for sockets after r195423 by returning POLLHUP instead of POLLIN

Re: svn commit: r196460 - head/sys/kern

2009-08-23 Thread Bruce Evans
On Sun, 23 Aug 2009, Jilles Tjoelker wrote: On Sun, Aug 23, 2009 at 12:44:15PM +, Konstantin Belousov wrote: ... Modified: head/sys/kern/uipc_socket.c == --- head/sys/kern/uipc_socket.c Sun Aug 23 12:23:24 2009