From 063caf3a90955709cb12d1c732099db052369512 Mon Sep 17 00:00:00 2001 From: Gregory Nutt Date: Tue, 16 Feb 2016 16:08:59 -0600 Subject: [PATCH] Add warnings and some comments --- arch/sim/src/up_schednote.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/arch/sim/src/up_schednote.c b/arch/sim/src/up_schednote.c index d77290885d..57c2ffe08a 100644 --- a/arch/sim/src/up_schednote.c +++ b/arch/sim/src/up_schednote.c @@ -135,6 +135,13 @@ void sched_note_switch(FAR struct tcb_s *from, FAR struct tcb_s *to) } #ifdef CONFIG_SCHED_INSTRUMENTATION_PREEMPTION +/* This does not work well... it interferes with the operation of the + * simulated /dev/console device which, of course, does disable preemption + * and does use critical sections. + */ + +#warning CONFIG_SCHED_INSTRUMENTATION_PREEMPTION is a bad idea + void sched_note_premption(FAR struct tcb_s *tcb, bool locked) { #ifdef CONFIG_SMP @@ -158,6 +165,13 @@ void sched_note_premption(FAR struct tcb_s *tcb, bool locked) #endif #ifdef CONFIG_SCHED_INSTRUMENTATION_CSECTION +/* This does not work well... it interferes with the operation of the + * simulated /dev/console device which, of course, does disable preemption + * and does use critical sections. + */ + +#warning CONFIG_SCHED_INSTRUMENTATION_CSECTION is a bad idea + void sched_note_csection(FAR struct tcb_s *tcb, bool enter) { #ifdef CONFIG_SMP