[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output |
Date: |
Sun, 17 Feb 2013 14:26:14 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
Il 15/02/2013 14:14, Alexander Graf ha scritto:
>> >
>> > I'm assuming that changes to ppc logging go through ppc-next, changes to
>> > sparc code go through Blue etc. All those potentially conflict since
>> > they're adding to the bottom of the same text file, thus coordination.
>> > It's a long-standing criticism of our centralistic tracing
>> > infrastructure compared to the totally decentral and inhomogeneous
>> > DPRINTFs and what-nots on the other hand.
I rarely found conflicts in trace-events, but in this case you would
indeed have them. Trivial to solve, however.
> In parallel to the completely disastrous user experience when using trace
> points.
What exactly is disastrous? We could have improvements (such as having
multiple trace backends compiled in an executable, e.g. simple+stderr,
or having gdb magic to enable/disable them from command breakpoints).
But in general, I find trace points to be quite easy to use, and
unintrusive enough to let you debug performance problems.
Paolo
- [Qemu-devel] [PATCH for-next v3 0/5] qtest: tmp105 cleanups and MMIO support, Andreas Färber, 2013/02/11
- [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/11
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/14
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Stefan Hajnoczi, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Stefan Hajnoczi, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Alexander Graf, 2013/02/15
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Andreas Färber, 2013/02/16
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output, Alexander Graf, 2013/02/25
- Re: [Qemu-devel] [PATCH for-next v3 2/5] tmp105: Add debug output,
Paolo Bonzini <=
[Qemu-devel] [PATCH for-next v3 4/5] libqtest: Introduce qtest_qmpv() and convert remaining macro, Andreas Färber, 2013/02/11
[Qemu-devel] [PATCH for-next v3 3/5] libqtest: Convert macros to functions and clean up documentation, Andreas Färber, 2013/02/11
[Qemu-devel] [PATCH for-next v3 1/5] tmp105-test: Combine assertions of 16-bit responses, Andreas Färber, 2013/02/11
[Qemu-devel] [PATCH for-next v3 5/5] qtest: Add MMIO support, Andreas Färber, 2013/02/11