]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "drivers:tty:pty: Fix a race causing data loss on close"
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 4 Mar 2021 16:18:02 +0000 (17:18 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 10 Mar 2021 08:27:56 +0000 (09:27 +0100)
commit53d971cfef1efbc05bfc808bc44971dff69d0432
tree02479c6cabaa846b78332e0bf5202b79b9625ae5
parent0ae116a8a2b24db00f4044a126e86e2c870f7ed4
Revert "drivers:tty:pty: Fix a race causing data loss on close"

This reverts commit 0937f505a919d76c97c191cadb6b3579db449f0b.

Pierre-Louis writes:

Our SOF/audio CI shows an across-the-board regression when we try v5.12-rc1,
specifically on pause/resume tests with an interactive terminal running 'aplay
-i' commands managed by expect to simulate the user pressing the space bar to
pause/unpause. It turns out the processes are not longer killed and the audio
devices remain busy (see publicly available test results listed below).

git bisect points to commit 0937f505a919d ("drivers:tty:pty: Fix a race
causing data loss on close"). Reverting the patch fixes the issue on all test
devices.

Further analysis with Corey Minyard points to a problem where a slave tty will
not get a SIGHUP when the master is closed.

So revert this for now:

Reported-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/00154592-c5ee-aaba-956e-b265473b53bc@linux.intel.com
Cc: Corey Minyard <cminyard@mvista.com>
Cc: Jiri Slaby <jirislaby@kernel.org>
Cc: Mark Brown <broonie@kernel.org>,
Fixes: 0937f505a919 ("drivers:tty:pty: Fix a race causing data loss on close")
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/pty.c
drivers/tty/tty_io.c