X-Recipient: archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status: No, hits=-3.5 required=5.0	tests=AWL,BAYES_00,KHOP_THREADED,RP_MATCHES_RCVD,SPF_HELO_PASS
X-Spam-Check-By: sourceware.org
Message-ID: <506C5844.4020906@malth.us>
Date: Wed, 03 Oct 2012 08:22:44 -0700
From: "Gregory M. Turner" <gmt AT malth DOT us>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/20120907 Thunderbird/15.0.1
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: [PATCH] Re: Fifo blocking and performance issues
References: <2415374 DOT xBCzgxA7JH AT bob-kubuntu> <20121002201947 DOT GA5314 AT ednor DOT casa DOT cgf DOT cx> <506B5254 DOT 3060708 AT malth DOT us> <20121003005540 DOT GA17560 AT ednor DOT casa DOT cgf DOT cx> <506C0C5F DOT 6020602 AT malth DOT us> <20121003130252 DOT GA19620 AT ednor DOT casa DOT cgf DOT cx>
In-Reply-To: <20121003130252.GA19620@ednor.casa.cgf.cx>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com

On 10/3/2012 6:02 AM, Christopher Faylor wrote:
> On Wed, Oct 03, 2012 at 02:58:55AM -0700, Gregory M. Turner wrote:
>> On 10/2/2012 5:55 PM, Christopher Faylor wrote:
>>> FYI, a quick scan seems to show that you've pretty much undone a lot of
>>> what I've recently added for fifos so I don't expect to really be too
>>> keen on applying the patch.  But, I don't want to spend too much time
>>> inspecting a change which can't yet be applied.
>>
>> Yeah, I'm aware of this; I'd be happy to assign -- I'm always easy when
>> it comes to IP stuff.  FTR, that doesn't mean I expect this patch to go
>> in or even be looked at; I'd just like for it to be a non-issue in the
>> future.
>>
>> Is your tree available somewhere for perusal, Chris?  Or is there some
>> way we could get a look at your WIP pipe/fifo deltas?  I'm on
>> -developers and -patches if you want to change venues.
>
> Maybe you are misinterpreting "What I've recently added".> I made big
> changes to fifo handling in the last couple of Cygwin releases.  That is
> what I was referring to.  Otherwise, no, you can't look at my sandbox.

Yep, I thought what you think I thought :)

Unfortunately, with my patch, once the first RDWR handle is created, and 
until it and all of its clones are closed, subsequent attempts to create 
additional non-cloned handles don't work.

The root cause is my patch's use of "platonic" half-duplex win32 pipe 
handles, analogous to pipe2().  Although difficult to "prove" without a 
lot of verbiage, I believe that this can't be solved in that approach 
without changing the way fhandler_fifo coordinates readers and writers. 
  So yeah, the patch probably starts fhandler_fifo down a path that 
would require some sort of significant semantic change.

I used to believe this also proved that the current approach was flawed; 
but after after some alone-time with your comments, cvs commit history, 
and a nerf-brand clue-by-four, I'm feeling a lot more agnostic about 
that... so back to the drawing board, if anywhere; I wouldn't encourage 
you to put much effort into understanding my patch.

Hopefully, though, I'm starting to get what you're trying to do with 
PIPE_ACCESS_DUPLEX.  If I ever have a good enough question about this 
that I can formulate it and press the send button without figuring out 
the answer, I won't hesitate.

-gmt

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple