Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mem: optimize debug logging of enqueued pages #2389

Merged
merged 1 commit into from Apr 16, 2024

Conversation

rst0git
Copy link
Member

@rst0git rst0git commented Apr 16, 2024

During restore, CRIU prints "Enqueue page-read" messages for each page-read request (91388fc). However, this message does not provide useful information, increases performance overhead during restore and the size of log file.

$ ./zdtm.py run -t zdtm/static/maps06 -f h -k always
$ grep 'Enqueue page-read' dump/zdtm/static/maps06/56/1/restore.log | wc -l
20493

This pull request replaces these messages with a single message that shows the number of enqueued page-read requests.

$ grep 'enqueued' dump/zdtm/static/maps06/56/1/restore.log
(00.061449)     56: nr_enqueued:   20493

@rst0git rst0git changed the title mem: optimize debug logging enqueued pages mem: optimize debug logging of enqueued pages Apr 16, 2024
@rst0git rst0git force-pushed the mem-page-read branch 3 times, most recently from 324bdc2 to eafd1a6 Compare April 16, 2024 12:26
During restore, CRIU prints "Enqueue page-read" messages for
each page-read request [1]. However, this message does not
provide useful information, increases performance overhead
during restore and the size of log file.

$ ./zdtm.py run -t zdtm/static/maps06 -f h -k always
$ grep 'Enqueue page-read' dump/zdtm/static/maps06/56/1/restore.log | wc -l
20493

This commit replaces these log messages with a single message
that shows the number of enqueued page-read requests.

$ grep 'enqueued' dump/zdtm/static/maps06/56/1/restore.log
(00.061449)     56: nr_enqueued:   20493

[1] checkpoint-restore@91388fc

Signed-off-by: Radostin Stoyanov <rstoyanov@fedoraproject.org>
@avagin avagin merged commit 18dcf15 into checkpoint-restore:criu-dev Apr 16, 2024
35 of 39 checks passed
@rst0git rst0git deleted the mem-page-read branch April 17, 2024 04:33
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.

None yet

2 participants