Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 22 Aug 2018 13:57:07 -0700
From:   Eduardo Valentin <eduval@amazon.com>
To:     Andi Kleen <ak@linux.intel.com>
CC:     Eduardo Valentin <eduval@amazon.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>,
        "H. Peter Anvin" <hpa@zytor.com>, <x86@kernel.org>,
        Peter Zijlstra <peterz@infradead.org>,
        "Kan Liang" <kan.liang@linux.intel.com>,
        Dan Carpenter <dan.carpenter@oracle.com>,
        Jia Zhang <qianyue.zj@alibaba-inc.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/1] perf/x86/intel: make error messages less confusing
Message-ID: <20180822205707.GA16015@u40b0340c692b58f6553c.ant.amazon.com>
References: <20180821211528.8575-1-eduval@amazon.com>
 <20180821220937.GV12066@tassilo.jf.intel.com>
 <20180821230522.GA10816@u40b0340c692b58f6553c.ant.amazon.com>
 <20180821235908.GW12066@tassilo.jf.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20180821235908.GW12066@tassilo.jf.intel.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hey,

On Tue, Aug 21, 2018 at 04:59:08PM -0700, Andi Kleen wrote:
> On Tue, Aug 21, 2018 at 04:05:22PM -0700, Eduardo Valentin wrote:
> > On Tue, Aug 21, 2018 at 03:09:37PM -0700, Andi Kleen wrote:
> > > On Tue, Aug 21, 2018 at 02:15:28PM -0700, Eduardo Valentin wrote:
> > > > On a system with X86_FEATURE_ARCH_PERFMON disabled
> > > > and with a model not known by family PMU drivers,
> > > > user gets a kernel message log like the following:
> > > > [ 0.100114] Performance Events: unsupported p6 CPU model 85 no PMU driver, software events only.
> > > > 
> > > > The "unsupported .. CPU" part may be confusing for some
> > > > users. Rewording the messages on the failure path to:
> > > > [ 0.667154] Performance Events: unknown p6 PMU on CPU model 85: !X86_FEATURE_ARCH_PERFMON: no PMU driver, software events only.
> > > 
> > > Are you sure users even know what ARCH_PERFMON is?
> > > 
> > > Maybe it is confusing (why exactly?), but it doesn't seem to me that your
> > > new message is any better.
> > 
> > Yeah, the part that says "unsupported CPU" is the confusing part,
> 
> That makes sense.
> 
> > I get people thinking that the specific reported CPU model is not
> > supported by the kernel :-)
> > 
> > > 
> > > If you refer to VMs not exposing the PMU perhaps that should be explicitely mentioned.
> > > 
> > > Of course the real fix is to always expose the PMU, not improve the error messages...
> > 
> > I agree that best is simply to enable PMU. But it does not hurt to improve the error messaging, does it?
> > 
> > Any suggestions there, given that the initial attempt seams to make it even worse :-)
> 
> Perhaps just say
> 
> "CPU does not support PMU"
>

Ok.
 
> which is really what the problem is here.
> 

Fair enough.

> The other option would be to move this message after the big model switch,
> but would need to be very careful that it doesn't have any unintended
> side effects. 

I will simply remove those messages with CPU model details and on
the failure path build the message to look like:
[    0.666785] Performance Events: CPU does not support PMU: no PMU driver, software events only.


> 
> -Andi
> 

-- 
All the best,
Eduardo Valentin