hamradio, ppp: change semaphore to completion
ppp and hamradio have copies of the same code that uses a semaphore in place of a completion for historic reasons. Make it use the proper interface instead in all copies. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:

committed by
David S. Miller

parent
2aa55dccf8
commit
c2c79a32fb
@@ -81,7 +81,7 @@ struct mkiss {
|
||||
#define CRC_MODE_SMACK_TEST 4
|
||||
|
||||
atomic_t refcnt;
|
||||
struct semaphore dead_sem;
|
||||
struct completion dead;
|
||||
};
|
||||
|
||||
/*---------------------------------------------------------------------------*/
|
||||
@@ -687,7 +687,7 @@ static struct mkiss *mkiss_get(struct tty_struct *tty)
|
||||
static void mkiss_put(struct mkiss *ax)
|
||||
{
|
||||
if (atomic_dec_and_test(&ax->refcnt))
|
||||
up(&ax->dead_sem);
|
||||
complete(&ax->dead);
|
||||
}
|
||||
|
||||
static int crc_force = 0; /* Can be overridden with insmod */
|
||||
@@ -715,7 +715,7 @@ static int mkiss_open(struct tty_struct *tty)
|
||||
|
||||
spin_lock_init(&ax->buflock);
|
||||
atomic_set(&ax->refcnt, 1);
|
||||
sema_init(&ax->dead_sem, 0);
|
||||
init_completion(&ax->dead);
|
||||
|
||||
ax->tty = tty;
|
||||
tty->disc_data = ax;
|
||||
@@ -795,7 +795,7 @@ static void mkiss_close(struct tty_struct *tty)
|
||||
* we have to wait for all existing users to finish.
|
||||
*/
|
||||
if (!atomic_dec_and_test(&ax->refcnt))
|
||||
down(&ax->dead_sem);
|
||||
wait_for_completion(&ax->dead);
|
||||
/*
|
||||
* Halt the transmit queue so that a new transmit cannot scribble
|
||||
* on our buffers
|
||||
|
Reference in New Issue
Block a user