Re: [PATCH] tracing/markers: make markers select tracepoints

From: Frank Ch. Eigler
Date: Sun Feb 22 2009 - 12:38:30 EST


Hi -

On Sun, Feb 22, 2009 at 06:13:44PM +0100, Ingo Molnar wrote:
> [...]
> > It would be even easier converted to the markers API directly,
> > without the KVMTRACE* macro intermediary:
> >
> > before:
> > KVMTRACE_3D(MSR_READ, &svm->vcpu, ecx, (u32)data,
> > (u32)(data >> 32), handler);
> > after:
> > trace_mark(kvmtrace, "MSR_READ: %p, %08lx, %016Lx\n",
> > &svm->vcpu, ecx, data);
> >
> > All this already "just works".
> except that we are removing markers.

Perhaps that intention should be justified and reexamined, especially
considering the KVMTRACE* macro-replacement solution you suggested is
almost the same.


- FChE
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/