From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932179AbcFUShh (ORCPT ); Tue, 21 Jun 2016 14:37:37 -0400 Received: from mail-pa0-f47.google.com ([209.85.220.47]:34627 "EHLO mail-pa0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752010AbcFUShc (ORCPT ); Tue, 21 Jun 2016 14:37:32 -0400 From: David Carrillo-Cisneros To: linux-kernel@vger.kernel.org Cc: "x86@kernel.org" , Ingo Molnar , Andi Kleen , Kan Liang , Peter Zijlstra , David Carrillo-Cisneros Subject: [PATCH v02 0/5] fix MSR_LAST_BRANCH_FROM Haswell support Date: Tue, 21 Jun 2016 11:31:09 -0700 Message-Id: <1466533874-52003-1-git-send-email-davidcc@google.com> X-Mailer: git-send-email 2.8.0.rc3.226.g39d4020 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org commit 338b522ca43c ("perf/x86/intel: Protect LBR and extra_regs against KVM lying") introduced an extra test for LBR support but did not move the dmesg accordingly. This problem is fixed in first patch in this series. When a machine that used LBR is rebooted using kexec, the extra test for LBR support may fail due to a hw bug/quirk in Haswell that generates a #GPF when restoring a value of MSR_LAST_BRANCH_FROM_* msrs that has sign extension (e.g. kernel addresses). This hw bug/quirk currently does not manifest in the context switch of LBR callstack mode because of a workaround for another LBR bug (bug in FREEZE_LBRS_ON_PMI, more details in second patch of this series). The workaround deactivates LBR callstack in kernel mode. The second and fourth patches in this series contain workarounds for the MSR_LAST_BRANCH_FROM_* hw bug/quirk. The third patch contains a trivial format fix for aesthetic uniformity. The last patch is not to be committed, but to test the fourth patch. It removes the effect of the FREEZE_LBRS_ON_PMI work-around by allowing LBR callstack for kernel addresses. This series is rebased at torvalds/linux/master . Changes in 2nd version: - Remove branch from quirk (as pointed by Peter Z.). - Format fixes. David Carrillo-Cisneros (5): perf/x86/intel: output LBR support statement after validation perf/x86/intel: fix for MSR_LAST_BRANCH_FROM_x bug when no TSX perf/x86/intel: trivial format and style fix perf/x86/intel: MSR_LAST_BRANCH_FROM_x quirk for ctx switch not required, used to test ctxsw, do not merge arch/x86/events/intel/core.c | 20 ++++++++++ arch/x86/events/intel/lbr.c | 90 ++++++++++++++++++++++++++++++++++++-------- arch/x86/events/perf_event.h | 2 + tools/perf/util/evsel.c | 17 +++++++-- 4 files changed, 111 insertions(+), 18 deletions(-) -- 2.8.0.rc3.226.g39d4020