[PATCH] Re: Fifo blocking and performance issues

Gregory M. Turner gmt@malth.us
Wed Oct 3 09:59:00 GMT 2012


On 10/2/2012 5:55 PM, Christopher Faylor wrote:
> On Tue, Oct 02, 2012 at 01:45:08PM -0700, Gregory M. Turner wrote:
>> On 10/2/2012 1:19 PM, Christopher Faylor wrote:
>>> On Tue, Oct 02, 2012 at 03:15:37PM -0400, bob wrote:
>>>> Any suggestions on how we can achieve a higher performance blocking read on a
>>>> Cygwin RDWR fifo?
>>>
>>> As always, if you can provide test cases of bugs we will endeavor to fix problems.
>>
>> I didn't think the RDWR fifo's worked at all -- they certainly don't if
>> you create them via bash.
>
> If you want to submit an assignment I'll be happy to look at the patch.
> It is too big to be included without an assignment.  See:
>
> http://cygwin.com/contrib.html
>
> 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.

I'd definitely be keen to know where that's headed, since it impacts 
some decisions I need to make for my cygwin gentoo work -- plus, I may 
have managed to glean some insight into the limitations of what's in 
cvs... but I obviously shouldn't go much further with it if I'm just 
creating a merge situation or, worse, addressing bugs that you already 
fixed.

-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



More information about the Cygwin mailing list