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
Unknown Object (File)
Sat, Apr 27, 11:01 PM
Unknown Object (File)
Thu, Apr 25, 4:03 AM
Unknown Object (File)
Tue, Apr 23, 8:24 PM
Unknown Object (File)
Jan 3 2024, 4:51 AM
Unknown Object (File)
Dec 30 2023, 10:59 AM
Unknown Object (File)
Dec 30 2023, 10:10 AM
Unknown Object (File)
Dec 30 2023, 9:49 AM
Unknown Object (File)
Dec 10 2023, 12:17 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.