Thread (19 messages) 19 messages, 2 authors, 2014-11-05

Re: [PATCH 03/12] serial_core: Handle TIOC[GS]RS485 ioctls.

From: Alan Cox <hidden>
Date: 2014-10-09 14:16:46
Also in: lkml

On Thu, 2014-10-09 at 10:07 +0200, Ricardo Ribalda Delgado wrote:
Hello Alan
quoted
What is the locking between setting/getting/driver use of the config ?
This really needs a lock (termios sem I think is perhaps appropriate
given when the values are normally referenced).
I tried implementing it with the sermios sem
((&(uart_port)->state->port.tty->termios_rwsem)), but some drivers
access the rs485 structure inside their irq handler. So I have see
options here

1) Protect the structure with uart_port->lock spinlock
2) Assume that an assignment is atomic on critical sections where I
cannot hold the rwsem.

I think 1) is more correct. Any issues that I continue in this
direction? Any better idea?
For uart #1 sounds right to me too.

Alan

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help