From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752024AbdBFMlW (ORCPT ); Mon, 6 Feb 2017 07:41:22 -0500 Received: from mail.skyhub.de ([78.46.96.112]:57582 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751979AbdBFMlV (ORCPT ); Mon, 6 Feb 2017 07:41:21 -0500 Date: Mon, 6 Feb 2017 13:41:16 +0100 From: Borislav Petkov To: Ingo Molnar Cc: Arnaldo Carvalho de Melo , Peter Zijlstra , Robert Richter , Vince Weaver , lkml Subject: Re: [RFC PATCH] perf/stat: Add --disable-hwdt Message-ID: <20170206124116.7jmrmufthqtnd3dc@pd.tnic> References: <20170206121506.mtknwusus4djp2sx@pd.tnic> <20170206122231.GA9404@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20170206122231.GA9404@gmail.com> User-Agent: NeoMutt/20161014 (1.7.1) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 06, 2017 at 01:22:31PM +0100, Ingo Molnar wrote: > Looks sensible, and I'd in fact make this the new default behavior (if root runs > perf stat) - i.e. add a flag to re-enable it, for the rare case where we want to > debug a hard deadlock while running perf stat ... I'd probably only need to save the previous state, in case it was disabled for whatever reason. So actually, I'll make it so that we restore the previous state at exit. Whatever it was. But while perf stat runs as root, we will disable it by default. /me scratches head a bit... Oh ok, even better - no need for a cmdline option then. Or does it *ever* make sense to have watchdog running *while* perf stat runs too? Because if it doesn't, turning it off by default *if* it was enabled [ should be, we do enable HW WDT by default ] and restoring it should be the proper thing to do. Without cmdline option. Thanks! -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.