From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 63B1EC46CA1 for ; Mon, 18 Sep 2023 03:42:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230456AbjIRDmY (ORCPT ); Sun, 17 Sep 2023 23:42:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41412 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238080AbjIRDlw (ORCPT ); Sun, 17 Sep 2023 23:41:52 -0400 Received: from mail-pg1-x52d.google.com (mail-pg1-x52d.google.com [IPv6:2607:f8b0:4864:20::52d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DC3CE11C for ; Sun, 17 Sep 2023 20:41:46 -0700 (PDT) Received: by mail-pg1-x52d.google.com with SMTP id 41be03b00d2f7-577fff1cae6so2344322a12.1 for ; Sun, 17 Sep 2023 20:41:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=telus.net; s=google; t=1695008506; x=1695613306; darn=vger.kernel.org; h=thread-index:content-language:content-transfer-encoding :mime-version:message-id:date:subject:in-reply-to:references:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=/LChi52F8W125oBVQox7w/tsu4dA5p1CWPdYepEsEMA=; b=HeEhc5cj6uTCEweUUrxkp38SGf0W7FprBeZqJe342Avdd3+p4+eGyT5xM8PXvlUcNZ JwYhr7pyeceRXcPJlMzz4/X8WMT3aBN07xSreSh5OkpjfeG6WA30KpDuF8MXQyRy6eSt bNcC6S9wxmdk0hjuGO37yPENLvcnSyVSMTQ5fHN5KNexRtcvli5R253BkjGFfDQ2w6+g tKincdCyD5IEj0fUPMKob3N9dmnPDtIaci/TDYvGXkNujHX+VYlGL54um0DrhSUy5Y2B UMNzm3ifEsawCmUCqXBhvAdwyYpdowxti2xh6J0trnGoyBBH1Vp2dZbghzORWe13XVVJ T0aA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1695008506; x=1695613306; h=thread-index:content-language:content-transfer-encoding :mime-version:message-id:date:subject:in-reply-to:references:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=/LChi52F8W125oBVQox7w/tsu4dA5p1CWPdYepEsEMA=; b=tELPiGr3doEn7WC/WpdF/YjHBmLcbXnoTuOrwrbnnbeA78ydIyqRfthvjLTHBPgseu FASyNgGedoWBGmdQL4BoQ1UbVc9YCnHHfjRzQmle1CcsjciU1dUlW3nSw6cwRgT+nSKb ethibLhOpIESVxPpQpNxwiA3uQA7ZRWJkNd2sQt8uPMkxC8o7zUgdzNSJUr64XXufY+O hHYPwQUvBddFk71ZZQ8lPS1AfciQEml/draJNOIQUXUTcz7fcROgIn7amceVegdEnvI1 VqRbKsZDNgByLZRT/I40tMW77fB+poGblYLXg2ZjSa+gpigMpdOFSmMoo2+k5xR++SIT 8RNw== X-Gm-Message-State: AOJu0Yw+5m1DHBLCm2ZrsZO/IH3jfBMMIMbAHDSW69Cu0UtlSnLvw166 NoNQdy11DSq+PFzjcJz7UVigBw== X-Google-Smtp-Source: AGHT+IGn4FTGbH1QQhmBd7k+Pa26rEKOe+zz97KcSNNTL00qwRvLl5Krgy9F/0sHC6c9u6hycUN4ig== X-Received: by 2002:a17:90a:5d92:b0:26b:374f:97c2 with SMTP id t18-20020a17090a5d9200b0026b374f97c2mr10633316pji.6.1695008506328; Sun, 17 Sep 2023 20:41:46 -0700 (PDT) Received: from DougS18 (s66-183-142-209.bc.hsia.telus.net. [66.183.142.209]) by smtp.gmail.com with ESMTPSA id 10-20020a17090a08ca00b00262e604724dsm7900314pjn.50.2023.09.17.20.41.45 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 17 Sep 2023 20:41:45 -0700 (PDT) From: "Doug Smythies" To: "'Mario Limonciello'" Cc: , , "'Swapnil Sapkal'" , , "'Srinivas Pandruvada'" , , "Doug Smythies" References: <001701d9e9ae$02438120$06ca8360$@telus.net> In-Reply-To: Subject: RE: [PATCH] tools/power/x86/intel_pstate_tracer: python minimum version Date: Sun, 17 Sep 2023 20:41:49 -0700 Message-ID: <002201d9e9e2$0ea11a20$2be34e60$@telus.net> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Mailer: Microsoft Outlook 16.0 Content-Language: en-ca Thread-Index: AQHZ2qIP8m8g/xv1WN/ewv41PaKNWAIKXfLusA+3S2A= Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org On 2023.09.17 17:57 Mario wrote: > On 9/17/2023 16:29, Doug Smythies wrote: >> From: Doug Smythies >>=20 >> Change the minimum python version from 2.7 to 3.6. >> Remove a 2.X backwards compatibility line. >>=20 >> Signed-off-by: Doug Smythies >> --- >> .../x86/intel_pstate_tracer/intel_pstate_tracer.py | 10 = +++++----- >> 1 file changed, 5 insertions(+), 5 deletions(-) >>=20 >> diff --git = a/tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py = b/tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py >> index ec3323100e1a..6b0790e6465a 100755 >> --- a/tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py >> +++ b/tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py >> @@ -1,4 +1,4 @@ >> -#!/usr/bin/env python >> +#!/usr/bin/env python3 >> # SPDX-License-Identifier: GPL-2.0-only >> # -*- coding: utf-8 -*- >> # >> @@ -11,9 +11,9 @@ then this utility enables and collects trace data = for a user specified interval >> and generates performance plots. >>=20 >> Prerequisites: >> - Python version 2.7.x or higher >> + Python version 3.6.x or higher >> gnuplot 5.0 or higher >> - gnuplot-py 1.8 or higher >> + phython3-gnuplot 1.8 or higher > > Presumably you mean 'python3-gnuplot'. Oh Darn. Yes. I'll send a V2. >> (Most of the distributions have these required packages. They = may be called >> gnuplot-py, phython-gnuplot or phython3-gnuplot, gnuplot-nox, = ... ) >>=20 > > Likewise here too. > >> @@ -23,7 +23,7 @@ Prerequisites: >> see print_help(): for Usage and Output details >>=20 >> """ >> -from __future__ import print_function >> + >> from datetime import datetime >> import subprocess >> import os >> @@ -562,7 +562,7 @@ if __name__ =3D=3D "__main__": >>=20 >> # Temporary (or perhaps not) >> cur_version =3D sys.version_info >> - print('python version (should be >=3D 2.7):') >> + print('python version (should be >=3D 3.6):') >> print(cur_version) > > Is this block printing the version really needed at all? > I don't see 'cur_version' used anywhere else in the file. Well, it is there to make it easy to know what the requirements are and what the actual version is. The minimum version is not enforced. >> # Left as "cleanup" for potential future re-run ability. >> -- >> 2.25.1