From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755318Ab0LHOjv (ORCPT ); Wed, 8 Dec 2010 09:39:51 -0500 Received: from mx1.redhat.com ([209.132.183.28]:57297 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755248Ab0LHOju (ORCPT ); Wed, 8 Dec 2010 09:39:50 -0500 Date: Wed, 8 Dec 2010 09:39:22 -0500 From: Vivek Goyal To: Don Zickus Cc: Peter Zijlstra , "Eric W. Biederman" , Yinghai Lu , Ingo Molnar , Jason Wessel , "linux-kernel@vger.kernel.org" , Haren Myneni Subject: Re: perf hw in kexeced kernel broken in tip Message-ID: <20101208143922.GA31703@redhat.com> References: <20101201194644.GD2511@redhat.com> <1291232989.32004.1987.camel@laptop> <20101201195835.GE2511@redhat.com> <1291234036.32004.2008.camel@laptop> <20101202052321.GH18100@redhat.com> <1291275270.4023.20.camel@twins> <20101202161502.GL18100@redhat.com> <1291764620.2032.1293.camel@laptop> <20101208140103.GM21786@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101208140103.GM21786@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 08, 2010 at 09:01:03AM -0500, Don Zickus wrote: > On Wed, Dec 08, 2010 at 12:30:20AM +0100, Peter Zijlstra wrote: > > On Thu, 2010-12-02 at 11:15 -0500, Don Zickus wrote: > > > > > Vivek suggested to me this morning that I should just blantantly disable the > > > perf counter during init when running my test. > > > > Nah, we should actively scan for that during the bring-up and kill > > hw-perf when we find an enable bit set, some BIOSes actively use the > > PMU, this is something that should be discouraged. > > Ok, the reboot notifier addresses the kexec problem but doesn't fix it > though (I have to test to confirm that, comments below). The bios check > should catch those situations (ironically I stumbled upon a machine with > this problem, so I will test your patch with it, though it only uses perf > counter 0). The kdump problem will still exist, not sure if we care and > perhaps we should document in the changelog that we know kdump is still > broken (unless we do care). Can't think why would somebody like to use performance counters in kdump kernel. So that probably should not be a concern. Vivek