diff options
author | Jacob Meuser <jakemsr@cvs.openbsd.org> | 2007-07-06 04:10:35 +0000 |
---|---|---|
committer | Jacob Meuser <jakemsr@cvs.openbsd.org> | 2007-07-06 04:10:35 +0000 |
commit | 067c425101eda3d6b956f9d0eef6602958d4cfda (patch) | |
tree | 15604b45d4db828074b77fdd09478edf52421620 /sys/dev/auconv.h | |
parent | be5066ce5b065fbdd61bd469418b7d591674a31b (diff) |
- there is no need for audio_init_ringbuffer() to always set the "pause"
attribute to '0'. this function may be called when executing an
AUDIO_SETINFO ioctl where the pause attribute was set to '1', and
setting it to '0' in this function will cause unexpected bahaviour.
- according to audio(4), the AUDIO_FLUSH ioctl "restarts recording
and playback". therefor, it should set the pause attributes to '0'.
this was previously accomplished indirectly, because it calls
audio_init_ringbuffer(), which was setting pause to '0'.
- the pause attribute must be set to '0' in audio_open() so that
recording and playback buffers can be "activated" when the device
is opened. this was apparently forgotten when audiosetinfo()
was split off of audio_open() and the logic in audiosetinfo()
was changed to accomodate being run at times other than open().
also, this only really affects read() with poll(), which may
explain why the problem was not noticed for some time.
originally part of a patch Alexandre Ratchov sent to tech@ a
while back
fixes PR 3813
"patches look correct to me" marco@
Diffstat (limited to 'sys/dev/auconv.h')
0 files changed, 0 insertions, 0 deletions