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

Regression with last autofan update - All drives ignored #63

Open
Richardk2n opened this issue Oct 15, 2022 · 0 comments
Open

Regression with last autofan update - All drives ignored #63

Richardk2n opened this issue Oct 15, 2022 · 0 comments

Comments

@Richardk2n
Copy link

Richardk2n commented Oct 15, 2022

[[ -n $HDD ]] && SLEEPING=sdspin ${DISK}; echo $? || SLEEPING=hdparm -C ${DISK}|grep -c standby

This line is giving me trouble. (sorry for the formatting).
Apparently this never worked properly for me as hdparm always returns unknown status, which counts as not standby, and thus I did not notice.

The issue now is, that sdspin returns 1 for my HDDs (they never spin down) and 25 for my nvme (whatever this is supposed to mean) and thus all drives are excluded from the max temp calculation.

I understand there is little you can do about that (aside from maybe a config option to ignore the state of the disk; commenting this line fixed my issues).

What I would like very much is a check if all disks are excluded, because as it is now, all fans are set to 0 if no temp is read. This of course causes issues.

There is already a post on the support forum about this, so I guess multiple people are affected.

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

No branches or pull requests

1 participant