From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=50424 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OTuPO-0007tr-LC for qemu-devel@nongnu.org; Wed, 30 Jun 2010 06:20:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OTuPG-0002FD-BY for qemu-devel@nongnu.org; Wed, 30 Jun 2010 06:20:46 -0400 Received: from e28smtp03.in.ibm.com ([122.248.162.3]:43270) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OTuPF-0002ER-Jj for qemu-devel@nongnu.org; Wed, 30 Jun 2010 06:20:38 -0400 Received: from d28relay03.in.ibm.com (d28relay03.in.ibm.com [9.184.220.60]) by e28smtp03.in.ibm.com (8.14.4/8.13.1) with ESMTP id o5UAKUsu023935 for ; Wed, 30 Jun 2010 15:50:30 +0530 Received: from d28av02.in.ibm.com (d28av02.in.ibm.com [9.184.220.64]) by d28relay03.in.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id o5UAKTwP3973342 for ; Wed, 30 Jun 2010 15:50:29 +0530 Received: from d28av02.in.ibm.com (loopback [127.0.0.1]) by d28av02.in.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id o5UAKTe1018516 for ; Wed, 30 Jun 2010 20:20:29 +1000 Message-ID: <4C2B1A6C.3000109@linux.vnet.ibm.com> Date: Wed, 30 Jun 2010 15:50:28 +0530 From: Prerna Saxena MIME-Version: 1.0 Subject: Re: [Qemu-devel] Tracing: outstanding tasks References: <20100626080657.GA4806@stefan-thinkpad.transitives.com> In-Reply-To: <20100626080657.GA4806@stefan-thinkpad.transitives.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Maneesh Soni , Ananth , qemu-devel@nongnu.org On 06/26/2010 01:36 PM, Stefan Hajnoczi wrote: > Here are the outstanding tasks for QEMU tracing, which Prerna and I have > been working on. Tracing aids debugging, profiling, and observing > execution via lightweight logging at key points in the code path. > > The current prototype is available from the 'tracing' branch at: > > http://repo.or.cz/w/qemu/stefanha.git/shortlog/refs/heads/tracing > > This email is both to help Prerna and me focus our efforts, as well as a > roadmap for the QEMU community to discuss and comment on. > > Here are the outstanding tasks for a tracing patchset that can be > proposed for merge: > > 1. Integration with QMP > > Owner: Prerna > > Currently the trace commands are available from the monitor but a QMP interface > is needed. > Agree. I'd suggest we let this be a TODO till the trace infrastructure makes its way upstream. > 2. More tracepoints need to be added for instrumenting other qemu components > such as virtio drivers, etc. > > Owner: ? > > QEMU must come with a useful set of trace events that allows people to get up > and running quickly. Focus areas include: > > * guest device emulation > * host devices > * lifecycle and runloop > * memory management > * live migration > > 3. Documentation > > Owner: Stefan > > User documentation that explains how to collect traces and add new trace > events. I have committed documentation to the tracing branch here: > > http://repo.or.cz/w/qemu/stefanha.git/blob_plain/91fde34dc6bfe01af6d5e9265f6a81535d6add15:/docs/tracing.txt > > 4. Fix i386-linux-user build > > Owner: Prerna > > LINK i386-linux-user/qemu-i386 > ../simpletrace.o: In function `do_info_all_trace_events': > /home/stefanha/qemu/simpletrace.c:88: undefined reference to `monitor_printf' > ../simpletrace.o: In function `do_info_trace': > /home/stefanha/qemu/simpletrace.c:77: undefined reference to `monitor_printf' > I'm looking into this. > 5. Out-of-line trace file write-out > > Owner: Stefan > > Trace buffers are written out to file synchronously. The vcpu thread should > not be blocked so an async write-out mechanism is needed. > > 6. Trace file command > > Owner: ? > > Traces are written out to hardcoded /tmp/trace.log. This must be configurable. > Tracing at startup time should still be possible so configuration needs to > happen early. Agree, it is a good-to-have feature. > > 7. Binary trace format finalization > > Owner: Stefan > > We should leave room for extension. I suggest partitioning the Event ID > namespace into normal events and special events. The __trace_begin special > event is defined to contain the file format version and/or trace record size in > bytes as the first trace record in the file. > > This way, post-processing tools can check the format of the binary trace file. > > 8. QMP/monitor command review > > Owner: Prerna, Stefan > > 9. In its present format, the tracing infrastructure causes at least two function calls even for trace events that are disabled. Ideally, there should be minimal performance overhead for disabled trace events. Investigating scope for further optimization. Owner : Prerna Thanks, -- Prerna Saxena Linux Technology Centre, IBM Systems and Technology Lab, Bangalore, India