libgo patch committed: Fixes for select based pollster

Rainer Orth ro@CeBiTec.Uni-Bielefeld.DE
Thu Feb 7 16:51:00 GMT 2013


Ian Lance Taylor <iant@google.com> writes:

> On Solaris the libgo networking library uses select.  This patch
> implements some fixes for that code.  The Close method is changed to
> actually do something; this is mainly for testing purposes.  More
> importantly, when a socket is closed, the select needs to be restarted
> so that the callers see the close.  Bootstrapped and ran Go testsuite on
> x86_64-unknown-linux-gnu.  Committed to mainline.
>

Thanks, this got net to almost passing on Solaris: the only remaining
failure is

pollServer WaitFD: pollster closed
--- FAIL: TestMulticastListener (0.00 seconds)
multicast_posix_test.go:70:     "224.0.0.254:12345" not found in RIB
multicast_posix_test.go:75:     "224.0.0.254:12345" not found in RIB
multicast_posix_test.go:70:     "224.0.0.254:12345" not found in RIB
multicast_posix_test.go:75:     "224.0.0.254:12345" not found in RIB
Select unexpected fd=13 for write
FAIL
FAIL: net

which is no wonder since Solaris still uses interface_stub.go.  I've
checked what it takes to implement those interfaces, but it's quite an
effort, so for the moment we should just skip the test.

The following patch does this and allows the net test to pass.

	Rainer


2013-02-07  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

	PR go/56172
	* go/net/multicast_posix_test.go (TestMulticastListener): Skip on
	solaris.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: libgo-net.patch
Type: text/x-patch
Size: 606 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20130207/0fc145d5/attachment.bin>
-------------- next part --------------


-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University


More information about the Gcc-patches mailing list