3985f0151a
* Added a way to notify a Module that it's been attached to a ProcessThread. The benefit of this is to give the module a way to wake up the thread when it needs work to happen on the worker thread, immediately. Today, module instances are typically registered with a process thread outside the control of the modules themselves. I.e. they typically don't know about the process thread they're attached to. * Improve ProcessThread's WakeUp algorithm to not call TimeUntilNextProcess when a WakeUp call is requested. This is an optimization for the above case which avoids the module having to acquire a lock or do an interlocked operation before calling WakeUp(), which would ensure the module's TimeUntilNextProcess() implementation would return 0. BUG=2822 R=stefan@webrtc.org Review URL: https://webrtc-codereview.appspot.com/39239004 Cr-Commit-Position: refs/heads/master@{#8527} git-svn-id: http://webrtc.googlecode.com/svn/trunk@8527 4adac7df-926f-26a2-2b94-8c16560cd09d
80 lines
2.5 KiB
C++
80 lines
2.5 KiB
C++
/*
|
|
* Copyright (c) 2011 The WebRTC project authors. All Rights Reserved.
|
|
*
|
|
* Use of this source code is governed by a BSD-style license
|
|
* that can be found in the LICENSE file in the root of the source
|
|
* tree. An additional intellectual property rights grant can be found
|
|
* in the file PATENTS. All contributing project authors may
|
|
* be found in the AUTHORS file in the root of the source tree.
|
|
*/
|
|
|
|
#ifndef WEBRTC_MODULES_UTILITY_SOURCE_PROCESS_THREAD_IMPL_H_
|
|
#define WEBRTC_MODULES_UTILITY_SOURCE_PROCESS_THREAD_IMPL_H_
|
|
|
|
#include <list>
|
|
|
|
#include "webrtc/base/criticalsection.h"
|
|
#include "webrtc/base/thread_checker.h"
|
|
#include "webrtc/modules/utility/interface/process_thread.h"
|
|
#include "webrtc/system_wrappers/interface/event_wrapper.h"
|
|
#include "webrtc/system_wrappers/interface/thread_wrapper.h"
|
|
#include "webrtc/typedefs.h"
|
|
|
|
namespace webrtc {
|
|
|
|
class ProcessThreadImpl : public ProcessThread {
|
|
public:
|
|
ProcessThreadImpl();
|
|
~ProcessThreadImpl() override;
|
|
|
|
void Start() override;
|
|
void Stop() override;
|
|
|
|
void WakeUp(Module* module) override;
|
|
|
|
void RegisterModule(Module* module) override;
|
|
void DeRegisterModule(Module* module) override;
|
|
|
|
protected:
|
|
static bool Run(void* obj);
|
|
bool Process();
|
|
|
|
private:
|
|
struct ModuleCallback {
|
|
ModuleCallback() : module(nullptr), next_callback(0) {}
|
|
ModuleCallback(const ModuleCallback& cb)
|
|
: module(cb.module), next_callback(cb.next_callback) {}
|
|
ModuleCallback(Module* module) : module(module), next_callback(0) {}
|
|
bool operator==(const ModuleCallback& cb) const {
|
|
return cb.module == module;
|
|
}
|
|
|
|
Module* const module;
|
|
int64_t next_callback; // Absolute timestamp.
|
|
|
|
private:
|
|
ModuleCallback& operator=(ModuleCallback&);
|
|
};
|
|
|
|
typedef std::list<ModuleCallback> ModuleList;
|
|
|
|
// Warning: For some reason, if |lock_| comes immediately before |modules_|
|
|
// with the current class layout, we will start to have mysterious crashes
|
|
// on Mac 10.9 debug. I (Tommi) suspect we're hitting some obscure alignemnt
|
|
// issues, but I haven't figured out what they are, if there are alignment
|
|
// requirements for mutexes on Mac or if there's something else to it.
|
|
// So be careful with changing the layout.
|
|
rtc::CriticalSection lock_; // Used to guard modules_ and stop_.
|
|
|
|
rtc::ThreadChecker thread_checker_;
|
|
const rtc::scoped_ptr<EventWrapper> wake_up_;
|
|
rtc::scoped_ptr<ThreadWrapper> thread_;
|
|
|
|
ModuleList modules_;
|
|
bool stop_;
|
|
};
|
|
|
|
} // namespace webrtc
|
|
|
|
#endif // WEBRTC_MODULES_UTILITY_SOURCE_PROCESS_THREAD_IMPL_H_
|