Bump version

CHANGELOG.md

@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+
+## [v5.0.8]
 ### Changed
 - Reintroduce persisting dark/light mode in config file
 - Added ability to change what EPP is linked with each throttle profile

Cargo.lock

@@ -199,7 +199,7 @@ checksum = "96d30a06541fbafbc7f82ed10c06164cfbd2c401138f6addd8404629c4b16711"
 
 [[package]]
 name = "asusctl"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "asusd",
  "cargo-husky",
@@ -219,7 +219,7 @@ dependencies = [
 
 [[package]]
 name = "asusd"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "cargo-husky",
  "concat-idents",
@@ -244,7 +244,7 @@ dependencies = [
 
 [[package]]
 name = "asusd-user"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "cargo-husky",
  "config-traits",
@@ -847,7 +847,7 @@ dependencies = [
 
 [[package]]
 name = "config-traits"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "cargo-husky",
  "log",
@@ -900,7 +900,7 @@ dependencies = [
 
 [[package]]
 name = "cpuctl"
-version = "5.0.8-RC4"
+version = "5.0.8"
 
 [[package]]
 name = "cpufeatures"
@@ -1024,7 +1024,7 @@ dependencies = [
 
 [[package]]
 name = "dmi_id"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "log",
  "udev",
@@ -2834,7 +2834,7 @@ checksum = "c08c74e62047bb2de4ff487b251e4a92e24f48745648451635cec7d591162d9f"
 
 [[package]]
 name = "rog-control-center"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "asusd",
  "cargo-husky",
@@ -2867,7 +2867,7 @@ dependencies = [
 
 [[package]]
 name = "rog_anime"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "cargo-husky",
  "dmi_id",
@@ -2884,7 +2884,7 @@ dependencies = [
 
 [[package]]
 name = "rog_aura"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "cargo-husky",
  "dmi_id",
@@ -2898,7 +2898,7 @@ dependencies = [
 
 [[package]]
 name = "rog_dbus"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "asusd",
  "cargo-husky",
@@ -2911,7 +2911,7 @@ dependencies = [
 
 [[package]]
 name = "rog_platform"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "cargo-husky",
  "concat-idents",
@@ -2928,7 +2928,7 @@ dependencies = [
 
 [[package]]
 name = "rog_profiles"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "cargo-husky",
  "log",
@@ -2942,7 +2942,7 @@ dependencies = [
 
 [[package]]
 name = "rog_simulators"
-version = "5.0.8-RC4"
+version = "5.0.8"
 dependencies = [
  "glam",
  "log",

Cargo.toml

@@ -4,7 +4,7 @@ default-members = ["asusctl", "asusd", "asusd-user", "cpuctl", "rog-control-cent
 resolver = "2"
 
 [workspace.package]
-version = "5.0.8-RC4"
+version = "5.0.9"
 rust-version = "1.75"
 
 [workspace.dependencies]

asusctl/src/main.rs

@@ -20,7 +20,6 @@ use rog_dbus::RogDbusClientBlocking;
 use rog_platform::error::PlatformError;
 use rog_platform::platform::{GpuMode, Properties, ThrottlePolicy};
 use rog_profiles::error::ProfileError;
-use rog_profiles::fan_curve_set::CurveData;
 
 use crate::aura_cli::{AuraPowerStates, LedBrightness};
 use crate::cli_opts::*;

asusd/src/ctrl_fancurves.rs

@@ -21,9 +21,7 @@ pub const FAN_CURVE_ZBUS_PATH: &str = "/org/asuslinux/FanCurves";
 
 #[derive(Deserialize, Serialize, Debug, Default)]
 pub struct FanCurveConfig {
-    pub balanced: Vec<CurveData>,
+    pub profiles: FanCurveProfiles,
-    pub performance: Vec<CurveData>,
-    pub quiet: Vec<CurveData>,
     #[serde(skip)]
     pub current: u8,
 }
@@ -49,7 +47,6 @@ impl StdConfigLoad for FanCurveConfig {}
 #[derive(Debug, Clone)]
 pub struct CtrlFanCurveZbus {
     config: Arc<Mutex<FanCurveConfig>>,
-    fan_curves: Arc<Mutex<FanCurveProfiles>>,
     platform: RogPlatform,
 }
 
@@ -64,8 +61,8 @@ impl CtrlFanCurveZbus {
             let mut config = FanCurveConfig::new();
             let mut fan_curves = FanCurveProfiles::default();
 
-            // Only do defaults if the config doesn't already exist
+            // Only do defaults if the config doesn't already exist\
-            if !config.file_path().exists() {
+            if config.profiles.balanced.is_empty() || !config.file_path().exists() {
                 info!("{MOD_NAME}: Fetching default fan curves");
 
                 for this in [
@@ -82,45 +79,28 @@ impl CtrlFanCurveZbus {
                     let active = platform
                         .get_throttle_thermal_policy()
                         .map_or(ThrottlePolicy::Balanced, |t| t.into());
+                    fan_curves.read_from_dev_profile(active, &find_fan_curve_node()?)?;
 
                     info!("{MOD_NAME}: {active:?}:");
                     for curve in fan_curves.get_fan_curves_for(active) {
                         info!("{}", String::from(curve));
                     }
                 }
+                config.profiles = fan_curves;
                 config.write();
             } else {
                 info!("{MOD_NAME}: Fan curves previously stored, loading...");
                 config = config.load();
-                fan_curves.balanced = config.balanced.clone();
-                fan_curves.performance = config.performance.clone();
-                fan_curves.quiet = config.quiet.clone();
             }
 
             return Ok(Self {
                 config: Arc::new(Mutex::new(config)),
-                fan_curves: Arc::new(Mutex::new(fan_curves)),
                 platform,
             });
         }
 
         Err(ProfileError::NotSupported.into())
     }
-
-    pub async fn update_profiles_from_config(&self) {
-        self.fan_curves.lock().await.balanced = self.config.lock().await.balanced.clone();
-        self.fan_curves.lock().await.performance = self.config.lock().await.performance.clone();
-        self.fan_curves.lock().await.quiet = self.config.lock().await.quiet.clone();
-    }
-
-    /// Because this locks both config and fan_curves, it means nothing else can
-    /// hold a lock across this function call. Stupid choice to do this and
-    /// needs to be fixed.
-    pub async fn update_config_from_profiles(&self) {
-        self.config.lock().await.balanced = self.fan_curves.lock().await.balanced.clone();
-        self.config.lock().await.performance = self.fan_curves.lock().await.performance.clone();
-        self.config.lock().await.quiet = self.fan_curves.lock().await.quiet.clone();
-    }
 }
 
 #[dbus_interface(name = "org.asuslinux.Daemon")]
@@ -132,15 +112,16 @@ impl CtrlFanCurveZbus {
         profile: ThrottlePolicy,
         enabled: bool,
     ) -> zbus::fdo::Result<()> {
-        self.fan_curves
+        self.config
             .lock()
             .await
+            .profiles
             .set_profile_curves_enabled(profile, enabled);
-        self.fan_curves
+        self.config
             .lock()
             .await
+            .profiles
             .write_profile_curve_to_platform(profile, &mut find_fan_curve_node()?)?;
-        self.update_config_from_profiles().await;
         self.config.lock().await.write();
         Ok(())
     }
@@ -153,15 +134,16 @@ impl CtrlFanCurveZbus {
         fan: FanCurvePU,
         enabled: bool,
     ) -> zbus::fdo::Result<()> {
-        self.fan_curves
+        self.config
             .lock()
             .await
+            .profiles
             .set_profile_fan_curve_enabled(profile, fan, enabled);
-        self.fan_curves
+        self.config
             .lock()
             .await
+            .profiles
             .write_profile_curve_to_platform(profile, &mut find_fan_curve_node()?)?;
-        self.update_config_from_profiles().await;
         self.config.lock().await.write();
         Ok(())
     }
@@ -172,9 +154,10 @@ impl CtrlFanCurveZbus {
         profile: ThrottlePolicy,
     ) -> zbus::fdo::Result<Vec<CurveData>> {
         let curve = self
-            .fan_curves
+            .config
             .lock()
             .await
+            .profiles
             .get_fan_curves_for(profile)
             .to_vec();
         Ok(curve)
@@ -187,15 +170,16 @@ impl CtrlFanCurveZbus {
         profile: ThrottlePolicy,
         curve: CurveData,
     ) -> zbus::fdo::Result<()> {
-        self.fan_curves
+        self.config
             .lock()
             .await
+            .profiles
             .save_fan_curve(curve, profile)?;
-        self.fan_curves
+        self.config
             .lock()
             .await
+            .profiles
             .write_profile_curve_to_platform(profile, &mut find_fan_curve_node()?)?;
-        self.update_config_from_profiles().await;
         self.config.lock().await.write();
         Ok(())
     }
@@ -207,11 +191,11 @@ impl CtrlFanCurveZbus {
     /// read only for the currently active profile.
     async fn set_active_curve_to_defaults(&mut self) -> zbus::fdo::Result<()> {
         let active = self.platform.get_throttle_thermal_policy()?;
-        self.fan_curves
+        self.config
             .lock()
             .await
+            .profiles
             .set_active_curve_to_defaults(active.into(), &mut find_fan_curve_node()?)?;
-        self.update_config_from_profiles().await;
         self.config.lock().await.write();
         Ok(())
     }
@@ -225,13 +209,13 @@ impl CtrlFanCurveZbus {
         let active = self.platform.get_throttle_thermal_policy()?;
 
         self.platform.set_throttle_thermal_policy(profile.into())?;
-        self.fan_curves
+        self.config
             .lock()
             .await
+            .profiles
             .set_active_curve_to_defaults(active.into(), &mut find_fan_curve_node()?)?;
         self.platform.set_throttle_thermal_policy(active)?;
 
-        self.update_config_from_profiles().await;
         self.config.lock().await.write();
         Ok(())
     }
@@ -252,7 +236,7 @@ impl CtrlTask for CtrlFanCurveZbus {
         let watch_throttle_thermal_policy = self.platform.monitor_throttle_thermal_policy()?;
         let platform = self.platform.clone();
         let config = self.config.clone();
-        let fan_curves = self.fan_curves.clone();
+        let fan_curves = self.config.clone();
 
         tokio::spawn(async move {
             let mut buffer = [0; 32];
@@ -266,6 +250,7 @@ impl CtrlTask for CtrlFanCurveZbus {
                             fan_curves
                                 .lock()
                                 .await
+                                .profiles
                                 .write_profile_curve_to_platform(
                                     profile.into(),
                                     &mut find_fan_curve_node().unwrap(),
@@ -294,8 +279,10 @@ impl crate::Reloadable for CtrlFanCurveZbus {
             // initialises the data from system read and we need to save it
             // after
             loop {
-                if let Ok(mut curves) = self.fan_curves.try_lock() {
+                if let Ok(mut config) = self.config.try_lock() {
-                    curves.write_profile_curve_to_platform(active, &mut device)?;
+                    config
+                        .profiles
+                        .write_profile_curve_to_platform(active, &mut device)?;
                     break;
                 }
             }