From: Clark Williams <williams@redhat.com>
To: John Kacur <jkacur@redhat.com>
Cc: rt-users <linux-rt-users@vger.kernel.org>
Subject: Re: [PATCH] rt-tests: hwlatdetect.py: Code clean-up
Date: Mon, 13 May 2019 17:06:18 -0500 [thread overview]
Message-ID: <20190513170618.7dfd59ba@torg> (raw)
In-Reply-To: <20190510140144.6817-1-jkacur@redhat.com>
On Fri, 10 May 2019 16:01:44 +0200
John Kacur <jkacur@redhat.com> wrote:
> - Remove obsolete from __future__
> - Fix spacing around calls to print, open and brackets
> - Fix spacing around assignments
>
> Signed-off-by: John Kacur <jkacur@redhat.com>
> ---
> src/hwlatdetect/hwlatdetect.py | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/src/hwlatdetect/hwlatdetect.py b/src/hwlatdetect/hwlatdetect.py
> index 368079a158b1..b72bdc3aa930 100755
> --- a/src/hwlatdetect/hwlatdetect.py
> +++ b/src/hwlatdetect/hwlatdetect.py
> @@ -8,7 +8,7 @@
> # modify it under the terms of the GNU General Public License Version 2
> # as published by the Free Software Foundation.
>
> -from __future__ import print_function
> +
>
> import sys
> import os
> @@ -84,7 +84,7 @@ class DebugFS(object):
> try:
> val = f.readline()
> except OSError as e:
> - print ("errno: %s" % e)
> + print("errno: %s" % e)
> if e.errno == errno.EAGAIN:
> val = None
> else:
> @@ -159,7 +159,7 @@ class Kmod(object):
> return
>
> # now look for already loaded module
> - for l in open ('/proc/modules'):
> + for l in open('/proc/modules'):
> field = l.split()
> if self.name in field[0]:
> self.preloaded = True
> @@ -210,7 +210,7 @@ class Detector(object):
> if self.have_msr:
> p = subprocess.Popen(['/usr/sbin/rdmsr', '-a', '-d', '0x34'], stdout=subprocess.PIPE)
> p.wait()
> - counts = [ int(x.strip()) for x in p.stdout.readlines()]
> + counts = [int(x.strip()) for x in p.stdout.readlines()]
> return counts
>
> # methods for preventing/enabling c-state transitions
> @@ -295,11 +295,11 @@ class Tracer(Detector):
>
> class Sample(object):
> 'private class for tracer sample data'
> - __slots__= 'timestamp', 'inner', 'outer',
> + __slots__ = 'timestamp', 'inner', 'outer',
> def __init__(self, line):
> fields = line.split()
> i,o = fields[6].split('/')
> - ts=fields[7][3:]
> + ts = fields[7][3:]
> self.timestamp = str(ts)
> self.inner = int(i)
> self.outer = int(o)
> @@ -332,14 +332,14 @@ class Tracer(Detector):
> self.set('current_tracer', 'hwlat')
>
> def set(self, field, val):
> - path=self.translate(field)
> + path = self.translate(field)
> self.debugfs.putval(path, str(val))
>
> def get(self, field):
> if field == "count":
> return len(self.samples)
> elif field == "max":
> - max=0
> + max = 0
> for values in self.samples:
> s = int(values.largest())
> if s > max:
> @@ -435,7 +435,7 @@ class Hwlat(Detector):
>
> def display(self):
> for s in self.samples:
> - print (s)
> + print(s)
>
> def save(self, output=None):
> if output:
> --
> 2.20.1
>
Acked-by: Clark Williams <williams@redhat.com>
--
The United States Coast Guard
Ruining Natural Selection since 1790
prev parent reply other threads:[~2019-05-13 22:06 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-10 14:01 [PATCH] rt-tests: hwlatdetect.py: Code clean-up John Kacur
2019-05-13 22:06 ` Clark Williams [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190513170618.7dfd59ba@torg \
--to=williams@redhat.com \
--cc=jkacur@redhat.com \
--cc=linux-rt-users@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).