From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Hurley Subject: Re: [PATCH 00/26] tty/serial flow control fixes Date: Mon, 08 Sep 2014 19:33:51 -0400 Message-ID: <540E3CDF.8020607@hurleysoftware.com> References: <1409693975-1028-1-git-send-email-peter@hurleysoftware.com> <20140908232445.GA28204@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout32.mail01.mtsvc.net ([216.70.64.70]:59287 "EHLO n23.mail01.mtsvc.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754481AbaIHXd4 (ORCPT ); Mon, 8 Sep 2014 19:33:56 -0400 In-Reply-To: <20140908232445.GA28204@kroah.com> Sender: linux-serial-owner@vger.kernel.org List-Id: linux-serial@vger.kernel.org To: Greg Kroah-Hartman Cc: Jiri Slaby , One Thousand Gnomes , linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org On 09/08/2014 07:24 PM, Greg Kroah-Hartman wrote: > On Tue, Sep 02, 2014 at 05:39:09PM -0400, Peter Hurley wrote: >> Hi Greg, >> >> This patch series reworks the internals of tty and serial flow control to >> fix multiple races in both START/STOP flow control and RTS/CTS flow control. >> >> The main changes in this series are: >> Patch 1 >> Backs out the UPF_HARD_FLOW kludge for 8250. This revert should be >> for mainline and -next >> >> Patches 3-7 >> Fixes to x_char handling (ie., sending START/STOP) both in the serial >> core and to several UART drivers >> >> Patches 10-15 >> Rename and move ASYNC_CTS_FLOW and ASYNC_CHECK_CD statuses into >> a private field for the serial core, and add helper functions to test >> those statuses for UART drivers. >> >> This is a necessary step toward making tty port->flags SMP-safe, without >> introducing a new lock. > > I stopped applying at patch 13, due to the objections to that patch. > Please redo the rest of the series and resend. I'm going to wait for the fallout from patch 21 to shake out first. Regards, Peter Hurley