smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2026-01-08 14:45:43 +01:00
Code Issues Projects Releases Wiki Activity

Fix false-positive under-run messages

Browse source 
Port (or Tracks) can be safely added during playback, however
the disk-reader's playback buffer is initially empty. This lead to
false-positive Underrun() signals when processing takes place
before or concurrently with re-filling the disk-buffer for the new
channels.

Now new empty buffers are ignored, and produce silence until the
initial refill is complete. There is however no per-channel
de-click in, yet.

This fixes: play some audio track, ctrl+drag a region to the
drop-zone, creating a new track while playing.
This commit is contained in:
Robin Gareus 2020-04-08 01:08:57 +02:00
parent e893c31701
commit 648beb94a0
No known key found for this signature in database
GPG key ID: A090BCE02CF57F04
2 changed files with 10 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
libs/ardour/ardour/disk_reader.h
View file

@ -123,6 +123,7 @@ protected:
: DiskIOProcessor::ChannelInfo (buffer_size)
, pre_loop_buffer (0)
, pre_loop_buffer_size (0)
, initialized (false)
{
resize (buffer_size);
resize_preloop (preloop_size);
@ -134,6 +135,7 @@ protected:
Sample* pre_loop_buffer;
samplecnt_t pre_loop_buffer_size;
bool initialized;
};
XMLNode& state ();

10
libs/ardour/disk_reader.cc
View file

@ -83,6 +83,7 @@ DiskReader::ReaderChannelInfo::resize (samplecnt_t bufsize)
rbuf = new PlaybackBuffer<Sample> (bufsize);
/* touch memory to lock it */
memset (rbuf->buffer (), 0, sizeof (Sample) * rbuf->bufsize ());
initialized = false;
}
void
@ -366,7 +367,7 @@ DiskReader::run (BufferSet& bufs, samplepos_t start_sample, samplepos_t end_samp
const sampleoffset_t declick_offs = _declick_offs;
for (n = 0, chan = c->begin (); chan != c->end (); ++chan, ++n) {
ChannelInfo* chaninfo (*chan);
ReaderChannelInfo* chaninfo = dynamic_cast<ReaderChannelInfo*> (*chan);
AudioBuffer& output (bufs.get_audio (n % n_buffers));
AudioBuffer& disk_buf ((ms & MonitoringInput) ? scratch_bufs.get_audio (n) : output);
@ -391,7 +392,9 @@ DiskReader::run (BufferSet& bufs, samplepos_t start_sample, samplepos_t end_samp
if (!declick_out) {
const samplecnt_t available = chaninfo->rbuf->read (disk_buf.data (), disk_samples_to_consume);
if (disk_samples_to_consume > available) {
if (available == 0 && !chaninfo->initialized) {
disk_buf.silence (disk_samples_to_consume);
} else if (disk_samples_to_consume > available) {
cerr << "underrun for " << _name << " Available samples: " << available << " required: " << disk_samples_to_consume << endl;
DEBUG_TRACE (DEBUG::Butler, string_compose ("%1 underrun in %2, total space = %3 vs %4\n", DEBUG_THREAD_SELF, name (), available, disk_samples_to_consume));
Underrun ();
@ -673,6 +676,8 @@ DiskReader::overwrite_existing_audio ()
ret = false;
}
}
rci->initialized = true;
}
return ret;
@ -1231,6 +1236,7 @@ DiskReader::refill_audio (Sample* sum_buffer, Sample* mixdown_buffer, float* gai
ret = -1;
}
}
rci->initialized = true;
}
if (zero_fill) {