From 224e23bef7a46c823c4b3d41efdf9dbf98853ed0 Mon Sep 17 00:00:00 2001 From: "Frank Ch. Eigler" Date: Wed, 26 Nov 2008 17:42:45 -0500 Subject: safety: add another schedule_sched, ahead of the per-context release loop --- ChangeLog | 4 ++++ translate.cxx | 7 ++++++- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index e8821811..fde0f158 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +2008-11-26 Frank Ch. Eigler + + * translate.cxx (emit_module_exit): Add another synchronize_sched. + 2008-11-26 Frank Ch. Eigler * main.cxx (main): Advise users to try '--vp ...' after a pass error. diff --git a/translate.cxx b/translate.cxx index 49a86160..43299791 100644 --- a/translate.cxx +++ b/translate.cxx @@ -1273,6 +1273,11 @@ c_unparser::emit_module_exit () // But some other probes may have launched too during unregistration. // Let's wait a while to make sure they're all done, done, done. + // cargo cult prologue + o->newline() << "#ifdef STAPCONF_SYNCHRONIZE_SCHED"; + o->newline() << "synchronize_sched();"; + o->newline() << "#endif"; + // NB: systemtap_module_exit is assumed to be called from ordinary // user context, say during module unload. Among other things, this // means we can sleep a while. @@ -1287,7 +1292,7 @@ c_unparser::emit_module_exit () o->newline () << "yield ();"; // aka schedule() and then some o->newline(-2) << "} while (holdon);"; - // + // cargo cult epilogue o->newline() << "#ifdef STAPCONF_SYNCHRONIZE_SCHED"; o->newline() << "synchronize_sched();"; o->newline() << "#endif"; -- cgit