Skip to content

Commit

Permalink
santad: Bump QoS of notify handling queue (#1342)
Browse files Browse the repository at this point in the history
The use of the background queue is a historical artifact from when Santa had its own kernel extension with separate in-kernel queues for processing AUTH & NOTIFY type events. With the move to ES and the larger number of event types that we now notify on, running at the background QoS carries a small risk that the thread processing these events is not given a chance to run often enough that the queue grows and increases memory usage.
  • Loading branch information
russellhancox committed May 9, 2024
1 parent 7b0d2fd commit f00ad32
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions Source/santad/EventProviders/SNTEndpointSecurityClient.mm
Original file line number Diff line number Diff line change
Expand Up @@ -89,10 +89,8 @@ - (instancetype)initWithESAPI:(std::shared_ptr<EndpointSecurityAPI>)esApi
dispatch_queue_attr_make_with_qos_class(DISPATCH_QUEUE_CONCURRENT_WITH_AUTORELEASE_POOL,
QOS_CLASS_USER_INTERACTIVE, 0));

_notifyQueue = dispatch_queue_create(
"com.google.santa.daemon.notify_queue",
dispatch_queue_attr_make_with_qos_class(DISPATCH_QUEUE_CONCURRENT_WITH_AUTORELEASE_POOL,
QOS_CLASS_BACKGROUND, 0));
_notifyQueue = dispatch_queue_create("com.google.santa.daemon.notify_queue",
DISPATCH_QUEUE_CONCURRENT_WITH_AUTORELEASE_POOL);
}
return self;
}
Expand Down

0 comments on commit f00ad32

Please sign in to comment.