ZoneMinder
/
zoneminder
mirror of https://github.com/ZoneMinder/zoneminder.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Redo the body of event::Run() so that we don't hold the lock while actually processing the packets. Only around queue manipulation. Should free up analysis thread.

pull/4202/head
Isaac Connor 2022-04-27 10:59:52 -04:00
parent c6f4af0dca
commit 305042ed67
1 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/zm_event.cpp
View File

@ -729,22 +729,24 @@ void Event::Run() {
if (storage != monitor->getStorage())
delete storage;
std::unique_lock<std::mutex> lck(packet_queue_mutex);
// The idea is to process the queue no matter what so that all packets get processed.
// We only break if the queue is empty
while (true) {
if (!packet_queue.empty()) {
ZMLockedPacket * packet_lock = nullptr;
{
std::unique_lock<std::mutex> lck(packet_queue_mutex);
if (packet_queue.empty()) {
packet_queue_condition.wait(lck);
if (terminate_ or zm_terminate) break;
continue;
}
// Packets on this queue are locked. They are locked by analysis thread
const ZMLockedPacket * packet_lock = packet_queue.front();
this->AddPacket_(packet_lock->packet_);
delete packet_lock;
packet_lock = packet_queue.front();
packet_queue.pop();
} else {
if (terminate_ or zm_terminate) {
break;
}
packet_queue_condition.wait(lck);
}
}
} // end lock scope
this->AddPacket_(packet_lock->packet_);
delete packet_lock;
} // end while
} // end Run()