ExternalVendorCode/genode
1
0
Fork 0
mirror of https://github.com/genodelabs/genode.git synced 2025-04-08 20:05:54 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

base: separate the API's 'Thread_base' constructor

Browse Source 
The 'Thread_base' class is constructed differently in some special cases
like the main thread or a thread that use a distinct CPU session. The
official API, however, should be clean from such artifacts. Hence, I
separated the official constructor from the other cases.
This commit is contained in:
Norman Feske 2015-03-24 21:14:44 +01:00 committed by Christian Helmuth
parent 0d74435448
commit 13a0c336d9
1 changed files with 49 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

65
repos/base/include/base/thread.h
View File

@ -340,20 +340,7 @@ class Genode::Thread_base
/**
* Constructor
*
* \param quota CPU quota that shall be granted to the thread
* \param name thread name for debugging
* \param stack_size stack size
* \param type enables selection of special construction
*
* \throw Stack_too_large
* \throw Stack_alloc_failed
* \throw Context_alloc_failed
*
* The stack for the new thread will be allocated from the RAM
* session of the process environment. A small portion of the
* stack size is internally used by the framework for storing
* thread-context information such as the thread's name (see
* 'struct Context').
* \noapi
*
* FIXME: With type = Forked_main_thread the whole
* Context::_alloc_context call gets skipped but we should
@ -361,7 +348,7 @@ class Genode::Thread_base
* the dataspace of the already attached stack.
*/
Thread_base(size_t quota, const char *name, size_t stack_size,
Type type = NORMAL);
Type type);
/**
* Constructor
@ -369,6 +356,34 @@ class Genode::Thread_base
* \param quota CPU quota that shall be granted to the thread
* \param name thread name for debugging
* \param stack_size stack size
*
* \throw Stack_too_large
* \throw Stack_alloc_failed
* \throw Context_alloc_failed
*
* The stack for the new thread will be allocated from the RAM session
* of the process environment. A small portion of the stack size is
* internally used by the framework for storing thread-context
* information such as the thread's name ('Context').
*
*/
Thread_base(size_t quota, const char *name, size_t stack_size)
:
Thread_base(quota, name, stack_size, NORMAL)
{ }
/**
* Constructor
*
* Variant of the constructor that allows the use of a different
* CPU session.
*
* \noapi Using multiple CPU sessions within a single component is
* an experimental feature.
*
* \param quota CPU quota that shall be granted to the thread
* \param name thread name for debugging
* \param stack_size stack size
* \param type enables selection of special construction
* \param cpu_session capability to cpu session used for construction
*
@ -524,7 +539,19 @@ class Genode::Thread : public Thread_base
* \param name thread name (for debugging)
* \param type enables selection of special construction
*/
explicit Thread(size_t quota, const char *name, Type type = NORMAL)
explicit Thread(size_t quota, const char *name)
: Thread_base(quota, name, STACK_SIZE, Type::NORMAL) { }
/**
* Constructor
*
* \param quota CPU quota that shall be granted to the thread
* \param name thread name (for debugging)
* \param type enables selection of special construction
*
* \noapi
*/
explicit Thread(size_t quota, const char *name, Type type)
: Thread_base(quota, name, STACK_SIZE, type) { }
/**
@ -533,6 +560,8 @@ class Genode::Thread : public Thread_base
* \param quota CPU quota that shall be granted to the thread
* \param name thread name (for debugging)
* \param cpu_session thread created via specific cpu session
*
* \noapi
*/
explicit Thread(size_t quota, const char *name, Cpu_session * cpu_session)
: Thread_base(quota, name, STACK_SIZE, Type::NORMAL, cpu_session)
@ -540,12 +569,16 @@ class Genode::Thread : public Thread_base
/**
* Shortcut for 'Thread(0, name, type)'
*
* \noapi
*/
explicit Thread(const char *name, Type type = NORMAL)
: Thread_base(0, name, STACK_SIZE, type) { }
/**
* Shortcut for 'Thread(0, name, cpu_session)'
*
* \noapi
*/
explicit Thread(const char *name, Cpu_session * cpu_session)
: Thread_base(0, name, STACK_SIZE, Type::NORMAL, cpu_session)