Wert von MAXCPU in C auf amd64 (prometheus cpu collector)

Hallo Liste,


ich knobel an einer compile Frage, in die ich reingestolpert bin, 
nachdem ich mir Prometheus auf meinem Server installiert habe.
Der Autor des cpu collectors probiert scheinbar rauszubekommen, wie groß 
die maximale Anzahl an CPUs ist.

https://github.com/prometheus/node_exporter/blob/master/collector/cpu_freebsd.go#L106

Zeile 109: const maxCPUTimesLen = C.MAXCPU * C.CPUSTATES


MAXCPU ist wohl als konstante in /usr/include/machine/params.h 
definiert. Unter amd64, wenn SMP definiert ist, dann 64, else 1.
Was mich wundert ist, dass auf meinem 10.3-RELEASE das ganze im else 
part landet. Obwohl amd64.

[prometheus ~]$ uname -srm
FreeBSD 10.3-RELEASE-p11 amd64

Wenn ich den Golang (oder halb C halb cgo halb golang?!) code richtig 
verstehe, will er diese Berechnung moeglichst genau machen, damit er 
keinen unnötig großen Array in go anlegt:

Zeile 159 und 160

	// Convert C.double array to Go array 
(https://github.com/golang/go/wiki/cgo#turning-c-arrays-into-go-slices).
	cpuTimes := 
(*[maxCPUTimesLen]C.double)(unsafe.Pointer(cpuTimesC))[:cpuTimesLength:cpuTimesLength]

Das ist ja an sich erstmal löblich.
Ich finde nur keinen Weg, wie C.MAXCPU auf 64 kommt, ausser über Zeile 
30 in seinem Code ein '#define SMP 1' reinzuwerfen.
Aber das kann ja nicht die Lösung sein.

So wie es jetzt ist, egal ob node_exporter via package oder via github 
und go build installiert wird, kann es nur funktionieren, falls das 
system nur 1 Prozessor hat. Ansonsten geht sein vergleich in Zeile 154 
bis 157:
	defer C.freeCPUTimes(cpuTimesC)
	if cpuTimesLength > maxCPUTimesLen {
		return errors.New("more CPU's than MAXCPU?")
	}
einfach schief und falls man node_exporter startet und dann via curl 
http://...:9100/metrics abfragt, endet das mit:
More CPU's than MAXCPU?
Weil er CPU's tatsächlich richtig berechnet. In meinem Fall bei 
hw.ncpu=8 kommt sein Code auf 40. Aber maxCPUTimesLen nur bei 1 * 5 
landet.
Und 40 ist ja tatsache größer als 5 :)


Hat jemand ne Idee warum MAXCPU 1 ist, bzw wie man das sauber lösen 
kann?

BeSDe Grüße,
Marian

PS: Spaß am Rande. Was die go Leute unter 
https://github.com/golang/go/wiki/cgo#turning-c-arrays-into-go-slices 
angeben klingt auch eher abenteuerlich:
To create a Go slice backed by a C array (without copying the original 
data), one needs to acquire this length at runtime and use a type 
conversion to a pointer to a very big array and then slice it to the 
length that you want.
Klingt wie ein Backrezept: Man nehme eine große Schüssel (a pointer to a 
very big array), werfe die Zutaten da rein und messe dann mal nach (and 
slice it to the length that you want).
Immerhin wollte der Autor von cpu_freebsd.go scheinbar die Schüsselgröße 
nicht zu groß dimensionieren.
Mein workaround mit dem #define SMP 1 macht halt ne große Schüssel. Aber 
als pr will ich das auf github eigentlich nicht rübergeben...



-- 
the problem with troubleshooting is that trouble shoots back.


To Unsubscribe: send mail to [email protected]
with "unsubscribe de-bsd-questions" in the body of the message