From b3f156b7a5bf668a248dd79a078d58c731c5fc6a Mon Sep 17 00:00:00 2001 From: daniel-k Date: Fri, 1 Jul 2016 18:54:09 +0200 Subject: [PATCH 1/4] hermit/stddef.h: remove excess ';' --- hermit/include/hermit/stddef.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hermit/include/hermit/stddef.h b/hermit/include/hermit/stddef.h index 9edecfbb0..ee212f058 100644 --- a/hermit/include/hermit/stddef.h +++ b/hermit/include/hermit/stddef.h @@ -47,9 +47,9 @@ extern "C" { /// represents a task identifier typedef unsigned int tid_t; -#define DECLARE_PER_CORE(type, name) extern type name __attribute__ ((section (".percore"))); -#define DEFINE_PER_CORE(type, name, def_value) type name __attribute__ ((section (".percore"))) = def_value; -#define DEFINE_PER_CORE_STATIC(type, name, def_value) static type name __attribute__ ((section (".percore"))) = def_value; +#define DECLARE_PER_CORE(type, name) extern type name __attribute__ ((section (".percore"))) +#define DEFINE_PER_CORE(type, name, def_value) type name __attribute__ ((section (".percore"))) = def_value +#define DEFINE_PER_CORE_STATIC(type, name, def_value) static type name __attribute__ ((section (".percore"))) = def_value /* needed to find the task, which is currently running on this core */ struct task; From 53331b3daa1b07a2d1064040dfcd8348fd26ee75 Mon Sep 17 00:00:00 2001 From: daniel-k Date: Fri, 1 Jul 2016 18:54:56 +0200 Subject: [PATCH 2/4] hermit/tasks.h: typo --- hermit/include/hermit/tasks.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hermit/include/hermit/tasks.h b/hermit/include/hermit/tasks.h index f11e2c4b3..32bd213b0 100644 --- a/hermit/include/hermit/tasks.h +++ b/hermit/include/hermit/tasks.h @@ -230,7 +230,7 @@ void check_ticks(void); extern volatile uint32_t go_down; -/** @brief shutdown the hole system +/** @brief shutdown the whole system */ void shutdown_system(void); From 3d752ed73720ffa4625004e529c135d517a90604 Mon Sep 17 00:00:00 2001 From: daniel-k Date: Fri, 1 Jul 2016 18:58:04 +0200 Subject: [PATCH 3/4] kernel/main: get rid of unused variable warning --- hermit/kernel/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hermit/kernel/main.c b/hermit/kernel/main.c index e9fbaeb72..bd2c35599 100644 --- a/hermit/kernel/main.c +++ b/hermit/kernel/main.c @@ -212,7 +212,7 @@ int network_shutdown(void) #if MAX_CORES > 1 int smp_main(void) { - int32_t cpu = atomic_int32_inc(&cpu_online); + atomic_int32_inc(&cpu_online); #ifdef DYNAMIC_TICKS enable_dynticks(); From cd30d0969bc98ee814de969e3d89c5a7b4fdc5f0 Mon Sep 17 00:00:00 2001 From: daniel-k Date: Fri, 1 Jul 2016 19:03:51 +0200 Subject: [PATCH 4/4] kernel/tasks: add missing include for get_tls() --- hermit/kernel/tasks.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hermit/kernel/tasks.c b/hermit/kernel/tasks.c index fe45ba017..c58c6aab8 100644 --- a/hermit/kernel/tasks.c +++ b/hermit/kernel/tasks.c @@ -37,6 +37,7 @@ #include #include #include +#include /* * Note that linker symbols are not variables, they have no memory allocated for