Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752354AbbEKQgX (ORCPT ); Mon, 11 May 2015 12:36:23 -0400 Received: from casper.infradead.org ([85.118.1.10]:57619 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751607AbbEKQgT (ORCPT ); Mon, 11 May 2015 12:36:19 -0400 Date: Mon, 11 May 2015 18:36:05 +0200 From: Peter Zijlstra To: Stephane Eranian Cc: Andi Kleen , LKML , Andi Kleen Subject: Re: [PATCH 9/9] x86, perf: Move PMU ACK after LBR read Message-ID: <20150511163605.GB21418@twins.programming.kicks-ass.net> References: <1431285767-27027-1-git-send-email-andi@firstfloor.org> <1431285767-27027-10-git-send-email-andi@firstfloor.org> <20150511162354.GA21418@twins.programming.kicks-ass.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2012-12-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1274 Lines: 31 On Mon, May 11, 2015 at 09:32:33AM -0700, Stephane Eranian wrote: > >> This is a minimal change. In principle the ACK could be moved much later. > > > > Right, so the more complete change would be to use the new and improved > > FREEZE_ON_PMI and reenable both the LBRs and the CTRs with the > > STATUS_RESET MSR, right? > > > > Does it make sense to have a new handle_irq() routine for that? > > Were we not already using FREEZE_ON_PMI with LBR (except for one > erratum on HSW)? That's FREEZE_LBRS_ON_PMI, I was referring to FREEZE_PERFMON_ON_PMI, which we've not used so far. I think Andi tried using it before, but there's some issues with it on v3, but v4 should have fixed all that. Andi can you perhaps explain what the problem with FREEZE_PERFMON_ON_PMI on v3 was again? > It would make sense to me to have an "optimized" and clean handle_irq > for the newer PMU. > We the caveat that any change to the core of it would now have to be done twice. We could pull that out in a shared function of course, if possible. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/