From fc4efa2f102705bad7301a3c0e63543228c72a94 Mon Sep 17 00:00:00 2001 From: Gregory Nutt Date: Tue, 30 Sep 2014 17:45:42 -0600 Subject: [PATCH] Update some comments --- arch/sim/src/up_simuart.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/arch/sim/src/up_simuart.c b/arch/sim/src/up_simuart.c index 7d48d39877..cd14700d42 100644 --- a/arch/sim/src/up_simuart.c +++ b/arch/sim/src/up_simuart.c @@ -150,6 +150,14 @@ static void *simuart_thread(void *arg) } } + /* REVISIT: This is very weird and scary here. When sched_unlock() + * is called, we may do a lonjmp() style context switch meaning + * that the logic will be run running on this thread! (but with a + * different stack). So we do not get back here until the task + * sleeps again. I can't help but believe that that is going to + * be a problem someday. + */ + sched_unlock(); } }