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
xen: add v 4.10 #36292
xen: add v 4.10 #36292
Conversation
Thanks! Will test.
I also think we should switch the default from 4.5 to 4.8 as soon as I fix 4.8 build, gcc update broke it again :(
|
I agree that 4.5 should be deprecated since security support ended in January. |
now all 3 versions (-vanilla, -slim, -light) build. Still needs testing. |
@oxij regarding xen 4.8, I believe this will fix the gcc7 build error: https://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=f49fa658b53580cf2ad354d2bf1796766cc11222 |
May I kindly ask you to fetchpatch it into the 4.8 expression and to check it compiles? My computational resources are busy with libpulseaudio/pressureaudio/cardiacarrest mass-rebuild ATM.
|
sure, will do and let you know/ |
4.8: we're on the right track. Patch fixed previous error but no a similar error occurs in another place. I'll keep picking patches and open a separate PR when it works |
rebased on master |
4.10 works now. Tested in a VM with no obvious problems. Could use some more testing (maybe on real hardware) to make sure it's ready to merge. |
From a glance, it looks like it doesn't change 4.8. If it is a pure addition, maybe it should be just merged? Or maybe @oxij wants to test it on real hardware… |
I'm rebuilding everything the 6th time with libcardiacarrest... So not yet.
I will test it eventually, but if it boots I say it should be merged now.
I would squash (and remove [WIP]) 7a3b1949fdc, bf9426cb6e2, and c4dd6904cee into a single commit, though.
And then maybe I'd repeat that for 387ef24572f and f1541f1475a too.
|
Sounds good. I'll clean up the commits first, then we can merge. |
string compare breaks with xen 4.10 (because "4.10" < "4.8")
this is required for oxenstored to start
Ready to merge. |
LGTM. Thanks!
|
Motivation for this change
Would like to experiment with latest xen version (in particular, try to enable xen-uefi boot in NixOS - which appears to require xen >= 4.9).
This change only adds xen 4.10. It does not modify the other versions (4.5 and 4.8) and leaves the default at 4.5 to prevent breaking existing installations.
Current Status
Things done
Things to do
/CC xen maintainers: @edolstra @oxij @tstrobel