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)
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
Unknown Object (File)
Nov 1 2023, 3:36 PM
Unknown Object (File)
Sep 6 2023, 7:07 AM
Unknown Object (File)
Aug 23 2023, 4:31 PM

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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
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.