paraslash.git
10 years agodccp: sort out help text
Gerrit Renker [Sat, 13 Mar 2010 12:09:35 +0000 (13:09 +0100)]
dccp: sort out help text

This improves the formatting of the dccp help for para_recv: help2man
interprets words starting with '--...' as new options, not as examples.

10 years agonet: fix broken dccp_available_ccids()
Gerrit Renker [Sat, 13 Mar 2010 12:08:32 +0000 (13:08 +0100)]
net: fix broken dccp_available_ccids()

This fixes a dumb programming error in dccp_available_ccids(), where a
pointer to 1-byte address was type cast to a socklent_t (typically 4 byte)
pointer. Changed the interface to not expose internals of call.

In addition, this patch adds a missing close() in the error path of makesock().

10 years agoaudiod: Don't init slot info unless opening the receiver succeeds.
Andre Noll [Wed, 10 Mar 2010 07:56:55 +0000 (08:56 +0100)]
audiod: Don't init slot info unless opening the receiver succeeds.

Currently we set the slot's audio format number even if the receiver could not
be opened. This confuses the slot clearing code as no writer is active in
this case which triggers the assertion

para_audiod: audiod.c:384: close_writers: Assertion `s->wns'

Fix it by not touching the slot info structure unless we have successfully started
the receiver. Also perform some minor cleanups while we're at it.

Thanks to Gerrit Renker for pointing out this bug.

10 years agoMerge branch 'maint'
Andre Noll [Sun, 7 Mar 2010 15:34:40 +0000 (16:34 +0100)]
Merge branch 'maint'

10 years agosetup_next_fec_group(): Return early on eof.
Andre Noll [Sun, 7 Mar 2010 15:34:25 +0000 (16:34 +0100)]
setup_next_fec_group(): Return early on eof.

The old code updated the group start via set_group_timing() even if the
end of the file was reached. This could confuse the logic in vss_send()
as it uses the goup start timestamp to find out whether all FEC clients
have sent all remaining slices.

Fix this by moving the check for the eof condition before the call to
set_group_timing() that updates the group start.

10 years agoMinor doxygen fixes.
Andre Noll [Fri, 5 Mar 2010 08:18:49 +0000 (09:18 +0100)]
Minor doxygen fixes.

Fixes

net.c:279: Warning: Found unknown command `\opt'
net.h:104: Warning: argument 'l3type' of command @param is not found in the argument list of makesock...

10 years agoNEWS update.
Andre Noll [Fri, 5 Mar 2010 07:47:37 +0000 (08:47 +0100)]
NEWS update.

10 years agoaudiod: Make default filters work also for udp streams.
Andre Noll [Fri, 5 Mar 2010 07:42:13 +0000 (08:42 +0100)]
audiod: Make default filters work also for udp streams.

Currently one has to use --no_default_filters and must specify fecdec as the
first filter for UDP streams, a fact that is not documented anywhere. Make decoding
of udp streams work if no filters were given by changing init_default_filters() to
add fecdec automatically if the udp receiver is used.

10 years agonet: revert restricting SO_REUSEADDR
Gerrit Renker [Wed, 3 Mar 2010 16:57:14 +0000 (17:57 +0100)]
net: revert restricting SO_REUSEADDR

This reverts part of commit a5927501e41fa3fca2975452617474e78ffecc48
(08_refactor-makesock.diff, 25 Feb 2010), which restricted by default
SO_REUSEADDR to connection-oriented protocols only - it used to be
set on all passive sockets.

However, the same problem exists also for UDP, whenever connecting
more than one listener on the same host, hence revert it.

10 years agobtr: Fix a nasty thinko in merge_input_pool().
Andre Noll [Tue, 2 Mar 2010 06:10:00 +0000 (07:10 +0100)]
btr: Fix a nasty thinko in merge_input_pool().

We must take into account the number of bytes already consumed from the wrap buffer.
Otherwise, we might end up not merging as much as we must,  which may cause errors
in the child nodes.

This bug was quite hard to hit, but it did bite the aac decoder reliably when used in audiod
with UDP + FEC.

10 years agobtr: Kill an unused variable.
Andre Noll [Tue, 2 Mar 2010 05:54:18 +0000 (06:54 +0100)]
btr: Kill an unused variable.

10 years agonet.h: Fic compilation on systems with partial DCCP support.
Andre Noll [Thu, 25 Feb 2010 16:33:54 +0000 (17:33 +0100)]
net.h: Fic compilation on systems with partial DCCP support.

There exists at least one system that defines IPPROTO_DCCP but not
DCCP_SOCKOPT_CCID and friends. On such systems compilation fails due
to to undefined symbols.

Fix this problem by checking each value separately.

10 years ago11_recv-segfault.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:46 +0000 (17:05 +0100)]
11_recv-segfault.diff

This fixes a segmentation fault when trying to access a NULL rn.conf:

vongole> para_recv -r 'dccp -i localhost -c 253'
dccp_recv_ccid_support_check: 'CCID-253' not supported on this host.
main: parse failed
Segmentation fault

10 years ago10_set-client-ccid.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:46 +0000 (17:05 +0100)]
10_set-client-ccid.diff

This adds a commandline option to either
 * select the CCID in use for the sender-receiver half-connection or
 * advertise the (para_)client's priorities for the CCID;
 * the feature is optional and can be used to fine-tune a connection.

The arguments are both range-checked by gengetopt, and against the CCIDs
available on the host. Currently known CCIDs are 2-4, 248-254. Trying a
CCID which is not supported by the host exits via the following route:

 dccp_recv_ccid_support_check: 'CCID-253' not supported on this host.
 main: parse failed
 main: recv syntax error

The CCID negotiation can be watched in wireshark, between the 'Request'
packet sent by the para_client and the corresponding 'Response' packet of
the para_server. Here are some example negotiations:

     +-------------+---------------+----------------+
     | client-list |  server-list  | resulting CCID |
     +-------------+---------------+----------------+
     |  3, 2       |   2, 3, 4     |       2        |
     |  4, 3, 2, 4 |   3, 2, 4     |       3        |
     |  2, 4       |   2, 3, 4     |       2        |
     |  4, 3, 2, 4 |   2, 3, 4     |       2        |
     |  2          |   2, 3, 4     |       2        |
     |  3          |   2, 3, 4     |       3        |
     |  4          |   2, 3, 4     |       4        |
     +-------------+---------------+----------------+

The server list of (2, 3, 4) is the unaltered default, it can be changed on
the server using the same mechanism.

As shown, the easiest way is to use just a single '--ccid <n>' argument;
when passing multiple values, the outcome is determined by the algorithm
described in RFC 4340, 6.3.1:
 * 'server-priority' means that the server overrids the result;
 * the server traverses its list of preferences in order of priority and
 * uses the first entry which also appears in the client-list;
 * resets connection with negotiation failure when there is no common value.

Applying this algorithm on the above lists can be used to verify the
table. This also illustrates that repetition of CCID values has no effect
on the outcome: if the repeated value is first in the server list, it will
be chosen; otherwise all instances of that value are ignored.

10 years ago09_fix-errno-usage.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:45 +0000 (17:05 +0100)]
09_fix-errno-usage.diff

This fixes a bug in the use of errno which corresponds to the 'common mistake'
in the use of errno(3): to not save it before making new system calls. Thanks.

10 years ago08_refactor-makesock.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:45 +0000 (17:05 +0100)]
08_refactor-makesock.diff

This is a refactoring of makesock():
 * the ubiquitous AF_UNSPEC is promoted as the default;
 * a wrapper for active sockets, para_connect_simple();
 * para_listen() now supports flowopts;
 * a wrapper for passive sockets, para_listen_simple();
 * for consistency, port numbers (which are 2 byte in
   UDP, TCP, and DCCP) have all been set to 'uint16_t'.

10 years ago07_flow-opt-support.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:45 +0000 (17:05 +0100)]
07_flow-opt-support.diff

In order to support a class of socket options which have to be set
before commiting a socket to a connection, this patch adds an API
for "pre-connection" socket options, called "flowopts" here.

The API is used as follows:
 * to provide queue storage, a new head is allocated using flowopt_new();
 * this queue is then populated with pre-connection socket options via
   - generic 'add' method flowopt_add(),
   - a specific variant for adding Boolean options - flowopt_add_bool(),
   - convenience macros - OPT_ADD(), OPT_ENABLE(), OPT_DISABLE();
 * the rest is then dealt with automatically by makesock():
   - any provided flowopts are set before making the connection,
   - due to using a queue, the order of invocation is preserved,
   - after making the connection, makesock() frees storage that has been
     previously allocated by flowopt_new() and flowopt_add.*().

10 years ago06_get-tx-ccid.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:45 +0000 (17:05 +0100)]
06_get-tx-ccid.diff

This patch adds support for querying the TX half-connection CCID
in use to from sender to client.

On more recent kernels (> 2.6.30) supporting this functionality,
connected clients are now displayed in the manner
          <IP-address>"#"<port>"~"<CCID-number>

10 years ago05_dccp-supported-ccid-lookup.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:45 +0000 (17:05 +0100)]
05_dccp-supported-ccid-lookup.diff

This adds a self-contained function to return the space-separated list of
CCIDs available on the para_server. Visible via 'para_client si'.

Since feature negotiation for CCIDs is supported only from v2.6.30-rc1
(released 7th April 2009), a more recent kernel (>= 2.6.30) is needed to
support this feature, as well as subsequent per-connection use of CCIDs.

Note: This patch still uses a hardcoded upper bound (DCCP_MAX_HOST_CCIDS),
      support dynamically detecting the number of CCIDs will be in 2.6.33.

10 years ago04_per-connection-ccid-constants.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:45 +0000 (17:05 +0100)]
04_per-connection-ccid-constants.diff

This adds symbolic constants for per-connection CCID support; these are used
by subsequent patches.

Per-connection CCID support is available in the Linux kernel from v2.6.30-rc1
(7th April 2009) onwards, further information is available at:
 * symbolic names are taken from include/linux/dccp.h,
 * API documentation is in Documentation/networking/dccp.txt.

The patch also corrects the manner of using doxygen comments in the #defines.

10 years ago03_resolve-port-names.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:45 +0000 (17:05 +0100)]
03_resolve-port-names.diff

This patch allows to resolve port numbers into names defined
in the services(5) /etc/services database:
 (a) it provides a standalone function which will fall back to
     stringifying a port number;
 (b) it resets the 'numeric service' flag of getnameinfo(3) to
     analogously consider a lookup in /etc/services.

Assuming the following had been added to /etc/services:
para_server_control 2990/tcp # para_server control
paraslash_http_service 8000/tcp        # paraslash http sender
paraslash_dccp_service 8000/dccp       # paraslash dccp sender
paraslash_udp_service 8000/udp        # paraslash udp sender,

then the output of 'para_client si' would look like:

http sender:
        status: on
        port: paraslash_http_service
        ...
dccp sender:
        status: on
        port: paraslash_dccp_service
        ...
udp sender:
        status: on
        port: paraslash_udp_service
        ...

Note: On some systems the service lookup may involve LDAP or NIS lookups,
depending on the 'services' configuration in /etc/nsswitch.conf. This does
not seem to cause a problem, as it implies a working network connection.

10 years ago02_udp-sender-info.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:44 +0000 (17:05 +0100)]
02_udp-sender-info.diff

udp sender: simplify port information of 'si' command

The 'udp' information is already in the transport protocol name,
hence removed by this patch.

http sender:
status: on
port: 8000
...
dccp sender:
status: on
port: 8000
...
udp sender:
status: on
port: udp 8000

For DCCP, the case is similar, http relies on TCP.

10 years ago01_convert-v6-mapped-v4-addresses.diff
Gerrit Renker [Thu, 25 Feb 2010 16:05:44 +0000 (17:05 +0100)]
01_convert-v6-mapped-v4-addresses.diff

This updates the resolution of v4/v6 addresses to present v6-mapped-v4
addresses in dot-quad format rather than as subtype of v6 address.

The getnameinfo(3) is in use for both address classes, where potential
DNS lookups (requiring Internet connectivity) are avoided by using the
NI_NUMERICHOST flag.
The use of this flag reduces getnameinfo() to inet_ntop(3). In the 'Bugs'
section of the inet_ntop() manpage it is stated that v6-mapped-v4 addresses
are converted into an IPv6 format, which is what had been observed here.

The patch works around the issue by casting the v6-mapped-v4 address into
a genuine AF_INET IPv4 socket address.

10 years agoAdd some missing documentation of public functions.
Andre Noll [Sun, 21 Feb 2010 20:32:06 +0000 (21:32 +0100)]
Add some missing documentation of public functions.

10 years agoREADME.afs: Update the Troubles section.
Andre Noll [Wed, 17 Feb 2010 07:01:51 +0000 (08:01 +0100)]
README.afs: Update the Troubles section.

It still talked about para_fsck which was replaced by oslfsck in paraslash-0.4.0.

10 years agoREQUIREMENTS update.
Andre Noll [Wed, 17 Feb 2010 06:40:41 +0000 (07:40 +0100)]
REQUIREMENTS update.

10 years agoNEWS update.
Andre Noll [Wed, 17 Feb 2010 06:06:34 +0000 (07:06 +0100)]
NEWS update.

10 years agoFix doxygen reference to struct writer.
Andre Noll [Wed, 17 Feb 2010 05:28:36 +0000 (06:28 +0100)]
Fix doxygen reference to struct writer.

Fixes
Warning: unable to resolve reference to `struct' for \ref command

10 years agoRemove doxygen reference to private struct mood.
Andre Noll [Wed, 17 Feb 2010 05:27:32 +0000 (06:27 +0100)]
Remove doxygen reference to private struct mood.

This fixes the doxygen warning

Warning: unable to resolve reference to `mood' for \ref command

10 years agoclient_open(): Add documentation of missing parameters.
Andre Noll [Wed, 17 Feb 2010 05:23:41 +0000 (06:23 +0100)]
client_open(): Add documentation of missing parameters.

10 years agosched: Add missing documentation for timeout-request functions.
Andre Noll [Wed, 17 Feb 2010 05:16:10 +0000 (06:16 +0100)]
sched: Add missing documentation for timeout-request functions.

10 years agooggdec: Kill unused ->stream_start of struct private_oggdec_data().
Andre Noll [Sun, 14 Feb 2010 22:00:00 +0000 (23:00 +0100)]
oggdec: Kill unused ->stream_start of struct private_oggdec_data().

10 years agooggdec: Fix a thinko in post_select().
Andre Noll [Sun, 14 Feb 2010 21:58:12 +0000 (22:58 +0100)]
oggdec: Fix a thinko in post_select().

The old code broke out of the decoding loop if btr_node_status() returned zero
which is true if  no more input data is available for decoding or we already
have decoded more than 640K. However, in the first case there might still be
decoded data available to be passed to the child nodes and we should pass
that data as quick as possible to avoid buffer underruns.

So call btr_get_output_queue_size() rather than btr_node_status() to detect
the second case.

10 years agobtr: Rename btr_bytes_pending() and make it public.
Andre Noll [Sun, 14 Feb 2010 21:37:06 +0000 (22:37 +0100)]
btr: Rename btr_bytes_pending() and make it public.

The oggvorbis decoder needs to know this info, so make it public. As we have already
btr_get_input_queue_size() it is natural to rename btr_bytes_pending() to
btr_get_output_queue_size().

10 years agodccp_recv: Use para_readv() instead of recv_bin_buffer().
Andre Noll [Sat, 13 Feb 2010 12:12:47 +0000 (13:12 +0100)]
dccp_recv: Use para_readv() instead of recv_bin_buffer().

When the end of the buffer pool area is reached, the old code ended up calling recv()
with a buffer size smaller than a full datagram. This results in data being lost
because the remaining part of the datagram will be discarded rather than returned
at the next call to recv().

By using the new btr_pool_get_buffers() we obtain two buffers in this case, a small
one pointing to the end of the area and a larger one pointing to the area start. Passing
both buffers to para_readv() instead of using recv_bin_buffer() receives the full
datagram and therefore avoids the above mentioned loss of data.

10 years agoAdd para_readv(), a simple wrapper for readv().
Andre Noll [Sat, 13 Feb 2010 11:58:40 +0000 (12:58 +0100)]
Add para_readv(), a simple wrapper for readv().

This new function will be used by the dccp receiver in a subsequent patch.

10 years agoIntroduce btr_pool_get_buffers().
Andre Noll [Sat, 13 Feb 2010 11:57:35 +0000 (12:57 +0100)]
Introduce btr_pool_get_buffers().

This allows the users of the buffer pool API to obtain references to both parts
of the buffer pool area in case there is free space available at the end of the
area as well as at the beginning.

This is needed for reading from file descriptors where reading less than a full
datagram would discard the remaining part (DCCP and UDP).

10 years agoconfigure: Check for broken snprintf().
Andre Noll [Thu, 11 Feb 2010 21:56:35 +0000 (22:56 +0100)]
configure: Check for broken snprintf().

We rely on a working snprintf implementation at several places, so abort early
of the system has no fully C99 compliant version.

10 years agoalsa: Remove xrun().
Andre Noll [Mon, 8 Feb 2010 13:46:48 +0000 (14:46 +0100)]
alsa: Remove xrun().

This function tried to compute the duration of the underun but never
really worked.  The values were more or less random (up to hundreds
of seconds) and thus of little value.

10 years agoFix a couple of memory leaks.
Andre Noll [Sat, 6 Feb 2010 23:23:47 +0000 (00:23 +0100)]
Fix a couple of memory leaks.

Found by valgrinds memory checker.

10 years agoFix make clean and make install.
Gerrit Renker [Wed, 3 Feb 2010 14:05:33 +0000 (15:05 +0100)]
Fix make clean and make install.

Commit 77aa6680 replaced the hardcoded list of binaries in Makefile.in
by the autoconf macro @executables@. However, the "install" and the
"clean" targets still used the now undefined $(BINARIES) variable. This
caused "make clean" to not remove any executables and "make install"
to fail with

/usr/bin/install -c -s -m 755  /usr/local/bin
/usr/bin/install: missing destination file operand after `/usr/local/bin'
Try `/usr/bin/install --help' for more information.

Fix this bug by using @executables@ rather than $(BINARIES).

10 years agoMerge branch 'maint'
Andre Noll [Sun, 31 Jan 2010 22:25:16 +0000 (23:25 +0100)]
Merge branch 'maint'

Conflicts:
recv_common.c

10 years agorecv: Don't set argv to NULL when parsing receiver args.
Andre Noll [Sun, 31 Jan 2010 22:16:15 +0000 (23:16 +0100)]
recv: Don't set argv to NULL when parsing receiver args.

This leads to error messages like

(null): unrecognized option '--foo'

in case an invalid option was given. Fix it by setting argv[0] to
the receiver name instead.

Thanks to Gerrit Renker for pointing out this bug.

10 years agoMerge remote branch 'fml/next'
Andre Noll [Sat, 30 Jan 2010 21:57:09 +0000 (22:57 +0100)]
Merge remote branch 'fml/next'

10 years agoclient: Remove the buffer tree node on errors.
Andre Noll [Sat, 30 Jan 2010 13:24:17 +0000 (14:24 +0100)]
client: Remove the buffer tree node on errors.

This fixes restarting of the status client task in case para_server is being restarted.

10 years agosched: Add sched_request_barrier_or_min_delay().
Andre Noll [Sat, 30 Jan 2010 13:22:32 +0000 (14:22 +0100)]
sched: Add sched_request_barrier_or_min_delay().

This is what some tasks need, so add this public function to the scheduler.

10 years agoaudiod: Split open_current_receiver().
Andre Noll [Sat, 30 Jan 2010 13:20:19 +0000 (14:20 +0100)]
audiod: Split open_current_receiver().

We need to check whether the current receiver should be opened from both
pre_select() and post_select(). So rename this function to must_start_decoder(),
make it return bool and do not start the decoder there.

This change makes audiod start the decoder as soon as possible.

10 years agoaudiod: Fix time diff log message.
Andre Noll [Sat, 30 Jan 2010 13:15:10 +0000 (14:15 +0100)]
audiod: Fix time diff log message.

The average time diff was always displayed as "+".

10 years agotime.c: tv_convex_combination() fixes.
Andre Noll [Sat, 30 Jan 2010 13:13:35 +0000 (14:13 +0100)]
time.c: tv_convex_combination() fixes.

Fix some typos, make it return zero if both coefficients are zero.

10 years agowma: Defer decoding until two data blocks are available.
Andre Noll [Sat, 30 Jan 2010 13:07:03 +0000 (14:07 +0100)]
wma: Defer decoding until two data blocks are available.

This avoids buffer underruns caused by frames that span two blocks.

10 years agoAdd buffer tree support for the osx writer.
Andre Noll [Wed, 27 Jan 2010 11:56:06 +0000 (12:56 +0100)]
Add buffer tree support for the osx writer.

Fix makes paraslash build again on Max OS.

10 years agoaudiod: Request a minimal delay if the status buffer tree node is in error state.
Andre Noll [Sat, 23 Jan 2010 13:29:57 +0000 (14:29 +0100)]
audiod: Request a minimal delay if the status buffer tree node is in error state.

This happens during startup and causes a large delay.

10 years ago[btr] Complete source code documentation.
Andre Noll [Tue, 19 Jan 2010 11:38:29 +0000 (12:38 +0100)]
[btr] Complete source code documentation.

10 years ago[btr] Add more documentation.
Andre Noll [Tue, 19 Jan 2010 01:44:40 +0000 (02:44 +0100)]
[btr] Add more documentation.

10 years agoconfigure.ac: Remove some unused AC_SUBST() exports.
Andre Noll [Tue, 19 Jan 2010 00:54:29 +0000 (01:54 +0100)]
configure.ac: Remove some unused AC_SUBST() exports.

10 years agoconfigure: Don't mention the list of supported senders/receivers.
Andre Noll [Tue, 19 Jan 2010 00:38:28 +0000 (01:38 +0100)]
configure: Don't mention the list of supported senders/receivers.

It is the constant list http, dccp, udp.

10 years agoDon't hardcode the list of executables in Makefile.in.
Andre Noll [Tue, 19 Jan 2010 00:21:32 +0000 (01:21 +0100)]
Don't hardcode the list of executables in Makefile.in.

Just export the list via the $all_executables variable from configure.ac. Rename it to
$ececutables while we're at it and remove two unused variables in configure.ac.

10 years agoBuild para_audiod unconditionally.
Andre Noll [Mon, 18 Jan 2010 23:53:52 +0000 (00:53 +0100)]
Build para_audiod unconditionally.

The wma decoder will always be build, so the case of no supported
audio files can not happen.

10 years agoRemove some unused error codes.
Andre Noll [Mon, 18 Jan 2010 23:36:15 +0000 (00:36 +0100)]
Remove some unused error codes.

10 years agoDon't use ENAVAIL.
Andre Noll [Mon, 18 Jan 2010 09:52:32 +0000 (10:52 +0100)]
Don't use ENAVAIL.

NetBSD doesn't know it, and the error text does not fit well. So replace
it by a paraslash error message.

10 years agoFix some format string warnings on x86_64.
Andre Noll [Mon, 18 Jan 2010 09:28:33 +0000 (10:28 +0100)]
Fix some format string warnings on x86_64.

10 years agoMerge branch 'master' into next
Andre Noll [Sun, 17 Jan 2010 20:36:41 +0000 (21:36 +0100)]
Merge branch 'master' into next

Conflicts:
stdin.c

10 years agoMerge branch 'maint'
Andre Noll [Sun, 17 Jan 2010 20:32:19 +0000 (21:32 +0100)]
Merge branch 'maint'

10 years agoMerge remote branch 'athcx/maint' into maint
Andre Noll [Sun, 17 Jan 2010 20:32:11 +0000 (21:32 +0100)]
Merge remote branch 'athcx/maint' into maint

10 years ago[btr]: Avoid a gcc warning.
Andre Noll [Sun, 17 Jan 2010 20:30:27 +0000 (21:30 +0100)]
[btr]: Avoid a gcc warning.

Although the warning
warning: 'brs[1]' may be used uninitialized in this function
is clearly bogus, it can't hurt to assert that i == 2 which also makes the warning to away.

10 years ago[btr] Only print debug message if we're really increasing the wrap buffer.
Andre Noll [Sun, 17 Jan 2010 12:29:05 +0000 (13:29 +0100)]
[btr] Only print debug message if we're really increasing the wrap buffer.

10 years ago[btr] Documentation update.
Andre Noll [Sun, 17 Jan 2010 12:27:46 +0000 (13:27 +0100)]
[btr] Documentation update.

10 years agoaudiod cleanups.
Andre Noll [Fri, 15 Jan 2010 20:41:06 +0000 (21:41 +0100)]
audiod cleanups.

It's pointless to change the scheduler's timeout during post_select().

10 years ago[btr] Add more documentation.
Andre Noll [Fri, 15 Jan 2010 19:22:27 +0000 (20:22 +0100)]
[btr] Add more documentation.

10 years agoclient: Fix a memory leak in client_post_select().
Andre Noll [Fri, 15 Jan 2010 19:21:57 +0000 (20:21 +0100)]
client: Fix a memory leak in client_post_select().

10 years agosched: Reset the descriptor sets on timeouts.
Andre Noll [Fri, 15 Jan 2010 19:21:13 +0000 (20:21 +0100)]
sched: Reset the descriptor sets on timeouts.

This is certainly not neccessary on linux, but it does not hurt
and is clearly more portable.

10 years agoalsa: Replace check for pending frames on eof by a barrier.
Andre Noll [Fri, 15 Jan 2010 19:19:30 +0000 (20:19 +0100)]
alsa: Replace check for pending frames on eof by a barrier.

The old code could lead to a an endless loop where the alsa writer
would never be unscheduled.

10 years ago[btr] Add more documentation.
Andre Noll [Fri, 15 Jan 2010 06:59:54 +0000 (07:59 +0100)]
[btr] Add more documentation.

10 years ago[btr] Add more documentation.
Andre Noll [Fri, 15 Jan 2010 06:41:19 +0000 (07:41 +0100)]
[btr] Add more documentation.

10 years agostdin/stdout cleanup.
Andre Noll [Fri, 15 Jan 2010 06:06:56 +0000 (07:06 +0100)]
stdin/stdout cleanup.

para_client was the last user of the non-btr code in stdin.[ch]
and stdout.[ch]. Remove this unused code and update the documentation.

10 years agobtr support for para_client.
Andre Noll [Fri, 15 Jan 2010 05:46:03 +0000 (06:46 +0100)]
btr support for para_client.

The use of the client code in audiod requires a node to be inserted
as the _parent_ of some existing node, which was not neccessary
before. Therefore a new btrn pointer for the child node is added to
struct btr_node_description and btr_new_node() is adjusted accordingly.

The patch also adds a small helper sched_request_barrier() to
the scheduler.

10 years ago[btr] log message cleanups.
Andre Noll [Thu, 14 Jan 2010 22:59:05 +0000 (23:59 +0100)]
[btr] log message cleanups.

10 years ago[btr] Add a comment to btr_pool_available().
Andre Noll [Thu, 14 Jan 2010 22:58:00 +0000 (23:58 +0100)]
[btr] Add a comment to btr_pool_available().

10 years ago[btr] btr_node_status(): Node pointer should never be NULL.
Andre Noll [Thu, 14 Jan 2010 22:57:22 +0000 (23:57 +0100)]
[btr] btr_node_status(): Node pointer should never be NULL.

10 years agofilter.h: Document that the ->close method need not be provided.
Andre Noll [Thu, 14 Jan 2010 22:49:57 +0000 (23:49 +0100)]
filter.h: Document that the ->close method need not be provided.

10 years agoaudiod: Add a paranoia check to close_writers().
Andre Noll [Thu, 14 Jan 2010 22:48:24 +0000 (23:48 +0100)]
audiod: Add a paranoia check to close_writers().

10 years agoAdd documentation of struct btr_node and btr_pool.
Andre Noll [Thu, 14 Jan 2010 22:24:24 +0000 (23:24 +0100)]
Add documentation of struct btr_node and btr_pool.

10 years ago[btr] Fix btr_get_input_queue_size().
Andre Noll [Thu, 14 Jan 2010 21:58:54 +0000 (22:58 +0100)]
[btr] Fix btr_get_input_queue_size().

We must take care of wrap buffers when computing the input queue size.

10 years agoIntroduce btr_node_description.
Andre Noll [Thu, 14 Jan 2010 03:19:59 +0000 (04:19 +0100)]
Introduce btr_node_description.

As the number of arguments to btr_new_node() grows, the code becomes
hard to read, especially since some callers must set many of the
arguments to NULL.

Using a pointer to a struct has the additional advantage that adding
new parameters does no longer require to touch all callers.

10 years agofile_write: Use para_open() and set the fd to non-blocking mode.
Andre Noll [Thu, 14 Jan 2010 00:19:12 +0000 (01:19 +0100)]
file_write: Use para_open() and set the fd to non-blocking mode.

10 years ago[btr] Fix merge_input_pool().
Andre Noll [Wed, 13 Jan 2010 23:11:46 +0000 (00:11 +0100)]
[btr] Fix merge_input_pool().

This function contained the following bug: wbr, buf1 and sz1 could
be used without being initialized. This was sometimes hitting the
assertion (buf1 && buf2).

Fix this by initializing these variables at the top.

10 years agoSkip btr suffix from {pre,post}_select_btr().
Andre Noll [Wed, 13 Jan 2010 22:39:32 +0000 (23:39 +0100)]
Skip btr suffix from {pre,post}_select_btr().

Everything uses btr now, so there's no need for a suffix any more.
Update the documentation a bit.

10 years agoKill unused writer_node_group code.
Andre Noll [Wed, 13 Jan 2010 22:31:17 +0000 (23:31 +0100)]
Kill unused writer_node_group code.

10 years agofile_write: Use sched_min_delay().
Andre Noll [Wed, 13 Jan 2010 22:17:58 +0000 (23:17 +0100)]
file_write: Use sched_min_delay().

10 years agofile_write: Kill ->fd of struct private_file_write_data.
Andre Noll [Wed, 13 Jan 2010 22:16:01 +0000 (23:16 +0100)]
file_write: Kill ->fd of struct private_file_write_data.

Also, simplify the logic in file_write_post_select_btr() a bit.

10 years agofile_write: Kill non-btr code.
Andre Noll [Wed, 13 Jan 2010 22:08:30 +0000 (23:08 +0100)]
file_write: Kill non-btr code.

Also, don't free the config on close.

10 years agooss fixes and cleanups.
Andre Noll [Wed, 13 Jan 2010 21:40:23 +0000 (22:40 +0100)]
oss fixes and cleanups.

- Kill ->samplerate and ->channels of struct private_oss_write_data.
- Don't invalidate the config on close().
- Simplify oss_pre_select_btr().

10 years agooss: Kill non-btr code.
Andre Noll [Wed, 13 Jan 2010 21:01:55 +0000 (22:01 +0100)]
oss: Kill non-btr code.

10 years agoalsa: Kill non-btr code.
Andre Noll [Wed, 13 Jan 2010 20:57:50 +0000 (21:57 +0100)]
alsa: Kill non-btr code.

10 years agowrite: Kill non-btr mode.
Andre Noll [Wed, 13 Jan 2010 20:46:28 +0000 (21:46 +0100)]
write: Kill non-btr mode.

Yes, this breaks the build on Macs. Sue me.

10 years agoMake writers remove btr node on errors.
Andre Noll [Wed, 13 Jan 2010 20:44:35 +0000 (21:44 +0100)]
Make writers remove btr node on errors.

This is what receivers/filters do as well.

10 years agofilter_common: Use sched_min_delay() helper.
Andre Noll [Wed, 13 Jan 2010 06:40:45 +0000 (07:40 +0100)]
filter_common: Use sched_min_delay() helper.

10 years agoclient.c: Add comment on the supervisor task.
Andre Noll [Wed, 13 Jan 2010 06:39:57 +0000 (07:39 +0100)]
client.c: Add comment on the supervisor task.

10 years agomp3dec: Don't check for bad data delay on errors.
Andre Noll [Wed, 13 Jan 2010 06:37:00 +0000 (07:37 +0100)]
mp3dec: Don't check for bad data delay on errors.

10 years agomp3dec does not need --buffer_tree.
Andre Noll [Wed, 13 Jan 2010 06:35:38 +0000 (07:35 +0100)]
mp3dec does not need --buffer_tree.