On Tue, Jul 07, 2009 at 01:06:00PM +0300, Avi Kivity wrote:
> On 07/06/2009 04:17 PM, Gleb Natapov wrote:
>> Add tracepoint in msi/ioapic/pic set_irq() functions,
>> in IPI sending and in the point where IRQ is placed into
>> apic's IRR.
>>
>> Signed-off-by: Gleb Natapov<gleb@xxxxxxxxxx>
>> diff --git a/arch/x86/kvm/i8259.c b/arch/x86/kvm/i8259.c
>> index 1d1bb75..e4bcbdd 100644
>> --- a/arch/x86/kvm/i8259.c
>> +++ b/arch/x86/kvm/i8259.c
>> @@ -30,6 +30,7 @@
>> #include "irq.h"
>>
>> #include<linux/kvm_host.h>
>> +#include "trace.h"
>>
>> static void pic_lock(struct kvm_pic *s)
>> __acquires(&s->lock)
>> @@ -190,6 +191,8 @@ int kvm_pic_set_irq(void *opaque, int irq, int level)
>> if (irq>= 0&& irq< PIC_NUM_PINS) {
>> ret = pic_set_irq1(&s->pics[irq>> 3], irq& 7, level);
>> pic_update_irq(s);
>> + trace_kvm_pic_set_irq(irq>> 3, irq& 7, s->pics[irq>> 3].elcr,
>> + s->pics[irq>> 3].imr, ret == 0);
>>
>
> It's better to send undecoded arguments (irq, s, ret) and do the
> shifting/masking/derefing in the TP_fast_assign. This is because
> argument preparation is always compiled inline and executed (so I
> understand; not sure) but TP_fast_assign is out-of-line and only
> executed if the tracepoint is enabled.
This will move knowledge how gsi is mapped to pic chips into tracing
code. This will break if we will add emulation to some fantastic HW with
number of pics different from 2.
--
Gleb.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
