[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20130831012327.22813171@gandalf.local.home>
Date: Sat, 31 Aug 2013 01:23:27 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Borislav Petkov <bp@...en8.de>
Cc: "H. Peter Anvin" <hpa@...or.com>,
Jan Kiszka <jan.kiszka@...mens.com>,
Fenghua Yu <fenghua.yu@...el.com>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
x86 <x86@...nel.org>, Ingo Molnar <mingo@...e.hu>,
Thomas Gleixner <tglx@...utronix.de>
Subject: Re: x86-32: Early microcode loading stumbles over disabled
DYNAMIC_FTRACE
On Sat, 31 Aug 2013 07:07:37 +0200
Borislav Petkov <bp@...en8.de> wrote:
> On Fri, Aug 30, 2013 at 08:17:41PM -0400, Steven Rostedt wrote:
> > On Fri, 30 Aug 2013 13:48:55 -0700
> > "H. Peter Anvin" <hpa@...or.com> wrote:
> >
> > > >> You *could* bail out of the calling address is < PAGE_OFFSET.
> > >
> > > Anyway... can we add this to the static tracing code? It is two
> > > instructions and is only needed on x86-32. For performance, use dynamic.
> >
> > Sure, do you want to add it ?
>
> Another, simpler option could be to make static tracing depend on
> !MICROCODE until the first has been removed.
>
Might be trickier than you think. Static tracing is the default.
Dynamic tracing is the extension. By making static tracing depend
on !MICROCODE, you basically made function tracing depend on !MICROCODE.
I guess you could make MICROCODE select DYNAMIC_FTRACE, but that could
get messy.
-- Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists