Thread (5 messages) 5 messages, 2 authors, 2011-09-14

Re: close() handler of tty driver called for port that's not open?

From: Alan Cox <hidden>
Date: 2011-09-14 19:34:29

On Wed, 14 Sep 2011 16:21:03 +0000 (UTC)
Grant Edwards [off-list ref] wrote:
Why does the tty layer call my tty driver's close() handler on a port
that's not open?
Because a lot of the tty drivers it makes sense to structure the cleanup
this way.
 
Why does the tty layer try to close a device that isn't open?
Because it was originally designed that way
Does the tty consider a device to be open even if the open() handler
returned an error?
Define "open". The answer depends upon what you mean by "open"
The problem is that my driver keeps a usage count on the port so it
can clean up some internal stuff when the last user closes a port.
Use the tty_port helpers and it'll do that for you including the locking,
blocking, POSIX open blocking/waiting semantics, error handling, callbacks
and hangup. Trying to open code this stuff is a world of pain and the
locking rules are now much more your problem as we strip out the remains
of the big kernel lock.
Does the tty layer guarantee that the number of close() calls will
match the number of open() calls regardless of whether the open()
calls succeed or not?

If so, has that always been the policy, or is calling close() on
non-open device a new feature?
The tty layer has always worked that way. It's I think unique in that
behaviour in the kernel.

If you use the tty_port_open/hangup/close boilerplate then you can
provide shutdown and activate port methods. The guarantee they provide is
that they will be called in matching pairs, they will be called in the
correct places, they will not be called in parallel with one another and
they will properly handle hangup() events - where you shutdown on the
hangup but the close occurs afterwards.

In an ideal world eventually everything will be using the tty port
helpers and then we can make that the underlying (sane) tty interface and
clean up the rest.

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