asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #76533 from lawnjelly/message_queue_perf

Browse source 
MessageQueue - Fix max usage performance statistic
This commit is contained in:
Rémi Verschelde 2023-04-28 18:37:37 +02:00
commit adee8cfee6
No known key found for this signature in database
GPG key ID: C3336907360768E1
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
core/message_queue.cpp
View file

@ -251,8 +251,7 @@ void MessageQueue::statistics() {
}
int MessageQueue::get_max_buffer_usage() const {
// Note this may be better read_buffer, or a combination, depending when this is read.
return buffers[write_buffer].data.size();
return _buffer_size_monitor.max_size_overall;
}
void MessageQueue::_call_function(Object *p_target, const StringName &p_func, const Variant *p_args, int p_argcount, bool p_show_error) {
@ -392,6 +391,7 @@ void MessageQueue::flush() {
_THREAD_SAFE_LOCK_
// keep track of the maximum used size, so we can downsize buffers when appropriate
_buffer_size_monitor.max_size = MAX(buffer_data_size, _buffer_size_monitor.max_size);
_buffer_size_monitor.max_size_overall = MAX(buffer_data_size, _buffer_size_monitor.max_size_overall);
// flip buffers, this is the only part that requires a lock
SWAP(read_buffer, write_buffer);

4
core/message_queue.h
View file

@ -70,6 +70,9 @@ class MessageQueue {
struct BufferSizeMonitor {
uint32_t max_size = 0;
uint32_t flush_count = 0;
// Only used for performance statistics.
uint32_t max_size_overall = 0;
} _buffer_size_monitor;
void _call_function(Object *p_target, const StringName &p_func, const Variant *p_args, int p_argcount, bool p_show_error);
@ -97,6 +100,7 @@ public:
bool is_flushing() const;
int get_max_buffer_usage() const;
int get_current_buffer_usage() const;
MessageQueue();
~MessageQueue();