[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [ltt-dev] [PATCH 2/3] trace: [ust] Do not use 'm' in ev
Re: [Qemu-devel] [ltt-dev] [PATCH 2/3] trace: [ust] Do not use 'm' in event argument names (used by ust macros)
Tue, 20 Sep 2011 10:31:12 -0400
* Lluís Vilanova (address@hidden) wrote:
> Stefan Hajnoczi writes:
> > On Fri, Sep 16, 2011 at 06:59:38PM +0200, Lluís Vilanova wrote:
> >> Signed-off-by: Lluís Vilanova <address@hidden>
> >> ---
> >> trace-events | 4 ++--
> >> 1 files changed, 2 insertions(+), 2 deletions(-)
> >> diff --git a/trace-events b/trace-events
> >> index 9d1fbbb..b653d70 100644
> >> --- a/trace-events
> >> +++ b/trace-events
> >> @@ -418,8 +418,8 @@ milkymist_pfpu_pulse_irq(void) "Pulse IRQ"
> >> # hw/milkymist-softusb.c
> >> milkymist_softusb_memory_read(uint32_t addr, uint32_t value) "addr %08x
> >> value %08x"
> >> milkymist_softusb_memory_write(uint32_t addr, uint32_t value) "addr %08x
> >> value %08x"
> >> -milkymist_softusb_mevt(uint8_t m) "m %d"
> >> -milkymist_softusb_kevt(uint8_t m) "m %d"
> >> +milkymist_softusb_mevt(uint8_t _m) "m %d"
> >> +milkymist_softusb_kevt(uint8_t _m) "m %d"
> > The LTTng community has been very responsive in addressing namespace
> > issues with libust. Let's post more details and see if it can be fixed
> > in libust.
> > Could you please post your gcc and libust versions?
> > I have not been able to reproduce the problem on Debian libust-dev
> > 0.15-3. My gcc version is Debian gcc 4.6.1-4.
> Yup, I was using libust-dev 0.5 (debian stable). After switching to 0.15
> testing) other problems arise.
> For example, in my box compiling osdep.c yields lots of "variable
> set but not used", but using a minimal example does not raise such problems,
> I'm assuming this is related to some namespace problems related to some other
> code in qemu's headers.
If you can find a few minutes to provide:
1) a link to the git repository/commit ID you are testing
2) the exact configuration you use (detailed way to reproduce the
problem, as a sequence of commands from a pristine repository)
It will allow me to look into this warning and fix it.
> Unfortunately, right now I have no time to debug the causes, so I'll just
> the two patches I sent (this naming issue and the zero-length printf format)
> just no longer necessary with recent versions of libust.
> "And it's much the same thing with knowledge, for whenever you learn
> something new, the whole world becomes that much richer."
> -- The Princess of Pure Reason, as told by Norton Juster in The Phantom
> ltt-dev mailing list
Operating System Efficiency R&D Consultant
Re: [Qemu-devel] [PATCH 2/3] trace: [ust] Do not use 'm' in event argument names (used by ust macros), Stefan Hajnoczi, 2011/09/20
[Qemu-devel] [PATCH 3/3] trace: [ust] Do not use zero-length format strings (avoid compiler warning), Lluís Vilanova, 2011/09/16
Re: [Qemu-devel] [PATCH 1/3] trace: Fix print format for "mipsnet_write", Stefan Hajnoczi, 2011/09/20