Use new kernel ioctl() to get accurate reading of queue length and
latency on kernel + controller side.

Use the latency information to workaround controller issues where
streams are desynchronized due to tx problems or spontaneously when some
packets that should have been sent are left sitting in the queue, and
transmission is off by a multiple of the ISO interval.  This state is
visible in the latency information, so if we see streams in a group have
persistently different latencies, drop packets to resynchronize them.

Also make corrections if the kernel/controller queues get too long, so
that we don't have too big latency there.

Link: https://github.com/bluez/bluez/issues/515

Setting it to NULL will remove it from the properties and will not
send the updated property to the server, keeping things freewheeling
forever.

Don't just overwite the node.group property when changing the freewheel
thate but append the frewheel group and remove it again to preserve
the original node.group.

See #3873

Keep track of the valid ports and don't emit port info for
invalid ports. When a listener is added while the ports are being
created, it is possible that the ports are still NULL or invalid.

The info is NULL when the port is removed, don't crash on that.

The conf argument was unused before, but it is correctly populated
using the context's conf in pw_context_new(), so the code behaves
the same effectively.

This allows parsing standard conf sections from other configuration
sources, as long as the sections are placed in a dictionary.

Barnabás Pőcze authored 1 year ago and Wim Taymans committed 1 year ago

Monitoring does not interact with the normal modes of pw-link,
so separate it from them.

Barnabás Pőcze authored 1 year ago and Wim Taymans committed 1 year ago

This is safer than using a bare array of `uint32_t` with hard-coded indices.

Barnabás Pőcze authored 1 year ago and Wim Taymans committed 1 year ago

Using an enum provides better debugging experience and
indicates the purpose better than a bare `uint32_t`.

dsee authored 1 year ago and Wim Taymans committed 1 year ago

Use `getaddrinfo` in `parse_address` instead of `inet_pton`.
Display Ipv6 addresses with scope identifiers correctly in `get_ip`
functions using `if_indextoname`.

Sabri Ünal authored 1 year ago and Wim Taymans committed 1 year ago

Add missing strings

There wasn't a MIDI example, so add one.

We can deduce this from the way the node is configured with the
PortParam now.

Add some option to set manually group IDs of unicast streams
of devices.

Intel controllers don't appear to support more than one group at a time,
but in principle this can be done.

Keep track of the active number of mixer ports and update the global mix
io in sync with the data thread because that is where we will check the
state of the global mix io.

This is mostly important for output ports. When removing all links from
an output port, we first will clear all the mixer io and then remove the
global mixer with client_node_port_set_mix_info(). If we don't clear the
io before that, the data thread will be using that buffers as they are
cleared.

See !1915

When we clear the port io, pause the core until the invoke call
completed. This way we don't start processing other messages until we
have safely removed the port io.

Normally, when clearing a link on a port, first the mix io will be set
to NULL and then the format will be cleared, which clears the buffers
as well. By delaying the processing of the format clear until the io
is removed from the data thread we avoid taking away the buffer memory
from the processing thread prematurely.

When creating a link, first the format and buffers are configured and
then the io is set, which should be safe in all cases.

See !1915

You can now do this to permanently enable htimestamp:

pw-cli s 38 Props '{ params: [ api.alsa.htimestamp.max-errors: 0, api.alsa.htimestamp: true ]}'

Disable graphs in the layout file. Doxygen seems to ignore Doxyfile
settings if they're enabled in layout.

Add some sections to index + fix typos and section levels.

Clarify discussion of device properties.

Use 'unknown' when we can't find the peer_name. Otherwise we will just
get a protocol error when the node.name is missing.

Jonas Holmberg authored 1 year ago and Wim Taymans committed 1 year ago

Propagate the error if spa_system_eventfd_create() fails. Also copy
errno before calling spa_log_debug() in spa_system_eventfd_create() to
make sure it is not overwritten.

When we emit a new EnumFormat, the adapter will renegotiate a new format
that will be applied when we restart.

Fixes #3858

When the follower has new EnumFormat, make sure we recheck the formats
the next time the node is started.