mirror of
				https://gitlab.freedesktop.org/pipewire/pipewire.git
				synced 2025-11-03 09:01:54 -05:00 
			
		
		
		
	alsa-seq: handle large control events
Handle the case where the control event is large and needs to be encoded into multiple midi events.
This commit is contained in:
		
							parent
							
								
									7dcfe81690
								
							
						
					
					
						commit
						cb8330c798
					
				
					 1 changed files with 41 additions and 34 deletions
				
			
		| 
						 | 
					@ -644,29 +644,35 @@ static int process_write(struct seq_state *state)
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
		SPA_POD_SEQUENCE_FOREACH(pod, c) {
 | 
							SPA_POD_SEQUENCE_FOREACH(pod, c) {
 | 
				
			||||||
			long s;
 | 
								long s, body_size;
 | 
				
			||||||
 | 
								uint8_t *body;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
			if (c->type != SPA_CONTROL_Midi)
 | 
								if (c->type != SPA_CONTROL_Midi)
 | 
				
			||||||
				continue;
 | 
									continue;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
								body = SPA_POD_BODY(&c->value);
 | 
				
			||||||
 | 
								body_size = SPA_POD_BODY_SIZE(&c->value);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
								while (body_size > 0) {
 | 
				
			||||||
				if (size == 0)
 | 
									if (size == 0)
 | 
				
			||||||
					/* only reset when we start decoding a new message */
 | 
										/* only reset when we start decoding a new message */
 | 
				
			||||||
					snd_seq_ev_clear(&ev);
 | 
										snd_seq_ev_clear(&ev);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
				if ((s = snd_midi_event_encode(stream->codec,
 | 
									if ((s = snd_midi_event_encode(stream->codec,
 | 
				
			||||||
						SPA_POD_BODY(&c->value),
 | 
												body, body_size, &ev)) < 0) {
 | 
				
			||||||
						SPA_POD_BODY_SIZE(&c->value), &ev)) < 0) {
 | 
					 | 
				
			||||||
					spa_log_warn(state->log, "failed to encode event: %s",
 | 
										spa_log_warn(state->log, "failed to encode event: %s",
 | 
				
			||||||
							snd_strerror(s));
 | 
												snd_strerror(s));
 | 
				
			||||||
					snd_midi_event_reset_encode(stream->codec);
 | 
										snd_midi_event_reset_encode(stream->codec);
 | 
				
			||||||
					size = 0;
 | 
										size = 0;
 | 
				
			||||||
	                        continue;
 | 
										break;
 | 
				
			||||||
				}
 | 
									}
 | 
				
			||||||
 | 
									body += s;
 | 
				
			||||||
 | 
									body_size -= s;
 | 
				
			||||||
				size += s;
 | 
									size += s;
 | 
				
			||||||
				if (ev.type == SND_SEQ_EVENT_NONE)
 | 
									if (ev.type == SND_SEQ_EVENT_NONE)
 | 
				
			||||||
					/* this can happen when the event is not complete yet, like
 | 
										/* this can happen when the event is not complete yet, like
 | 
				
			||||||
				 * a sysex message and we need to get some more data. */
 | 
										 * a sysex message and we need to encode some more data. */
 | 
				
			||||||
				continue;
 | 
										break;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
				snd_seq_ev_set_source(&ev, state->event.addr.port);
 | 
									snd_seq_ev_set_source(&ev, state->event.addr.port);
 | 
				
			||||||
				snd_seq_ev_set_dest(&ev, port->addr.client, port->addr.port);
 | 
									snd_seq_ev_set_dest(&ev, port->addr.client, port->addr.port);
 | 
				
			||||||
| 
						 | 
					@ -677,7 +683,7 @@ static int process_write(struct seq_state *state)
 | 
				
			||||||
				out_rt.tv_sec = out_time / SPA_NSEC_PER_SEC;
 | 
									out_rt.tv_sec = out_time / SPA_NSEC_PER_SEC;
 | 
				
			||||||
				snd_seq_ev_schedule_real(&ev, state->event.queue_id, 0, &out_rt);
 | 
									snd_seq_ev_schedule_real(&ev, state->event.queue_id, 0, &out_rt);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
			spa_log_trace_fp(state->log, "event %d time:%"PRIu64" offset:%d size:%ld port:%d.%d",
 | 
									spa_log_info(state->log, "event %d time:%"PRIu64" offset:%d size:%ld port:%d.%d",
 | 
				
			||||||
					ev.type, out_time, c->offset, size, port->addr.client, port->addr.port);
 | 
										ev.type, out_time, c->offset, size, port->addr.client, port->addr.port);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
				if ((err = snd_seq_event_output(state->event.hndl, &ev)) < 0) {
 | 
									if ((err = snd_seq_event_output(state->event.hndl, &ev)) < 0) {
 | 
				
			||||||
| 
						 | 
					@ -687,6 +693,7 @@ static int process_write(struct seq_state *state)
 | 
				
			||||||
				size = 0;
 | 
									size = 0;
 | 
				
			||||||
			}
 | 
								}
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
	snd_seq_drain_output(state->event.hndl);
 | 
						snd_seq_drain_output(state->event.hndl);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	return res;
 | 
						return res;
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue