Page MenuHomeFreeBSD

xen/control: disable 15226522304868a8312a84f8ff549fdc6bbf54db on !x86
ClosedPublic

Authored by ehem_freebsd_m5p.com on Mar 17 2021, 12:59 AM.
Tags
None
Referenced Files
F106328897: D29306.diff
Sat, Dec 28, 7:08 PM
Unknown Object (File)
Oct 5 2024, 7:08 AM
Unknown Object (File)
Sep 26 2024, 12:08 PM
Unknown Object (File)
Sep 24 2024, 4:45 PM
Unknown Object (File)
Sep 22 2024, 1:32 AM
Unknown Object (File)
Sep 19 2024, 6:48 PM
Unknown Object (File)
Sep 18 2024, 1:38 PM
Unknown Object (File)
Sep 13 2024, 5:00 AM

Details

Summary

Commit 15226522304868a8312a84f8ff549fdc6bbf54db was implemented strictly
for x86. Unfortunately one of the pieces was mixed into a common area
breaking other architectures. For now disable these bits on !x86, this
should be cleaned up later.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 37888
Build 34777: arc lint + arc unit

Event Timeline

I'm fine with this, if in the future other arches require some arch specific stuff to be executed at a similar place we could introduce an arch-specific hook (ie: xen_cpu_resumed or some such, I'm bad at naming).

This revision is now accepted and ready to land.Mar 25 2021, 3:41 PM

Then what I realized D29599 heavily overlaps with this. I suspect I was trying to see whether xctrl_suspend() could be made to build on ARM and found this spot was particularly problematic. When you're shuffling 82 patches around you can lose track of overlapping ones.