feat: betterperformancelogic

Issue: linuxdeepin/developer-center#6514
Log:

Author: Decodetalkers

system/power1/cpu_handler.go

@@ -26,6 +26,7 @@ const (
 	globalAvailableGovernorFileName = "scaling_available_governors"
 	globalBoostFilePath             = "/sys/devices/system/cpu/cpufreq/boost"
 	globalDefaultPath               = "/sys/devices/system/cpu/cpu0/cpufreq/scaling_governor"
+	globalDefaultDriverPath         = "/sys/devices/system/cpu/cpu0/cpufreq/scaling_driver"
 	_isHuaWei                       = `dmidecode -t 1 | awk "/Product Name:/{print $NF}" | cut -d ":" -f 2`
 )
 
@@ -103,13 +104,11 @@ func getIsBalanceSupported(isPstate bool) bool {
 	}
 }
 
-// TODO: 之后是否需要判断boost 的逻辑需要讨论
 func getIsHighPerformanceSupported(isPstate bool) bool {
-	cpus := CpuHandlers{}
 	if isPstate {
 		return strv.Strv(getSupportGovernors()).Contains("performance")
 	}
-	return cpus.IsBoostFileExist() && strv.Strv(getSupportGovernors()).Contains("performance")
+	return strv.Strv(getSupportGovernors()).Contains("performance")
 }
 
 func getIsPowerSaveSupported(isPstate bool) bool {
@@ -340,6 +339,22 @@ func (cpus *CpuHandlers) getCpuGovernorPath(isPstate bool) string {
 	return path
 }
 
+func cpuHasPstate() bool {
+	if !dutils.IsFileExist(globalDefaultDriverPath) {
+		return false
+	}
+	driverData, err := os.ReadFile(globalDefaultDriverPath)
+	if err != nil {
+		return false
+	}
+	driver := string(driverData)
+	return driverHasPstate(driver) && dutils.IsFileExist(pstateConfPath)
+}
+
+func driverHasPstate(driver string) bool {
+	return strings.Contains(driver, "pstate")
+}
+
 // 通过写文件的返回情况，获取非scaling_available_governors的值是否支持
 func (cpus *CpuHandlers) tryWriteGovernor(lines []string, isPstate bool) []string {
 	path := cpus.getCpuGovernorPath(isPstate)

system/power1/cpu_handler_test.go

@@ -5,11 +5,25 @@
 package power
 
 import (
-	"testing"
-
 	"github.com/stretchr/testify/assert"
+	"os"
+	"testing"
 )
 
+func Test_IsPstate(t *testing.T) {
+	driverData, _ := os.ReadFile("./testdata/scaling_driver_intel_pstate")
+	driver := string(driverData)
+	assert.Equal(t, driverHasPstate(driver), true)
+
+	driverData, _ = os.ReadFile("./testdata/scaling_driver_amd_pstate")
+	driver = string(driverData)
+	assert.Equal(t, driverHasPstate(driver), true)
+
+	driverData, _ = os.ReadFile("./testdata/scaling_driver_intel_cpufreq")
+	driver = string(driverData)
+	assert.Equal(t, driverHasPstate(driver), false)
+}
+
 func Test_GetGovernor(t *testing.T) {
 	expectGovernor := "performance"
 	cpu := CpuHandler{}

system/power1/manager.go

@@ -151,7 +151,7 @@ func newManager(service *dbusutil.Service) (*Manager, error) {
 	}
 	// check pstate , if has pstate, it is intel pstate mode , then
 	// we need another logic
-	m.hasPstate = (dutils.IsFileExist(intelPstatePath) || dutils.IsFileExist(amdPstatePath)) && dutils.IsFileExist(pstateConfPath) // check if amd is used
+	m.hasPstate = cpuHasPstate() // check if amd is used
 	m.hasAmddpm = dutils.IsFileExist(amdGPUPath)
 
 	m.refreshSystemPowerPerformance()

system/power1/testdata/scaling_driver_amd_pstate

@@ -0,0 +1 @@
+amd_pstatepp

system/power1/testdata/scaling_driver_intel_cpufreq

@@ -0,0 +1 @@
+intel_cpufreq

system/power1/testdata/scaling_driver_intel_pstate

@@ -0,0 +1 @@
+intel_pstate