Skip to content

Commit

Permalink
Fix trace.h
Browse files Browse the repository at this point in the history
It looks like the variable "pc" is defined. At least the current code always
failed on me stating that "pc" is already defined somewhere else.

Let's use _pc instead, because that doesn't collide.

Is this the right approach? Does it break on 440 too? If not, why not?

Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
  • Loading branch information
Alexander Graf authored and Benjamin Herrenschmidt committed Nov 5, 2009
1 parent c4f9c77 commit 346b276
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions arch/powerpc/kvm/trace.h
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@
* Tracepoint for guest mode entry.
*/
TRACE_EVENT(kvm_ppc_instr,
TP_PROTO(unsigned int inst, unsigned long pc, unsigned int emulate),
TP_ARGS(inst, pc, emulate),
TP_PROTO(unsigned int inst, unsigned long _pc, unsigned int emulate),
TP_ARGS(inst, _pc, emulate),

TP_STRUCT__entry(
__field( unsigned int, inst )
Expand All @@ -23,7 +23,7 @@ TRACE_EVENT(kvm_ppc_instr,

TP_fast_assign(
__entry->inst = inst;
__entry->pc = pc;
__entry->pc = _pc;
__entry->emulate = emulate;
),

Expand Down

0 comments on commit 346b276

Please sign in to comment.