Thanks! On Wed, Nov 4, 2020 at 12:16 AM Takashi Iwai <tiwai@xxxxxxx> wrote: > > On Tue, 03 Nov 2020 23:33:35 +0100, > Yu Hao wrote: > > > > The function snd_seq_queue_client_termination() is only called from > > the function seq_free_client1(). The function seq_free_client1() calls > > the function snd_seq_queue_client_leave() and the function > > snd_seq_queue_client_termination() together. Because the function > > snd_seq_queue_client_leave() does all things, so the function > > snd_seq_queue_client_termination() is a useless function. > > How poetic, rhymes awfully well :) > > > Signed-off-by: Yu Hao <yuhaobehappy@xxxxxxxxx> > > Applied now. Thanks! > > > Takashi > > > sound/core/seq/seq_clientmgr.c | 1 - > > sound/core/seq/seq_queue.c | 27 --------------------------- > > sound/core/seq/seq_queue.h | 3 --- > > 3 files changed, 31 deletions(-) > > > > diff --git a/sound/core/seq/seq_clientmgr.c b/sound/core/seq/seq_clientmgr.c > > index cc93157fa950..f9f2fea58b32 100644 > > --- a/sound/core/seq/seq_clientmgr.c > > +++ b/sound/core/seq/seq_clientmgr.c > > @@ -279,7 +279,6 @@ static int seq_free_client1(struct snd_seq_client *client) > > snd_seq_delete_all_ports(client); > > snd_seq_queue_client_leave(client->number); > > snd_use_lock_sync(&client->use_lock); > > - snd_seq_queue_client_termination(client->number); > > if (client->pool) > > snd_seq_pool_delete(&client->pool); > > spin_lock_irq(&clients_lock); > > diff --git a/sound/core/seq/seq_queue.c b/sound/core/seq/seq_queue.c > > index 71a6ea62c3be..13cfc2d47fa7 100644 > > --- a/sound/core/seq/seq_queue.c > > +++ b/sound/core/seq/seq_queue.c > > @@ -537,33 +537,6 @@ int snd_seq_queue_is_used(int queueid, int client) > > > > /*----------------------------------------------------------------*/ > > > > -/* notification that client has left the system - > > - * stop the timer on all queues owned by this client > > - */ > > -void snd_seq_queue_client_termination(int client) > > -{ > > - unsigned long flags; > > - int i; > > - struct snd_seq_queue *q; > > - bool matched; > > - > > - for (i = 0; i < SNDRV_SEQ_MAX_QUEUES; i++) { > > - if ((q = queueptr(i)) == NULL) > > - continue; > > - spin_lock_irqsave(&q->owner_lock, flags); > > - matched = (q->owner == client); > > - if (matched) > > - q->klocked = 1; > > - spin_unlock_irqrestore(&q->owner_lock, flags); > > - if (matched) { > > - if (q->timer->running) > > - snd_seq_timer_stop(q->timer); > > - snd_seq_timer_reset(q->timer); > > - } > > - queuefree(q); > > - } > > -} > > - > > /* final stage notification - > > * remove cells for no longer exist client (for non-owned queue) > > * or delete this queue (for owned queue) > > diff --git a/sound/core/seq/seq_queue.h b/sound/core/seq/seq_queue.h > > index 9254c8dbe5e3..1c3a8d3254d9 100644 > > --- a/sound/core/seq/seq_queue.h > > +++ b/sound/core/seq/seq_queue.h > > @@ -59,9 +59,6 @@ struct snd_seq_queue *snd_seq_queue_alloc(int client, int locked, unsigned int f > > /* delete queue (destructor) */ > > int snd_seq_queue_delete(int client, int queueid); > > > > -/* notification that client has left the system */ > > -void snd_seq_queue_client_termination(int client); > > - > > /* final stage */ > > void snd_seq_queue_client_leave(int client); > > > > -- > > 2.17.1 > >