Page MenuHomeFreeBSD

xen/control: disable 15226522304868a8312a84f8ff549fdc6bbf54db on !x86

Authored by on Mar 17 2021, 12:59 AM.
Referenced Files
Unknown Object (File)
Thu, May 16, 2:12 AM
Unknown Object (File)
Mon, May 13, 11:48 AM
Unknown Object (File)
Sun, May 12, 3:50 AM
Unknown Object (File)
Fri, May 10, 10:35 PM
Unknown Object (File)
Thu, May 9, 5:52 PM
Unknown Object (File)
Sun, May 5, 1:47 AM
Unknown Object (File)
Sat, Apr 27, 11:01 PM
Unknown Object (File)
Apr 25 2024, 4:03 AM



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

rG FreeBSD src repository
Lint Not Applicable
Tests Not Applicable

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.