Re: [PATCH] trace/simple: Enable tracing on startup only if the user specifies a trace option

2020-10-20 Thread Josh DuBois
t; On Sun, Aug 16, 2020 at 12:46:10PM -0500, dubo...@gmail.com wrote: >>>> From: Josh DuBois >>>> >>>> Tracing can be enabled at the command line or via the >>>> monitor. Command-line trace options are recorded during >>>> trace_opt_par

Re: [PATCH] trace/simple: Allow enabling simple traces from command line

2020-08-04 Thread Josh DuBois
On Aug 3, 2020, at 4:08 AM, Markus Armbruster wrote: > >> >> - prior to db25d56c014aa1a96319c663e0a60346a223b31e, just like today, >> QEMU built with simple tracing will always produce a trace- file, >> regardless of whether the user asks for traces at runtime. > > When you send a patch with a

Re: [PATCH] trace/simple: Allow enabling simple traces from command line

2020-07-30 Thread Josh DuBois
If I resubmit, I'll try to test a little more next time.  I just wanted my traces to work. ;) On 7/29/20 8:05 AM, Stefan Hajnoczi wrote: On Thu, Jul 23, 2020 at 12:33:59AM -0500, Josh DuBois wrote: The simple trace backend is enabled / disabled with a call to st_set_trace_file_enabled(

[PATCH] trace/simple: Allow enabling simple traces from command line

2020-07-23 Thread Josh DuBois
enabled tracing.) Fixes: db25d56c014aa1a96319c663e0a60346a223b31e Signed-off-by: Josh DuBois --- trace/control.c | 1 + 1 file changed, 1 insertion(+) diff --git a/trace/control.c b/trace/control.c index 2ffe000818..6558b5c906 100644 --- a/trace/control.c +++ b/trace/control.c @@ -225,6 +225,7