Ryzen 9 - ksysguard - CPU freq

15.3.2020 10:00 | Přečteno: 1486× | Ryzen 9 | poslední úprava: 16.3.2020 11:28

V aktuálním OpenSUSE Leap 15.1 v ksysguard nelze zobrazit frekvence CPU. Je to v důsledku nedostatečné velikosti buferu, jak je popsáno zde v hlášení bugu včetně návrhu dočasného řešení. Pro Ryzen 9 3900X a 3950X stačí zvětšit velikost buferu na řádku 43 v cpuinfo.c na dvojnásobek. Přikládám patch a upravený spec file pro OpenSUSE Leap 15.1, kdyby to někomu přišlo vhod. Pokud nejste zběhlí v v tvorbě rpm balíčků, tady je malý návod. Stáhněte si přiložené soubory, a pak jako root proveďte (je třeba mít přidaný repozitář pro zdrojové balíčky):

zypper ar -f http://download.opensuse.org/source/distribution/leap/$releasever/repo/oss/ repo-source

zypper ref

zypper si ksysguard5

cp ksysguard5.spec /usr/src/packages/SPECS/

cp CPUINFOBUFSIZE.patch /usr/src/packages/SOURCES/

rpmbuild -ba /usr/src/packages/SPECS/ksysguard5.spec

rpm -Uhv --force /usr/src/packages/RPMS/x86_64ksysguard5-5.12.8-lp151.1.1.pf.x86_64.rpm

systemctl restart ksysguardd.service

CPUINFOBUFSIZE.patch

diff -Naur ksysguard-5.12.8.orig/ksysguardd/Linux/cpuinfo.c ksysguard-5.12.8/ksysguardd/Linux/cpuinfo.c
--- ksysguard-5.12.8.orig/ksysguardd/Linux/cpuinfo.c	2019-03-05 13:20:46.000000000 +0100
+++ ksysguard-5.12.8/ksysguardd/Linux/cpuinfo.c	2020-01-30 10:53:59.604577796 +0100
@@ -40,7 +40,7 @@
 static int HighNumCores = 0; /* Highest # of cores ever seen */
 static float* Clocks = 0; /* Array with one entry per core */
 
-#define CPUINFOBUFSIZE (32 * 1024)
+#define CPUINFOBUFSIZE (64 * 1024)
 static char CpuInfoBuf[ CPUINFOBUFSIZE ];
 static int Dirty = 0;
 static struct SensorModul *CpuInfoSM;

V ksysguard5.spec je nutno přidat řádek odkazující na CPUINFOBUFSIZE.patch

Source:         https://download.kde.org/stable/plasma/%{version}/ksysguard-%{version}.tar.xz
#Source:		ksysguard-5.12.8.tar.gz	
Source1:        ksysguardd.service
# PATCH-FIX-OPENSUSE 0001-Use-run-for-ksysguardd-s-pid-file.patch
Patch0:         0001-Use-run-for-ksysguardd-s-pid-file.patch
Patch1:		CPUINFOBUFSIZE.patch
BuildRequires:  extra-cmake-modules >= 1.5.0
BuildRequires:  kf5-filesystem
BuildRequires:  libsensors4-devel
BuildRequires:  systemd-rpm-macros
BuildRequires:  xz
BuildRequires:  cmake(KF5Config)
BuildRequires:  cmake(KF5CoreAddons)
BuildRequires:  cmake(KF5I18n)
BuildRequires:  cmake(KF5IconThemes)
BuildRequires:  cmake(KF5ItemViews)
BuildRequires:  cmake(KF5KDELibs4Support)
BuildRequires:  cmake(KF5NewStuff)
BuildRequires:  cmake(KF5SysGuard) >= %{_plasma5_version}
BuildRequires:  cmake(Qt5Core) >= 5.4.0
BuildRequires:  cmake(Qt5WebKitWidgets) >= 5.4.0
BuildRequires:  cmake(Qt5Widgets) >= 5.4.0
Requires:       libksysguard5-helper

       

Komentáře

[Michal Kubeček]

Chápu, že se chcete o informace podělit, ale IMHO by bylo užitečnější otevřít bug a - protože víte, jak problém (aspoň částečně) řešit - poslat submit request.

[Petr Tomášek]

A to se snad vylučuje?

[Michal Kubeček]

Nevylučuje… ale žádný bug report ani submit request jsem nenašel. Kdyby ano, ten komentář bych nepsal.

[Kate]

Co ten odkaz v druhé větě blogpostu?

[Michal Kubeček]

Mluvil jsem o openSUSE bugzille, když je post o tom, jak si udělat balíček pro openSUSE Leap 15.1.

[Michal Kubeček]

(a stejně tak jsem měl na mysli submit request pro Leap 15.1 - a případně i 15.2, je-li potřeba)

[doron]

Proč jsem to nereportoval v openSUSE bugzille? Jde jen o workaround, až to někdo u ksysguard korektně opraví, bude to opravené i openSUSE.

Chápu, že zkušený uživatel neocení návod jak vytvořit balíček pro openSUSE, ale nelze vyloučit, že to bude číst někdo, komu to přijde vhod.

[Michal Kubeček]

Pokud to měl být návod, jak si udělat upravený balíček pro openSUSE, pak bych spíš doporučil

  osc checkout openSUSE:Leap:15.2:ksysguard5
  # upravit
  osc build

nebo to rovnou udělat v OBS (a pak už je submit request otázkou jednoho příkazu navíc). Distribuční balíčky vesměs moc nepočítají s tím, že by někdo chtěl používat přímo rpmbuild, takže můžete narazit na různé záhadné problémy.

[Michal Kubeček]

Ten první příkaz měl být

  osc checkout openSUSE:Leap:15.1:Update ksysguard5

[doron]

Díky za typ. OBS jsem prozatím nepoužíval. Tak jsem si alespoň zřídil účet, a pokusím se to tam během dne (příštích dní) umístit.

[Nuphar]

Říkal jsem si, že se mi na desktopu s Ryzenem nějak neukazuje frekvence CPU, ale neměl jsem čas to řešit. I takovýto workaround je rozhodně lepší rovnou v OBS, protože výsledek je pak rozhodně přístupnější více lidem a spíš to rozproudí nějakou diskuzi, aby se to pořádně opravilo.

[retroslava]

https://phabricator.kde.org/D27362

[BFU]

Proc se ten soubor read()uje a neudela se na to treba radeji mmap() s PROT_READ ?

[Michal Kubeček]

Protože je to /proc/cpuinfo, kde něco takového nemá šanci fungovat. Správné řešení je nesnažit se odhadnout velikost a načíst celý soubor najednou, ale zpracovávat ho postupně.

[BFU]

No to je teda vysvetleni jak remen, nedik.

Takze jsem si to vygrepoval, ze kernelovy fs/proc/cpuinfo.c + arch-specific kod plni userspacovy buffer pres seq_printf() . Coz by ale zrejme taky znamenalo, ze nejlepsi je to cist po radcich fgets() , jak to dela treba util-linux.

[Michal Kubeček]

Prostě jsem předpokládal, že někomu, kdo používá mmap(), nemusím do detailu vysvětlovat, proč na procfs nepůjde použít.

[BFU]

To jste predpokladal spatne, protoze to, ze jeden soubor v nejakem kernelovem virtualnim FS neimplementuje .mmap neznamena, ze jiny soubor - klidne ve stejnem virtualnim FS - ten .mmap neimplementuje taky. Ono by vesmes stacilo ten mmap do fs/proc/cpuinfo.c pridat, struct file_operations {} ma callbacky na vsechno mozne, ale ten duvod proc to tam neni jsem popsal vyse.

[Michal Kubeček]

No tak se zkuste podívat, kolik těch proc_mmap celkem v jádře existuje. Já tam vidím celé čtyři kusy - z toho tři jsou věci, které mají k něčemu jako /proc/cpuinfo zatraceně daleko a jeden navíc existuje jen na powerpc.

[BFU]

To ja nevim, rychly git grep tvrdi neco pres 500:

linux-2.6$ git describe HEAD

next-20200316

linux-2.6$ git grep '[[:space:]]\.mmap[[:space:]]*=' | wc -l

516

Cely ten proc_* je wrapper okolo genericke implementace virtualniho FS v Linuxu, nad kterou je postaveny i sysfs, debugfs, configfs atd.

[Michal Kubeček]

To jste si ale vymyslel úplně jinou otázku, než na jakou jsem se ptal.

Zkuste se nad tím trochu zamyslet. Ten soubor vzniká on demand a dopředu nezjistíte ani jeho velikost (dostanete nulu). Jak ho chcete mmapovat? Co tomu mmap() předáte jako velikost?

[BFU]

Viz vyse muj komentar o seq_printf().