Skip to content

Skip one-wire bus temperature sensors; reading takes a lot of time and conflicts with other readouts #2452

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Skip one-wire bus temperature sensors; reading takes a lot of time and conflicts with other readouts #2452

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 13 additions & 6 deletions psutil/_pslinux.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1410,10 +1410,15 @@ def sensors_temperatures():

for base in basenames:
try:
path = base + '_input'
current = float(bcat(path)) / 1000.0
# First find the name. Skip if it's a w1-bus sensor as reading such sensors will
# take a lot of time, and makes no sense for us
path = os.path.join(os.path.dirname(base), 'name')
unit_name = cat(path).strip()
if unit_name == "w1_slave_temp":
continue
# If it was not a w1-bus sensor it's ours.
path = base + '_input'
current = float(bcat(path)) / 1000.0
except (IOError, OSError, ValueError):
# A lot of things can go wrong here, so let's just skip the
# whole entry. Sure thing is Linux's /sys/class/hwmon really
Expand Down Expand Up @@ -1458,10 +1463,12 @@ def sensors_temperatures():
continue

trip_paths = glob.glob(base + '/trip_point*')
trip_points = set([
'_'.join(os.path.basename(p).split('_')[0:3])
for p in trip_paths
])
trip_points = set(
[
'_'.join(os.path.basename(p).split('_')[0:3])
for p in trip_paths
]
)
critical = None
high = None
for trip_point in trip_points:
Expand Down