ExternalVendorCode/genode
1
0
Fork 0
mirror of https://github.com/genodelabs/genode.git synced 2025-01-18 18:56:29 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

libc: enforce suspend is not called from initial stack

Browse Source 
In other words: Loosen the former property that suspend can only be
called from the first user stack and, thus, enable additional user
stacks, e.g., in coroutine libraries.

Fix #2737
This commit is contained in:
Emery Hemingway 2018-03-29 15:22:33 +02:00 committed by Christian Helmuth
parent 04516a0d39
commit 7091f5febe
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
repos/libports/src/lib/libc/task.cc
View File

@ -6,7 +6,7 @@
*/
/*
* Copyright (C) 2016-2017 Genode Labs GmbH
* Copyright (C) 2016-2018 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.
@ -394,6 +394,8 @@ struct Libc::Kernel
Genode::Thread &_myself { *Genode::Thread::myself() };
addr_t _kernel_stack = Thread::mystack().top;
void *_user_stack = {
_myself.alloc_secondary_stack(_myself.name().string(),
Component::stack_size()) };
@ -546,8 +548,8 @@ struct Libc::Kernel
unsigned long _suspend_main(Suspend_functor &check,
unsigned long timeout_ms)
{
/* check if we're running on the user context */
if (Thread::myself()->mystack().top != (Genode::addr_t)_user_stack) {
/* check that we're not running on libc kernel context */
if (Thread::mystack().top == _kernel_stack) {
error("libc suspend() called from non-user context (",
__builtin_return_address(0), ") - aborting");
exit(1);