Messages in this thread Patch in this message | | | Date | Wed, 11 Mar 2020 12:19:38 +0900 | From | Takashi Sakamoto <> | Subject | Re: [PATCH 7/8] ALSA: firewire-tascam: Add missing annotation for tscm_hwdep_read_queue() |
| |
On Wed, Mar 11, 2020 at 01:09:07AM +0000, Jules Irenge wrote: > Sparse reports a warning at tscm_hwdep_read_queue() > > warning: context imbalance in tscm_hwdep_read_queue() - unexpected unlock > > The root cause is the missing annotation at tscm_hwdep_read_queue() > Add the missing __releases(&tscm->lock) annotation > > Signed-off-by: Jules Irenge <jbi.octave@gmail.com> > --- > sound/firewire/tascam/tascam-hwdep.c | 1 + > 1 file changed, 1 insertion(+) This looks good.
Acked-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
> diff --git a/sound/firewire/tascam/tascam-hwdep.c b/sound/firewire/tascam/tascam-hwdep.c > index c29a97f6f638..9801e33e7f2a 100644 > --- a/sound/firewire/tascam/tascam-hwdep.c > +++ b/sound/firewire/tascam/tascam-hwdep.c > @@ -36,6 +36,7 @@ static long tscm_hwdep_read_locked(struct snd_tscm *tscm, char __user *buf, > > static long tscm_hwdep_read_queue(struct snd_tscm *tscm, char __user *buf, > long remained, loff_t *offset) > + __releases(&tscm->lock) > { > char __user *pos = buf; > unsigned int type = SNDRV_FIREWIRE_EVENT_TASCAM_CONTROL; > -- > 2.24.1
Regards
Takashi Sakamoto
| |