Re: [PATCH v9 2/3] trace,x86: add x86 irq vector tracepoints

From: Steven Rostedt
Date: Mon Feb 18 2013 - 23:41:30 EST


On Mon, 2013-02-18 at 20:39 -0800, H. Peter Anvin wrote:

> > static void switch_to_trace_idt(void *arg)
> > {
> > load_idt(&trace_idt_descr);
> > }
> >
> > static void restore_original_idt(void *arg)
> > {
> > load_idt(this_cpu_ptr(&idt_descr));
> > }
> >
>
> Yes. If there needs to be handshaking about whose IDT is currently
> installed I'm much happier if that handshake is done explicitly, rather
> than as a save/restore function which will break when we have to add
> another one which for whatever reason isn't nesting.

Agreed.

Seiji, can you make the update?

Thanks,

-- Steve


--
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/