On 02/05/2018 21:16, Sebastian Andrzej Siewior wrote: > On 2018-04-27 12:31:52 [+0200], Richard Genoud wrote: >> Hi Sebastian, > Hi, > >>> If this is unknown and a bisect is requested, please let me know. >> Indeed, this will be appreciated. >> I'm quite curious to find the commit that led to this. > > commit 761ed4a94582ab291aa24dcbea4e01e8936488c8 > Author: Rob Herring <robh@xxxxxxxxxx> > Date: Mon Aug 22 17:39:10 2016 -0500 > > tty: serial_core: convert uart_close to use tty_port_close > > tty_port_close handles much of the common parts of tty close. Convert > uart_close to use it and move the serial_core specific parts into > tty_port.shutdown function. This will be needed to use tty_port functions > directly from in kernel clients. > > This change causes ops->stop_rx() to be called after uart_wait_until_sent() > is called which I think should be fine. Otherwise, the sequence of the > close should be the same. > > Cc: Peter Hurley <peter@xxxxxxxxxxxxxxxxxx> > Signed-off-by: Rob Herring <robh@xxxxxxxxxx> > Acked-by: Alan Cox <alan@xxxxxxxxxxxxxxx> > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > The thread starts at > https://www.spinics.net/lists/linux-serial/msg30070.html > http://lkml.kernel.org/r/20180426150625.q5tqcb7fzchvkb5d@xxxxxxxxxxxxx Thanks for this hunting session ! Could resend your patch with Fixes: in the commit message ? BTW, I didn't manage to reproduce the behavior you describe, could you give me your .config and describe a little more how you manage to trigger this bug ? (do you use the console on serial debug ? which board ? ) Thanks ! -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
