PVHv1 was officially removed from Xen in 4.9, so just axe the related
code from FreeBSD.
Note FreeBSD supports PVHv2, which is the replacement for PVHv1.
Sponsored by: Citrix Systems R&D
Paths
| Differential D30228 Authored by royger on May 12 2021, 11:53 AM.
Details
Summary PVHv1 was officially removed from Xen in 4.9, so just axe the related Note FreeBSD supports PVHv2, which is the replacement for PVHv1. Sponsored by: Citrix Systems R&D
Diff Detail
Event TimelineHerald added subscribers: rpokala, imp. · View Herald TranscriptMay 12 2021, 11:53 AM2021-05-12 11:53:37 (UTC+0) Harbormaster completed remote builds in B39138: Diff 89066.May 12 2021, 11:53 AM2021-05-12 11:53:41 (UTC+0) This revision is now accepted and ready to land.May 12 2021, 12:59 PM2021-05-12 12:59:39 (UTC+0) Comment Actions This isn't quite what I had expected, I was expecting more of cfa0b7b82fbdda56d7160569def5c6133eb045aa to be reverted. That though appears to have been done in a separate commit which isn't here.
Comment Actions
Yes, this was done by @mhorne, I have his patches as prereqs to this one. There's also more that can be removed from ELFNOTES. I've fixed the now redundant stuff that you pointer out. Thanks for the review. This revision now requires review to proceed.May 14 2021, 10:11 AM2021-05-14 10:11:57 (UTC+0) Harbormaster completed remote builds in B39186: Diff 89187.May 14 2021, 10:11 AM2021-05-14 10:11:59 (UTC+0) Comment Actions This causes a whole bunch of #if defined(__amd64__) || defined(__i386__)/#endif pairs to disappear from unsubmitted commits. For my attempts at getting FreeBSD/ARM operational on Xen/ARM this is a noticeable improvement. This revision is now accepted and ready to land.May 15 2021, 2:05 AM2021-05-15 02:05:48 (UTC+0) Closed by commit rGac3ede5371af: x86/xen: remove PVHv1 code (authored by royger). · Explain WhyMay 17 2021, 9:43 AM2021-05-17 09:43:23 (UTC+0) This revision was automatically updated to reflect the committed changes. ehem_freebsd_m5p.com added a child revision: D29873: etc/ttys: add the xen console.Jul 15 2021, 5:13 AM2021-07-15 05:13:12 (UTC+0)
Revision Contents
Diff 89187 sys/amd64/amd64/machdep.c
|
ste[1] is reinitialized few lines bellow.