Skip to content

vmm: temporarily decrease info logging for PIO access#177

Merged
amphi merged 1 commit into
cyberus-technology:gardenlinuxfrom
amphi:less-logging
Jun 29, 2026
Merged

vmm: temporarily decrease info logging for PIO access#177
amphi merged 1 commit into
cyberus-technology:gardenlinuxfrom
amphi:less-logging

Conversation

@amphi

@amphi amphi commented Jun 29, 2026

Copy link
Copy Markdown

The "Guest PIO read/write to unregistered address" messages are spamming the logs of our customer. As a temporary solution we thus turn those info messages into debug messages.

This is only a temporary solution, we will try to find a better long-term solution with out customer.

@amphi amphi requested review from arctic-alpaca and phip1611 June 29, 2026 13:04
@amphi amphi self-assigned this Jun 29, 2026
@amphi

amphi commented Jun 29, 2026

Copy link
Copy Markdown
Author

When this is merged, I will bump Cloud Hypervisor in libvirt and create releases.

Comment thread vmm/src/vm.rs
The "Guest PIO read/write to unregistered address" messages are spamming
the logs of our customer. As a temporary solution we thus turn those
info messages into debug messages.

This is only a temporary solution, we will try to find a better
long-term solution with out customer.

On-behalf-of: SAP sebastian.eydam@sap.com
Signed-off-by: Sebastian Eydam <sebastian.eydam@cyberus-technology.de>

@arctic-alpaca arctic-alpaca left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks!

@phip1611 phip1611 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.

Lgtm

@amphi amphi merged commit 7c88aec into cyberus-technology:gardenlinux Jun 29, 2026
19 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