diff options
author | Blue Swirl <blauwirbel@gmail.com> | 2010-10-13 19:14:29 +0000 |
---|---|---|
committer | Blue Swirl <blauwirbel@gmail.com> | 2010-10-13 19:14:29 +0000 |
commit | f871d6893a7bf8c14db162f6e75a5f8157e4c2bb (patch) | |
tree | 6b68bc6aed69b8725f4454ec83c577056f206869 /simpletrace.c | |
parent | 2abf314dddcfc1792a981ef6ecda343384b2a1a2 (diff) | |
download | qemu-f871d6893a7bf8c14db162f6e75a5f8157e4c2bb.tar.gz qemu-f871d6893a7bf8c14db162f6e75a5f8157e4c2bb.tar.bz2 qemu-f871d6893a7bf8c14db162f6e75a5f8157e4c2bb.zip |
trace: print a warning if user tries to enable an unknown trace event
There was no warning if a bad trace event name was given to
'trace-event' command, thus the user could think that the command
was successful even if this was not the case.
Print a warning if the user tries to enable a trace event which is not
known.
Acked-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
Diffstat (limited to 'simpletrace.c')
-rw-r--r-- | simpletrace.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/simpletrace.c b/simpletrace.c index f849e42914..deb1e0742e 100644 --- a/simpletrace.c +++ b/simpletrace.c @@ -246,12 +246,14 @@ static TraceEvent* find_trace_event_by_name(const char *tname) return NULL; /* indicates end of list reached without a match */ } -void st_change_trace_event_state(const char *tname, bool tstate) +bool st_change_trace_event_state(const char *tname, bool tstate) { TraceEvent *tp; tp = find_trace_event_by_name(tname); if (tp) { tp->state = tstate; + return true; } + return false; } |