X-Recipient: archive-cygwin AT delorie DOT com X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,RCVD_IN_HOSTKARMA_YE X-Spam-Check-By: sourceware.org X-Mail-Handler: Dyn Standard SMTP by Dyn X-Report-Abuse-To: abuse AT dyndns DOT com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX18d2Eo2Bhbt46kWGrDxw7ZS Date: Wed, 3 Oct 2012 09:02:52 -0400 From: Christopher Faylor To: cygwin AT cygwin DOT com Subject: Re: [PATCH] Re: Fifo blocking and performance issues Message-ID: <20121003130252.GA19620@ednor.casa.cgf.cx> Reply-To: cygwin AT cygwin DOT com Mail-Followup-To: cygwin AT cygwin DOT com 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> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <506C0C5F.6020602@malth.us> User-Agent: Mutt/1.5.20 (2009-06-14) Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , 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 Wed, Oct 03, 2012 at 02:58:55AM -0700, Gregory M. Turner wrote: >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. 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. cgf -- 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