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
Unknown Object (File)
Thu, Dec 5, 11:22 AM
Unknown Object (File)
Sep 30 2024, 2:02 PM
Unknown Object (File)
Sep 25 2024, 10:26 AM
Unknown Object (File)
Sep 25 2024, 10:26 AM
Unknown Object (File)
Sep 25 2024, 10:26 AM
Unknown Object (File)
Sep 24 2024, 8:54 PM
Unknown Object (File)
Sep 22 2024, 6:01 PM
Unknown Object (File)
Sep 17 2024, 4:11 AM

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.