Page MenuHomeFreeBSD

xen/xenusb: always include xen/xen-os.h rather than machine/xen/xen-os.h
ClosedPublic

Authored by ehem_freebsd_m5p.com on Mar 3 2021, 4:35 PM.
Tags
None
Referenced Files
F110929201: D29043.diff
Tue, Feb 25, 1:03 AM
Unknown Object (File)
Sat, Feb 8, 10:12 PM
Unknown Object (File)
Sat, Feb 8, 2:13 PM
Unknown Object (File)
Jan 25 2025, 10:05 PM
Unknown Object (File)
Jan 25 2025, 4:53 PM
Unknown Object (File)
Jan 14 2025, 12:50 AM
Unknown Object (File)
Jan 10 2025, 9:47 PM
Unknown Object (File)
Jan 10 2025, 6:14 PM

Details

Summary

Fix compilation since machine/xen/xen-os.h is requiring definition
existing in xen/xen-os.h.

In general machine/xen/xen-os.h should never be included

Adapted-to-HEAD-by: Elliott Mitchell <ehem+freebsd@m5p.com>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Mar 4 2021, 8:57 AM

This has the same issue of D29040, I had previously dealt with patch submission via mailing list and other review systems where the git author field was preserved, so crediting someone else you simply left them in the author field. Phabricator is the inverse where you need to add the original author as the footer, and the person submitting it for review gets the author field.

One issue, what are the conventional footer names? This particular one, this is simply a forward port where it is 99% the original author (simply a forward rebase). Some of this series needed rather substantial rework by me before submission (and perhaps only 75% original author's work). There are a few which were merely inspired by the original author.

I also wonder if there should be a "Group"/"Project"/"Umbrella" for Xen.