source: mainline/uspace/srv/audio/hound/audio_sink.h

Last change on this file was 2fbd49c, checked in by Jiri Svoboda <jiri@…>, 3 years ago

Audio synk needs locking

Without locking the list of sink connections, we are exposed
to a race between removing a connection at the end of playback
(typically while destroying a hound context) and audio device
event PCM_EVENT_FRAMES_PLAYED which causes audio mixing to occur
via audio_sink_mix_inputs(), causing hound to crash often at
the end of playback.

  • Property mode set to 100644
File size: 3.0 KB
RevLine 
[737b4c0]1/*
2 * Copyright (c) 2012 Jan Vesely
3 * All rights reserved.
4 *
5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions
7 * are met:
8 *
9 * - Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer.
11 * - Redistributions in binary form must reproduce the above copyright
12 * notice, this list of conditions and the following disclaimer in the
13 * documentation and/or other materials provided with the distribution.
14 * - The name of the author may not be used to endorse or promote products
15 * derived from this software without specific prior written permission.
16 *
17 * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
18 * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
19 * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
20 * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
21 * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
22 * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
23 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
24 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
25 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
26 * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 */
28
29/** @addtogroup audio
30 * @brief HelenOS sound server
31 * @{
32 */
33/** @file
34 */
35
36#ifndef AUDIO_SINK_H_
37#define AUDIO_SINK_H_
38
39#include <adt/list.h>
[1df3018a]40#include <async.h>
[03362fbd]41#include <stdbool.h>
[737b4c0]42#include <fibril.h>
[2fbd49c]43#include <fibril_synch.h>
[ea6c838]44#include <pcm/format.h>
[737b4c0]45
46#include "audio_source.h"
[1df3018a]47
48typedef struct audio_sink audio_sink_t;
[737b4c0]49
[c799138]50/** Audio sink abstraction structure */
[1df3018a]51struct audio_sink {
[c799138]52 /** Link in hound's sink list */
[737b4c0]53 link_t link;
[2fbd49c]54 /** Protect sink including list of connections */
55 fibril_mutex_t lock;
[c799138]56 /** List of all related connections */
[fa60cd69]57 list_t connections;
[c799138]58 /** Sink's name */
[33b8d024]59 char *name;
[c799138]60 /** Consumes data in this format */
[ea6c838]61 pcm_format_t format;
[c799138]62 /** Backend data */
[1df3018a]63 void *private_data;
[c799138]64 /** Connect/disconnect callback */
[b7fd2a0]65 errno_t (*connection_change)(audio_sink_t *, bool);
[c799138]66 /** Backend callback to check data */
[b7fd2a0]67 errno_t (*check_format)(audio_sink_t *);
[bee5349]68 /** new data notifier */
[b7fd2a0]69 errno_t (*data_available)(audio_sink_t *);
[1df3018a]70};
[737b4c0]71
[c799138]72/**
73 * List instance helper.
74 * @param l link
75 * @return pointer to a sink structure, NULL on failure.
76 */
[1433ecda]77static inline audio_sink_t *audio_sink_list_instance(link_t *l)
[737b4c0]78{
[b893cd6]79 return l ? list_get_instance(l, audio_sink_t, link) : NULL;
[1df3018a]80}
81
[b7fd2a0]82errno_t audio_sink_init(audio_sink_t *sink, const char *name, void *private_data,
83 errno_t (*connection_change)(audio_sink_t *, bool),
84 errno_t (*check_format)(audio_sink_t *), errno_t (*data_available)(audio_sink_t *),
[bee5349]85 const pcm_format_t *f);
[1df3018a]86
[c799138]87void audio_sink_fini(audio_sink_t *sink);
[b7fd2a0]88errno_t audio_sink_set_format(audio_sink_t *sink, const pcm_format_t *format);
[1433ecda]89void audio_sink_mix_inputs(audio_sink_t *sink, void *dest, size_t size);
[737b4c0]90
91#endif
92/**
93 * @}
94 */
Note: See TracBrowser for help on using the repository browser.