Skip to content

Change logging mechanism to syslog#74

Merged
liulog merged 4 commits intosyswonder:mainfrom
KouweiLee:main
Jan 27, 2026
Merged

Change logging mechanism to syslog#74
liulog merged 4 commits intosyswonder:mainfrom
KouweiLee:main

Conversation

@KouweiLee
Copy link
Collaborator

@KouweiLee KouweiLee commented Jan 27, 2026

This pr mainly fixes #72 and #69

This commit refactors the logging system by removing the callback mechanism and direct file logging. The logging function now primarily outputs to syslog, streamlining the logging process. Additionally, the log_event structure and related functions have been eliminated to reduce complexity.

BREAKING CHANGE: The log_log function signature has changed; it no longer includes the 'with_enter' parameter.
This commit fixes a typo in the function name multithread_log_exit, which was incorrectly spelled as mutithread_log_exit in the log.c and virtio.c files. The header file log.h has also been updated to reflect this correction.
@liulog
Copy link
Contributor

liulog commented Jan 27, 2026

You need to install rsyslog first in root zone.

Then you should execute rsyslogd to start daemon syslog service.

Then all logs are saved in /var/log/syslog .

You can start virtio backend and start non-root zonex later.

@liulog
Copy link
Contributor

liulog commented Jan 27, 2026

I have tested in rk3588.

It works well.

This commit updates the README files in both English and Chinese to clarify the command for starting the Virtio daemon and the logging mechanisms. The `nohup ... &` command has been simplified to `&`, and instructions for viewing logs have been added for both `systemd` and non-`systemd` systems.

Additionally, the reference to `nohup.out` has been replaced with `syslog` for log retrieval instructions.
@liulog liulog self-requested a review January 27, 2026 11:15
This commit updates the README files in both English and Chinese to specify the use of `nohup` when starting the Virtio daemon. This change ensures that the daemon continues running even after the terminal is closed, improving usability for users.

No other changes were made to the content.
@liulog liulog merged commit 7e55b1b into syswonder:main Jan 27, 2026
1 check 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.

Change current hvisor-tool userspace logging framework

2 participants