d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

- Properly unlock mutex on failure in channel_trigger. Before this

Browse Source 
the function just returned with the mutex held.

MFC after:	1 week
This commit is contained in:
stas 2009-04-20 12:43:07 +00:00
parent 588e47016e
commit a780a09a2b
1 changed files with 15 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
sys/dev/sound/pci/envy24.c
View File

@ -1766,6 +1766,7 @@ envy24chan_trigger(kobj_t obj, void *data, int go)
struct sc_info *sc = ch->parent;
u_int32_t ptr;
int slot;
int error = 0;
#if 0
int i;
@ -1787,8 +1788,10 @@ envy24chan_trigger(kobj_t obj, void *data, int go)
sc->caps[0].minspeed = sc->caps[0].maxspeed = sc->speed;
sc->caps[1].minspeed = sc->caps[1].maxspeed = sc->speed;
}
else if (ch->speed != 0 && ch->speed != sc->speed)
return -1;
else if (ch->speed != 0 && ch->speed != sc->speed) {
error = -1;
goto fail;
}
if (ch->speed == 0)
ch->channel->speed = sc->speed;
/* start or enable channel */
@ -1818,16 +1821,20 @@ envy24chan_trigger(kobj_t obj, void *data, int go)
#if(0)
device_printf(sc->dev, "envy24chan_trigger(): emldmawr\n");
#endif
if (ch->run != 1)
return -1;
if (ch->run != 1) {
error = -1;
goto fail;
}
ch->emldma(ch);
break;
case PCMTRIG_EMLDMARD:
#if(0)
device_printf(sc->dev, "envy24chan_trigger(): emldmard\n");
#endif
if (ch->run != 1)
return -1;
if (ch->run != 1) {
error = -1;
goto fail;
}
ch->emldma(ch);
break;
case PCMTRIG_ABORT:
@ -1859,9 +1866,9 @@ envy24chan_trigger(kobj_t obj, void *data, int go)
}
break;
}
fail:
snd_mtxunlock(sc->lock);
return 0;
return (error);
}
static int