Steven, Thank you for reviewing.
I will create a patch just renaming variables and adding the is_debug_idt_enabled() and load_debug_idt(). Seiji > -----Original Message----- > From: Steven Rostedt [mailto:[email protected]] > Sent: Thursday, June 06, 2013 4:01 PM > To: Seiji Aguchi > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; Tomoki Sekiyama > Subject: Re: [PATCH v14 3/3] trace,x86: Add irq vector tracepoints > > On Thu, 2013-06-06 at 15:18 -0400, Steven Rostedt wrote: > > On Thu, 2013-06-06 at 11:24 -0400, Seiji Aguchi wrote: > > > Since we are basically down to cosmetic changes, can you make the > > renames of these variables into a separate patch. As they are now more > > about the debug IDT than a NMI idt, it should have been done before, and > > is a separate change to what you are doing here. > > I would even add the is_debug_idt_enabled() and load_debug_idt() > interface to that patch as well. > > -- Steve >
