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)
Feb 15 2024, 9:11 PM
Unknown Object (File)
Feb 8 2024, 2:58 PM
Unknown Object (File)
Jan 11 2024, 8:08 PM
Unknown Object (File)
Dec 24 2023, 3:52 AM
Unknown Object (File)
Dec 23 2023, 12:20 AM
Unknown Object (File)
Dec 10 2023, 8:47 PM
Unknown Object (File)
Nov 28 2023, 3:53 PM
Unknown Object (File)
Nov 28 2023, 3:53 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.