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)
Mon, Oct 20, 9:17 AM
Unknown Object (File)
Mon, Oct 20, 12:16 AM
Unknown Object (File)
Thu, Oct 16, 11:37 AM
Unknown Object (File)
Thu, Oct 16, 9:24 AM
Unknown Object (File)
Thu, Oct 16, 6:58 AM
Unknown Object (File)
Thu, Oct 16, 4:18 AM
Unknown Object (File)
Thu, Oct 16, 3:07 AM
Unknown Object (File)
Thu, Oct 16, 2:24 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.