public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Norton Allen <allen@huarp.harvard.edu>
To: Ken Brown <kbrown@cornell.edu>, cygwin <cygwin@cygwin.com>
Subject: Re: Unix Domain Socket Limitation?
Date: Fri, 4 Dec 2020 08:51:02 -0500	[thread overview]
Message-ID: <a13ab85d-bee7-71e3-41d0-1a67422a859f@huarp.harvard.edu> (raw)
In-Reply-To: <1c1e875a-40a0-ff9e-a119-ba77203e43ea@cornell.edu>

On 12/3/2020 8:11 PM, Ken Brown wrote:
> On 12/2/2020 12:30 PM, Norton Allen wrote:
>> On 11/30/2020 9:22 PM, Norton Allen wrote:
>>> Yeah, so now the example no longer blocks for me. Unfortunately 
>>> these bugs are not present in my application, so I will need to keep 
>>> working on this.
>>>
>>
>> After paring the main application down and back up, I finally 
>> narrowed in on the condition that was causing this blocking behavior. 
>> The issue arises when a client connect()s twice to the same server 
>> with non-blocking unix-domain sockets before calling select().
>>
>> There are a few pieces to this. With the client configured to 
>> connect() just once, I can see that the server's select() returns as 
>> soon as the client calls connect(), but then the server's accept() 
>> blocks until the client calls select(). That is not proper 
>> non-blocking behavior, but it appears that the implementation under 
>> Cygwin does require that client and server both be communicating 
>> synchronously to accomplish the connect() operation.
>>
>> I tried running this under Ubuntu 16.04 and found that connect() 
>> succeeded immediately, so no subsequent select() is required, and 
>> there does not appear to be a possibility for this collision. That 
>> proves to hold true even if the server is not waiting in select() to 
>> process the connect() with accept().
>>
>> A workaround for this issue may be to keep the socket blocking until 
>> after connect().
>>
>> I have pushed the new minimal example program,  'rapid_connects' to 
>> https://github.com/nthallen/cygwin_unix
>>
>> The server is run like before as:
>>
>>     $ ./rapid_connects server
>>
>> The client can be run in two different modes. To connect with just 
>> one socket:
>>
>>     $ ./rapid_connects client1
>>
>> To connect with two:
>>
>>     $ ./rapid_connects client2
>>
>> My immediate strategy will be to develop a workaround for my project. 
>> Having spent a day inside cygwin1.dll, I can see that I have a steep 
>> learning curve to make much of a contribution there.
>
> I'm traveling at the moment and unable to do any testing, but I wonder 
> if you're bumping into an issue that was just discussed on the 
> cygwin-developers list:
>
> https://cygwin.com/pipermail/cygwin-developers/2020-December/012015.html
>
> A different workaround is described there.
>
> If it's the same issue, then I don't think it will happen with the new 
> AF_UNIX implementation.  More in a few days.
>
It does seem related.

A work around that is working for me is to do a blocking connect() and 
switch to non-blocking when that completes. In my application, the 
connect() generally occurs once at the beginning of a run, so blocking 
for a few milliseconds does not impact responsiveness.



  reply	other threads:[~2020-12-04 13:51 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-25 21:47 Norton Allen
2020-11-25 22:27 ` Ken Brown
     [not found]   ` <4260ad1b-4ab2-fa36-fd0e-7c9644560114@huarp.harvard.edu>
2020-11-26 17:13     ` Ken Brown
2020-11-30 17:19       ` Norton Allen
2020-11-30 18:14         ` Ken Brown
2020-11-30 18:26           ` Norton Allen
2020-11-30 23:19             ` Ken Brown
2020-12-01  2:14               ` Norton Allen
2020-12-01  2:22                 ` Norton Allen
2020-12-02 17:30                   ` Norton Allen
2020-12-04  1:11                     ` Ken Brown
2020-12-04 13:51                       ` Norton Allen [this message]
2020-12-05 23:52                         ` Ken Brown
2020-12-06 17:17                           ` Norton Allen
2020-12-06 22:32                             ` Ken Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=a13ab85d-bee7-71e3-41d0-1a67422a859f@huarp.harvard.edu \
    --to=allen@huarp.harvard.edu \
    --cc=cygwin@cygwin.com \
    --cc=kbrown@cornell.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).