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

fix: allow scttempsts probing method for asleep drives #56

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

georgeto
Copy link
Contributor

@georgeto georgeto commented Aug 3, 2024

The smartctl scttempsts probing method works for asleep drives, without waking them up.
Stated here and it matches the observations from my own experiments.

PS:
With smartctl,5 -l devstat /dev/sda there seems to be an additional way to query the driver temperature without waking asleep drives. This command queries the Temperature Statistics log page from the Device Statistics log (see ATA/ATAPI Command Set).

0x05  =====  =               =  ===  == Temperature Statistics (rev 1) ==
0x05  0x008  1              29  ---  Current Temperature
0x05  0x010  1              29  ---  Average Short Term Temperature
0x05  0x018  1               -  ---  Average Long Term Temperature
0x05  0x020  1              41  ---  Highest Temperature
0x05  0x028  1              25  ---  Lowest Temperature
0x05  0x030  1              37  ---  Highest Average Short Term Temperature
0x05  0x038  1              29  ---  Lowest Average Short Term Temperature
0x05  0x040  1               -  ---  Highest Average Long Term Temperature
0x05  0x048  1               -  ---  Lowest Average Long Term Temperature
0x05  0x050  4               0  ---  Time in Over-Temperature
0x05  0x058  1              60  ---  Specified Maximum Operating Temperature
0x05  0x060  4               0  ---  Time in Under-Temperature
0x05  0x068  1               5  ---  Specified Minimum Operating Temperature

The smartctl scttempsts probing method works for asleep drives,
without waking them up.
@desbma desbma merged commit 39ca614 into desbma:master Aug 5, 2024
6 checks passed
@desbma
Copy link
Owner

desbma commented Aug 5, 2024

Thank you.

I have made a similar change in the v2 branch.

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.

2 participants