Skip to content

Add support for /sys/class/mei/mei0 - #792

Merged
SuperQ merged 12 commits into
prometheus:masterfrom
micgor32:master
Jun 21, 2026
Merged

SuperQ merged 12 commits into
prometheus:masterfrom
micgor32:master

Conversation

@micgor32

@micgor32 micgor32 commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

The structure is based on sysfs/class_dmi.go.

Assumes mei0 instead of checking for meiN since in practice (afaik), there are never more than one ME interfaces. But I am happy to get rid of that assumption if needed.

@micgor32

Copy link
Copy Markdown
Contributor Author

@SuperQ could you take a look?

Comment thread sysfs/class_mei.go
Comment thread sysfs/class_mei.go Outdated
Comment thread testdata/fixtures.ttar Outdated
Comment thread sysfs/class_mei_test.go Outdated
Comment thread sysfs/class_mei.go Outdated
Comment thread sysfs/class_mei.go Outdated
@micgor32
micgor32 force-pushed the master branch 2 times, most recently from ec15f47 to 3c27180 Compare April 20, 2026 19:39
@micgor32

Copy link
Copy Markdown
Contributor Author

Resolved the mentioned issues and rebased.

Comment thread sysfs/class_mei.go Outdated

@roidelapluie roidelapluie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we have a test with multiple mei (e.g. mei1) ?

Comment thread sysfs/class_mei.go Outdated
@micgor32

Copy link
Copy Markdown
Contributor Author

Can we have a test with multiple mei (e.g. mei1) ?

Yep, added sample data for it, plus extended the test.

@micgor32

micgor32 commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

rebased.

@micgor32

Copy link
Copy Markdown
Contributor Author

rebased.

micgor32 added 11 commits June 2, 2026 12:14
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
@micgor32

micgor32 commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

@SuperQ could you please take a look if you have some time? Thanks :)

@SuperQ SuperQ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@SuperQ
SuperQ merged commit fe02e56 into prometheus:master Jun 21, 2026
7 checks passed
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.

3 participants