mirror of
				https://gitlab.freedesktop.org/pipewire/pipewire.git
				synced 2025-11-03 09:01:54 -05:00 
			
		
		
		
	audioconvert: Improve buffer params
Scale the default size of the buffer with the sample rate conversion factor.
This commit is contained in:
		
							parent
							
								
									6144d037ce
								
							
						
					
					
						commit
						6c5ec409bf
					
				
					 1 changed files with 31 additions and 1 deletions
				
			
		| 
						 | 
					@ -1725,21 +1725,51 @@ impl_node_port_enum_params(void *object, int seq,
 | 
				
			||||||
			param = spa_format_audio_raw_build(&b, id, &port->format.info.raw);
 | 
								param = spa_format_audio_raw_build(&b, id, &port->format.info.raw);
 | 
				
			||||||
		break;
 | 
							break;
 | 
				
			||||||
	case SPA_PARAM_Buffers:
 | 
						case SPA_PARAM_Buffers:
 | 
				
			||||||
 | 
						{
 | 
				
			||||||
 | 
							uint32_t size;
 | 
				
			||||||
 | 
							struct dir *dir;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
		if (!port->have_format)
 | 
							if (!port->have_format)
 | 
				
			||||||
			return -EIO;
 | 
								return -EIO;
 | 
				
			||||||
		if (result.index > 0)
 | 
							if (result.index > 0)
 | 
				
			||||||
			return 0;
 | 
								return 0;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
							dir = &this->dir[direction];
 | 
				
			||||||
 | 
							if (dir->mode == SPA_PARAM_PORT_CONFIG_MODE_dsp) {
 | 
				
			||||||
 | 
								/* DSP ports always use the quantum_limit as the buffer
 | 
				
			||||||
 | 
								 * size. */
 | 
				
			||||||
 | 
								size = this->quantum_limit;
 | 
				
			||||||
 | 
							} else {
 | 
				
			||||||
 | 
								uint32_t irate, orate;
 | 
				
			||||||
 | 
								/* Convert ports are scaled so that they can always
 | 
				
			||||||
 | 
								 * provide one quantum of data */
 | 
				
			||||||
 | 
								irate = dir->format.info.raw.rate;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
								/* collect the other port rate */
 | 
				
			||||||
 | 
								dir = &this->dir[SPA_DIRECTION_REVERSE(direction)];
 | 
				
			||||||
 | 
								if (dir->mode == SPA_PARAM_PORT_CONFIG_MODE_dsp)
 | 
				
			||||||
 | 
									orate = this->io_position ?  this->io_position->clock.rate.denom : DEFAULT_RATE;
 | 
				
			||||||
 | 
								else
 | 
				
			||||||
 | 
									orate = dir->format.info.raw.rate;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
								/* always keep some extra room for adaptive resampling */
 | 
				
			||||||
 | 
								size = this->quantum_limit * 2;
 | 
				
			||||||
 | 
								/*  scale the buffer size when we can. */
 | 
				
			||||||
 | 
								if (irate != 0 && orate != 0)
 | 
				
			||||||
 | 
									size = size * (irate + orate - 1) / orate;
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
		param = spa_pod_builder_add_object(&b,
 | 
							param = spa_pod_builder_add_object(&b,
 | 
				
			||||||
			SPA_TYPE_OBJECT_ParamBuffers, id,
 | 
								SPA_TYPE_OBJECT_ParamBuffers, id,
 | 
				
			||||||
			SPA_PARAM_BUFFERS_buffers, SPA_POD_CHOICE_RANGE_Int(2, 1, MAX_BUFFERS),
 | 
								SPA_PARAM_BUFFERS_buffers, SPA_POD_CHOICE_RANGE_Int(2, 1, MAX_BUFFERS),
 | 
				
			||||||
			SPA_PARAM_BUFFERS_blocks,  SPA_POD_Int(port->blocks),
 | 
								SPA_PARAM_BUFFERS_blocks,  SPA_POD_Int(port->blocks),
 | 
				
			||||||
			SPA_PARAM_BUFFERS_size,    SPA_POD_CHOICE_RANGE_Int(
 | 
								SPA_PARAM_BUFFERS_size,    SPA_POD_CHOICE_RANGE_Int(
 | 
				
			||||||
								this->quantum_limit * port->stride,
 | 
													size * port->stride,
 | 
				
			||||||
								16 * port->stride,
 | 
													16 * port->stride,
 | 
				
			||||||
								INT32_MAX),
 | 
													INT32_MAX),
 | 
				
			||||||
			SPA_PARAM_BUFFERS_stride,  SPA_POD_Int(port->stride));
 | 
								SPA_PARAM_BUFFERS_stride,  SPA_POD_Int(port->stride));
 | 
				
			||||||
		break;
 | 
							break;
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
	case SPA_PARAM_Meta:
 | 
						case SPA_PARAM_Meta:
 | 
				
			||||||
		switch (result.index) {
 | 
							switch (result.index) {
 | 
				
			||||||
		case 0:
 | 
							case 0:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue