Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 1 | /** |
| 2 | * Copyright 2017 Google Inc. |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 17 | #include "pidthread.hpp" |
| 18 | |
James Feist | 22c257a | 2018-08-31 14:07:12 -0700 | [diff] [blame] | 19 | #include "pid/pidcontroller.hpp" |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 20 | #include "pid/tuning.hpp" |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 21 | #include "sensors/sensor.hpp" |
| 22 | |
Patrick Venture | da4a5dd | 2018-08-31 09:42:48 -0700 | [diff] [blame] | 23 | #include <chrono> |
| 24 | #include <map> |
| 25 | #include <memory> |
| 26 | #include <thread> |
| 27 | #include <vector> |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 28 | |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 29 | static void processThermals(PIDZone* zone) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 30 | { |
| 31 | // Get the latest margins. |
| 32 | zone->updateSensors(); |
| 33 | // Zero out the RPM set point goals. |
| 34 | zone->clearRPMSetPoints(); |
James Feist | 608304d | 2019-02-25 10:01:42 -0800 | [diff] [blame] | 35 | zone->clearRPMCeilings(); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 36 | // Run the margin PIDs. |
Patrick Venture | 563a356 | 2018-10-30 09:31:26 -0700 | [diff] [blame] | 37 | zone->processThermals(); |
Patrick Venture | 7280e27 | 2019-02-11 10:45:32 -0800 | [diff] [blame] | 38 | // Get the maximum RPM setpoint. |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 39 | zone->determineMaxRPMRequest(); |
| 40 | } |
| 41 | |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 42 | void pidControlThread(PIDZone* zone) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 43 | { |
| 44 | int ms100cnt = 0; |
| 45 | /* |
| 46 | * This should sleep on the conditional wait for the listen thread to tell |
| 47 | * us it's in sync. But then we also need a timeout option in case |
| 48 | * phosphor-hwmon is down, we can go into some weird failure more. |
| 49 | * |
| 50 | * Another approach would be to start all sensors in worst-case values, |
| 51 | * and fail-safe mode and then clear out of fail-safe mode once we start |
| 52 | * getting values. Which I think it is a solid approach. |
| 53 | * |
| 54 | * For now this runs before it necessarily has any sensor values. For the |
| 55 | * host sensors they start out in fail-safe mode. For the fans, they start |
| 56 | * out as 0 as input and then are adjusted once they have values. |
| 57 | * |
| 58 | * If a fan has failed, it's value will be whatever we're told or however |
| 59 | * we retrieve it. This program disregards fan values of 0, so any code |
| 60 | * providing a fan speed can set to 0 on failure and that fan value will be |
| 61 | * effectively ignored. The PID algorithm will be unhappy but nothing bad |
| 62 | * will happen. |
| 63 | * |
| 64 | * TODO(venture): If the fan value is 0 should that loop just be skipped? |
Patrick Venture | 563a356 | 2018-10-30 09:31:26 -0700 | [diff] [blame] | 65 | * Right now, a 0 value is ignored in FanController::inputProc() |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 66 | */ |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 67 | if (tuningLoggingEnabled) |
| 68 | { |
| 69 | zone->initializeLog(); |
| 70 | } |
| 71 | |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 72 | zone->initializeCache(); |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 73 | processThermals(zone); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 74 | |
| 75 | while (true) |
| 76 | { |
| 77 | using namespace std::literals::chrono_literals; |
| 78 | std::this_thread::sleep_for(0.1s); |
| 79 | |
| 80 | // Check if we should just go back to sleep. |
| 81 | if (zone->getManualMode()) |
| 82 | { |
| 83 | continue; |
| 84 | } |
| 85 | |
| 86 | // Get the latest fan speeds. |
| 87 | zone->updateFanTelemetry(); |
| 88 | |
| 89 | if (10 <= ms100cnt) |
| 90 | { |
| 91 | ms100cnt = 0; |
| 92 | |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 93 | processThermals(zone); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 94 | } |
| 95 | |
| 96 | // Run the fan PIDs every iteration. |
Patrick Venture | 563a356 | 2018-10-30 09:31:26 -0700 | [diff] [blame] | 97 | zone->processFans(); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 98 | |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 99 | if (tuningLoggingEnabled) |
| 100 | { |
| 101 | zone->getLogHandle() << "," << zone->getFailSafeMode(); |
| 102 | zone->getLogHandle() << std::endl; |
| 103 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 104 | |
| 105 | ms100cnt += 1; |
| 106 | } |
| 107 | |
| 108 | return; |
| 109 | } |