]> err.no Git - linux-2.6/commit
[SERIAL] Remove unconditional enable of TX irq for console
authorRussell King <rmk@dyn-67.arm.linux.org.uk>
Sun, 30 Apr 2006 10:30:15 +0000 (11:30 +0100)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Sun, 30 Apr 2006 10:30:15 +0000 (11:30 +0100)
commita88d75b257b2b28b26d7d4d2b640f05feb00ad53
tree559fff21b74d16d946b33333396dcd435da2e1b4
parentb32b19b8ffc05cbd3bf91c65e205f6a912ca15d9
[SERIAL] Remove unconditional enable of TX irq for console

A bug report from Gerd Hoffmann has highlighted that unconditionally
enabling the transmit interrupt at the end of console writes is very
bad.

In Gerd's case, it causes the test for buggy UARTs to give false
positives, incorrectly identifying ports as buggy when they are not.

Moreover, if we unconditionally enable the interrupt, and the port
is sharing it's interrupt with other ports, there is the very real
possibility that we'll cause an interrupt storm.  (Not all ports use
OUT2 as an interrupt mask.)

Hence, revert part of f91a3715db2bb44fcf08cec642e68f919b70f7f4 and
all of f5968b37b3ad35b682b574b578843a0361218aff until a better solution
can be found.

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
drivers/serial/8250.c