X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fdata%2Fdecoderstack.cpp;h=1f158c230b7f55c53630c0eb2e78f5f09ded6ed4;hp=5e30faa73c810e7d64e9bfb936e7cb6e8656daa8;hb=326cf6feb8598aa03a35fd6f678e4f536f168149;hpb=2acdb232d6bb452cfdfaea3ef5218fb4da592329 diff --git a/pv/data/decoderstack.cpp b/pv/data/decoderstack.cpp index 5e30faa7..1f158c23 100644 --- a/pv/data/decoderstack.cpp +++ b/pv/data/decoderstack.cpp @@ -14,8 +14,7 @@ * GNU General Public License for more details. * * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + * along with this program; if not, see . */ #include @@ -27,10 +26,10 @@ #include "decoderstack.hpp" #include -#include +#include #include #include -#include +#include #include using std::lock_guard; @@ -45,6 +44,7 @@ using std::list; using std::map; using std::pair; using std::shared_ptr; +using std::make_shared; using std::vector; using namespace pv::data::decode; @@ -54,14 +54,16 @@ namespace data { const double DecoderStack::DecodeMargin = 1.0; const double DecoderStack::DecodeThreshold = 0.2; -const int64_t DecoderStack::DecodeChunkLength = 4096; -const unsigned int DecoderStack::DecodeNotifyPeriod = 65536; +const int64_t DecoderStack::DecodeChunkLength = 10 * 1024 * 1024; +const unsigned int DecoderStack::DecodeNotifyPeriod = 1024; -mutex DecoderStack::global_decode_mutex_; +mutex DecoderStack::global_srd_mutex_; -DecoderStack::DecoderStack(pv::SigSession &session, +DecoderStack::DecoderStack(pv::Session &session, const srd_decoder *const dec) : session_(session), + start_time_(0), + samplerate_(0), sample_count_(0), frame_complete_(false), samples_decoded_(0) @@ -73,8 +75,7 @@ DecoderStack::DecoderStack(pv::SigSession &session, connect(&session_, SIGNAL(frame_ended()), this, SLOT(on_frame_ended())); - stack_.push_back(shared_ptr( - new decode::Decoder(dec))); + stack_.push_back(make_shared(dec)); } DecoderStack::~DecoderStack() @@ -105,13 +106,23 @@ void DecoderStack::remove(int index) // Find the decoder in the stack auto iter = stack_.begin(); - for(int i = 0; i < index; i++, iter++) + for (int i = 0; i < index; i++, iter++) assert(iter != stack_.end()); // Delete the element stack_.erase(iter); } +double DecoderStack::samplerate() const +{ + return samplerate_; +} + +const pv::util::Timestamp& DecoderStack::start_time() const +{ + return start_time_; +} + int64_t DecoderStack::samples_decoded() const { lock_guard decode_lock(output_mutex_); @@ -124,8 +135,7 @@ std::vector DecoderStack::get_visible_rows() const vector rows; - for (const shared_ptr &dec : stack_) - { + for (const shared_ptr &dec : stack_) { assert(dec); if (!dec->shown()) continue; @@ -135,15 +145,14 @@ std::vector DecoderStack::get_visible_rows() const // Add a row for the decoder if it doesn't have a row list if (!decc->annotation_rows) - rows.push_back(Row(decc)); + rows.emplace_back(decc); // Add the decoder rows - for (const GSList *l = decc->annotation_rows; l; l = l->next) - { + for (const GSList *l = decc->annotation_rows; l; l = l->next) { const srd_decoder_annotation_row *const ann_row = (srd_decoder_annotation_row *)l->data; assert(ann_row); - rows.push_back(Row(decc, ann_row)); + rows.emplace_back(decc, ann_row); } } @@ -181,9 +190,6 @@ void DecoderStack::clear() void DecoderStack::begin_decode() { - shared_ptr logic_signal; - shared_ptr data; - if (decode_thread_.joinable()) { interrupt_ = true; input_cond_.notify_one(); @@ -201,8 +207,7 @@ void DecoderStack::begin_decode() } // Add classes - for (const shared_ptr &dec : stack_) - { + for (const shared_ptr &dec : stack_) { assert(dec); const srd_decoder *const decc = dec->decoder(); assert(dec->decoder()); @@ -212,8 +217,7 @@ void DecoderStack::begin_decode() rows_[Row(decc)] = decode::RowData(); // Add the decoder rows - for (const GSList *l = decc->annotation_rows; l; l = l->next) - { + for (const GSList *l = decc->annotation_rows; l; l = l->next) { const srd_decoder_annotation_row *const ann_row = (srd_decoder_annotation_row *)l->data; assert(ann_row); @@ -233,26 +237,29 @@ void DecoderStack::begin_decode() // We get the logic data of the first channel in the list. // This works because we are currently assuming all - // LogicSignals have the same data/snapshot + // logic signals have the same data/segment + pv::data::SignalBase *signalbase; + pv::data::Logic *data = nullptr; + for (const shared_ptr &dec : stack_) if (dec && !dec->channels().empty() && - ((logic_signal = (*dec->channels().begin()).second)) && - ((data = logic_signal->logic_data()))) + ((signalbase = (*dec->channels().begin()).second.get())) && + ((data = signalbase->logic_data().get()))) break; if (!data) return; - // Check we have a snapshot of data - const deque< shared_ptr > &snapshots = - data->get_snapshots(); - if (snapshots.empty()) + // Check we have a segment of data + const deque< shared_ptr > &segments = + data->logic_segments(); + if (segments.empty()) return; - snapshot_ = snapshots.front(); + segment_ = segments.front(); // Get the samplerate and start time - start_time_ = data->get_start_time(); - samplerate_ = data->samplerate(); + start_time_ = segment_->start_time(); + samplerate_ = segment_->samplerate(); if (samplerate_ == 0.0) samplerate_ = 1.0; @@ -260,13 +267,13 @@ void DecoderStack::begin_decode() decode_thread_ = std::thread(&DecoderStack::decode_proc, this); } -uint64_t DecoderStack::get_max_sample_count() const +uint64_t DecoderStack::max_sample_count() const { uint64_t max_sample_count = 0; - for (auto i = rows_.cbegin(); i != rows_.end(); i++) + for (const auto& row : rows_) max_sample_count = max(max_sample_count, - (*i).second.get_max_sample()); + row.second.get_max_sample()); return max_sample_count; } @@ -274,34 +281,41 @@ uint64_t DecoderStack::get_max_sample_count() const optional DecoderStack::wait_for_data() const { unique_lock input_lock(input_mutex_); - while(!interrupt_ && !frame_complete_ && - samples_decoded_ >= sample_count_) + + // Do wait if we decoded all samples but we're still capturing + // Do not wait if we're done capturing + while (!interrupt_ && !frame_complete_ && + (samples_decoded_ >= sample_count_) && + (session_.get_capture_state() != Session::Stopped)) { + input_cond_.wait(input_lock); + } + + // Return value is valid if we're not aborting the decode, return boost::make_optional(!interrupt_ && - (samples_decoded_ < sample_count_ || !frame_complete_), + // and there's more work to do... + (samples_decoded_ < sample_count_ || !frame_complete_) && + // and if the end of the data hasn't been reached yet + (!((samples_decoded_ >= sample_count_) && (session_.get_capture_state() == Session::Stopped))), sample_count_); } void DecoderStack::decode_data( - const int64_t sample_count, const unsigned int unit_size, + const int64_t abs_start_samplenum, const int64_t sample_count, const unsigned int unit_size, srd_session *const session) { - uint8_t chunk[DecodeChunkLength]; - const unsigned int chunk_sample_count = - DecodeChunkLength / snapshot_->unit_size(); + DecodeChunkLength / segment_->unit_size(); - for (int64_t i = 0; !interrupt_ && i < sample_count; - i += chunk_sample_count) - { - lock_guard decode_lock(global_decode_mutex_); + for (int64_t i = abs_start_samplenum; !interrupt_ && i < sample_count; + i += chunk_sample_count) { const int64_t chunk_end = min( i + chunk_sample_count, sample_count); - snapshot_->get_samples(chunk, i, chunk_end); + const uint8_t* chunk = segment_->get_samples(i, chunk_end); - if (srd_session_send(session, i, i + sample_count, chunk, - (chunk_end - i) * unit_size) != SRD_OK) { + if (srd_session_send(session, i, chunk_end, chunk, + (chunk_end - i) * unit_size, unit_size) != SRD_OK) { error_message_ = tr("Decoder reported an error"); break; } @@ -322,23 +336,24 @@ void DecoderStack::decode_proc() { optional sample_count; srd_session *session; - srd_decoder_inst *prev_di = NULL; + srd_decoder_inst *prev_di = nullptr; - assert(snapshot_); + assert(segment_); + + // Prevent any other decode threads from accessing libsigrokdecode + lock_guard srd_lock(global_srd_mutex_); // Create the session srd_session_new(&session); assert(session); // Create the decoders - const unsigned int unit_size = snapshot_->unit_size(); + const unsigned int unit_size = segment_->unit_size(); - for (const shared_ptr &dec : stack_) - { - srd_decoder_inst *const di = dec->create_decoder_inst(session, unit_size); + for (const shared_ptr &dec : stack_) { + srd_decoder_inst *const di = dec->create_decoder_inst(session); - if (!di) - { + if (!di) { error_message_ = tr("Failed to create decoder instance"); srd_session_destroy(session); return; @@ -353,7 +368,7 @@ void DecoderStack::decode_proc() // Get the intial sample count { unique_lock input_lock(input_mutex_); - sample_count = sample_count_ = snapshot_->get_sample_count(); + sample_count = sample_count_ = segment_->get_sample_count(); } // Start the session @@ -365,9 +380,11 @@ void DecoderStack::decode_proc() srd_session_start(session); + int64_t abs_start_samplenum = 0; do { - decode_data(*sample_count, unit_size, session); - } while(error_message_.isEmpty() && (sample_count = wait_for_data())); + decode_data(abs_start_samplenum, *sample_count, unit_size, session); + abs_start_samplenum = *sample_count; + } while (error_message_.isEmpty() && (sample_count = wait_for_data())); // Destroy the session srd_session_destroy(session); @@ -392,13 +409,12 @@ void DecoderStack::annotation_callback(srd_proto_data *pdata, void *decoder) assert(decc); auto row_iter = d->rows_.end(); - + // Try looking up the sub-row of this class const auto r = d->class_rows_.find(make_pair(decc, a.format())); if (r != d->class_rows_.end()) row_iter = d->rows_.find((*r).second); - else - { + else { // Failing that, use the decoder as a key row_iter = d->rows_.find(Row(decc)); } @@ -424,8 +440,8 @@ void DecoderStack::on_data_received() { { unique_lock lock(input_mutex_); - if (snapshot_) - sample_count_ = snapshot_->get_sample_count(); + if (segment_) + sample_count_ = segment_->get_sample_count(); } input_cond_.notify_one(); } @@ -434,7 +450,7 @@ void DecoderStack::on_frame_ended() { { unique_lock lock(input_mutex_); - if (snapshot_) + if (segment_) frame_complete_ = true; } input_cond_.notify_one();