RE: [PATCH v16 4/4] trace,x86: Add irq vector tracepoints

From: Seiji Aguchi
Date: Wed Jun 19 2013 - 17:02:26 EST



> On Tue, 2013-06-18 at 15:01 -0400, Seiji Aguchi wrote:
> > /*
> > * the load_current_idt() is called with interrupt disabled by
> > local_irq_save()
> > * to avoid races. That way the IDT will always be set back to the
> > expected
> > @@ -442,6 +508,8 @@ static inline void load_current_idt(void)
> > local_irq_save(flags);
> > if (is_debug_idt_enabled())
> > load_debug_idt();
> > + if (is_trace_idt_enabled())
> > + load_trace_idt();
>
> This should be:
>
> else if (is_trace_idt_enabled())
>
> Otherwise, you just totally negated the previous if.

Thank you for testing my patch.
I will fix it.

Seiji
èº{.nÇ+‰·Ÿ®‰­†+%ŠËlzwm…ébëæìr¸›zX§»®w¥Š{ayºÊÚë,j­¢f£¢·hš‹àz¹®w¥¢¸ ¢·¦j:+v‰¨ŠwèjØm¶Ÿÿ¾«‘êçzZ+ƒùšŽŠÝj"ú!¶iO•æ¬z·švØ^¶m§ÿðà nÆàþY&—