mirror of
https://github.com/genodelabs/genode.git
synced 2024-12-25 08:21:08 +00:00
eba9c15746
The patch adjust the code of the base, base-<kernel>, and os repository. To adapt existing components to fix violations of the best practices suggested by "Effective C++" as reported by the -Weffc++ compiler argument. The changes follow the patterns outlined below: * A class with virtual functions can no longer publicly inherit base classed without a vtable. The inherited object may either be moved to a member variable, or inherited privately. The latter would be used for classes that inherit 'List::Element' or 'Avl_node'. In order to enable the 'List' and 'Avl_tree' to access the meta data, the 'List' must become a friend. * Instead of adding a virtual destructor to abstract base classes, we inherit the new 'Interface' class, which contains a virtual destructor. This way, single-line abstract base classes can stay as compact as they are now. The 'Interface' utility resides in base/include/util/interface.h. * With the new warnings enabled, all member variables must be explicitly initialized. Basic types may be initialized with '='. All other types are initialized with braces '{ ... }' or as class initializers. If basic types and non-basic types appear in a row, it is nice to only use the brace syntax (also for basic types) and align the braces. * If a class contains pointers as members, it must now also provide a copy constructor and assignment operator. In the most cases, one would make them private, effectively disallowing the objects to be copied. Unfortunately, this warning cannot be fixed be inheriting our existing 'Noncopyable' class (the compiler fails to detect that the inheriting class cannot be copied and still gives the error). For now, we have to manually add declarations for both the copy constructor and assignment operator as private class members. Those declarations should be prepended with a comment like this: /* * Noncopyable */ Thread(Thread const &); Thread &operator = (Thread const &); In the future, we should revisit these places and try to replace the pointers with references. In the presence of at least one reference member, the compiler would no longer implicitly generate a copy constructor. So we could remove the manual declaration. Issue #465
351 lines
9.5 KiB
C++
351 lines
9.5 KiB
C++
/*
|
|
* \brief Connection to timer service and timeout scheduler
|
|
* \author Norman Feske
|
|
* \date 2008-08-22
|
|
*/
|
|
|
|
/*
|
|
* Copyright (C) 2008-2017 Genode Labs GmbH
|
|
*
|
|
* This file is part of the Genode OS framework, which is distributed
|
|
* under the terms of the GNU Affero General Public License version 3.
|
|
*/
|
|
|
|
#ifndef _INCLUDE__TIMER_SESSION__CONNECTION_H_
|
|
#define _INCLUDE__TIMER_SESSION__CONNECTION_H_
|
|
|
|
/* Genode includes */
|
|
#include <timer_session/client.h>
|
|
#include <base/connection.h>
|
|
#include <util/reconstructible.h>
|
|
#include <base/entrypoint.h>
|
|
#include <timer/timeout.h>
|
|
#include <trace/timestamp.h>
|
|
|
|
namespace Timer
|
|
{
|
|
class Connection;
|
|
template <typename> class Periodic_timeout;
|
|
template <typename> class One_shot_timeout;
|
|
}
|
|
|
|
|
|
/**
|
|
* Periodic timeout that is linked to a custom handler, scheduled when constructed
|
|
*/
|
|
template <typename HANDLER>
|
|
struct Timer::Periodic_timeout : private Genode::Noncopyable
|
|
{
|
|
private:
|
|
|
|
using Duration = Genode::Duration;
|
|
using Timeout = Genode::Timeout;
|
|
using Timeout_scheduler = Genode::Timeout_scheduler;
|
|
using Microseconds = Genode::Microseconds;
|
|
|
|
typedef void (HANDLER::*Handler_method)(Duration);
|
|
|
|
Timeout _timeout;
|
|
|
|
struct Handler : Timeout::Handler
|
|
{
|
|
HANDLER &object;
|
|
Handler_method const method;
|
|
|
|
Handler(HANDLER &object, Handler_method method)
|
|
: object(object), method(method) { }
|
|
|
|
|
|
/**********************
|
|
** Timeout::Handler **
|
|
**********************/
|
|
|
|
void handle_timeout(Duration curr_time) override {
|
|
(object.*method)(curr_time); }
|
|
|
|
} _handler;
|
|
|
|
public:
|
|
|
|
Periodic_timeout(Timeout_scheduler &timeout_scheduler,
|
|
HANDLER &object,
|
|
Handler_method method,
|
|
Microseconds duration)
|
|
:
|
|
_timeout(timeout_scheduler), _handler(object, method)
|
|
{
|
|
_timeout.schedule_periodic(duration, _handler);
|
|
}
|
|
};
|
|
|
|
|
|
/**
|
|
* One-shot timeout that is linked to a custom handler, scheduled manually
|
|
*/
|
|
template <typename HANDLER>
|
|
class Timer::One_shot_timeout : private Genode::Noncopyable
|
|
{
|
|
private:
|
|
|
|
using Duration = Genode::Duration;
|
|
using Timeout = Genode::Timeout;
|
|
using Timeout_scheduler = Genode::Timeout_scheduler;
|
|
using Microseconds = Genode::Microseconds;
|
|
|
|
typedef void (HANDLER::*Handler_method)(Duration);
|
|
|
|
Timeout _timeout;
|
|
|
|
struct Handler : Timeout::Handler
|
|
{
|
|
HANDLER &object;
|
|
Handler_method const method;
|
|
|
|
Handler(HANDLER &object, Handler_method method)
|
|
: object(object), method(method) { }
|
|
|
|
|
|
/**********************
|
|
** Timeout::Handler **
|
|
**********************/
|
|
|
|
void handle_timeout(Duration curr_time) override {
|
|
(object.*method)(curr_time); }
|
|
|
|
} _handler;
|
|
|
|
public:
|
|
|
|
One_shot_timeout(Timeout_scheduler &timeout_scheduler,
|
|
HANDLER &object,
|
|
Handler_method method)
|
|
: _timeout(timeout_scheduler), _handler(object, method) { }
|
|
|
|
void schedule(Microseconds duration) {
|
|
_timeout.schedule_one_shot(duration, _handler); }
|
|
|
|
void discard() { _timeout.discard(); }
|
|
|
|
bool scheduled() { return _timeout.scheduled(); }
|
|
};
|
|
|
|
|
|
/**
|
|
* Connection to timer service and timeout scheduler
|
|
*
|
|
* Multiplexes a timer session amongst different timeouts.
|
|
*/
|
|
class Timer::Connection : public Genode::Connection<Session>,
|
|
public Session_client,
|
|
private Genode::Time_source,
|
|
public Genode::Timeout_scheduler
|
|
{
|
|
private:
|
|
|
|
using Timeout = Genode::Timeout;
|
|
using Timeout_handler = Genode::Time_source::Timeout_handler;
|
|
using Timestamp = Genode::Trace::Timestamp;
|
|
using Duration = Genode::Duration;
|
|
using Lock = Genode::Lock;
|
|
using Microseconds = Genode::Microseconds;
|
|
using Milliseconds = Genode::Milliseconds;
|
|
using Entrypoint = Genode::Entrypoint;
|
|
|
|
/*
|
|
* Noncopyable
|
|
*/
|
|
Connection(Connection const &);
|
|
Connection &operator = (Connection const &);
|
|
|
|
/*
|
|
* The mode determines which interface of the timer connection is
|
|
* enabled. Initially, a timer connection is in LEGACY mode. When in
|
|
* MODERN mode, a call to the LEGACY interface causes an exception.
|
|
* When in LEGACY mode, a call to the MODERN interface causes a switch
|
|
* to the MODERN mode. The LEGACY interface is deprecated. Please
|
|
* prefer using the MODERN interface.
|
|
*
|
|
* LEGACY = Timer Session interface, blocking calls usleep and msleep
|
|
* MODERN = more precise curr_time, non-blocking and multiplexed
|
|
* handling with Periodic_timeout resp. One_shot_timeout
|
|
*/
|
|
enum Mode { LEGACY, MODERN };
|
|
|
|
Mode _mode { LEGACY };
|
|
Genode::Lock _lock { };
|
|
Genode::Signal_receiver _sig_rec { };
|
|
Genode::Signal_context _default_sigh_ctx { };
|
|
|
|
Genode::Signal_context_capability
|
|
_default_sigh_cap = _sig_rec.manage(&_default_sigh_ctx);
|
|
|
|
Genode::Signal_context_capability _custom_sigh_cap { };
|
|
|
|
void _enable_modern_mode();
|
|
|
|
void _sigh(Signal_context_capability sigh)
|
|
{
|
|
Session_client::sigh(sigh);
|
|
}
|
|
|
|
|
|
/*************************
|
|
** Time_source helpers **
|
|
*************************/
|
|
|
|
enum { MIN_TIMEOUT_US = 5000 };
|
|
enum { REAL_TIME_UPDATE_PERIOD_US = 500000 };
|
|
enum { MAX_INTERPOLATION_QUALITY = 3 };
|
|
enum { MAX_REMOTE_TIME_LATENCY_US = 500 };
|
|
enum { MAX_REMOTE_TIME_TRIALS = 5 };
|
|
enum { NR_OF_INITIAL_CALIBRATIONS = 3 * MAX_INTERPOLATION_QUALITY };
|
|
enum { MIN_FACTOR_LOG2 = 8 };
|
|
|
|
Genode::Io_signal_handler<Connection> _signal_handler;
|
|
|
|
Timeout_handler *_handler { nullptr };
|
|
Lock _real_time_lock { Lock::UNLOCKED };
|
|
unsigned long _us { elapsed_us() };
|
|
Timestamp _ts { _timestamp() };
|
|
Duration _real_time { Microseconds(_us) };
|
|
Duration _interpolated_time { _real_time };
|
|
unsigned _interpolation_quality { 0 };
|
|
unsigned long _us_to_ts_factor { 1UL };
|
|
unsigned _us_to_ts_factor_shift { 0 };
|
|
|
|
Timestamp _timestamp();
|
|
|
|
void _update_interpolation_quality(unsigned long min_factor,
|
|
unsigned long max_factor);
|
|
|
|
unsigned long _ts_to_us_ratio(Timestamp ts,
|
|
unsigned long us,
|
|
unsigned shift);
|
|
|
|
void _update_real_time();
|
|
|
|
Duration _update_interpolated_time(Duration &interpolated_time);
|
|
|
|
void _handle_timeout();
|
|
|
|
|
|
/*****************
|
|
** Time_source **
|
|
*****************/
|
|
|
|
void schedule_timeout(Microseconds duration, Timeout_handler &handler) override;
|
|
Microseconds max_timeout() const override { return Microseconds(REAL_TIME_UPDATE_PERIOD_US); }
|
|
|
|
|
|
/*******************************
|
|
** Timeout_scheduler helpers **
|
|
*******************************/
|
|
|
|
Genode::Alarm_timeout_scheduler _scheduler { *this };
|
|
|
|
|
|
/***********************
|
|
** Timeout_scheduler **
|
|
***********************/
|
|
|
|
void _schedule_one_shot(Timeout &timeout, Microseconds duration) override;
|
|
void _schedule_periodic(Timeout &timeout, Microseconds duration) override;
|
|
void _discard(Timeout &timeout) override;
|
|
|
|
public:
|
|
|
|
struct Cannot_use_both_legacy_and_modern_interface : Genode::Exception { };
|
|
|
|
/**
|
|
* Constructor
|
|
*/
|
|
Connection(Genode::Env &env, char const *label = "");
|
|
|
|
/**
|
|
* Constructor
|
|
*
|
|
* \noapi
|
|
* \deprecated Use the constructor with 'Env &' as first
|
|
* argument instead
|
|
*/
|
|
Connection() __attribute__((deprecated));
|
|
|
|
~Connection() { _sig_rec.dissolve(&_default_sigh_ctx); }
|
|
|
|
/*
|
|
* Intercept 'sigh' to keep track of customized signal handlers
|
|
*
|
|
* \noapi
|
|
* \deprecated Use One_shot_timeout (or Periodic_timeout) instead
|
|
*/
|
|
void sigh(Signal_context_capability sigh) override
|
|
{
|
|
if (_mode == MODERN) {
|
|
throw Cannot_use_both_legacy_and_modern_interface();
|
|
}
|
|
_custom_sigh_cap = sigh;
|
|
Session_client::sigh(_custom_sigh_cap);
|
|
}
|
|
|
|
/*
|
|
* Block for a time span of 'us' microseconds
|
|
*
|
|
* \noapi
|
|
* \deprecated Use One_shot_timeout (or Periodic_timeout) instead
|
|
*/
|
|
void usleep(unsigned us)
|
|
{
|
|
if (_mode == MODERN) {
|
|
throw Cannot_use_both_legacy_and_modern_interface();
|
|
}
|
|
/*
|
|
* Omit the interaction with the timer driver for the corner case
|
|
* of not sleeping at all. This corner case may be triggered when
|
|
* polling is combined with sleeping (as some device drivers do).
|
|
* If we passed the sleep operation to the timer driver, the
|
|
* timer would apply its policy about a minimum sleep time to
|
|
* the sleep operation, which is not desired when polling.
|
|
*/
|
|
if (us == 0)
|
|
return;
|
|
|
|
/* serialize sleep calls issued by different threads */
|
|
Genode::Lock::Guard guard(_lock);
|
|
|
|
/* temporarily install to the default signal handler */
|
|
if (_custom_sigh_cap.valid())
|
|
Session_client::sigh(_default_sigh_cap);
|
|
|
|
/* trigger timeout at default signal handler */
|
|
trigger_once(us);
|
|
_sig_rec.wait_for_signal();
|
|
|
|
/* revert custom signal handler if registered */
|
|
if (_custom_sigh_cap.valid())
|
|
Session_client::sigh(_custom_sigh_cap);
|
|
}
|
|
|
|
/*
|
|
* Block for a time span of 'ms' milliseconds
|
|
*
|
|
* \noapi
|
|
* \deprecated Use One_shot_timeout (or Periodic_timeout) instead
|
|
*/
|
|
void msleep(unsigned ms)
|
|
{
|
|
if (_mode == MODERN) {
|
|
throw Cannot_use_both_legacy_and_modern_interface();
|
|
}
|
|
usleep(1000*ms);
|
|
}
|
|
|
|
|
|
/***********************************
|
|
** Timeout_scheduler/Time_source **
|
|
***********************************/
|
|
|
|
Duration curr_time() override;
|
|
};
|
|
|
|
#endif /* _INCLUDE__TIMER_SESSION__CONNECTION_H_ */
|