Skip to content

Not enough fans #39

@alexforencich

Description

@alexforencich

On my 7920 (workstation tower), there are at least 10 fans. I added this to fan_speed in i8kctl.c:

for (int x = 0; x < 20; x++) printf("%d: %d\n", x, i8k_get_fan_speed(x));

And with that modification, when I run i8kctl speed I get:

$ ./i8kctl speed
0: 0
1: 819
2: 787
3: 777
4: 800
5: 808
6: 788
7: 802
8: 1502
9: 1843
10: 0
11: 0
12: 0
13: 0
14: 0
15: 0
16: 0
17: 0
18: 0
19: 0
819 0

Running the same on a 7820 also shows several additional fans:

$ ./i8kctl speed
0: 0
1: 0
2: 694
3: 971
4: 1037
5: 815
6: 810
7: 0
8: 0
9: 0
10: 0
11: 0
12: 0
13: 0
14: 0
15: 0
16: 0
17: 0
18: 0
19: 0
0 0

So, both the kernel module and the userspace tools should be updated to support a lot more fans. Not sure how many would be the max across all Dell machines, and I have not looked at temp sensors or PWM controls at all. Perhaps it makes sense to autodetect the number of fans that are actually present in some way.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions