Skip to content
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.

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

Add temperature support to NVME drives #377

Conversation

PartialVolume
Copy link
Collaborator

Patch #1 (first check) @Firminator If you could check whether the NVME temperatures now show up. If they don't, could you anonymize/verbose the log (nwipe -v -q) and paste the results.. Thanks

@PartialVolume PartialVolume merged commit 40ae117 into martijnvanbrummelen:master Nov 17, 2021
@PartialVolume PartialVolume deleted the add_temperature_support_to_nvme_drives branch November 17, 2021 14:45
@PartialVolume
Copy link
Collaborator Author

Thanks,

Can you type the following and post the result.

cd /sys/class/hwmon/hwmon3/device/nvme
ls -al

Thanks

@Firminator
Copy link
Contributor

Firminator commented Nov 17, 2021

total 0
drwxr-xr-x 3 root root 0 Nov 17 12:20 .
drwxr-xr-x 7 root root 0 Nov 17 12:15 ..
drwxr-xr-x 4 root root 0 Nov 17 12:20 nvme0

@PartialVolume
Copy link
Collaborator Author

And if you the do

cd /sys/class/hwmon/hwmon3/device/nvme/nvme0
ls -al

@Firminator
Copy link
Contributor

total 0
drwxr-xr-x 3 root root    0 Nov 17 12:15 .
drwxr-xr-x 3 root root    0 Nov 17 12:15 ..
lrwxrwxrwx 1 root root    0 Nov 17 12:15 device -> ../../../0000:3d:00.0
-r--r--r-- 1 root root 4096 Nov 17 12:15 name
drwxr-xr-x 2 root root    0 Nov 17 12:45 power
lrwxrwxrwx 1 root root    0 Nov 17 12:45 subsystem -> ../../../../../../class/hwmon
-r--r--r-- 1 root root 4096 Nov 17 12:15 temp1_alarm
-r--r--r-- 1 root root 4096 Nov 17 12:15 temp1_input
-r--r--r-- 1 root root 4096 Nov 17 12:15 temp1_label
-rw-r--r-- 1 root root 4096 Nov 17 12:45 uevent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants