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-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-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.

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 g

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 | POLLH

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

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

2009-08-23 Thread Kostik Belousov
On Sun, Aug 23, 2009 at 11:38:00PM +0200, Jilles Tjoelker wrote: > 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 conform

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 POL

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 clo