From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Hurley Date: Tue, 11 Mar 2014 22:59:12 +0000 Subject: Re: serial core: crash / race condition on unbind Message-Id: <531F9540.4020906@hurleysoftware.com> List-Id: References: <531E7F81.20400@hurleysoftware.com> <531EF864.5020104@hurleysoftware.com> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Geert Uytterhoeven Cc: Greg Kroah-Hartman , linux-serial@vger.kernel.org, "linux-kernel@vger.kernel.org" , Linux-sh list , One Thousand Gnomes On 03/11/2014 11:35 AM, Geert Uytterhoeven wrote: > On Tue, Mar 11, 2014 at 12:49 PM, Peter Hurley wrote: >> On 03/11/2014 06:58 AM, Geert Uytterhoeven wrote: >>> On Tue, Mar 11, 2014 at 4:14 AM, Peter Hurley >>> wrote: >>>>> >>>>> >>>>> [...] >>>>> >>>>> /* >>>>> * Indicate that there isn't a port here anymore. >>>>> */ >>>>> uport->type = PORT_UNKNOWN; >>>>> >>>>> state->uart_port = NULL; >>>> >>>> >>>> How did this ever work? >>>> >>>> Detaching the ll driver from the tty port in this manner is not ok; >>>> as you already note, it blows up if consoles are still running. >>> >>> No one unbinds serial drivers using serial_core, as all these drivers are >>> for fixed hardware? >> >> Yep, never tested until now :) >> Do you need this to work? > > Well, "need" may be a bit strong. Crashes are not so nice. :) What I meant was, 'is this a debug situation that I can eventually get to?' or 'are you on the verge of shipping product and this is a priority?' Regards, Peter Hurley