mirrors/gst-plugins-rs
mirrors/gst-plugins-rs
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs.git synced 2024-06-23 00:20:34 +00:00
Code Issues Projects Releases Wiki Activity

threadshare: Task: reduce memory usage

Browse source 
Using dynamic dispatch instead of monomorphization reduces the lib size
and memory use without affecting CPU nor throughput significantly.
This commit is contained in:
François Laignel 2020-04-30 18:49:32 +02:00
parent 1bea2ad279
commit 5c9bbc6818
1 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
generic/threadshare/src/runtime/task.rs
View file

@ -311,7 +311,7 @@ impl Task {
// FIXME allow configuration of the channel buffer size,
// this determines the contention on the Task.
let (transition_tx, transition_rx) = async_mpsc::channel(4);
let state_machine = StateMachine::new(task_impl, transition_rx);
let state_machine = StateMachine::new(Box::new(task_impl), transition_rx);
let (transition, _) = Transition::new(TransitionKind::Prepare);
inner.state_machine_handle = Some(inner.state_machine_context.spawn(state_machine.run(
Arc::clone(&self.0),
@ -515,9 +515,8 @@ impl Task {
}
}
#[derive(Debug)]
struct StateMachine<Impl> {
task_impl: Impl,
struct StateMachine {
task_impl: Box<dyn TaskImpl>,
transition_rx: async_mpsc::Receiver<Transition>,
pending_transition: Option<Transition>,
}
@ -546,8 +545,10 @@ macro_rules! spawn_hook {
}};
}
impl<Impl: TaskImpl> StateMachine<Impl> {
fn new(task_impl: Impl, transition_rx: async_mpsc::Receiver<Transition>) -> Self {
impl StateMachine {
// Use dynamic dispatch for TaskImpl as it reduces memory usage compared to monomorphization
// without inducing any significant performance penalties.
fn new(task_impl: Box<dyn TaskImpl>, transition_rx: async_mpsc::Receiver<Transition>) -> Self {
StateMachine {
task_impl,
transition_rx,