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

x86 paravirt code uses alternatives v6.8+ #1380

Closed
joe-lawrence opened this issue Mar 21, 2024 · 1 comment · Fixed by #1392
Closed

x86 paravirt code uses alternatives v6.8+ #1380

joe-lawrence opened this issue Mar 21, 2024 · 1 comment · Fixed by #1392

Comments

@joe-lawrence
Copy link
Contributor

Upstream kernel commit torvalds/linux@f7af697 removes struct paravirt_patch_site as it's no longer used to implement the feature.

Kpatch-build and friends will need to handle accordingly for upstream v6.8+ and future downstream backports.

Copy link

This issue has been open for 30 days with no activity and no assignee. It will be closed in 7 days unless a comment is added.

@github-actions github-actions bot added the stale label Apr 21, 2024
ryanbsull added a commit to ryanbsull/kpatch that referenced this issue May 1, 2024
Adds linux kernel version checks to all functions pertaining to struct
paravirt_patch_site as it is removed in Linux v6.8+

Fixes: dynup#1380

Signed-off-by: Ryan Sullivan <rysulliv@redhat.com>
ryanbsull added a commit to ryanbsull/kpatch that referenced this issue May 2, 2024
Adds linux kernel version checks to all functions pertaining to struct
paravirt_patch_site as it is removed in Linux v6.8+

Fixes: dynup#1380

Signed-off-by: Ryan Sullivan <rysulliv@redhat.com>
ryanbsull added a commit to ryanbsull/kpatch that referenced this issue May 2, 2024
Adds linux kernel version checks to all functions pertaining to struct
paravirt_patch_site as it is removed in Linux v6.8+

Fixes: dynup#1380

Signed-off-by: Ryan Sullivan <rysulliv@redhat.com>
ryanbsull added a commit to ryanbsull/kpatch that referenced this issue May 3, 2024
Adds linux kernel version checks to all functions pertaining to struct
paravirt_patch_site as it is removed in Linux v6.8+

Fixes: dynup#1380

Signed-off-by: Ryan Sullivan <rysulliv@redhat.com>
ryanbsull added a commit to ryanbsull/kpatch that referenced this issue May 3, 2024
Adds linux kernel version checks to all functions pertaining to struct
paravirt_patch_site as it is removed in Linux v6.8+

Fixes: dynup#1380

Signed-off-by: Ryan Sullivan <rysulliv@redhat.com>
ryanbsull added a commit to ryanbsull/kpatch that referenced this issue May 3, 2024
Adds linux kernel version checks to all functions pertaining to struct
paravirt_patch_site as it is removed in Linux v6.8+

Fixes: dynup#1380

Signed-off-by: Ryan Sullivan <rysulliv@redhat.com>
ryanbsull added a commit to ryanbsull/kpatch that referenced this issue May 3, 2024
Upstream kernel commit f7af6977621a ("x86/paravirt: Remove no longer
needed paravirt patching code") v6.8+ removed the .parainstructions
section and its paravirt_patch_site struct. Therefore this checks the
kernel version and does not export the struct size if the kernel
version is >= v6.8.0, avoiding the code path for it in
create-diff-object.c entirely.

Fixes: dynup#1380

Signed-off-by: Ryan Sullivan <rysulliv@redhat.com>
swine pushed a commit to swine/kpatch that referenced this issue May 31, 2024
Upstream kernel commit f7af6977621a ("x86/paravirt: Remove no longer
needed paravirt patching code") v6.8+ removed the .parainstructions
section and its paravirt_patch_site struct. Therefore this checks the
kernel version and does not export the struct size if the kernel
version is >= v6.8.0, avoiding the code path for it in
create-diff-object.c entirely.

Fixes: dynup#1380

Signed-off-by: Ryan Sullivan <rysulliv@redhat.com>
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 a pull request may close this issue.

1 participant