2008-07-22 10:16:55 +00:00
|
|
|
#include <linux/types.h>
|
|
|
|
|
#include <linux/errno.h>
|
2011-11-16 15:27:10 +00:00
|
|
|
#include <linux/kmod.h>
|
2008-07-22 10:16:55 +00:00
|
|
|
#include <linux/sched.h>
|
|
|
|
|
#include <linux/interrupt.h>
|
|
|
|
|
#include <linux/tty.h>
|
|
|
|
|
#include <linux/tty_driver.h>
|
|
|
|
|
#include <linux/file.h>
|
|
|
|
|
#include <linux/mm.h>
|
|
|
|
|
#include <linux/string.h>
|
|
|
|
|
#include <linux/slab.h>
|
|
|
|
|
#include <linux/poll.h>
|
|
|
|
|
#include <linux/proc_fs.h>
|
|
|
|
|
#include <linux/init.h>
|
|
|
|
|
#include <linux/module.h>
|
|
|
|
|
#include <linux/device.h>
|
|
|
|
|
#include <linux/wait.h>
|
|
|
|
|
#include <linux/bitops.h>
|
|
|
|
|
#include <linux/seq_file.h>
|
|
|
|
|
#include <linux/uaccess.h>
|
2011-11-16 15:27:09 +00:00
|
|
|
#include <linux/ratelimit.h>
|
2008-07-22 10:16:55 +00:00
|
|
|
|
2013-03-11 20:44:38 +00:00
|
|
|
#undef LDISC_DEBUG_HANGUP
|
|
|
|
|
|
|
|
|
|
#ifdef LDISC_DEBUG_HANGUP
|
|
|
|
|
#define tty_ldisc_debug(tty, f, args...) ({ \
|
|
|
|
|
char __b[64]; \
|
|
|
|
|
printk(KERN_DEBUG "%s: %s: " f, __func__, tty_name(tty, __b), ##args); \
|
|
|
|
|
})
|
|
|
|
|
#else
|
|
|
|
|
#define tty_ldisc_debug(tty, f, args...)
|
|
|
|
|
#endif
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/*
|
|
|
|
|
* This guards the refcounted line discipline lists. The lock
|
|
|
|
|
* must be taken with irqs off because there are hangup path
|
|
|
|
|
* callers who will do ldisc lookups and cannot sleep.
|
|
|
|
|
*/
|
|
|
|
|
|
2012-10-17 12:03:14 +00:00
|
|
|
static DEFINE_RAW_SPINLOCK(tty_ldisc_lock);
|
2008-07-22 10:16:55 +00:00
|
|
|
static DECLARE_WAIT_QUEUE_HEAD(tty_ldisc_wait);
|
|
|
|
|
/* Line disc dispatch table */
|
|
|
|
|
static struct tty_ldisc_ops *tty_ldiscs[NR_LDISCS];
|
|
|
|
|
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
static inline struct tty_ldisc *get_ldisc(struct tty_ldisc *ld)
|
|
|
|
|
{
|
|
|
|
|
if (ld)
|
|
|
|
|
atomic_inc(&ld->users);
|
|
|
|
|
return ld;
|
|
|
|
|
}
|
|
|
|
|
|
2009-08-03 21:54:56 +00:00
|
|
|
static void put_ldisc(struct tty_ldisc *ld)
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
{
|
2009-08-03 21:54:56 +00:00
|
|
|
unsigned long flags;
|
|
|
|
|
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
if (WARN_ON_ONCE(!ld))
|
|
|
|
|
return;
|
|
|
|
|
|
|
|
|
|
/*
|
|
|
|
|
* If this is the last user, free the ldisc, and
|
|
|
|
|
* release the ldisc ops.
|
2009-08-03 21:54:56 +00:00
|
|
|
*
|
2012-10-17 12:03:14 +00:00
|
|
|
* We really want an "atomic_dec_and_raw_lock_irqsave()",
|
2009-08-03 21:54:56 +00:00
|
|
|
* but we don't have it, so this does it by hand.
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
*/
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_lock_irqsave(&tty_ldisc_lock, flags);
|
|
|
|
|
if (atomic_dec_and_test(&ld->users)) {
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
struct tty_ldisc_ops *ldo = ld->ops;
|
|
|
|
|
|
|
|
|
|
ldo->refcount--;
|
|
|
|
|
module_put(ldo->owner);
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_unlock_irqrestore(&tty_ldisc_lock, flags);
|
2009-08-03 21:54:56 +00:00
|
|
|
|
|
|
|
|
kfree(ld);
|
|
|
|
|
return;
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
}
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_unlock_irqrestore(&tty_ldisc_lock, flags);
|
2012-12-18 14:48:50 +00:00
|
|
|
|
|
|
|
|
if (waitqueue_active(&ld->wq_idle))
|
|
|
|
|
wake_up(&ld->wq_idle);
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
}
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/**
|
|
|
|
|
* tty_register_ldisc - install a line discipline
|
|
|
|
|
* @disc: ldisc number
|
|
|
|
|
* @new_ldisc: pointer to the ldisc object
|
|
|
|
|
*
|
|
|
|
|
* Installs a new line discipline into the kernel. The discipline
|
|
|
|
|
* is set up as unreferenced and then made available to the kernel
|
|
|
|
|
* from this point onwards.
|
|
|
|
|
*
|
|
|
|
|
* Locking:
|
|
|
|
|
* takes tty_ldisc_lock to guard against ldisc races
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
int tty_register_ldisc(int disc, struct tty_ldisc_ops *new_ldisc)
|
|
|
|
|
{
|
|
|
|
|
unsigned long flags;
|
|
|
|
|
int ret = 0;
|
|
|
|
|
|
|
|
|
|
if (disc < N_TTY || disc >= NR_LDISCS)
|
|
|
|
|
return -EINVAL;
|
|
|
|
|
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_lock_irqsave(&tty_ldisc_lock, flags);
|
2008-07-22 10:16:55 +00:00
|
|
|
tty_ldiscs[disc] = new_ldisc;
|
|
|
|
|
new_ldisc->num = disc;
|
|
|
|
|
new_ldisc->refcount = 0;
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_unlock_irqrestore(&tty_ldisc_lock, flags);
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
|
}
|
|
|
|
|
EXPORT_SYMBOL(tty_register_ldisc);
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_unregister_ldisc - unload a line discipline
|
|
|
|
|
* @disc: ldisc number
|
|
|
|
|
* @new_ldisc: pointer to the ldisc object
|
|
|
|
|
*
|
|
|
|
|
* Remove a line discipline from the kernel providing it is not
|
|
|
|
|
* currently in use.
|
|
|
|
|
*
|
|
|
|
|
* Locking:
|
|
|
|
|
* takes tty_ldisc_lock to guard against ldisc races
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
int tty_unregister_ldisc(int disc)
|
|
|
|
|
{
|
|
|
|
|
unsigned long flags;
|
|
|
|
|
int ret = 0;
|
|
|
|
|
|
|
|
|
|
if (disc < N_TTY || disc >= NR_LDISCS)
|
|
|
|
|
return -EINVAL;
|
|
|
|
|
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_lock_irqsave(&tty_ldisc_lock, flags);
|
2008-07-22 10:16:55 +00:00
|
|
|
if (tty_ldiscs[disc]->refcount)
|
|
|
|
|
ret = -EBUSY;
|
|
|
|
|
else
|
|
|
|
|
tty_ldiscs[disc] = NULL;
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_unlock_irqrestore(&tty_ldisc_lock, flags);
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
|
}
|
|
|
|
|
EXPORT_SYMBOL(tty_unregister_ldisc);
|
|
|
|
|
|
2009-08-03 23:00:15 +00:00
|
|
|
static struct tty_ldisc_ops *get_ldops(int disc)
|
|
|
|
|
{
|
|
|
|
|
unsigned long flags;
|
|
|
|
|
struct tty_ldisc_ops *ldops, *ret;
|
|
|
|
|
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_lock_irqsave(&tty_ldisc_lock, flags);
|
2009-08-03 23:00:15 +00:00
|
|
|
ret = ERR_PTR(-EINVAL);
|
|
|
|
|
ldops = tty_ldiscs[disc];
|
|
|
|
|
if (ldops) {
|
|
|
|
|
ret = ERR_PTR(-EAGAIN);
|
|
|
|
|
if (try_module_get(ldops->owner)) {
|
|
|
|
|
ldops->refcount++;
|
|
|
|
|
ret = ldops;
|
|
|
|
|
}
|
|
|
|
|
}
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_unlock_irqrestore(&tty_ldisc_lock, flags);
|
2009-08-03 23:00:15 +00:00
|
|
|
return ret;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static void put_ldops(struct tty_ldisc_ops *ldops)
|
|
|
|
|
{
|
|
|
|
|
unsigned long flags;
|
|
|
|
|
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_lock_irqsave(&tty_ldisc_lock, flags);
|
2009-08-03 23:00:15 +00:00
|
|
|
ldops->refcount--;
|
|
|
|
|
module_put(ldops->owner);
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_unlock_irqrestore(&tty_ldisc_lock, flags);
|
2009-08-03 23:00:15 +00:00
|
|
|
}
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_get - take a reference to an ldisc
|
|
|
|
|
* @disc: ldisc number
|
|
|
|
|
*
|
|
|
|
|
* Takes a reference to a line discipline. Deals with refcounts and
|
|
|
|
|
* module locking counts. Returns NULL if the discipline is not available.
|
|
|
|
|
* Returns a pointer to the discipline and bumps the ref count if it is
|
|
|
|
|
* available
|
|
|
|
|
*
|
|
|
|
|
* Locking:
|
|
|
|
|
* takes tty_ldisc_lock to guard against ldisc races
|
|
|
|
|
*/
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
static struct tty_ldisc *tty_ldisc_get(int disc)
|
2008-07-22 10:16:55 +00:00
|
|
|
{
|
2009-06-11 11:50:12 +00:00
|
|
|
struct tty_ldisc *ld;
|
2009-08-03 23:01:28 +00:00
|
|
|
struct tty_ldisc_ops *ldops;
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
if (disc < N_TTY || disc >= NR_LDISCS)
|
2009-06-11 11:50:12 +00:00
|
|
|
return ERR_PTR(-EINVAL);
|
2009-08-03 23:01:28 +00:00
|
|
|
|
|
|
|
|
/*
|
|
|
|
|
* Get the ldisc ops - we may need to request them to be loaded
|
|
|
|
|
* dynamically and try again.
|
|
|
|
|
*/
|
|
|
|
|
ldops = get_ldops(disc);
|
|
|
|
|
if (IS_ERR(ldops)) {
|
2008-07-22 10:16:55 +00:00
|
|
|
request_module("tty-ldisc-%d", disc);
|
2009-08-03 23:01:28 +00:00
|
|
|
ldops = get_ldops(disc);
|
|
|
|
|
if (IS_ERR(ldops))
|
|
|
|
|
return ERR_CAST(ldops);
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
ld = kmalloc(sizeof(struct tty_ldisc), GFP_KERNEL);
|
|
|
|
|
if (ld == NULL) {
|
|
|
|
|
put_ldops(ldops);
|
|
|
|
|
return ERR_PTR(-ENOMEM);
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
2009-08-03 23:01:28 +00:00
|
|
|
|
|
|
|
|
ld->ops = ldops;
|
|
|
|
|
atomic_set(&ld->users, 1);
|
2012-05-03 12:37:43 +00:00
|
|
|
init_waitqueue_head(&ld->wq_idle);
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
return ld;
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
|
2009-06-11 11:51:41 +00:00
|
|
|
static void *tty_ldiscs_seq_start(struct seq_file *m, loff_t *pos)
|
2008-07-22 10:16:55 +00:00
|
|
|
{
|
|
|
|
|
return (*pos < NR_LDISCS) ? pos : NULL;
|
|
|
|
|
}
|
|
|
|
|
|
2009-06-11 11:51:41 +00:00
|
|
|
static void *tty_ldiscs_seq_next(struct seq_file *m, void *v, loff_t *pos)
|
2008-07-22 10:16:55 +00:00
|
|
|
{
|
|
|
|
|
(*pos)++;
|
|
|
|
|
return (*pos < NR_LDISCS) ? pos : NULL;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static void tty_ldiscs_seq_stop(struct seq_file *m, void *v)
|
|
|
|
|
{
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static int tty_ldiscs_seq_show(struct seq_file *m, void *v)
|
|
|
|
|
{
|
|
|
|
|
int i = *(loff_t *)v;
|
2009-08-03 23:00:15 +00:00
|
|
|
struct tty_ldisc_ops *ldops;
|
2009-06-11 11:51:41 +00:00
|
|
|
|
2009-08-03 23:00:15 +00:00
|
|
|
ldops = get_ldops(i);
|
|
|
|
|
if (IS_ERR(ldops))
|
2008-07-22 10:16:55 +00:00
|
|
|
return 0;
|
2009-08-03 23:00:15 +00:00
|
|
|
seq_printf(m, "%-10s %2d\n", ldops->name ? ldops->name : "???", i);
|
|
|
|
|
put_ldops(ldops);
|
2008-07-22 10:16:55 +00:00
|
|
|
return 0;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static const struct seq_operations tty_ldiscs_seq_ops = {
|
|
|
|
|
.start = tty_ldiscs_seq_start,
|
|
|
|
|
.next = tty_ldiscs_seq_next,
|
|
|
|
|
.stop = tty_ldiscs_seq_stop,
|
|
|
|
|
.show = tty_ldiscs_seq_show,
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
static int proc_tty_ldiscs_open(struct inode *inode, struct file *file)
|
|
|
|
|
{
|
|
|
|
|
return seq_open(file, &tty_ldiscs_seq_ops);
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
const struct file_operations tty_ldiscs_proc_fops = {
|
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
|
.open = proc_tty_ldiscs_open,
|
|
|
|
|
.read = seq_read,
|
|
|
|
|
.llseek = seq_lseek,
|
|
|
|
|
.release = seq_release,
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_assign - set ldisc on a tty
|
|
|
|
|
* @tty: tty to assign
|
|
|
|
|
* @ld: line discipline
|
|
|
|
|
*
|
|
|
|
|
* Install an instance of a line discipline into a tty structure. The
|
2009-06-16 16:00:26 +00:00
|
|
|
* ldisc must have a reference count above zero to ensure it remains.
|
2008-07-22 10:16:55 +00:00
|
|
|
* The tty instance refcount starts at zero.
|
|
|
|
|
*
|
|
|
|
|
* Locking:
|
|
|
|
|
* Caller must hold references
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
static void tty_ldisc_assign(struct tty_struct *tty, struct tty_ldisc *ld)
|
|
|
|
|
{
|
2009-06-11 11:50:12 +00:00
|
|
|
tty->ldisc = ld;
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_try - internal helper
|
|
|
|
|
* @tty: the tty
|
|
|
|
|
*
|
|
|
|
|
* Make a single attempt to grab and bump the refcount on
|
|
|
|
|
* the tty ldisc. Return 0 on failure or 1 on success. This is
|
|
|
|
|
* used to implement both the waiting and non waiting versions
|
|
|
|
|
* of tty_ldisc_ref
|
|
|
|
|
*
|
|
|
|
|
* Locking: takes tty_ldisc_lock
|
|
|
|
|
*/
|
|
|
|
|
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
static struct tty_ldisc *tty_ldisc_try(struct tty_struct *tty)
|
2008-07-22 10:16:55 +00:00
|
|
|
{
|
|
|
|
|
unsigned long flags;
|
|
|
|
|
struct tty_ldisc *ld;
|
|
|
|
|
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_lock_irqsave(&tty_ldisc_lock, flags);
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
ld = NULL;
|
|
|
|
|
if (test_bit(TTY_LDISC, &tty->flags))
|
|
|
|
|
ld = get_ldisc(tty->ldisc);
|
2012-10-17 12:03:14 +00:00
|
|
|
raw_spin_unlock_irqrestore(&tty_ldisc_lock, flags);
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
return ld;
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_ref_wait - wait for the tty ldisc
|
|
|
|
|
* @tty: tty device
|
|
|
|
|
*
|
|
|
|
|
* Dereference the line discipline for the terminal and take a
|
|
|
|
|
* reference to it. If the line discipline is in flux then
|
|
|
|
|
* wait patiently until it changes.
|
|
|
|
|
*
|
|
|
|
|
* Note: Must not be called from an IRQ/timer context. The caller
|
|
|
|
|
* must also be careful not to hold other locks that will deadlock
|
|
|
|
|
* against a discipline change, such as an existing ldisc reference
|
|
|
|
|
* (which we check for)
|
|
|
|
|
*
|
|
|
|
|
* Locking: call functions take tty_ldisc_lock
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
struct tty_ldisc *tty_ldisc_ref_wait(struct tty_struct *tty)
|
|
|
|
|
{
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
struct tty_ldisc *ld;
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/* wait_event is a macro */
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
wait_event(tty_ldisc_wait, (ld = tty_ldisc_try(tty)) != NULL);
|
|
|
|
|
return ld;
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
EXPORT_SYMBOL_GPL(tty_ldisc_ref_wait);
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_ref - get the tty ldisc
|
|
|
|
|
* @tty: tty device
|
|
|
|
|
*
|
|
|
|
|
* Dereference the line discipline for the terminal and take a
|
|
|
|
|
* reference to it. If the line discipline is in flux then
|
|
|
|
|
* return NULL. Can be called from IRQ and timer functions.
|
|
|
|
|
*
|
|
|
|
|
* Locking: called functions take tty_ldisc_lock
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
struct tty_ldisc *tty_ldisc_ref(struct tty_struct *tty)
|
|
|
|
|
{
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
return tty_ldisc_try(tty);
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
EXPORT_SYMBOL_GPL(tty_ldisc_ref);
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_deref - free a tty ldisc reference
|
|
|
|
|
* @ld: reference to free up
|
|
|
|
|
*
|
|
|
|
|
* Undoes the effect of tty_ldisc_ref or tty_ldisc_ref_wait. May
|
|
|
|
|
* be called in IRQ context.
|
|
|
|
|
*
|
|
|
|
|
* Locking: takes tty_ldisc_lock
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
void tty_ldisc_deref(struct tty_ldisc *ld)
|
|
|
|
|
{
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
put_ldisc(ld);
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
EXPORT_SYMBOL_GPL(tty_ldisc_deref);
|
|
|
|
|
|
tty-ldisc: turn ldisc user count into a proper refcount
By using the user count for the actual lifetime rules, we can get rid of
the silly "wait_for_idle" logic, because any busy ldisc will
automatically stay around until the last user releases it. This avoids
a host of odd issues, and simplifies the code.
So now, when the last ldisc reference is dropped, we just release the
ldisc operations struct reference, and free the ldisc.
It looks obvious enough, and it does work for me, but the counting
_could_ be off. It probably isn't (bad counting in the new version would
generally imply that the old code did something really bad, like free an
ldisc with a non-zero count), but it does need some testing, and
preferably somebody looking at it.
With this change, both 'tty_ldisc_put()' and 'tty_ldisc_deref()' are
just aliases for the new ref-counting 'put_ldisc()'. Both of them
decrement the ldisc user count and free it if it goes down to zero.
They're identical functions, in other words.
But the reason they still exist as sepate functions is that one of them
was exported (tty_ldisc_deref) and had a stupid name (so I don't want to
use it as the main name), and the other one was used in multiple places
(and I didn't want to make the patch larger just to rename the users).
In addition to the refcounting, I did do some minimal cleanup. For
example, now "tty_ldisc_try()" actually returns the ldisc it got under
the lock, rather than returning true/false and then the caller would
look up the ldisc again (now without the protection of the lock).
That said, there's tons of dubious use of 'tty->ldisc' without obviously
proper locking or refcounting left. I expressly did _not_ want to try to
fix it all, keeping the patch minimal. There may or may not be bugs in
that kind of code, but they wouldn't be _new_ bugs.
That said, even if the bugs aren't new, the timing and lifetime will
change. For example, some silly code may depend on the 'tty->ldisc'
pointer not changing because they hold a refcount on the 'ldisc'. And
that's no longer true - if you hold a ref on the ldisc, the 'ldisc'
itself is safe, but tty->ldisc may change.
So the proper locking (remains) to hold tty->ldisc_mutex if you expect
tty->ldisc to be stable. That's not really a _new_ rule, but it's an
example of something that the old code might have unintentionally
depended on and hidden bugs.
Whatever. The patch _looks_ sensible to me. The only users of
ldisc->users are:
- get_ldisc() - atomically increment the count
- put_ldisc() - atomically decrements the count and releases if zero
- tty_ldisc_try_get() - creates the ldisc, and sets the count to 1.
The ldisc should then either be released, or be attached to a tty.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Tested-by: Sergey Senozhatsky <sergey.senozhatsky@mail.by>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2009-08-03 18:11:19 +00:00
|
|
|
static inline void tty_ldisc_put(struct tty_ldisc *ld)
|
|
|
|
|
{
|
|
|
|
|
put_ldisc(ld);
|
|
|
|
|
}
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/**
|
|
|
|
|
* tty_ldisc_enable - allow ldisc use
|
|
|
|
|
* @tty: terminal to activate ldisc on
|
|
|
|
|
*
|
|
|
|
|
* Set the TTY_LDISC flag when the line discipline can be called
|
2009-01-02 13:44:56 +00:00
|
|
|
* again. Do necessary wakeups for existing sleepers. Clear the LDISC
|
|
|
|
|
* changing flag to indicate any ldisc change is now over.
|
2008-07-22 10:16:55 +00:00
|
|
|
*
|
2009-01-02 13:44:56 +00:00
|
|
|
* Note: nobody should set the TTY_LDISC bit except via this function.
|
|
|
|
|
* Clearing directly is allowed.
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
|
|
|
|
|
2013-03-11 20:44:33 +00:00
|
|
|
static void tty_ldisc_enable(struct tty_struct *tty)
|
2008-07-22 10:16:55 +00:00
|
|
|
{
|
2013-03-11 20:44:21 +00:00
|
|
|
clear_bit(TTY_LDISC_HALTED, &tty->flags);
|
2008-07-22 10:16:55 +00:00
|
|
|
set_bit(TTY_LDISC, &tty->flags);
|
2009-01-02 13:44:56 +00:00
|
|
|
clear_bit(TTY_LDISC_CHANGING, &tty->flags);
|
2008-07-22 10:16:55 +00:00
|
|
|
wake_up(&tty_ldisc_wait);
|
|
|
|
|
}
|
|
|
|
|
|
2009-06-11 11:50:58 +00:00
|
|
|
/**
|
|
|
|
|
* tty_ldisc_flush - flush line discipline queue
|
|
|
|
|
* @tty: tty
|
|
|
|
|
*
|
|
|
|
|
* Flush the line discipline queue (if any) for this tty. If there
|
|
|
|
|
* is no line discipline active this is a no-op.
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
void tty_ldisc_flush(struct tty_struct *tty)
|
|
|
|
|
{
|
|
|
|
|
struct tty_ldisc *ld = tty_ldisc_ref(tty);
|
|
|
|
|
if (ld) {
|
|
|
|
|
if (ld->ops->flush_buffer)
|
|
|
|
|
ld->ops->flush_buffer(tty);
|
|
|
|
|
tty_ldisc_deref(ld);
|
|
|
|
|
}
|
|
|
|
|
tty_buffer_flush(tty);
|
|
|
|
|
}
|
|
|
|
|
EXPORT_SYMBOL_GPL(tty_ldisc_flush);
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/**
|
|
|
|
|
* tty_set_termios_ldisc - set ldisc field
|
|
|
|
|
* @tty: tty structure
|
|
|
|
|
* @num: line discipline number
|
|
|
|
|
*
|
|
|
|
|
* This is probably overkill for real world processors but
|
|
|
|
|
* they are not on hot paths so a little discipline won't do
|
|
|
|
|
* any harm.
|
|
|
|
|
*
|
|
|
|
|
* Locking: takes termios_mutex
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
static void tty_set_termios_ldisc(struct tty_struct *tty, int num)
|
|
|
|
|
{
|
|
|
|
|
mutex_lock(&tty->termios_mutex);
|
2012-07-14 14:31:47 +00:00
|
|
|
tty->termios.c_line = num;
|
2008-07-22 10:16:55 +00:00
|
|
|
mutex_unlock(&tty->termios_mutex);
|
|
|
|
|
}
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
/**
|
|
|
|
|
* tty_ldisc_open - open a line discipline
|
|
|
|
|
* @tty: tty we are opening the ldisc on
|
|
|
|
|
* @ld: discipline to open
|
|
|
|
|
*
|
|
|
|
|
* A helper opening method. Also a convenient debugging and check
|
|
|
|
|
* point.
|
2010-06-01 20:53:01 +00:00
|
|
|
*
|
|
|
|
|
* Locking: always called with BTM already held.
|
2009-06-11 11:50:12 +00:00
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
static int tty_ldisc_open(struct tty_struct *tty, struct tty_ldisc *ld)
|
|
|
|
|
{
|
|
|
|
|
WARN_ON(test_and_set_bit(TTY_LDISC_OPEN, &tty->flags));
|
2009-11-30 13:18:35 +00:00
|
|
|
if (ld->ops->open) {
|
|
|
|
|
int ret;
|
2010-06-01 20:53:01 +00:00
|
|
|
/* BTM here locks versus a hangup event */
|
2009-11-30 13:18:35 +00:00
|
|
|
ret = ld->ops->open(tty);
|
2010-11-24 23:27:54 +00:00
|
|
|
if (ret)
|
|
|
|
|
clear_bit(TTY_LDISC_OPEN, &tty->flags);
|
2009-11-30 13:18:35 +00:00
|
|
|
return ret;
|
|
|
|
|
}
|
2009-06-11 11:50:12 +00:00
|
|
|
return 0;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_close - close a line discipline
|
|
|
|
|
* @tty: tty we are opening the ldisc on
|
|
|
|
|
* @ld: discipline to close
|
|
|
|
|
*
|
|
|
|
|
* A helper close method. Also a convenient debugging and check
|
|
|
|
|
* point.
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
static void tty_ldisc_close(struct tty_struct *tty, struct tty_ldisc *ld)
|
|
|
|
|
{
|
|
|
|
|
WARN_ON(!test_bit(TTY_LDISC_OPEN, &tty->flags));
|
|
|
|
|
clear_bit(TTY_LDISC_OPEN, &tty->flags);
|
|
|
|
|
if (ld->ops->close)
|
|
|
|
|
ld->ops->close(tty);
|
|
|
|
|
}
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_restore - helper for tty ldisc change
|
|
|
|
|
* @tty: tty to recover
|
|
|
|
|
* @old: previous ldisc
|
|
|
|
|
*
|
|
|
|
|
* Restore the previous line discipline or N_TTY when a line discipline
|
|
|
|
|
* change fails due to an open error
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
static void tty_ldisc_restore(struct tty_struct *tty, struct tty_ldisc *old)
|
|
|
|
|
{
|
|
|
|
|
char buf[64];
|
2009-06-11 11:50:12 +00:00
|
|
|
struct tty_ldisc *new_ldisc;
|
|
|
|
|
int r;
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
/* There is an outstanding reference here so this is safe */
|
2009-06-11 11:50:12 +00:00
|
|
|
old = tty_ldisc_get(old->ops->num);
|
|
|
|
|
WARN_ON(IS_ERR(old));
|
2008-07-22 10:16:55 +00:00
|
|
|
tty_ldisc_assign(tty, old);
|
|
|
|
|
tty_set_termios_ldisc(tty, old->ops->num);
|
2009-06-11 11:50:12 +00:00
|
|
|
if (tty_ldisc_open(tty, old) < 0) {
|
|
|
|
|
tty_ldisc_put(old);
|
2008-07-22 10:16:55 +00:00
|
|
|
/* This driver is always present */
|
2009-06-11 11:51:41 +00:00
|
|
|
new_ldisc = tty_ldisc_get(N_TTY);
|
2009-06-11 11:50:12 +00:00
|
|
|
if (IS_ERR(new_ldisc))
|
2008-07-22 10:16:55 +00:00
|
|
|
panic("n_tty: get");
|
2009-06-11 11:50:12 +00:00
|
|
|
tty_ldisc_assign(tty, new_ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
tty_set_termios_ldisc(tty, N_TTY);
|
2009-06-11 11:50:12 +00:00
|
|
|
r = tty_ldisc_open(tty, new_ldisc);
|
|
|
|
|
if (r < 0)
|
|
|
|
|
panic("Couldn't open N_TTY ldisc for "
|
|
|
|
|
"%s --- error %d.",
|
|
|
|
|
tty_name(tty, buf), r);
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
TTY: restore tty_ldisc_wait_idle
It was removed in 65b770468e98 (tty-ldisc: turn ldisc user count into
a proper refcount), but we need to wait for last user to quit the
ldisc before we close it in tty_set_ldisc.
Otherwise weird things start to happen. There might be processes
waiting in tty_read->n_tty_read on tty->read_wait for input to appear
and at that moment, a change of ldisc is fatal. n_tty_close is called,
it frees read_buf and the waiting process is still in the middle of
reading and goes nuts after it is woken.
Previously we prevented close to happen when others are in ldisc ops
by tty_ldisc_wait_idle in tty_set_ldisc. But the commit above removed
that. So revoke the change and test whether there is 1 user (=we), and
allow the close then.
We can do that without ldisc/tty locks, because nobody else can open
the device due to TTY_LDISC_CHANGING bit set, so we in fact wait for
everybody to leave.
I don't understand why tty_ldisc_lock would be needed either when the
counter is an atomic variable, so this is a lockless
tty_ldisc_wait_idle.
On the other hand, if we fail to wait (timeout or signal), we have to
reenable the halted ldiscs, so we take ldisc lock and reuse the setup
path at the end of tty_set_ldisc.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: Sebastian Andrzej Siewior <bigeasy@breakpoint.cc>
LKML-Reference: <20101031104136.GA511@Chamillionaire.breakpoint.cc>
LKML-Reference: <1287669539-22644-1-git-send-email-jslaby@suse.cz>
Cc: Alan Cox <alan@linux.intel.com>
Cc: stable@kernel.org [32, 33, 36]
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-10-31 22:17:51 +00:00
|
|
|
/**
|
|
|
|
|
* tty_ldisc_wait_idle - wait for the ldisc to become idle
|
|
|
|
|
* @tty: tty to wait for
|
2011-11-16 15:27:07 +00:00
|
|
|
* @timeout: for how long to wait at most
|
TTY: restore tty_ldisc_wait_idle
It was removed in 65b770468e98 (tty-ldisc: turn ldisc user count into
a proper refcount), but we need to wait for last user to quit the
ldisc before we close it in tty_set_ldisc.
Otherwise weird things start to happen. There might be processes
waiting in tty_read->n_tty_read on tty->read_wait for input to appear
and at that moment, a change of ldisc is fatal. n_tty_close is called,
it frees read_buf and the waiting process is still in the middle of
reading and goes nuts after it is woken.
Previously we prevented close to happen when others are in ldisc ops
by tty_ldisc_wait_idle in tty_set_ldisc. But the commit above removed
that. So revoke the change and test whether there is 1 user (=we), and
allow the close then.
We can do that without ldisc/tty locks, because nobody else can open
the device due to TTY_LDISC_CHANGING bit set, so we in fact wait for
everybody to leave.
I don't understand why tty_ldisc_lock would be needed either when the
counter is an atomic variable, so this is a lockless
tty_ldisc_wait_idle.
On the other hand, if we fail to wait (timeout or signal), we have to
reenable the halted ldiscs, so we take ldisc lock and reuse the setup
path at the end of tty_set_ldisc.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: Sebastian Andrzej Siewior <bigeasy@breakpoint.cc>
LKML-Reference: <20101031104136.GA511@Chamillionaire.breakpoint.cc>
LKML-Reference: <1287669539-22644-1-git-send-email-jslaby@suse.cz>
Cc: Alan Cox <alan@linux.intel.com>
Cc: stable@kernel.org [32, 33, 36]
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-10-31 22:17:51 +00:00
|
|
|
*
|
|
|
|
|
* Wait for the line discipline to become idle. The discipline must
|
|
|
|
|
* have been halted for this to guarantee it remains idle.
|
|
|
|
|
*/
|
2011-11-16 15:27:07 +00:00
|
|
|
static int tty_ldisc_wait_idle(struct tty_struct *tty, long timeout)
|
TTY: restore tty_ldisc_wait_idle
It was removed in 65b770468e98 (tty-ldisc: turn ldisc user count into
a proper refcount), but we need to wait for last user to quit the
ldisc before we close it in tty_set_ldisc.
Otherwise weird things start to happen. There might be processes
waiting in tty_read->n_tty_read on tty->read_wait for input to appear
and at that moment, a change of ldisc is fatal. n_tty_close is called,
it frees read_buf and the waiting process is still in the middle of
reading and goes nuts after it is woken.
Previously we prevented close to happen when others are in ldisc ops
by tty_ldisc_wait_idle in tty_set_ldisc. But the commit above removed
that. So revoke the change and test whether there is 1 user (=we), and
allow the close then.
We can do that without ldisc/tty locks, because nobody else can open
the device due to TTY_LDISC_CHANGING bit set, so we in fact wait for
everybody to leave.
I don't understand why tty_ldisc_lock would be needed either when the
counter is an atomic variable, so this is a lockless
tty_ldisc_wait_idle.
On the other hand, if we fail to wait (timeout or signal), we have to
reenable the halted ldiscs, so we take ldisc lock and reuse the setup
path at the end of tty_set_ldisc.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: Sebastian Andrzej Siewior <bigeasy@breakpoint.cc>
LKML-Reference: <20101031104136.GA511@Chamillionaire.breakpoint.cc>
LKML-Reference: <1287669539-22644-1-git-send-email-jslaby@suse.cz>
Cc: Alan Cox <alan@linux.intel.com>
Cc: stable@kernel.org [32, 33, 36]
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-10-31 22:17:51 +00:00
|
|
|
{
|
2011-11-16 15:27:07 +00:00
|
|
|
long ret;
|
2012-05-03 12:37:43 +00:00
|
|
|
ret = wait_event_timeout(tty->ldisc->wq_idle,
|
2011-11-16 15:27:07 +00:00
|
|
|
atomic_read(&tty->ldisc->users) == 1, timeout);
|
TTY: restore tty_ldisc_wait_idle
It was removed in 65b770468e98 (tty-ldisc: turn ldisc user count into
a proper refcount), but we need to wait for last user to quit the
ldisc before we close it in tty_set_ldisc.
Otherwise weird things start to happen. There might be processes
waiting in tty_read->n_tty_read on tty->read_wait for input to appear
and at that moment, a change of ldisc is fatal. n_tty_close is called,
it frees read_buf and the waiting process is still in the middle of
reading and goes nuts after it is woken.
Previously we prevented close to happen when others are in ldisc ops
by tty_ldisc_wait_idle in tty_set_ldisc. But the commit above removed
that. So revoke the change and test whether there is 1 user (=we), and
allow the close then.
We can do that without ldisc/tty locks, because nobody else can open
the device due to TTY_LDISC_CHANGING bit set, so we in fact wait for
everybody to leave.
I don't understand why tty_ldisc_lock would be needed either when the
counter is an atomic variable, so this is a lockless
tty_ldisc_wait_idle.
On the other hand, if we fail to wait (timeout or signal), we have to
reenable the halted ldiscs, so we take ldisc lock and reuse the setup
path at the end of tty_set_ldisc.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: Sebastian Andrzej Siewior <bigeasy@breakpoint.cc>
LKML-Reference: <20101031104136.GA511@Chamillionaire.breakpoint.cc>
LKML-Reference: <1287669539-22644-1-git-send-email-jslaby@suse.cz>
Cc: Alan Cox <alan@linux.intel.com>
Cc: stable@kernel.org [32, 33, 36]
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-10-31 22:17:51 +00:00
|
|
|
return ret > 0 ? 0 : -EBUSY;
|
|
|
|
|
}
|
|
|
|
|
|
2013-03-11 20:44:27 +00:00
|
|
|
/**
|
|
|
|
|
* tty_ldisc_halt - shut down the line discipline
|
|
|
|
|
* @tty: tty device
|
2013-03-11 20:44:29 +00:00
|
|
|
* @o_tty: paired pty device (can be NULL)
|
2013-03-11 20:44:28 +00:00
|
|
|
* @timeout: # of jiffies to wait for ldisc refs to be released
|
2013-03-11 20:44:27 +00:00
|
|
|
*
|
2013-03-11 20:44:29 +00:00
|
|
|
* Shut down the line discipline and work queue for this tty device and
|
|
|
|
|
* its paired pty (if exists). Clearing the TTY_LDISC flag ensures
|
2013-03-11 20:44:34 +00:00
|
|
|
* no further references can be obtained, while waiting for existing
|
|
|
|
|
* references to be released ensures no more data is fed to the ldisc.
|
2013-03-11 20:44:28 +00:00
|
|
|
*
|
2013-03-11 20:44:27 +00:00
|
|
|
* You need to do a 'flush_scheduled_work()' (outside the ldisc_mutex)
|
|
|
|
|
* in order to make sure any currently executing ldisc work is also
|
|
|
|
|
* flushed.
|
|
|
|
|
*/
|
|
|
|
|
|
2013-03-11 20:44:29 +00:00
|
|
|
static int tty_ldisc_halt(struct tty_struct *tty, struct tty_struct *o_tty,
|
2013-03-11 20:44:34 +00:00
|
|
|
long timeout)
|
2013-03-11 20:44:27 +00:00
|
|
|
{
|
2013-03-11 20:44:34 +00:00
|
|
|
int retval;
|
2013-03-11 20:44:28 +00:00
|
|
|
|
2013-03-11 20:44:27 +00:00
|
|
|
clear_bit(TTY_LDISC, &tty->flags);
|
2013-03-11 20:44:29 +00:00
|
|
|
if (o_tty)
|
|
|
|
|
clear_bit(TTY_LDISC, &o_tty->flags);
|
|
|
|
|
|
2013-03-11 20:44:28 +00:00
|
|
|
retval = tty_ldisc_wait_idle(tty, timeout);
|
2013-03-11 20:44:29 +00:00
|
|
|
if (!retval && o_tty)
|
|
|
|
|
retval = tty_ldisc_wait_idle(o_tty, timeout);
|
2013-03-11 20:44:28 +00:00
|
|
|
if (retval)
|
|
|
|
|
return retval;
|
|
|
|
|
|
2013-03-11 20:44:27 +00:00
|
|
|
set_bit(TTY_LDISC_HALTED, &tty->flags);
|
2013-03-11 20:44:34 +00:00
|
|
|
if (o_tty)
|
2013-03-11 20:44:29 +00:00
|
|
|
set_bit(TTY_LDISC_HALTED, &o_tty->flags);
|
|
|
|
|
|
2013-03-11 20:44:28 +00:00
|
|
|
return 0;
|
2013-03-11 20:44:27 +00:00
|
|
|
}
|
|
|
|
|
|
2013-03-11 20:44:24 +00:00
|
|
|
/**
|
2013-03-11 20:44:26 +00:00
|
|
|
* tty_ldisc_hangup_halt - halt the line discipline for hangup
|
|
|
|
|
* @tty: tty being hung up
|
2013-03-11 20:44:24 +00:00
|
|
|
*
|
2013-03-11 20:44:26 +00:00
|
|
|
* Shut down the line discipline and work queue for the tty device
|
|
|
|
|
* being hungup. Clear the TTY_LDISC flag to ensure no further
|
2013-03-11 20:44:34 +00:00
|
|
|
* references can be obtained and wait for remaining references to be
|
|
|
|
|
* released to ensure no more data is fed to this ldisc.
|
2013-03-11 20:44:24 +00:00
|
|
|
* Caller must hold legacy and ->ldisc_mutex.
|
2013-03-11 20:44:25 +00:00
|
|
|
*
|
|
|
|
|
* NB: tty_set_ldisc() is prevented from changing the ldisc concurrently
|
|
|
|
|
* with this function by checking the TTY_HUPPING flag.
|
2013-03-11 20:44:24 +00:00
|
|
|
*/
|
2013-03-11 20:44:26 +00:00
|
|
|
static bool tty_ldisc_hangup_halt(struct tty_struct *tty)
|
2013-03-11 20:44:24 +00:00
|
|
|
{
|
2013-03-11 20:44:25 +00:00
|
|
|
char cur_n[TASK_COMM_LEN], tty_n[64];
|
|
|
|
|
long timeout = 3 * HZ;
|
|
|
|
|
|
2013-03-11 20:44:26 +00:00
|
|
|
clear_bit(TTY_LDISC, &tty->flags);
|
|
|
|
|
|
2013-03-11 20:44:25 +00:00
|
|
|
if (tty->ldisc) { /* Not yet closed */
|
|
|
|
|
tty_unlock(tty);
|
|
|
|
|
|
|
|
|
|
while (tty_ldisc_wait_idle(tty, timeout) == -EBUSY) {
|
|
|
|
|
timeout = MAX_SCHEDULE_TIMEOUT;
|
|
|
|
|
printk_ratelimited(KERN_WARNING
|
|
|
|
|
"%s: waiting (%s) for %s took too long, but we keep waiting...\n",
|
|
|
|
|
__func__, get_task_comm(cur_n, current),
|
|
|
|
|
tty_name(tty, tty_n));
|
2013-03-11 20:44:24 +00:00
|
|
|
}
|
2013-03-11 20:44:26 +00:00
|
|
|
|
|
|
|
|
set_bit(TTY_LDISC_HALTED, &tty->flags);
|
|
|
|
|
|
2013-03-11 20:44:25 +00:00
|
|
|
/* must reacquire both locks and preserve lock order */
|
|
|
|
|
mutex_unlock(&tty->ldisc_mutex);
|
|
|
|
|
tty_lock(tty);
|
|
|
|
|
mutex_lock(&tty->ldisc_mutex);
|
2013-03-11 20:44:24 +00:00
|
|
|
}
|
|
|
|
|
return !!tty->ldisc;
|
|
|
|
|
}
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/**
|
|
|
|
|
* tty_set_ldisc - set line discipline
|
|
|
|
|
* @tty: the terminal to set
|
|
|
|
|
* @ldisc: the line discipline
|
|
|
|
|
*
|
|
|
|
|
* Set the discipline of a tty line. Must be called from a process
|
2009-06-11 11:50:12 +00:00
|
|
|
* context. The ldisc change logic has to protect itself against any
|
|
|
|
|
* overlapping ldisc change (including on the other end of pty pairs),
|
|
|
|
|
* the close of one side of a tty/pty pair, and eventually hangup.
|
2008-07-22 10:16:55 +00:00
|
|
|
*
|
2009-06-11 11:50:12 +00:00
|
|
|
* Locking: takes tty_ldisc_lock, termios_mutex
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
int tty_set_ldisc(struct tty_struct *tty, int ldisc)
|
|
|
|
|
{
|
|
|
|
|
int retval;
|
2009-06-11 11:50:12 +00:00
|
|
|
struct tty_ldisc *o_ldisc, *new_ldisc;
|
2008-07-22 10:16:55 +00:00
|
|
|
struct tty_struct *o_tty;
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
new_ldisc = tty_ldisc_get(ldisc);
|
|
|
|
|
if (IS_ERR(new_ldisc))
|
|
|
|
|
return PTR_ERR(new_ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_lock(tty);
|
2008-07-22 10:16:55 +00:00
|
|
|
/*
|
2009-06-11 11:50:12 +00:00
|
|
|
* We need to look at the tty locking here for pty/tty pairs
|
|
|
|
|
* when both sides try to change in parallel.
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
o_tty = tty->link; /* o_tty is the pty side or NULL */
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
/*
|
|
|
|
|
* Check the no-op case
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
if (tty->ldisc->ops->num == ldisc) {
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_unlock(tty);
|
2009-06-11 11:50:12 +00:00
|
|
|
tty_ldisc_put(new_ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
return 0;
|
|
|
|
|
}
|
|
|
|
|
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_unlock(tty);
|
2009-06-11 11:50:12 +00:00
|
|
|
/*
|
|
|
|
|
* Problem: What do we do if this blocks ?
|
|
|
|
|
* We could deadlock here
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
tty_wait_until_sent(tty, 0);
|
|
|
|
|
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_lock(tty);
|
2009-06-11 11:50:12 +00:00
|
|
|
mutex_lock(&tty->ldisc_mutex);
|
|
|
|
|
|
|
|
|
|
/*
|
|
|
|
|
* We could be midstream of another ldisc change which has
|
|
|
|
|
* dropped the lock during processing. If so we need to wait.
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
while (test_bit(TTY_LDISC_CHANGING, &tty->flags)) {
|
|
|
|
|
mutex_unlock(&tty->ldisc_mutex);
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_unlock(tty);
|
2009-06-11 11:50:12 +00:00
|
|
|
wait_event(tty_ldisc_wait,
|
|
|
|
|
test_bit(TTY_LDISC_CHANGING, &tty->flags) == 0);
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_lock(tty);
|
2009-06-11 11:50:12 +00:00
|
|
|
mutex_lock(&tty->ldisc_mutex);
|
|
|
|
|
}
|
2009-11-30 13:18:29 +00:00
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
set_bit(TTY_LDISC_CHANGING, &tty->flags);
|
2009-06-11 11:51:41 +00:00
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/*
|
|
|
|
|
* No more input please, we are switching. The new ldisc
|
|
|
|
|
* will update this value in the ldisc open function
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
tty->receive_room = 0;
|
|
|
|
|
|
|
|
|
|
o_ldisc = tty->ldisc;
|
2009-11-30 13:18:29 +00:00
|
|
|
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_unlock(tty);
|
2008-07-22 10:16:55 +00:00
|
|
|
/*
|
|
|
|
|
* Make sure we don't change while someone holds a
|
|
|
|
|
* reference to the line discipline. The TTY_LDISC bit
|
|
|
|
|
* prevents anyone taking a reference once it is clear.
|
|
|
|
|
* We need the lock to avoid racing reference takers.
|
2009-01-02 13:44:56 +00:00
|
|
|
*
|
|
|
|
|
* We must clear the TTY_LDISC bit here to avoid a livelock
|
|
|
|
|
* with a userspace app continually trying to use the tty in
|
|
|
|
|
* parallel to the change and re-referencing the tty.
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
|
|
|
|
|
2013-03-11 20:44:34 +00:00
|
|
|
retval = tty_ldisc_halt(tty, o_tty, 5 * HZ);
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
/*
|
2013-03-11 20:44:35 +00:00
|
|
|
* Wait for hangup to complete, if pending.
|
2009-06-11 11:50:12 +00:00
|
|
|
* We must drop the mutex here in case a hangup is also in process.
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
2009-06-11 11:50:12 +00:00
|
|
|
|
|
|
|
|
mutex_unlock(&tty->ldisc_mutex);
|
|
|
|
|
|
2013-03-11 20:44:35 +00:00
|
|
|
flush_work(&tty->hangup_work);
|
2009-06-11 11:50:12 +00:00
|
|
|
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_lock(tty);
|
2010-06-01 20:53:06 +00:00
|
|
|
mutex_lock(&tty->ldisc_mutex);
|
TTY: restore tty_ldisc_wait_idle
It was removed in 65b770468e98 (tty-ldisc: turn ldisc user count into
a proper refcount), but we need to wait for last user to quit the
ldisc before we close it in tty_set_ldisc.
Otherwise weird things start to happen. There might be processes
waiting in tty_read->n_tty_read on tty->read_wait for input to appear
and at that moment, a change of ldisc is fatal. n_tty_close is called,
it frees read_buf and the waiting process is still in the middle of
reading and goes nuts after it is woken.
Previously we prevented close to happen when others are in ldisc ops
by tty_ldisc_wait_idle in tty_set_ldisc. But the commit above removed
that. So revoke the change and test whether there is 1 user (=we), and
allow the close then.
We can do that without ldisc/tty locks, because nobody else can open
the device due to TTY_LDISC_CHANGING bit set, so we in fact wait for
everybody to leave.
I don't understand why tty_ldisc_lock would be needed either when the
counter is an atomic variable, so this is a lockless
tty_ldisc_wait_idle.
On the other hand, if we fail to wait (timeout or signal), we have to
reenable the halted ldiscs, so we take ldisc lock and reuse the setup
path at the end of tty_set_ldisc.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: Sebastian Andrzej Siewior <bigeasy@breakpoint.cc>
LKML-Reference: <20101031104136.GA511@Chamillionaire.breakpoint.cc>
LKML-Reference: <1287669539-22644-1-git-send-email-jslaby@suse.cz>
Cc: Alan Cox <alan@linux.intel.com>
Cc: stable@kernel.org [32, 33, 36]
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-10-31 22:17:51 +00:00
|
|
|
|
|
|
|
|
/* handle wait idle failure locked */
|
|
|
|
|
if (retval) {
|
|
|
|
|
tty_ldisc_put(new_ldisc);
|
|
|
|
|
goto enable;
|
|
|
|
|
}
|
|
|
|
|
|
2012-07-10 04:54:13 +00:00
|
|
|
if (test_bit(TTY_HUPPING, &tty->flags)) {
|
2009-06-11 11:50:12 +00:00
|
|
|
/* We were raced by the hangup method. It will have stomped
|
|
|
|
|
the ldisc data and closed the ldisc down */
|
|
|
|
|
clear_bit(TTY_LDISC_CHANGING, &tty->flags);
|
|
|
|
|
mutex_unlock(&tty->ldisc_mutex);
|
|
|
|
|
tty_ldisc_put(new_ldisc);
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_unlock(tty);
|
2009-06-11 11:50:12 +00:00
|
|
|
return -EIO;
|
|
|
|
|
}
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/* Shutdown the current discipline. */
|
2009-06-11 11:50:12 +00:00
|
|
|
tty_ldisc_close(tty, o_ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
/* Now set up the new line discipline. */
|
2009-06-11 11:50:12 +00:00
|
|
|
tty_ldisc_assign(tty, new_ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
tty_set_termios_ldisc(tty, ldisc);
|
2009-06-11 11:50:12 +00:00
|
|
|
|
|
|
|
|
retval = tty_ldisc_open(tty, new_ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
if (retval < 0) {
|
2009-06-11 11:50:12 +00:00
|
|
|
/* Back to the old one or N_TTY if we can't */
|
|
|
|
|
tty_ldisc_put(new_ldisc);
|
|
|
|
|
tty_ldisc_restore(tty, o_ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
2009-06-11 11:50:12 +00:00
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/* At this point we hold a reference to the new ldisc and a
|
|
|
|
|
a reference to the old ldisc. If we ended up flipping back
|
|
|
|
|
to the existing ldisc we have two references to it */
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
if (tty->ldisc->ops->num != o_ldisc->ops->num && tty->ops->set_ldisc)
|
2008-07-22 10:16:55 +00:00
|
|
|
tty->ops->set_ldisc(tty);
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
tty_ldisc_put(o_ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
|
TTY: restore tty_ldisc_wait_idle
It was removed in 65b770468e98 (tty-ldisc: turn ldisc user count into
a proper refcount), but we need to wait for last user to quit the
ldisc before we close it in tty_set_ldisc.
Otherwise weird things start to happen. There might be processes
waiting in tty_read->n_tty_read on tty->read_wait for input to appear
and at that moment, a change of ldisc is fatal. n_tty_close is called,
it frees read_buf and the waiting process is still in the middle of
reading and goes nuts after it is woken.
Previously we prevented close to happen when others are in ldisc ops
by tty_ldisc_wait_idle in tty_set_ldisc. But the commit above removed
that. So revoke the change and test whether there is 1 user (=we), and
allow the close then.
We can do that without ldisc/tty locks, because nobody else can open
the device due to TTY_LDISC_CHANGING bit set, so we in fact wait for
everybody to leave.
I don't understand why tty_ldisc_lock would be needed either when the
counter is an atomic variable, so this is a lockless
tty_ldisc_wait_idle.
On the other hand, if we fail to wait (timeout or signal), we have to
reenable the halted ldiscs, so we take ldisc lock and reuse the setup
path at the end of tty_set_ldisc.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Tested-by: Sebastian Andrzej Siewior <bigeasy@breakpoint.cc>
LKML-Reference: <20101031104136.GA511@Chamillionaire.breakpoint.cc>
LKML-Reference: <1287669539-22644-1-git-send-email-jslaby@suse.cz>
Cc: Alan Cox <alan@linux.intel.com>
Cc: stable@kernel.org [32, 33, 36]
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-10-31 22:17:51 +00:00
|
|
|
enable:
|
2008-07-22 10:16:55 +00:00
|
|
|
/*
|
2009-06-11 11:50:12 +00:00
|
|
|
* Allow ldisc referencing to occur again
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
tty_ldisc_enable(tty);
|
|
|
|
|
if (o_tty)
|
|
|
|
|
tty_ldisc_enable(o_tty);
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
/* Restart the work queue in case no characters kick it off. Safe if
|
2008-07-22 10:16:55 +00:00
|
|
|
already running */
|
2013-03-11 20:44:34 +00:00
|
|
|
schedule_work(&tty->port->buf.work);
|
|
|
|
|
if (o_tty)
|
2012-10-18 20:26:47 +00:00
|
|
|
schedule_work(&o_tty->port->buf.work);
|
2013-03-11 20:44:34 +00:00
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
mutex_unlock(&tty->ldisc_mutex);
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_unlock(tty);
|
2008-07-22 10:16:55 +00:00
|
|
|
return retval;
|
|
|
|
|
}
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
/**
|
|
|
|
|
* tty_reset_termios - reset terminal state
|
|
|
|
|
* @tty: tty to reset
|
|
|
|
|
*
|
|
|
|
|
* Restore a terminal to the driver default state.
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
static void tty_reset_termios(struct tty_struct *tty)
|
|
|
|
|
{
|
|
|
|
|
mutex_lock(&tty->termios_mutex);
|
2012-07-14 14:31:47 +00:00
|
|
|
tty->termios = tty->driver->init_termios;
|
|
|
|
|
tty->termios.c_ispeed = tty_termios_input_baud_rate(&tty->termios);
|
|
|
|
|
tty->termios.c_ospeed = tty_termios_baud_rate(&tty->termios);
|
2009-06-11 11:50:12 +00:00
|
|
|
mutex_unlock(&tty->termios_mutex);
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_reinit - reinitialise the tty ldisc
|
|
|
|
|
* @tty: tty to reinit
|
2010-02-08 10:09:26 +00:00
|
|
|
* @ldisc: line discipline to reinitialize
|
2009-06-11 11:50:12 +00:00
|
|
|
*
|
2010-02-08 10:09:26 +00:00
|
|
|
* Switch the tty to a line discipline and leave the ldisc
|
|
|
|
|
* state closed
|
2009-06-11 11:50:12 +00:00
|
|
|
*/
|
|
|
|
|
|
2010-10-27 15:13:21 +00:00
|
|
|
static int tty_ldisc_reinit(struct tty_struct *tty, int ldisc)
|
2009-06-11 11:50:12 +00:00
|
|
|
{
|
2010-10-27 15:13:21 +00:00
|
|
|
struct tty_ldisc *ld = tty_ldisc_get(ldisc);
|
|
|
|
|
|
|
|
|
|
if (IS_ERR(ld))
|
|
|
|
|
return -1;
|
2009-06-11 11:50:12 +00:00
|
|
|
|
|
|
|
|
tty_ldisc_close(tty, tty->ldisc);
|
|
|
|
|
tty_ldisc_put(tty->ldisc);
|
|
|
|
|
tty->ldisc = NULL;
|
|
|
|
|
/*
|
|
|
|
|
* Switch the line discipline back
|
|
|
|
|
*/
|
|
|
|
|
tty_ldisc_assign(tty, ld);
|
2010-02-08 10:09:26 +00:00
|
|
|
tty_set_termios_ldisc(tty, ldisc);
|
2010-10-27 15:13:21 +00:00
|
|
|
|
|
|
|
|
return 0;
|
2009-06-11 11:50:12 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_hangup - hangup ldisc reset
|
|
|
|
|
* @tty: tty being hung up
|
|
|
|
|
*
|
|
|
|
|
* Some tty devices reset their termios when they receive a hangup
|
|
|
|
|
* event. In that situation we must also switch back to N_TTY properly
|
|
|
|
|
* before we reset the termios data.
|
|
|
|
|
*
|
|
|
|
|
* Locking: We can take the ldisc mutex as the rest of the code is
|
|
|
|
|
* careful to allow for this.
|
|
|
|
|
*
|
|
|
|
|
* In the pty pair case this occurs in the close() path of the
|
|
|
|
|
* tty itself so we must be careful about locking rules.
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
void tty_ldisc_hangup(struct tty_struct *tty)
|
|
|
|
|
{
|
|
|
|
|
struct tty_ldisc *ld;
|
2010-02-08 10:09:26 +00:00
|
|
|
int reset = tty->driver->flags & TTY_DRIVER_RESET_TERMIOS;
|
|
|
|
|
int err = 0;
|
2009-06-11 11:50:12 +00:00
|
|
|
|
2013-03-11 20:44:38 +00:00
|
|
|
tty_ldisc_debug(tty, "closing ldisc: %p\n", tty->ldisc);
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
/*
|
|
|
|
|
* FIXME! What are the locking issues here? This may me overdoing
|
|
|
|
|
* things... This question is especially important now that we've
|
|
|
|
|
* removed the irqlock.
|
|
|
|
|
*/
|
|
|
|
|
ld = tty_ldisc_ref(tty);
|
|
|
|
|
if (ld != NULL) {
|
|
|
|
|
/* We may have no line discipline at this point */
|
|
|
|
|
if (ld->ops->flush_buffer)
|
|
|
|
|
ld->ops->flush_buffer(tty);
|
|
|
|
|
tty_driver_flush_buffer(tty);
|
|
|
|
|
if ((test_bit(TTY_DO_WRITE_WAKEUP, &tty->flags)) &&
|
|
|
|
|
ld->ops->write_wakeup)
|
|
|
|
|
ld->ops->write_wakeup(tty);
|
|
|
|
|
if (ld->ops->hangup)
|
|
|
|
|
ld->ops->hangup(tty);
|
|
|
|
|
tty_ldisc_deref(ld);
|
|
|
|
|
}
|
|
|
|
|
/*
|
|
|
|
|
* FIXME: Once we trust the LDISC code better we can wait here for
|
|
|
|
|
* ldisc completion and fix the driver call race
|
|
|
|
|
*/
|
|
|
|
|
wake_up_interruptible_poll(&tty->write_wait, POLLOUT);
|
|
|
|
|
wake_up_interruptible_poll(&tty->read_wait, POLLIN);
|
|
|
|
|
/*
|
|
|
|
|
* Shutdown the current line discipline, and reset it to
|
2010-02-08 10:09:26 +00:00
|
|
|
* N_TTY if need be.
|
|
|
|
|
*
|
|
|
|
|
* Avoid racing set_ldisc or tty_ldisc_release
|
2009-06-11 11:50:12 +00:00
|
|
|
*/
|
2010-02-08 10:09:26 +00:00
|
|
|
mutex_lock(&tty->ldisc_mutex);
|
2010-06-01 20:53:06 +00:00
|
|
|
|
2013-03-11 20:44:26 +00:00
|
|
|
if (tty_ldisc_hangup_halt(tty)) {
|
2013-03-11 20:44:36 +00:00
|
|
|
|
|
|
|
|
/* At this point we have a halted ldisc; we want to close it and
|
|
|
|
|
reopen a new ldisc. We could defer the reopen to the next
|
|
|
|
|
open but it means auditing a lot of other paths so this is
|
|
|
|
|
a FIXME */
|
2010-02-08 10:09:26 +00:00
|
|
|
if (reset == 0) {
|
2010-10-27 15:13:21 +00:00
|
|
|
|
2012-07-14 14:31:47 +00:00
|
|
|
if (!tty_ldisc_reinit(tty, tty->termios.c_line))
|
2010-10-27 15:13:21 +00:00
|
|
|
err = tty_ldisc_open(tty, tty->ldisc);
|
|
|
|
|
else
|
|
|
|
|
err = 1;
|
2010-02-08 10:09:26 +00:00
|
|
|
}
|
|
|
|
|
/* If the re-open fails or we reset then go to N_TTY. The
|
|
|
|
|
N_TTY open cannot fail */
|
|
|
|
|
if (reset || err) {
|
2010-10-27 15:13:21 +00:00
|
|
|
BUG_ON(tty_ldisc_reinit(tty, N_TTY));
|
2009-07-16 15:05:08 +00:00
|
|
|
WARN_ON(tty_ldisc_open(tty, tty->ldisc));
|
|
|
|
|
}
|
2010-02-08 10:09:26 +00:00
|
|
|
tty_ldisc_enable(tty);
|
2009-06-11 11:50:12 +00:00
|
|
|
}
|
2010-02-08 10:09:26 +00:00
|
|
|
mutex_unlock(&tty->ldisc_mutex);
|
|
|
|
|
if (reset)
|
|
|
|
|
tty_reset_termios(tty);
|
2013-03-11 20:44:38 +00:00
|
|
|
|
|
|
|
|
tty_ldisc_debug(tty, "re-opened ldisc: %p\n", tty->ldisc);
|
2009-06-11 11:50:12 +00:00
|
|
|
}
|
2008-07-22 10:16:55 +00:00
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_setup - open line discipline
|
|
|
|
|
* @tty: tty being shut down
|
|
|
|
|
* @o_tty: pair tty for pty/tty pairs
|
|
|
|
|
*
|
|
|
|
|
* Called during the initial open of a tty/pty pair in order to set up the
|
2009-06-11 11:50:12 +00:00
|
|
|
* line disciplines and bind them to the tty. This has no locking issues
|
|
|
|
|
* as the device isn't yet active.
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
int tty_ldisc_setup(struct tty_struct *tty, struct tty_struct *o_tty)
|
|
|
|
|
{
|
2009-06-11 11:50:12 +00:00
|
|
|
struct tty_ldisc *ld = tty->ldisc;
|
2008-07-22 10:16:55 +00:00
|
|
|
int retval;
|
|
|
|
|
|
2009-06-11 11:50:12 +00:00
|
|
|
retval = tty_ldisc_open(tty, ld);
|
|
|
|
|
if (retval)
|
|
|
|
|
return retval;
|
|
|
|
|
|
|
|
|
|
if (o_tty) {
|
|
|
|
|
retval = tty_ldisc_open(o_tty, o_tty->ldisc);
|
2008-07-22 10:16:55 +00:00
|
|
|
if (retval) {
|
2009-06-11 11:50:12 +00:00
|
|
|
tty_ldisc_close(tty, ld);
|
2008-07-22 10:16:55 +00:00
|
|
|
return retval;
|
|
|
|
|
}
|
|
|
|
|
tty_ldisc_enable(o_tty);
|
|
|
|
|
}
|
|
|
|
|
tty_ldisc_enable(tty);
|
|
|
|
|
return 0;
|
|
|
|
|
}
|
2012-08-08 15:30:13 +00:00
|
|
|
|
|
|
|
|
static void tty_ldisc_kill(struct tty_struct *tty)
|
|
|
|
|
{
|
|
|
|
|
mutex_lock(&tty->ldisc_mutex);
|
|
|
|
|
/*
|
|
|
|
|
* Now kill off the ldisc
|
|
|
|
|
*/
|
|
|
|
|
tty_ldisc_close(tty, tty->ldisc);
|
|
|
|
|
tty_ldisc_put(tty->ldisc);
|
|
|
|
|
/* Force an oops if we mess this up */
|
|
|
|
|
tty->ldisc = NULL;
|
|
|
|
|
|
|
|
|
|
/* Ensure the next open requests the N_TTY ldisc */
|
|
|
|
|
tty_set_termios_ldisc(tty, N_TTY);
|
|
|
|
|
mutex_unlock(&tty->ldisc_mutex);
|
|
|
|
|
}
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
/**
|
|
|
|
|
* tty_ldisc_release - release line discipline
|
|
|
|
|
* @tty: tty being shut down
|
|
|
|
|
* @o_tty: pair tty for pty/tty pairs
|
|
|
|
|
*
|
2009-06-11 11:51:41 +00:00
|
|
|
* Called during the final close of a tty/pty pair in order to shut down
|
|
|
|
|
* the line discpline layer. On exit the ldisc assigned is N_TTY and the
|
2009-06-11 11:50:12 +00:00
|
|
|
* ldisc has not been opened.
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
void tty_ldisc_release(struct tty_struct *tty, struct tty_struct *o_tty)
|
|
|
|
|
{
|
|
|
|
|
/*
|
2013-03-11 20:44:35 +00:00
|
|
|
* Shutdown this line discipline. As this is the final close,
|
|
|
|
|
* it does not race with the set_ldisc code path.
|
2008-07-22 10:16:55 +00:00
|
|
|
*/
|
|
|
|
|
|
2013-03-11 20:44:38 +00:00
|
|
|
tty_ldisc_debug(tty, "closing ldisc: %p\n", tty->ldisc);
|
|
|
|
|
|
2013-03-11 20:44:34 +00:00
|
|
|
tty_ldisc_halt(tty, o_tty, MAX_SCHEDULE_TIMEOUT);
|
2012-07-27 17:02:54 +00:00
|
|
|
|
2012-12-25 22:02:48 +00:00
|
|
|
tty_lock_pair(tty, o_tty);
|
2012-07-14 14:31:27 +00:00
|
|
|
/* This will need doing differently if we need to lock */
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_ldisc_kill(tty);
|
2009-06-11 11:50:12 +00:00
|
|
|
if (o_tty)
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_ldisc_kill(o_tty);
|
2009-06-29 14:21:47 +00:00
|
|
|
|
2012-08-08 15:30:13 +00:00
|
|
|
tty_unlock_pair(tty, o_tty);
|
2009-06-29 14:21:47 +00:00
|
|
|
/* And the memory resources remaining (buffers, termios) will be
|
|
|
|
|
disposed of when the kref hits zero */
|
2013-03-11 20:44:38 +00:00
|
|
|
|
|
|
|
|
tty_ldisc_debug(tty, "ldisc closed\n");
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/**
|
|
|
|
|
* tty_ldisc_init - ldisc setup for new tty
|
|
|
|
|
* @tty: tty being allocated
|
|
|
|
|
*
|
|
|
|
|
* Set up the line discipline objects for a newly allocated tty. Note that
|
|
|
|
|
* the tty structure is not completely set up when this call is made.
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
void tty_ldisc_init(struct tty_struct *tty)
|
|
|
|
|
{
|
2009-06-11 11:50:12 +00:00
|
|
|
struct tty_ldisc *ld = tty_ldisc_get(N_TTY);
|
|
|
|
|
if (IS_ERR(ld))
|
2008-07-22 10:16:55 +00:00
|
|
|
panic("n_tty: init_tty");
|
2009-06-11 11:50:12 +00:00
|
|
|
tty_ldisc_assign(tty, ld);
|
2008-07-22 10:16:55 +00:00
|
|
|
}
|
|
|
|
|
|
2011-03-23 09:48:35 +00:00
|
|
|
/**
|
|
|
|
|
* tty_ldisc_init - ldisc cleanup for new tty
|
|
|
|
|
* @tty: tty that was allocated recently
|
|
|
|
|
*
|
|
|
|
|
* The tty structure must not becompletely set up (tty_ldisc_setup) when
|
|
|
|
|
* this call is made.
|
|
|
|
|
*/
|
|
|
|
|
void tty_ldisc_deinit(struct tty_struct *tty)
|
|
|
|
|
{
|
|
|
|
|
put_ldisc(tty->ldisc);
|
|
|
|
|
tty_ldisc_assign(tty, NULL);
|
|
|
|
|
}
|
|
|
|
|
|
2008-07-22 10:16:55 +00:00
|
|
|
void tty_ldisc_begin(void)
|
|
|
|
|
{
|
|
|
|
|
/* Setup the default TTY line discipline. */
|
|
|
|
|
(void) tty_register_ldisc(N_TTY, &tty_ldisc_N_TTY);
|
|
|
|
|
}
|