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

vdso-compat: Increase the reserved buffer for compat vdso #1938

Merged
merged 1 commit into from
Jul 26, 2022

Conversation

minhbq-99
Copy link
Member

On Arch Linux with 5.18.3-zen1-1-zen kernel, the vdso's size is 3 pages which
exceeds the current 2-page reserved buffer. This commit simply increases the
reserved buffer size to 4 pages.

Fixes: #1916

Signed-off-by: Bui Quang Minh minhquangbui99@gmail.com

On Arch Linux with 5.18.3-zen1-1-zen kernel, the vdso's size is 3 pages which
exceeds the current 2-page reserved buffer. This commit simply increases the
reserved buffer size to 4 pages.

Fixes: checkpoint-restore#1916

Signed-off-by: Bui Quang Minh <minhquangbui99@gmail.com>
@mihalicyn
Copy link
Member

Looks reasonable. Did you manage to find zen-kernel specific patch which changed compat vdso size?
I can't see any specific vdso patches in they kernel (https://github.com/zen-kernel/zen-kernel).

@minhbq-99
Copy link
Member Author

I can't see either, I guess it's because the compiler they use to build the kernel or maybe they set some configs.

@mihalicyn
Copy link
Member

cc @0x7f454c46

@0x7f454c46
Copy link
Member

Seems reasonable. We could probably mmap() a bigger buffer as likely most systems have overcommit enabled and it won't consume any memory before the vdso pages are copied out to the buffer.
Anyway, LGTM

@avagin avagin merged commit e15690b into checkpoint-restore:criu-dev Jul 26, 2022
@tuananh
Copy link

tuananh commented Oct 3, 2024

im getting this Error: Compatible vdso's size is bigger than reserved buf on kernel 6.11.1-zen1-1-zen (arch) still. not sure if that's related to this?

@avagin
Copy link
Member

avagin commented Oct 3, 2024

@tuananh could you file a new issue?

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.

Error: Compatible vdso's size is bigger than reserved buf
5 participants