vos: Remove pthread send/receive select()
authorAndrew Deason <adeason@sinenomine.net>
Wed, 23 Oct 2013 20:32:19 +0000 (15:32 -0500)
committerDerrick Brashear <shadow@your-file-system.com>
Sat, 26 Oct 2013 12:59:54 +0000 (05:59 -0700)
commit4af32591f05ce2d3237c0aeaa785799d11680152
treef830489ff6617a40e17ee7a41c5eddd0ea47b903
parentf7c31f865cd6cda9105a362f26fdd82a500adb37
vos: Remove pthread send/receive select()

Commit 65e701fee4968b17066bb81e25b7adaa4024d4f3 introduced a couple of
select() calls when the pthreaded vos writes to or reads from a usd
pipe, corresponding to the IOMGR_Select() calls in LWP. However, these
select() calls are unnecessary, since the read() or write() calls
themselves will block anyway. The reason they are there for LWP is so
the IOMGR can run another process while we're waiting for the file
descriptor to be ready.

The select() in ReceiveFile currently incorrectly waits for the output
to be ready for reading, even though we're trying to write to it. As a
result, if we try to 'vos dump' to a pipe with the pthreaded vos (such
as stdout), we will hang forever, since it will never be ready for
reading.

To fix this, just get rid of the select() calls, since they don't
really do anything.

FIXES 131749

Change-Id: Ibe8841e0c01f1e55ac4ca1a8a99ab71083654662
Reviewed-on: http://gerrit.openafs.org/10360
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
src/volser/vos.c