mirrors/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire.git synced 2025-10-29 05:40:27 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

jack: use the CPU max_alignment

Browse source 
Instead of aligning the buffers to 16 bytes, use the CPU max_align
value (32 on intel).

Move the mix function from a static global variable to a per client
member because this could change per client.
This commit is contained in:
Wim Taymans 2024-09-25 15:16:25 +02:00
parent fbbe983a05
commit c7c5b61dac
1 changed files with 14 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

24
pipewire-jack/src/pipewire-jack.c
View file

@ -55,7 +55,7 @@
#define MAX_MIX 1024
#define MAX_CLIENT_PORTS 768
#define MAX_ALIGN 16
#define MAX_ALIGN 32
#define MAX_BUFFERS 2
#define MAX_BUFFER_DATAS 1u
@ -132,8 +132,6 @@ static thread_local float midi_scratch[MIDI_SCRATCH_FRAMES];
typedef void (*mix_func) (float *dst, float *src[], uint32_t n_src, bool aligned, uint32_t n_samples);
static mix_func mix_function;
struct object {
struct spa_list link;
@ -472,6 +470,8 @@ struct client {
unsigned int async:1;
uint32_t max_frames;
uint32_t max_align;
mix_func mix_function;
jack_position_t jack_position;
jack_transport_state_t jack_state;
@ -737,7 +737,7 @@ static struct port * alloc_port(struct client *c, enum spa_direction direction)
}
if (spa_list_is_empty(&c->free_ports)) {
port_size = sizeof(struct port) + (c->max_frames * sizeof(float)) + MAX_ALIGN;
port_size = sizeof(struct port) + (c->max_frames * sizeof(float)) + c->max_align;
p = calloc(OBJECT_CHUNK, port_size);
if (p == NULL)
@ -768,7 +768,7 @@ static struct port * alloc_port(struct client *c, enum spa_direction direction)
p->props = pw_properties_new(NULL, NULL);
p->direction = direction;
p->emptyptr = SPA_PTR_ALIGN(p->empty, MAX_ALIGN, float);
p->emptyptr = SPA_PTR_ALIGN(p->empty, c->max_align, float);
p->port_id = pw_map_insert_new(&c->ports[direction], p);
c->n_ports++;
@ -4244,14 +4244,17 @@ jack_client_t * jack_client_open (const char *client_name,
support = pw_context_get_support(client->context.context, &n_support);
mix_function = mix_c;
client->mix_function = mix_c;
cpu_iface = spa_support_find(support, n_support, SPA_TYPE_INTERFACE_CPU);
if (cpu_iface) {
#if defined (__SSE__)
uint32_t flags = spa_cpu_get_flags(cpu_iface);
if (flags & SPA_CPU_FLAG_SSE)
mix_function = mix_sse;
client->mix_function = mix_sse;
#endif
client->max_align = spa_cpu_get_max_align(cpu_iface);
} else {
client->max_align = MAX_ALIGN;
}
client->context.old_thread_utils =
pw_context_get_object(client->context.context,
@ -5623,15 +5626,16 @@ static void *get_buffer_input_float(struct port *p, jack_nframes_t frames)
float *mix_ptr[MAX_MIX], *np;
uint32_t n_ptr = 0;
bool ptr_aligned = true;
struct client *c = p->client;
spa_list_for_each(mix, &p->mix, port_link) {
if (mix->id == SPA_ID_INVALID)
continue;
pw_log_trace_fp("%p: port %s mix %d.%d get buffer %d",
p->client, p->object->port.name, p->port_id, mix->id, frames);
c, p->object->port.name, p->port_id, mix->id, frames);
if ((b = get_mix_buffer(p->client, mix, frames)) == NULL)
if ((b = get_mix_buffer(c, mix, frames)) == NULL)
continue;
if ((np = get_buffer_data(b, frames)) == NULL)
@ -5648,7 +5652,7 @@ static void *get_buffer_input_float(struct port *p, jack_nframes_t frames)
ptr = mix_ptr[0];
} else if (n_ptr > 1) {
ptr = p->emptyptr;
mix_function(ptr, mix_ptr, n_ptr, ptr_aligned, frames);
c->mix_function(ptr, mix_ptr, n_ptr, ptr_aligned, frames);
p->zeroed = false;
}
if (ptr == NULL)