Skip to content

Update cpu collector for OpenBSD 6.4#1094

Merged
SuperQ merged 1 commit intoprometheus:masterfrom
ra1fh:openbsd-cpu-64
Oct 2, 2018
Merged

Update cpu collector for OpenBSD 6.4#1094
SuperQ merged 1 commit intoprometheus:masterfrom
ra1fh:openbsd-cpu-64

Conversation

@ra1fh
Copy link
Contributor

@ra1fh ra1fh commented Sep 30, 2018

Starting with (not yet released) OpenBSD 6.4, sysctl KERN_CPTIME2 will
return ENODEV for offline CPUs.

SMT siblings are reported as offline when hw.smt is disabled, which is
the default since one of the later Spectre variants. So this might
affect a few systems.

For more details see:
https://cvsweb.openbsd.org/src/sys/kern/kern_sysctl.c#rev1.348

Signed-off-by: Ralf Horstmann ralf@ackstorm.de

@SuperQ @discordianfish

@SuperQ
Copy link
Member

SuperQ commented Oct 1, 2018

Thanks!

Please add a [BUGFIX] entry to the CHANGELOG.md.

Starting with (not yet released) OpenBSD 6.4, sysctl KERN_CPTIME2 will
return ENODEV for offline CPUs.

SMT siblings are reported as offline when hw.smt is disabled, which is
the default since one of the later Spectre variants. So this might
affect a few systems.

For more details see:
https://cvsweb.openbsd.org/src/sys/kern/kern_sysctl.c#rev1.348

Signed-off-by: Ralf Horstmann <ralf+github@ackstorm.de>
@ra1fh
Copy link
Contributor Author

ra1fh commented Oct 1, 2018

Added CHANGELOG.md entry.

Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SuperQ SuperQ merged commit 9f820bd into prometheus:master Oct 2, 2018
@ra1fh ra1fh deleted the openbsd-cpu-64 branch February 3, 2019 20:12
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
Starting with (not yet released) OpenBSD 6.4, sysctl KERN_CPTIME2 will
return ENODEV for offline CPUs.

SMT siblings are reported as offline when hw.smt is disabled, which is
the default since one of the later Spectre variants. So this might
affect a few systems.

For more details see:
https://cvsweb.openbsd.org/src/sys/kern/kern_sysctl.c#rev1.348

Signed-off-by: Ralf Horstmann <ralf+github@ackstorm.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants