Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1284800

[PATCH LINUX v4 00/13] tty: xuartps: Fix lock ups

From Soren Brinkmann <soren.brinkmann@xilinx.com>
Newsgroups linux.kernel
Subject [PATCH LINUX v4 00/13] tty: xuartps: Fix lock ups
Date 2015-12-06 05:50 +0100
Message-ID <qCzqN-2b5-3@gated-at.bofh.it> (permalink)
Organization linux.* mail to news gateway

Show all headers | View raw


Hi,

I picked up all the review tags from Peter and Moritz and addressed
Peter's comments.
Most patches are unchanged but I added the info Peter suggested/provided
to some commit messages.
The patches "tty: xuartps: Move request_irq to after setting up the HW",
"tty: xuartps: Improve sysrq handling" and "tty: xuartps: Remove
'_OFFSET' suffix from #defines" are new, implementing the suggestions
Peter made in the v3 review (https://lkml.org/lkml/2015/11/21/172).
I pretty much rewrote "tty: xuartps: Refactor IRQ handling" (was: "tty:
xuartps: Only handle RX IRQs when RX is enabled"). Instead of adding a
flag, we now disable/enable the RX interrupts together with the
receiver. That should prevent the RX lock ups I saw.

	Thanks,
	Sören

Sören Brinkmann (13):
  tty: xuartps: Beautify read-modify writes
  tty: xuartps: Use spinlock to serialize HW access
  tty: xuartps: Don't consider circular buffer when enabling transmitter
  tty: xuartps: Clear interrupt status register in shutdown
  tty: xuartps: Improve startup function
  tty: xuartps: Move request_irq to after setting up the HW
  tty: xuartps: Keep lock for whole ISR
  tty: xuartps: Acquire port lock for shutdown
  tty: xuartps: Move RX path into helper function
  tty: xuartps: Refactor IRQ handling
  tty: xuartps: Cleanup: Reformat if-else
  tty: xuartps: Improve sysrq handling
  tty: xuartps: Remove '_OFFSET' suffix from #defines

 drivers/tty/serial/xilinx_uartps.c | 451 ++++++++++++++++++-------------------
 1 file changed, 224 insertions(+), 227 deletions(-)

-- 
2.6.3.3.g9bb996a

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread


Thread

[PATCH LINUX v4 00/13] tty: xuartps: Fix lock ups Soren Brinkmann <soren.brinkmann@xilinx.com> - 2015-12-06 05:50 +0100
  [PATCH LINUX v4 07/13] tty: xuartps: Keep lock for whole ISR Soren Brinkmann <soren.brinkmann@xilinx.com> - 2015-12-06 05:50 +0100
  [PATCH LINUX v4 13/13] tty: xuartps: Remove '_OFFSET' suffix from #defines Soren Brinkmann <soren.brinkmann@xilinx.com> - 2015-12-06 05:50 +0100
  [PATCH LINUX v4 05/13] tty: xuartps: Improve startup function Soren Brinkmann <soren.brinkmann@xilinx.com> - 2015-12-06 05:50 +0100
  [PATCH LINUX v4 03/13] tty: xuartps: Don't consider circular buffer when enabling transmitter Soren Brinkmann <soren.brinkmann@xilinx.com> - 2015-12-06 05:50 +0100

csiph-web