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
F153521333: D29306.id92868.diff
Tue, Apr 21, 2:56 PM
Unknown Object (File)
Wed, Apr 15, 2:38 PM
Unknown Object (File)
Wed, Apr 15, 12:48 AM
Unknown Object (File)
Sat, Apr 11, 4:16 PM
Unknown Object (File)
Sat, Apr 11, 3:07 AM
Unknown Object (File)
Fri, Apr 10, 10:11 PM
Unknown Object (File)
Fri, Apr 10, 6:47 AM
Unknown Object (File)
Wed, Apr 8, 12:52 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.