From 852c4baf00642fc8e48f2503e6f37081d4e6f496 Mon Sep 17 00:00:00 2001 From: smlng Date: Tue, 25 Sep 2018 21:47:08 +0200 Subject: [PATCH 1/3] tests: print missing thread_t attribute This adds the `stack_size` attribute which wasn't printed right now so for instance the numbers didn't add for the sizeof(thread_t) when DEVELHELP was on its of by 4 bytes (on 32-Bit). --- tests/bench_sizeof_coretypes/main.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/bench_sizeof_coretypes/main.c b/tests/bench_sizeof_coretypes/main.c index eaf5f07b0d..c923ae1af6 100644 --- a/tests/bench_sizeof_coretypes/main.c +++ b/tests/bench_sizeof_coretypes/main.c @@ -100,12 +100,13 @@ int main(void) P(msg_queue); P(msg_array); #endif -#ifdef DEVELHELP - P(name); -#endif #if defined(DEVELHELP) || defined(SCHED_TEST_STACK) || defined(MODULE_MPU_STACK_GUARD) P(stack_start); #endif +#ifdef DEVELHELP + P(name); + P(stack_size); +#endif puts("\n[SUCCESS]"); return 0; From 368ee45986c8b294d0b2e1e6fa073a30e91a9421 Mon Sep 17 00:00:00 2001 From: smlng Date: Tue, 25 Sep 2018 21:52:01 +0200 Subject: [PATCH 2/3] tests/bench_sizeof_coretypes: use testrunner from pylib path --- tests/bench_sizeof_coretypes/tests/01-run.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tests/bench_sizeof_coretypes/tests/01-run.py b/tests/bench_sizeof_coretypes/tests/01-run.py index b6b4d4d367..f7cc0da9a8 100755 --- a/tests/bench_sizeof_coretypes/tests/01-run.py +++ b/tests/bench_sizeof_coretypes/tests/01-run.py @@ -6,8 +6,8 @@ # General Public License v2.1. See the file LICENSE in the top level # directory for more details. -import os import sys +from testrunner import run def testfunc(child): @@ -15,6 +15,4 @@ def testfunc(child): if __name__ == "__main__": - sys.path.append(os.path.join(os.environ['RIOTTOOLS'], 'testrunner')) - from testrunner import run sys.exit(run(testfunc)) From 3c202d0ad24235cfa541d3c6c4813198e80fe642 Mon Sep 17 00:00:00 2001 From: smlng Date: Wed, 26 Sep 2018 08:05:06 +0200 Subject: [PATCH 3/3] tests/bench_sizeof_coretypes: fix ifdef for wait_data attribute --- tests/bench_sizeof_coretypes/main.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/bench_sizeof_coretypes/main.c b/tests/bench_sizeof_coretypes/main.c index c923ae1af6..76ac94fbcb 100644 --- a/tests/bench_sizeof_coretypes/main.c +++ b/tests/bench_sizeof_coretypes/main.c @@ -94,8 +94,10 @@ int main(void) P(flags); #endif P(rq_entry); -#ifdef MODULE_CORE_MSG +#if defined(MODULE_CORE_MSG) || defined(MODULE_CORE_THREAD_FLAGS) || defined(MODULE_CORE_MBOX) P(wait_data); +#endif +#ifdef MODULE_CORE_MSG P(msg_waiters); P(msg_queue); P(msg_array);