This patch replaces the former prominent use of pointers by references wherever feasible. This has the following benefits: * The contract between caller and callee becomes more obvious. When passing a reference, the contract says that the argument cannot be a null pointer. The caller is responsible to ensure that. Therefore, the use of reference eliminates the need to add defensive null-pointer checks at the callee site, which sometimes merely exist to be on the safe side. The bottom line is that the code becomes easier to follow. * Reference members must be initialized via an object initializer, which promotes a programming style that avoids intermediate object- construction states. Within core, there are still a few pointers as member variables left though. E.g., caused by the late association of 'Platform_thread' objects with their 'Platform_pd' objects. * If no pointers are present as member variables, we don't need to manually provide declarations of a private copy constructor and an assignment operator to avoid -Weffc++ errors "class ... has pointer data members [-Werror=effc++]". This patch also changes a few system bindings on NOVA and Fiasco.OC, e.g., the return value of the global 'cap_map' accessor has become a reference. Hence, the patch touches a few places outside of core. Fixes #3135
83 lines
1.7 KiB
C++
83 lines
1.7 KiB
C++
/*
|
|
* \brief Kernel object handling in core
|
|
* \author Stefan Kalkowski
|
|
* \date 2015-04-21
|
|
*/
|
|
|
|
/*
|
|
* Copyright (C) 2015-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 _CORE__OBJECT_H_
|
|
#define _CORE__OBJECT_H_
|
|
|
|
/* Genode includes */
|
|
#include <util/construct_at.h>
|
|
|
|
/* base-internal includes */
|
|
#include <base/internal/capability_space.h>
|
|
|
|
/* base-hw includes */
|
|
#include <kernel/interface.h>
|
|
#include <kernel/object.h>
|
|
|
|
namespace Genode {
|
|
/**
|
|
* Represents a kernel object in core
|
|
*
|
|
* \param T type of the kernel object
|
|
*/
|
|
template <typename T> class Kernel_object;
|
|
}
|
|
|
|
|
|
template <typename T>
|
|
class Genode::Kernel_object
|
|
{
|
|
private:
|
|
|
|
uint8_t _data[sizeof(Kernel::Core_object<T>)]
|
|
__attribute__((aligned(sizeof(addr_t))));
|
|
|
|
protected:
|
|
|
|
Untyped_capability _cap { };
|
|
|
|
public:
|
|
|
|
Kernel_object() {}
|
|
|
|
/**
|
|
* Creates a kernel object either via a syscall or directly
|
|
*/
|
|
template <typename... ARGS>
|
|
Kernel_object(bool syscall, ARGS &&... args)
|
|
: _cap(Capability_space::import(syscall ? T::syscall_create(&_data, args...)
|
|
: Kernel::cap_id_invalid()))
|
|
{
|
|
if (!syscall) construct_at<T>(&_data, args...);
|
|
}
|
|
|
|
~Kernel_object() { T::syscall_destroy(kernel_object()); }
|
|
|
|
T * kernel_object() { return reinterpret_cast<T*>(_data); }
|
|
|
|
Untyped_capability cap() { return _cap; }
|
|
|
|
/**
|
|
* Create the kernel object explicitely via this function
|
|
*/
|
|
template <typename... ARGS>
|
|
bool create(ARGS &&... args)
|
|
{
|
|
if (_cap.valid()) return false;
|
|
_cap = Capability_space::import(T::syscall_create(&_data, args...));
|
|
return _cap.valid();
|
|
}
|
|
};
|
|
|
|
#endif /* _CORE__OBJECT_H_ */
|