diff options
| author |   Sven Schnelle <svens@linux.ibm.com> | 2019-12-18 08:44:26 +0100 |
|---|---|---|
| committer |   Steven Rostedt (VMware) <rostedt@goodmis.org> | 2019-12-21 16:08:22 -0500 |
| commit | 01f36a554e3ef32f9fc4b81a4437cf08fd0e4742 (patch) | |
| tree | f0ef6325d08529b98aee0e88ff5f3612e2ec5709 /samples/trace_printk/trace-printk.c | |
| parent | tracing: Fix lock inversion in trace_event_enable_tgid_record() (diff) | |
| download | wireguard-linux-01f36a554e3ef32f9fc4b81a4437cf08fd0e4742.tar.xz wireguard-linux-01f36a554e3ef32f9fc4b81a4437cf08fd0e4742.zip | |
samples/trace_printk: Wait for IRQ work to finish
trace_printk schedules work via irq_work_queue(), but doesn't
wait until it was processed. The kprobe_module.tc testcase does:
:;: "Load module again, which means the event1 should be recorded";:
modprobe trace-printk
grep "event1:" trace
so the grep which checks the trace file might run before the irq work
was processed. Fix this by adding a irq_work_sync().
Link: http://lore.kernel.org/linux-trace-devel/20191218074427.96184-3-svens@linux.ibm.com
Cc: stable@vger.kernel.org
Fixes: af2a0750f3749 ("selftests/ftrace: Improve kprobe on module testcase to load/unload module")
Signed-off-by: Sven Schnelle <svens@linux.ibm.com>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Diffstat (limited to 'samples/trace_printk/trace-printk.c')
| -rw-r--r-- | samples/trace_printk/trace-printk.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/samples/trace_printk/trace-printk.c b/samples/trace_printk/trace-printk.c index 7affc3b50b61..cfc159580263 100644 --- a/samples/trace_printk/trace-printk.c +++ b/samples/trace_printk/trace-printk.c @@ -36,6 +36,7 @@ static int __init trace_printk_init(void) /* Kick off printing in irq context */ irq_work_queue(&irqwork); + irq_work_sync(&irqwork); trace_printk("This is a %s that will use trace_bprintk()\n", "static string"); |