Page MenuHomeFreeBSD

LinuxKPI: Add xen/xen.h header
ClosedPublic

Authored by wulf on Nov 28 2023, 8:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 11:22 PM
Unknown Object (File)
Jan 18 2024, 1:22 PM
Unknown Object (File)
Jan 5 2024, 10:37 PM
Unknown Object (File)
Jan 5 2024, 10:37 PM
Unknown Object (File)
Dec 25 2023, 5:30 PM
Unknown Object (File)
Dec 25 2023, 10:52 AM
Unknown Object (File)
Dec 24 2023, 10:02 AM
Unknown Object (File)
Dec 23 2023, 4:14 AM

Details

Summary

It contains proxy-implementation of xen_initial_domain() and
xen_pv_domain() required by latest drm-kmod.

Sponsored by: Serenity Cyber Security, LLC
MFC after: 1 week

Diff Detail

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

Event Timeline

wulf requested review of this revision.Nov 28 2023, 8:48 PM
This revision is now accepted and ready to land.Nov 29 2023, 6:10 AM
ehem_freebsd_m5p.com added inline comments.
sys/compat/linuxkpi/common/src/linux_compat.c
107–108

This is due to sys/amd64/include/xen/hypercall.h. I suspect the use is identical to what Linux headers used (the file originated with Linux as MIT/GPL2).

I think it would be reasonable to surround the #define __must_check in sys/amd64/include/xen/hypercall.h with #ifndef __must_check. This will interact with the merge from sys/amd64/include/xen/hypercall.h => sys/xen/hypercall.h though. @royger opinion on allowing use of the Linux-KPI's definition of __must_check?

While the hypercall headers where originally imported from Linux, I think there's very little chance we will even sync again with Linuxes version, so s/__must_check/__result_use_check/ in the Xen hypercall headers would be the best solution IMO. Playing games with ifdef/ifndef is usually not a good idea.

This revision was automatically updated to reflect the committed changes.