Page MenuHomeFreeBSD

bhyve: Make gdb support optional
ClosedPublic

Authored by markj on Jul 12 2023, 1:51 PM.
Tags
None
Referenced Files
F83929212: D40986.id128249.diff
Thu, May 16, 11:52 PM
F83929208: D40986.id.diff
Thu, May 16, 11:52 PM
F83929198: D40986.id124548.diff
Thu, May 16, 11:51 PM
Unknown Object (File)
Wed, May 15, 12:48 AM
Unknown Object (File)
Mar 10 2024, 5:32 PM
Unknown Object (File)
Jan 19 2024, 1:39 AM
Unknown Object (File)
Dec 20 2023, 8:05 AM
Unknown Object (File)
Dec 11 2023, 4:29 AM
Subscribers

Details

Summary

Add a BHYVE_GDB_SUPPORT make variable that can be set by per-arch
makefiles. When set, BHYVE_GDB is defined and can be used as a
preprocessor predicate. Use it to guard gdb stub calls in MI code.

The arm64 bhyve port currently does not have a functional gdb stub, but
that's not critical to landing the port, so this mechanism slightly
reduces the friction of adding support for a new platform.

Diff Detail

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

Event Timeline

markj requested review of this revision.Jul 12 2023, 1:51 PM
jhb added inline comments.
usr.sbin/bhyve/bhyverun.c
149

Do you want to make '-G' here conditional on BHYVE_GDB? (Not sure how messy the usage() function ends up at the end of all this)

This revision is now accepted and ready to land.Jul 12 2023, 6:33 PM
This revision was automatically updated to reflect the committed changes.