lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 30 Mar 2010 22:56:16 +0200 From: Arnd Bergmann <arnd@...db.de> To: LKML <linux-kernel@...r.kernel.org> Cc: Alan Cox <alan@...rguk.ukuu.org.uk>, Greg KH <gregkh@...e.de>, Frederic Weisbecker <fweisbec@...il.com>, Thomas Gleixner <tglx@...utronix.de>, Andrew Morton <akpm@...ux-foundation.org>, John Kacur <jkacur@...hat.com>, Al Viro <viro@...iv.linux.org.uk>, Ingo Molnar <mingo@...e.hu>, Arnd Bergmann <arnd@...db.de> Subject: [RFC 5/9] tty: make ldisc_mutex nest under tty_lock Signed-off-by: Arnd Bergmann <arnd@...db.de> --- drivers/char/tty_io.c | 2 +- drivers/char/tty_ldisc.c | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/char/tty_io.c b/drivers/char/tty_io.c index 91cc978..aeab6cb 100644 --- a/drivers/char/tty_io.c +++ b/drivers/char/tty_io.c @@ -1279,7 +1279,7 @@ static int tty_reopen(struct tty_struct *tty) tty->count++; tty->driver = driver; /* N.B. why do this every time?? */ - mutex_lock(&tty->ldisc_mutex); + mutex_lock_tty_on(&tty->ldisc_mutex); WARN_ON(!test_bit(TTY_LDISC, &tty->flags)); mutex_unlock(&tty->ldisc_mutex); diff --git a/drivers/char/tty_ldisc.c b/drivers/char/tty_ldisc.c index af5b4a7..946ac05 100644 --- a/drivers/char/tty_ldisc.c +++ b/drivers/char/tty_ldisc.c @@ -580,7 +580,7 @@ int tty_set_ldisc(struct tty_struct *tty, int ldisc) tty_wait_until_sent(tty, 0); - mutex_lock(&tty->ldisc_mutex); + mutex_lock_tty_off(&tty->ldisc_mutex); /* * We could be midstream of another ldisc change which has @@ -591,7 +591,7 @@ int tty_set_ldisc(struct tty_struct *tty, int ldisc) mutex_unlock(&tty->ldisc_mutex); wait_event(tty_ldisc_wait, test_bit(TTY_LDISC_CHANGING, &tty->flags) == 0); - mutex_lock(&tty->ldisc_mutex); + mutex_lock_tty_off(&tty->ldisc_mutex); } tty_lock(); @@ -632,7 +632,7 @@ int tty_set_ldisc(struct tty_struct *tty, int ldisc) flush_scheduled_work(); - mutex_lock(&tty->ldisc_mutex); + mutex_lock_tty_off(&tty->ldisc_mutex); tty_lock(); if (test_bit(TTY_HUPPED, &tty->flags)) { /* We were raced by the hangup method. It will have stomped @@ -779,7 +779,7 @@ void tty_ldisc_hangup(struct tty_struct *tty) * * Avoid racing set_ldisc or tty_ldisc_release */ - mutex_lock(&tty->ldisc_mutex); + mutex_lock_tty_on(&tty->ldisc_mutex); tty_ldisc_halt(tty); /* At this point we have a closed ldisc and we want to reopen it. We could defer this to the next open but @@ -854,7 +854,7 @@ void tty_ldisc_release(struct tty_struct *tty, struct tty_struct *o_tty) tty_ldisc_halt(tty); flush_scheduled_work(); - mutex_lock(&tty->ldisc_mutex); + mutex_lock_tty_on(&tty->ldisc_mutex); /* * Now kill off the ldisc */ -- 1.7.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists