[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140703073321.GU19379@twins.programming.kicks-ass.net>
Date: Thu, 3 Jul 2014 09:33:21 +0200
From: Peter Zijlstra <peterz@...radead.org>
To: kan.liang@...el.com
Cc: linux-kernel@...r.kernel.org, kvm@...r.kernel.org,
andi@...stfloor.org
Subject: Re: [PATCH V2 2/3] perf protect LBR when Intel PT is enabled.
On Wed, Jul 02, 2014 at 11:14:14AM -0700, kan.liang@...el.com wrote:
> From: Kan Liang <kan.liang@...el.com>
>
> If RTIT_CTL.TraceEn=1, any attempt to read or write the LBR or LER MSRs, including LBR_TOS, will result in a #GP.
> Since Intel PT can be enabled/disabled at runtime, LBR MSRs have to be protected by _safe() at runtime.
Lines are too long, and the reasoning is totally broken.
If there's active LBR users out there, we should refuse to enable PT and
vice versa. What we should not be doing is using _safe and fault and
generate crap.
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists